fixed more codacy warnings

rfiorella · Mar 28, 2019 · 04c33a0 · 04c33a0
1 parent 9027cff
commit 04c33a0
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/tests/cunit/test_iosystem2_simple2.c b/tests/cunit/test_iosystem2_simple2.c
@@ -36,10 +36,8 @@ int main(int argc, char **argv)
     int ntasks; /* Number of processors involved in current execution. */
     int iosysid; /* The ID for the parallel I/O system. */
     int iosysid_world; /* The ID for the parallel I/O system. */
-    int num_flavors; /* Number of PIO netCDF flavors in this build. */
-    int flavor[NUM_FLAVORS]; /* iotypes for the supported netCDF IO flavors. */
-    int ret; /* Return code. */
     MPI_Comm test_comm;
+    int ret; /* Return code. */
 
     /* Initialize test. */
     if ((ret = pio_test_init2(argc, argv, &my_rank, &ntasks, TARGET_NTASKS, TARGET_NTASKS,
@@ -49,13 +47,16 @@ int main(int argc, char **argv)
     /* Only do something on the first TARGET_NTASKS tasks. */
     if (my_rank < TARGET_NTASKS)
     {
+	int num_flavors; /* Number of PIO netCDF flavors in this build. */
+	int flavor[NUM_FLAVORS]; /* iotypes for the supported netCDF IO flavors. */
+
         /* Figure out iotypes. */
         if ((ret = get_iotypes(&num_flavors, flavor)))
             ERR(ret);
 
         /* Split world into odd and even. */
         MPI_Comm newcomm;
-        int even = my_rank % 2 ? 0 : 1;
+        int even = (my_rank % 2) ? 0 : 1;
         if ((ret = MPI_Comm_split(test_comm, even, 0, &newcomm)))
             MPIERR(ret);