Fix -Wstrict-overflow in domdec.c
authorAlexey Shvetsov <alexxy@omrb.pnpi.spb.ru>
Fri, 29 Nov 2013 18:22:31 +0000 (22:22 +0400)
committerAlexey Shvetsov <alexxy@omrb.pnpi.spb.ru>
Sat, 30 Nov 2013 17:45:58 +0000 (21:45 +0400)
d1 > d for all iterations except first one
so now we dont trigger warning

Change-Id: I581297bfafe4157659134aefc1df3c2c6a75e2fa
Signed-off-by: Alexey Shvetsov <alexxy@omrb.pnpi.spb.ru>
src/gromacs/mdlib/domdec.c

index 08eb9bfb1199f32514ea30a7bfc0bf90f3a0c558..5e2a60b4ead580b4076e9b0b3fdb2d97a98ed931 100644 (file)
@@ -1806,7 +1806,8 @@ static void dd_distribute_dfhist(gmx_domdec_t *dd, df_history_t *dfhist)
         dd_bcast(dd, sizeof(real)*nlam, dfhist->sum_minvar);
         dd_bcast(dd, sizeof(real)*nlam, dfhist->sum_variance);
 
-        for (i = 0; i<nlam; i++) {
+        for (i = 0; i < nlam; i++)
+        {
             dd_bcast(dd, sizeof(real)*nlam, dfhist->accum_p[i]);
             dd_bcast(dd, sizeof(real)*nlam, dfhist->accum_m[i]);
             dd_bcast(dd, sizeof(real)*nlam, dfhist->accum_p2[i]);
@@ -1839,7 +1840,7 @@ static void dd_distribute_state(gmx_domdec_t *dd, t_block *cgs,
         copy_mat(state->boxv, state_local->boxv);
         copy_mat(state->svir_prev, state_local->svir_prev);
         copy_mat(state->fvir_prev, state_local->fvir_prev);
-        copy_df_history(&state_local->dfhist,&state->dfhist);
+        copy_df_history(&state_local->dfhist, &state->dfhist);
         for (i = 0; i < state_local->ngtc; i++)
         {
             for (j = 0; j < nh; j++)
@@ -1874,7 +1875,7 @@ static void dd_distribute_state(gmx_domdec_t *dd, t_block *cgs,
     dd_bcast(dd, ((state_local->nnhpres*nh)*sizeof(double)), state_local->nhpres_vxi);
 
     /* communicate df_history -- required for restarting from checkpoint */
-    dd_distribute_dfhist(dd,&state_local->dfhist);
+    dd_distribute_dfhist(dd, &state_local->dfhist);
 
     if (dd->nat_home > state_local->nalloc)
     {
@@ -3676,7 +3677,7 @@ static void set_dd_cell_sizes_dlb_change(gmx_domdec_t *dd,
         {
             if (dd->ci[dd->dim[d1]] > 0)
             {
-                if (d1 > d)
+                if (d1 != d)
                 {
                     bRowMember = FALSE;
                 }