Fix clang-tidy warning
authorArtem Zhmurov <zhmurov@gmail.com>
Mon, 27 Jul 2020 10:38:11 +0000 (13:38 +0300)
committerArtem Zhmurov <zhmurov@gmail.com>
Mon, 27 Jul 2020 12:51:18 +0000 (12:51 +0000)
clang-tidy complains about comparison of int and gmx::index,
since they are of different type.

src/gromacs/gmxana/gmx_bar.cpp

index 0ce49fe1d792e13e10a2fc1a2f5a0389698d64a3..601717df3a5d96b841549bc05f4d8d9db4f77d1a 100644 (file)
@@ -983,9 +983,8 @@ static void sim_data_histogram(sim_data_t* sd, const char* filename, int nbin_de
     gmx_bool       first_set = FALSE;
     /* histogram data: */
     std::vector<int> hist;
-    double           dx     = 0;
-    double           minval = 0;
-    int              i;
+    double           dx      = 0;
+    double           minval  = 0;
     lambda_data_t*   bl_head = sd->lb;
 
     printf("\nWriting histogram to %s\n", filename);
@@ -1044,7 +1043,7 @@ static void sim_data_histogram(sim_data_t* sd, const char* filename, int nbin_de
 
             sample_coll_make_hist(sc, &hist, &dx, &minval, nbin_default);
 
-            for (i = 0; i < gmx::ssize(hist); i++)
+            for (gmx::index i = 0; i < gmx::ssize(hist); i++)
             {
                 double xmin = i * dx + minval;
                 double xmax = (i + 1) * dx + minval;