Code beautification with uncrustify
[alexxy/gromacs.git] / src / gromacs / mdlib / force.c
index 5167747a6d9c8d7eb8abee0dc674f0184d7ef6f2..c2301f8414a7fd64c99b64b3ab1be96c5041a75e 100644 (file)
 #include "gmx_omp_nthreads.h"
 
 
-void ns(FILE *fp,
-        t_forcerec *fr,
-        rvec       x[],
-        matrix     box,
-        gmx_groups_t *groups,
-        t_grpopts  *opts,
-        gmx_localtop_t *top,
-        t_mdatoms  *md,
-        t_commrec  *cr,
-        t_nrnb     *nrnb,
-        real       *lambda,
-        real       *dvdlambda,
+void ns(FILE              *fp,
+        t_forcerec        *fr,
+        rvec               x[],
+        matrix             box,
+        gmx_groups_t      *groups,
+        t_grpopts         *opts,
+        gmx_localtop_t    *top,
+        t_mdatoms         *md,
+        t_commrec         *cr,
+        t_nrnb            *nrnb,
+        real              *lambda,
+        real              *dvdlambda,
         gmx_grppairener_t *grppener,
-        gmx_bool       bFillGrid,
-        gmx_bool       bDoLongRangeNS)
+        gmx_bool           bFillGrid,
+        gmx_bool           bDoLongRangeNS)
 {
-  char   *ptr;
-  int    nsearch;
-
-
-  if (!fr->ns.nblist_initialized)
-  {
-      init_neighbor_list(fp, fr, md->homenr);
-  }
-
-  if (fr->bTwinRange)
-    fr->nlr=0;
-
-    nsearch = search_neighbours(fp,fr,x,box,top,groups,cr,nrnb,md,
-                                lambda,dvdlambda,grppener,
-                                bFillGrid,bDoLongRangeNS,TRUE);
-  if (debug)
-    fprintf(debug,"nsearch = %d\n",nsearch);
-
-  /* Check whether we have to do dynamic load balancing */
-  /*if ((nsb->nstDlb > 0) && (mod(step,nsb->nstDlb) == 0))
-    count_nb(cr,nsb,&(top->blocks[ebCGS]),nns,fr->nlr,
-    &(top->idef),opts->ngener);
-  */
-  if (fr->ns.dump_nl > 0)
-    dump_nblist(fp,cr,fr,fr->ns.dump_nl);
+    char   *ptr;
+    int     nsearch;
+
+
+    if (!fr->ns.nblist_initialized)
+    {
+        init_neighbor_list(fp, fr, md->homenr);
+    }
+
+    if (fr->bTwinRange)
+    {
+        fr->nlr = 0;
+    }
+
+    nsearch = search_neighbours(fp, fr, x, box, top, groups, cr, nrnb, md,
+                                lambda, dvdlambda, grppener,
+                                bFillGrid, bDoLongRangeNS, TRUE);
+    if (debug)
+    {
+        fprintf(debug, "nsearch = %d\n", nsearch);
+    }
+
+    /* Check whether we have to do dynamic load balancing */
+    /*if ((nsb->nstDlb > 0) && (mod(step,nsb->nstDlb) == 0))
+       count_nb(cr,nsb,&(top->blocks[ebCGS]),nns,fr->nlr,
+       &(top->idef),opts->ngener);
+     */
+    if (fr->ns.dump_nl > 0)
+    {
+        dump_nblist(fp, cr, fr, fr->ns.dump_nl);
+    }
 }
 
-static void reduce_thread_forces(int n,rvec *f,
+static void reduce_thread_forces(int n, rvec *f,
                                  tensor vir,
                                  real *Vcorr,
-                                 int efpt_ind,real *dvdl,
-                                 int nthreads,f_thread_t *f_t)
+                                 int efpt_ind, real *dvdl,
+                                 int nthreads, f_thread_t *f_t)
 {
-    int t,i;
+    int t, i;
 
     /* This reduction can run over any number of threads */
 #pragma omp parallel for num_threads(gmx_omp_nthreads_get(emntBonded)) private(t) schedule(static)
-    for(i=0; i<n; i++)
+    for (i = 0; i < n; i++)
     {
-        for(t=1; t<nthreads; t++)
+        for (t = 1; t < nthreads; t++)
         {
-            rvec_inc(f[i],f_t[t].f[i]);
+            rvec_inc(f[i], f_t[t].f[i]);
         }
     }
-    for(t=1; t<nthreads; t++)
+    for (t = 1; t < nthreads; t++)
     {
         *Vcorr += f_t[t].Vcorr;
         *dvdl  += f_t[t].dvdl[efpt_ind];
-        m_add(vir,f_t[t].vir,vir);
+        m_add(vir, f_t[t].vir, vir);
     }
 }
 
@@ -158,55 +164,55 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
                        int        flags,
                        float      *cycles_pme)
 {
-    int     i,j,status;
-    int     donb_flags;
-    gmx_bool    bDoEpot,bSepDVDL,bSB;
-    int     pme_flags;
-    matrix  boxs;
-    rvec    box_size;
-    real    Vsr,Vlr,Vcorr=0;
-    t_pbc   pbc;
-    real    dvdgb;
-    char    buf[22];
-    double  clam_i,vlam_i;
-    real    dvdl_dum[efptNR], dvdl, dvdl_nb[efptNR], lam_i[efptNR];
-    real    dvdlsum;
+    int         i, j, status;
+    int         donb_flags;
+    gmx_bool    bDoEpot, bSepDVDL, bSB;
+    int         pme_flags;
+    matrix      boxs;
+    rvec        box_size;
+    real        Vsr, Vlr, Vcorr = 0;
+    t_pbc       pbc;
+    real        dvdgb;
+    char        buf[22];
+    double      clam_i, vlam_i;
+    real        dvdl_dum[efptNR], dvdl, dvdl_nb[efptNR], lam_i[efptNR];
+    real        dvdlsum;
 
 #ifdef GMX_MPI
-    double  t0=0.0,t1,t2,t3; /* time measurement for coarse load balancing */
+    double  t0 = 0.0, t1, t2, t3; /* time measurement for coarse load balancing */
 #endif
 
-#define PRINT_SEPDVDL(s,v,dvdlambda) if (bSepDVDL) fprintf(fplog,sepdvdlformat,s,v,dvdlambda);
+#define PRINT_SEPDVDL(s, v, dvdlambda) if (bSepDVDL) {fprintf(fplog, sepdvdlformat, s, v, dvdlambda); }
 
 
-    set_pbc(&pbc,fr->ePBC,box);
+    set_pbc(&pbc, fr->ePBC, box);
 
     /* reset free energy components */
-    for (i=0;i<efptNR;i++)
+    for (i = 0; i < efptNR; i++)
     {
         dvdl_nb[i]  = 0;
         dvdl_dum[i] = 0;
     }
 
     /* Reset box */
-    for(i=0; (i<DIM); i++)
+    for (i = 0; (i < DIM); i++)
     {
-        box_size[i]=box[i][i];
+        box_size[i] = box[i][i];
     }
 
-    bSepDVDL=(fr->bSepDVDL && do_per_step(step,ir->nstlog));
+    bSepDVDL = (fr->bSepDVDL && do_per_step(step, ir->nstlog));
     debug_gmx();
 
     /* do QMMM first if requested */
-    if(fr->bQMMM)
+    if (fr->bQMMM)
     {
-        enerd->term[F_EQM] = calculate_QMMM(cr,x,f,fr,md);
+        enerd->term[F_EQM] = calculate_QMMM(cr, x, f, fr, md);
     }
 
     if (bSepDVDL)
     {
-        fprintf(fplog,"Step %s: non-bonded V and dVdl for node %d:\n",
-                gmx_step_str(step,buf),cr->nodeid);
+        fprintf(fplog, "Step %s: non-bonded V and dVdl for node %d:\n",
+                gmx_step_str(step, buf), cr->nodeid);
     }
 
     /* Call the short range functions all in one go. */
@@ -217,36 +223,36 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
     if (TAKETIME)
     {
         MPI_Barrier(cr->mpi_comm_mygroup);
-        t0=MPI_Wtime();
+        t0 = MPI_Wtime();
     }
 #endif
 
     if (ir->nwall)
     {
         /* foreign lambda component for walls */
-        dvdl = do_walls(ir,fr,box,md,x,f,lambda[efptVDW],
-                        enerd->grpp.ener[egLJSR],nrnb);
-        PRINT_SEPDVDL("Walls",0.0,dvdl);
+        dvdl = do_walls(ir, fr, box, md, x, f, lambda[efptVDW],
+                        enerd->grpp.ener[egLJSR], nrnb);
+        PRINT_SEPDVDL("Walls", 0.0, dvdl);
         enerd->dvdl_lin[efptVDW] += dvdl;
     }
 
-       /* If doing GB, reset dvda and calculate the Born radii */
-       if (ir->implicit_solvent)
-       {
+    /* If doing GB, reset dvda and calculate the Born radii */
+    if (ir->implicit_solvent)
+    {
         wallcycle_sub_start(wcycle, ewcsNONBONDED);
 
-               for(i=0;i<born->nr;i++)
-               {
-                       fr->dvda[i]=0;
-               }
+        for (i = 0; i < born->nr; i++)
+        {
+            fr->dvda[i] = 0;
+        }
 
-               if(bBornRadii)
-               {
-                       calc_gb_rad(cr,fr,ir,top,atype,x,&(fr->gblist),born,md,nrnb);
-               }
+        if (bBornRadii)
+        {
+            calc_gb_rad(cr, fr, ir, top, atype, x, &(fr->gblist), born, md, nrnb);
+        }
 
         wallcycle_sub_stop(wcycle, ewcsNONBONDED);
-       }
+    }
 
     where();
     /* We only do non-bonded calculation with group scheme here, the verlet
@@ -271,27 +277,27 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
         }
 
         wallcycle_sub_start(wcycle, ewcsNONBONDED);
-        do_nonbonded(cr,fr,x,f,f_longrange,md,excl,
-                    &enerd->grpp,box_size,nrnb,
-                    lambda,dvdl_nb,-1,-1,donb_flags);
+        do_nonbonded(cr, fr, x, f, f_longrange, md, excl,
+                     &enerd->grpp, box_size, nrnb,
+                     lambda, dvdl_nb, -1, -1, donb_flags);
 
         /* If we do foreign lambda and we have soft-core interactions
          * we have to recalculate the (non-linear) energies contributions.
          */
         if (fepvals->n_lambda > 0 && (flags & GMX_FORCE_DHDL) && fepvals->sc_alpha != 0)
         {
-            for(i=0; i<enerd->n_lambda; i++)
+            for (i = 0; i < enerd->n_lambda; i++)
             {
-                for (j=0;j<efptNR;j++)
+                for (j = 0; j < efptNR; j++)
                 {
-                    lam_i[j] = (i==0 ? lambda[j] : fepvals->all_lambda[j][i-1]);
+                    lam_i[j] = (i == 0 ? lambda[j] : fepvals->all_lambda[j][i-1]);
                 }
                 reset_foreign_enerdata(enerd);
-                do_nonbonded(cr,fr,x,f,f_longrange,md,excl,
-                             &(enerd->foreign_grpp),box_size,nrnb,
-                             lam_i,dvdl_dum,-1,-1,
+                do_nonbonded(cr, fr, x, f, f_longrange, md, excl,
+                             &(enerd->foreign_grpp), box_size, nrnb,
+                             lam_i, dvdl_dum, -1, -1,
                              (donb_flags & ~GMX_NONBONDED_DO_FORCE) | GMX_NONBONDED_DO_FOREIGNLAMBDA);
-                sum_epot(&ir->opts,&(enerd->foreign_grpp),enerd->foreign_term);
+                sum_epot(&ir->opts, &(enerd->foreign_grpp), enerd->foreign_term);
                 enerd->enerpart_lambda[i] += enerd->foreign_term[F_EPOT];
             }
         }
@@ -299,26 +305,26 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
         where();
     }
 
-       /* If we are doing GB, calculate bonded forces and apply corrections
-        * to the solvation forces */
+    /* If we are doing GB, calculate bonded forces and apply corrections
+     * to the solvation forces */
     /* MRS: Eventually, many need to include free energy contribution here! */
-       if (ir->implicit_solvent)
+    if (ir->implicit_solvent)
     {
         wallcycle_sub_start(wcycle, ewcsBONDED);
-               calc_gb_forces(cr,md,born,top,atype,x,f,fr,idef,
-                       ir->gb_algorithm,ir->sa_algorithm,nrnb,bBornRadii,&pbc,graph,enerd);
+        calc_gb_forces(cr, md, born, top, atype, x, f, fr, idef,
+                       ir->gb_algorithm, ir->sa_algorithm, nrnb, bBornRadii, &pbc, graph, enerd);
         wallcycle_sub_stop(wcycle, ewcsBONDED);
     }
 
 #ifdef GMX_MPI
     if (TAKETIME)
     {
-        t1=MPI_Wtime();
+        t1          = MPI_Wtime();
         fr->t_fnbf += t1-t0;
     }
 #endif
 
-    if (fepvals->sc_alpha!=0)
+    if (fepvals->sc_alpha != 0)
     {
         enerd->dvdl_nonlin[efptVDW] += dvdl_nb[efptVDW];
     }
@@ -327,10 +333,10 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
         enerd->dvdl_lin[efptVDW] += dvdl_nb[efptVDW];
     }
 
-    if (fepvals->sc_alpha!=0)
+    if (fepvals->sc_alpha != 0)
 
-        /* even though coulomb part is linear, we already added it, beacuse we
-           need to go through the vdw calculation anyway */
+    /* even though coulomb part is linear, we already added it, beacuse we
+       need to go through the vdw calculation anyway */
     {
         enerd->dvdl_nonlin[efptCOUL] += dvdl_nb[efptCOUL];
     }
@@ -342,7 +348,7 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
     Vsr = 0;
     if (bSepDVDL)
     {
-        for(i=0; i<enerd->grpp.nener; i++)
+        for (i = 0; i < enerd->grpp.nener; i++)
         {
             Vsr +=
                 (fr->bBHAM ?
@@ -351,14 +357,14 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
                 + enerd->grpp.ener[egCOULSR][i] + enerd->grpp.ener[egGB][i];
         }
         dvdlsum = dvdl_nb[efptVDW] + dvdl_nb[efptCOUL];
-        PRINT_SEPDVDL("VdW and Coulomb SR particle-p.",Vsr,dvdlsum);
+        PRINT_SEPDVDL("VdW and Coulomb SR particle-p.", Vsr, dvdlsum);
     }
     debug_gmx();
 
 
     if (debug)
     {
-        pr_rvecs(debug,0,"fshift after SR",fr->fshift,SHIFTS);
+        pr_rvecs(debug, 0, "fshift after SR", fr->fshift, SHIFTS);
     }
 
     /* Shift the coordinates. Must be done before bonded forces and PPPM,
@@ -371,14 +377,14 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
      */
     if (graph)
     {
-        shift_self(graph,box,x);
+        shift_self(graph, box, x);
         if (TRICLINIC(box))
         {
-            inc_nrnb(nrnb,eNR_SHIFTX,2*graph->nnodes);
+            inc_nrnb(nrnb, eNR_SHIFTX, 2*graph->nnodes);
         }
         else
         {
-            inc_nrnb(nrnb,eNR_SHIFTX,graph->nnodes);
+            inc_nrnb(nrnb, eNR_SHIFTX, graph->nnodes);
         }
     }
     /* Check whether we need to do bondeds or correct for exclusions */
@@ -389,18 +395,18 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
         /* Since all atoms are in the rectangular or triclinic unit-cell,
          * only single box vector shifts (2 in x) are required.
          */
-        set_pbc_dd(&pbc,fr->ePBC,cr->dd,TRUE,box);
+        set_pbc_dd(&pbc, fr->ePBC, cr->dd, TRUE, box);
     }
     debug_gmx();
 
     if (flags & GMX_FORCE_BONDED)
     {
         wallcycle_sub_start(wcycle, ewcsBONDED);
-        calc_bonds(fplog,cr->ms,
-                   idef,x,hist,f,fr,&pbc,graph,enerd,nrnb,lambda,md,fcd,
+        calc_bonds(fplog, cr->ms,
+                   idef, x, hist, f, fr, &pbc, graph, enerd, nrnb, lambda, md, fcd,
                    DOMAINDECOMP(cr) ? cr->dd->gatindex : NULL, atype, born,
                    flags,
-                   fr->bSepDVDL && do_per_step(step,ir->nstlog),step);
+                   fr->bSepDVDL && do_per_step(step, ir->nstlog), step);
 
         /* Check if we have to determine energy differences
          * at foreign lambda's.
@@ -412,16 +418,16 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
             {
                 gmx_incons("The bonded interactions are not sorted for free energy");
             }
-            for(i=0; i<enerd->n_lambda; i++)
+            for (i = 0; i < enerd->n_lambda; i++)
             {
                 reset_foreign_enerdata(enerd);
-                for (j=0;j<efptNR;j++)
+                for (j = 0; j < efptNR; j++)
                 {
-                    lam_i[j] = (i==0 ? lambda[j] : fepvals->all_lambda[j][i-1]);
+                    lam_i[j] = (i == 0 ? lambda[j] : fepvals->all_lambda[j][i-1]);
                 }
-                calc_bonds_lambda(fplog,idef,x,fr,&pbc,graph,&(enerd->foreign_grpp),enerd->foreign_term,nrnb,lam_i,md,
-                                  fcd,DOMAINDECOMP(cr) ? cr->dd->gatindex : NULL);
-                sum_epot(&ir->opts,&(enerd->foreign_grpp),enerd->foreign_term);
+                calc_bonds_lambda(fplog, idef, x, fr, &pbc, graph, &(enerd->foreign_grpp), enerd->foreign_term, nrnb, lam_i, md,
+                                  fcd, DOMAINDECOMP(cr) ? cr->dd->gatindex : NULL);
+                sum_epot(&ir->opts, &(enerd->foreign_grpp), enerd->foreign_term);
                 enerd->enerpart_lambda[i] += enerd->foreign_term[F_EPOT];
             }
         }
@@ -438,8 +444,8 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
         bSB = (ir->nwall == 2);
         if (bSB)
         {
-            copy_mat(box,boxs);
-            svmul(ir->wall_ewald_zfac,boxs[ZZ],boxs[ZZ]);
+            copy_mat(box, boxs);
+            svmul(ir->wall_ewald_zfac, boxs[ZZ], boxs[ZZ]);
             box_size[ZZ] *= ir->wall_ewald_zfac;
         }
 
@@ -461,23 +467,23 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
                 ir->ewald_geometry != eewg3D ||
                 ir->epsilon_surface != 0)
             {
-                int nthreads,t;
+                int nthreads, t;
 
                 wallcycle_sub_start(wcycle, ewcsEWALD_CORRECTION);
 
                 if (fr->n_tpi > 0)
                 {
-                    gmx_fatal(FARGS,"TPI with PME currently only works in a 3D geometry with tin-foil boundary conditions");
+                    gmx_fatal(FARGS, "TPI with PME currently only works in a 3D geometry with tin-foil boundary conditions");
                 }
 
                 nthreads = gmx_omp_nthreads_get(emntBonded);
 #pragma omp parallel for num_threads(nthreads) schedule(static)
-                for(t=0; t<nthreads; t++)
+                for (t = 0; t < nthreads; t++)
                 {
-                    int s,e,i;
-                    rvec *fnv;
+                    int     s, e, i;
+                    rvec   *fnv;
                     tensor *vir;
-                    real *Vcorrt,*dvdlt;
+                    real   *Vcorrt, *dvdlt;
                     if (t == 0)
                     {
                         fnv    = fr->f_novirsum;
@@ -491,32 +497,32 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
                         vir    = &fr->f_t[t].vir;
                         Vcorrt = &fr->f_t[t].Vcorr;
                         dvdlt  = &fr->f_t[t].dvdl[efptCOUL];
-                        for(i=0; i<fr->natoms_force; i++)
+                        for (i = 0; i < fr->natoms_force; i++)
                         {
                             clear_rvec(fnv[i]);
                         }
                         clear_mat(*vir);
                     }
-                    *dvdlt = 0;
+                    *dvdlt  = 0;
                     *Vcorrt =
                         ewald_LRcorrection(fplog,
-                                           fr->excl_load[t],fr->excl_load[t+1],
-                                           cr,t,fr,
+                                           fr->excl_load[t], fr->excl_load[t+1],
+                                           cr, t, fr,
                                            md->chargeA,
                                            md->nChargePerturbed ? md->chargeB : NULL,
                                            ir->cutoff_scheme != ecutsVERLET,
-                                           excl,x,bSB ? boxs : box,mu_tot,
+                                           excl, x, bSB ? boxs : box, mu_tot,
                                            ir->ewald_geometry,
                                            ir->epsilon_surface,
-                                           fnv,*vir,
-                                           lambda[efptCOUL],dvdlt);
+                                           fnv, *vir,
+                                           lambda[efptCOUL], dvdlt);
                 }
                 if (nthreads > 1)
                 {
-                    reduce_thread_forces(fr->natoms_force,fr->f_novirsum,
+                    reduce_thread_forces(fr->natoms_force, fr->f_novirsum,
                                          fr->vir_el_recip,
-                                         &Vcorr,efptCOUL,&dvdl,
-                                         nthreads,fr->f_t);
+                                         &Vcorr, efptCOUL, &dvdl,
+                                         nthreads, fr->f_t);
                 }
 
                 wallcycle_sub_stop(wcycle, ewcsEWALD_CORRECTION);
@@ -524,104 +530,104 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
 
             if (fr->n_tpi == 0)
             {
-                Vcorr += ewald_charge_correction(cr,fr,lambda[efptCOUL],box,
-                                                 &dvdl,fr->vir_el_recip);
+                Vcorr += ewald_charge_correction(cr, fr, lambda[efptCOUL], box,
+                                                 &dvdl, fr->vir_el_recip);
             }
 
-            PRINT_SEPDVDL("Ewald excl./charge/dip. corr.",Vcorr,dvdl);
+            PRINT_SEPDVDL("Ewald excl./charge/dip. corr.", Vcorr, dvdl);
             enerd->dvdl_lin[efptCOUL] += dvdl;
         }
 
         status = 0;
-        Vlr  = 0;
-        dvdl = 0;
+        Vlr    = 0;
+        dvdl   = 0;
         switch (fr->eeltype)
         {
-        case eelPME:
-        case eelPMESWITCH:
-        case eelPMEUSER:
-        case eelPMEUSERSWITCH:
-        case eelP3M_AD:
-            if (cr->duty & DUTY_PME)
-            {
-                assert(fr->n_tpi >= 0);
-                if (fr->n_tpi == 0 || (flags & GMX_FORCE_STATECHANGED))
+            case eelPME:
+            case eelPMESWITCH:
+            case eelPMEUSER:
+            case eelPMEUSERSWITCH:
+            case eelP3M_AD:
+                if (cr->duty & DUTY_PME)
                 {
-                    pme_flags = GMX_PME_SPREAD_Q | GMX_PME_SOLVE;
-                    if (flags & GMX_FORCE_FORCES)
-                    {
-                        pme_flags |= GMX_PME_CALC_F;
-                    }
-                    if (flags & (GMX_FORCE_VIRIAL | GMX_FORCE_ENERGY))
+                    assert(fr->n_tpi >= 0);
+                    if (fr->n_tpi == 0 || (flags & GMX_FORCE_STATECHANGED))
                     {
-                        pme_flags |= GMX_PME_CALC_ENER_VIR;
+                        pme_flags = GMX_PME_SPREAD_Q | GMX_PME_SOLVE;
+                        if (flags & GMX_FORCE_FORCES)
+                        {
+                            pme_flags |= GMX_PME_CALC_F;
+                        }
+                        if (flags & (GMX_FORCE_VIRIAL | GMX_FORCE_ENERGY))
+                        {
+                            pme_flags |= GMX_PME_CALC_ENER_VIR;
+                        }
+                        if (fr->n_tpi > 0)
+                        {
+                            /* We don't calculate f, but we do want the potential */
+                            pme_flags |= GMX_PME_CALC_POT;
+                        }
+                        wallcycle_start(wcycle, ewcPMEMESH);
+                        status = gmx_pme_do(fr->pmedata,
+                                            md->start, md->homenr - fr->n_tpi,
+                                            x, fr->f_novirsum,
+                                            md->chargeA, md->chargeB,
+                                            bSB ? boxs : box, cr,
+                                            DOMAINDECOMP(cr) ? dd_pme_maxshift_x(cr->dd) : 0,
+                                            DOMAINDECOMP(cr) ? dd_pme_maxshift_y(cr->dd) : 0,
+                                            nrnb, wcycle,
+                                            fr->vir_el_recip, fr->ewaldcoeff,
+                                            &Vlr, lambda[efptCOUL], &dvdl,
+                                            pme_flags);
+                        *cycles_pme = wallcycle_stop(wcycle, ewcPMEMESH);
+
+                        /* We should try to do as little computation after
+                         * this as possible, because parallel PME synchronizes
+                         * the nodes, so we want all load imbalance of the rest
+                         * of the force calculation to be before the PME call.
+                         * DD load balancing is done on the whole time of
+                         * the force call (without PME).
+                         */
                     }
                     if (fr->n_tpi > 0)
                     {
-                        /* We don't calculate f, but we do want the potential */
-                        pme_flags |= GMX_PME_CALC_POT;
+                        /* Determine the PME grid energy of the test molecule
+                         * with the PME grid potential of the other charges.
+                         */
+                        gmx_pme_calc_energy(fr->pmedata, fr->n_tpi,
+                                            x + md->homenr - fr->n_tpi,
+                                            md->chargeA + md->homenr - fr->n_tpi,
+                                            &Vlr);
                     }
-                    wallcycle_start(wcycle,ewcPMEMESH);
-                    status = gmx_pme_do(fr->pmedata,
-                                        md->start,md->homenr - fr->n_tpi,
-                                        x,fr->f_novirsum,
-                                        md->chargeA,md->chargeB,
-                                        bSB ? boxs : box,cr,
-                                        DOMAINDECOMP(cr) ? dd_pme_maxshift_x(cr->dd) : 0,
-                                        DOMAINDECOMP(cr) ? dd_pme_maxshift_y(cr->dd) : 0,
-                                        nrnb,wcycle,
-                                        fr->vir_el_recip,fr->ewaldcoeff,
-                                        &Vlr,lambda[efptCOUL],&dvdl,
-                                        pme_flags);
-                    *cycles_pme = wallcycle_stop(wcycle,ewcPMEMESH);
-
-                    /* We should try to do as little computation after
-                     * this as possible, because parallel PME synchronizes
-                     * the nodes, so we want all load imbalance of the rest
-                     * of the force calculation to be before the PME call.
-                     * DD load balancing is done on the whole time of
-                     * the force call (without PME).
-                     */
-                }
-                if (fr->n_tpi > 0)
-                {
-                    /* Determine the PME grid energy of the test molecule
-                     * with the PME grid potential of the other charges.
-                     */
-                    gmx_pme_calc_energy(fr->pmedata,fr->n_tpi,
-                                        x + md->homenr - fr->n_tpi,
-                                        md->chargeA + md->homenr - fr->n_tpi,
-                                        &Vlr);
+                    PRINT_SEPDVDL("PME mesh", Vlr, dvdl);
                 }
-                PRINT_SEPDVDL("PME mesh",Vlr,dvdl);
-            }
-            break;
-        case eelEWALD:
-            Vlr = do_ewald(fplog,FALSE,ir,x,fr->f_novirsum,
-                           md->chargeA,md->chargeB,
-                           box_size,cr,md->homenr,
-                           fr->vir_el_recip,fr->ewaldcoeff,
-                           lambda[efptCOUL],&dvdl,fr->ewald_table);
-            PRINT_SEPDVDL("Ewald long-range",Vlr,dvdl);
-            break;
-        default:
-            gmx_fatal(FARGS,"No such electrostatics method implemented %s",
-                      eel_names[fr->eeltype]);
+                break;
+            case eelEWALD:
+                Vlr = do_ewald(fplog, FALSE, ir, x, fr->f_novirsum,
+                               md->chargeA, md->chargeB,
+                               box_size, cr, md->homenr,
+                               fr->vir_el_recip, fr->ewaldcoeff,
+                               lambda[efptCOUL], &dvdl, fr->ewald_table);
+                PRINT_SEPDVDL("Ewald long-range", Vlr, dvdl);
+                break;
+            default:
+                gmx_fatal(FARGS, "No such electrostatics method implemented %s",
+                          eel_names[fr->eeltype]);
         }
         if (status != 0)
         {
-            gmx_fatal(FARGS,"Error %d in long range electrostatics routine %s",
-                      status,EELTYPE(fr->eeltype));
-               }
+            gmx_fatal(FARGS, "Error %d in long range electrostatics routine %s",
+                      status, EELTYPE(fr->eeltype));
+        }
         /* Note that with separate PME nodes we get the real energies later */
         enerd->dvdl_lin[efptCOUL] += dvdl;
-        enerd->term[F_COUL_RECIP] = Vlr + Vcorr;
+        enerd->term[F_COUL_RECIP]  = Vlr + Vcorr;
         if (debug)
         {
-            fprintf(debug,"Vlr = %g, Vcorr = %g, Vlr_corr = %g\n",
-                    Vlr,Vcorr,enerd->term[F_COUL_RECIP]);
-            pr_rvecs(debug,0,"vir_el_recip after corr",fr->vir_el_recip,DIM);
-            pr_rvecs(debug,0,"fshift after LR Corrections",fr->fshift,SHIFTS);
+            fprintf(debug, "Vlr = %g, Vcorr = %g, Vlr_corr = %g\n",
+                    Vlr, Vcorr, enerd->term[F_COUL_RECIP]);
+            pr_rvecs(debug, 0, "vir_el_recip after corr", fr->vir_el_recip, DIM);
+            pr_rvecs(debug, 0, "fshift after LR Corrections", fr->fshift, SHIFTS);
         }
     }
     else
@@ -633,15 +639,15 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
              */
             if (ir->cutoff_scheme != ecutsVERLET && fr->eeltype != eelRF_NEC)
             {
-                dvdl = 0;
+                dvdl                   = 0;
                 enerd->term[F_RF_EXCL] =
-                    RF_excl_correction(fplog,fr,graph,md,excl,x,f,
-                                       fr->fshift,&pbc,lambda[efptCOUL],&dvdl);
+                    RF_excl_correction(fplog, fr, graph, md, excl, x, f,
+                                       fr->fshift, &pbc, lambda[efptCOUL], &dvdl);
             }
 
             enerd->dvdl_lin[efptCOUL] += dvdl;
             PRINT_SEPDVDL("RF exclusion correction",
-                          enerd->term[F_RF_EXCL],dvdl);
+                          enerd->term[F_RF_EXCL], dvdl);
         }
     }
     where();
@@ -649,21 +655,21 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
 
     if (debug)
     {
-        print_nrnb(debug,nrnb);
+        print_nrnb(debug, nrnb);
     }
     debug_gmx();
 
 #ifdef GMX_MPI
     if (TAKETIME)
     {
-        t2=MPI_Wtime();
+        t2 = MPI_Wtime();
         MPI_Barrier(cr->mpi_comm_mygroup);
-        t3=MPI_Wtime();
+        t3          = MPI_Wtime();
         fr->t_wait += t3-t2;
         if (fr->timesteps == 11)
         {
-            fprintf(stderr,"* PP load balancing info: node %d, step %s, rel wait time=%3.0f%% , load string value: %7.2f\n",
-                    cr->nodeid, gmx_step_str(fr->timesteps,buf),
+            fprintf(stderr, "* PP load balancing info: node %d, step %s, rel wait time=%3.0f%% , load string value: %7.2f\n",
+                    cr->nodeid, gmx_step_str(fr->timesteps, buf),
                     100*fr->t_wait/(fr->t_wait+fr->t_fnbf),
                     (fr->t_fnbf+fr->t_wait)/fr->t_fnbf);
         }
@@ -673,44 +679,45 @@ void do_force_lowlevel(FILE       *fplog,   gmx_large_int_t step,
 
     if (debug)
     {
-        pr_rvecs(debug,0,"fshift after bondeds",fr->fshift,SHIFTS);
+        pr_rvecs(debug, 0, "fshift after bondeds", fr->fshift, SHIFTS);
     }
 
 }
 
-void init_enerdata(int ngener,int n_lambda,gmx_enerdata_t *enerd)
+void init_enerdata(int ngener, int n_lambda, gmx_enerdata_t *enerd)
 {
-    int i,n2;
+    int i, n2;
 
-    for(i=0; i<F_NRE; i++)
+    for (i = 0; i < F_NRE; i++)
     {
-        enerd->term[i] = 0;
+        enerd->term[i]         = 0;
         enerd->foreign_term[i] = 0;
     }
 
 
-    for(i=0; i<efptNR; i++) {
-        enerd->dvdl_lin[i]  = 0;
+    for (i = 0; i < efptNR; i++)
+    {
+        enerd->dvdl_lin[i]     = 0;
         enerd->dvdl_nonlin[i]  = 0;
     }
 
-    n2=ngener*ngener;
+    n2 = ngener*ngener;
     if (debug)
     {
-        fprintf(debug,"Creating %d sized group matrix for energies\n",n2);
+        fprintf(debug, "Creating %d sized group matrix for energies\n", n2);
     }
-    enerd->grpp.nener = n2;
+    enerd->grpp.nener         = n2;
     enerd->foreign_grpp.nener = n2;
-    for(i=0; (i<egNR); i++)
+    for (i = 0; (i < egNR); i++)
     {
-        snew(enerd->grpp.ener[i],n2);
-        snew(enerd->foreign_grpp.ener[i],n2);
+        snew(enerd->grpp.ener[i], n2);
+        snew(enerd->foreign_grpp.ener[i], n2);
     }
 
     if (n_lambda)
     {
         enerd->n_lambda = 1 + n_lambda;
-        snew(enerd->enerpart_lambda,enerd->n_lambda);
+        snew(enerd->enerpart_lambda, enerd->n_lambda);
     }
     else
     {
@@ -722,12 +729,12 @@ void destroy_enerdata(gmx_enerdata_t *enerd)
 {
     int i;
 
-    for(i=0; (i<egNR); i++)
+    for (i = 0; (i < egNR); i++)
     {
         sfree(enerd->grpp.ener[i]);
     }
 
-    for(i=0; (i<egNR); i++)
+    for (i = 0; (i < egNR); i++)
     {
         sfree(enerd->foreign_grpp.ener[i]);
     }
@@ -738,85 +745,88 @@ void destroy_enerdata(gmx_enerdata_t *enerd)
     }
 }
 
-static real sum_v(int n,real v[])
+static real sum_v(int n, real v[])
 {
-  real t;
-  int  i;
+    real t;
+    int  i;
 
-  t = 0.0;
-  for(i=0; (i<n); i++)
-    t = t + v[i];
+    t = 0.0;
+    for (i = 0; (i < n); i++)
+    {
+        t = t + v[i];
+    }
 
-  return t;
+    return t;
 }
 
 void sum_epot(t_grpopts *opts, gmx_grppairener_t *grpp, real *epot)
 {
-  int i;
-
-  /* Accumulate energies */
-  epot[F_COUL_SR]  = sum_v(grpp->nener,grpp->ener[egCOULSR]);
-  epot[F_LJ]       = sum_v(grpp->nener,grpp->ener[egLJSR]);
-  epot[F_LJ14]     = sum_v(grpp->nener,grpp->ener[egLJ14]);
-  epot[F_COUL14]   = sum_v(grpp->nener,grpp->ener[egCOUL14]);
-  epot[F_COUL_LR]  = sum_v(grpp->nener,grpp->ener[egCOULLR]);
-  epot[F_LJ_LR]    = sum_v(grpp->nener,grpp->ener[egLJLR]);
-  /* We have already added 1-2,1-3, and 1-4 terms to F_GBPOL */
-  epot[F_GBPOL]   += sum_v(grpp->nener,grpp->ener[egGB]);
+    int i;
+
+    /* Accumulate energies */
+    epot[F_COUL_SR]  = sum_v(grpp->nener, grpp->ener[egCOULSR]);
+    epot[F_LJ]       = sum_v(grpp->nener, grpp->ener[egLJSR]);
+    epot[F_LJ14]     = sum_v(grpp->nener, grpp->ener[egLJ14]);
+    epot[F_COUL14]   = sum_v(grpp->nener, grpp->ener[egCOUL14]);
+    epot[F_COUL_LR]  = sum_v(grpp->nener, grpp->ener[egCOULLR]);
+    epot[F_LJ_LR]    = sum_v(grpp->nener, grpp->ener[egLJLR]);
+    /* We have already added 1-2,1-3, and 1-4 terms to F_GBPOL */
+    epot[F_GBPOL]   += sum_v(grpp->nener, grpp->ener[egGB]);
 
 /* lattice part of LR doesnt belong to any group
  * and has been added earlier
  */
-  epot[F_BHAM]     = sum_v(grpp->nener,grpp->ener[egBHAMSR]);
-  epot[F_BHAM_LR]  = sum_v(grpp->nener,grpp->ener[egBHAMLR]);
-
-  epot[F_EPOT] = 0;
-  for(i=0; (i<F_EPOT); i++)
-  {
-      if (i != F_DISRESVIOL && i != F_ORIRESDEV)
-      {
-          epot[F_EPOT] += epot[i];
-      }
-  }
+    epot[F_BHAM]     = sum_v(grpp->nener, grpp->ener[egBHAMSR]);
+    epot[F_BHAM_LR]  = sum_v(grpp->nener, grpp->ener[egBHAMLR]);
+
+    epot[F_EPOT] = 0;
+    for (i = 0; (i < F_EPOT); i++)
+    {
+        if (i != F_DISRESVIOL && i != F_ORIRESDEV)
+        {
+            epot[F_EPOT] += epot[i];
+        }
+    }
 }
 
 void sum_dhdl(gmx_enerdata_t *enerd, real *lambda, t_lambda *fepvals)
 {
-    int i,j,index;
+    int    i, j, index;
     double dlam;
 
     enerd->dvdl_lin[efptVDW] += enerd->term[F_DVDL_VDW];  /* include dispersion correction */
-    enerd->term[F_DVDL] = 0.0;
-    for (i=0;i<efptNR;i++)
+    enerd->term[F_DVDL]       = 0.0;
+    for (i = 0; i < efptNR; i++)
     {
         if (fepvals->separate_dvdl[i])
         {
             /* could this be done more readably/compactly? */
-            switch (i) {
-            case (efptCOUL):
-                index = F_DVDL_COUL;
-                break;
-            case (efptVDW):
-                index = F_DVDL_VDW;
-                break;
-            case (efptBONDED):
-                index = F_DVDL_BONDED;
-                break;
-            case (efptRESTRAINT):
-                index = F_DVDL_RESTRAINT;
-                break;
-            case (efptMASS):
-                index = F_DKDL;
-                break;
-            default:
-                index = F_DVDL;
-                break;
+            switch (i)
+            {
+                case (efptCOUL):
+                    index = F_DVDL_COUL;
+                    break;
+                case (efptVDW):
+                    index = F_DVDL_VDW;
+                    break;
+                case (efptBONDED):
+                    index = F_DVDL_BONDED;
+                    break;
+                case (efptRESTRAINT):
+                    index = F_DVDL_RESTRAINT;
+                    break;
+                case (efptMASS):
+                    index = F_DKDL;
+                    break;
+                default:
+                    index = F_DVDL;
+                    break;
             }
             enerd->term[index] = enerd->dvdl_lin[i] + enerd->dvdl_nonlin[i];
             if (debug)
             {
-                fprintf(debug,"dvdl-%s[%2d]: %f: non-linear %f + linear %f\n",
-                        efpt_names[i],i,enerd->term[index],enerd->dvdl_nonlin[i],enerd->dvdl_lin[i]);
+                fprintf(debug, "dvdl-%s[%2d]: %f: non-linear %f + linear %f\n",
+                        efpt_names[i], i, enerd->term[index], enerd->dvdl_nonlin[i], enerd->dvdl_lin[i]);
             }
         }
         else
@@ -824,8 +834,8 @@ void sum_dhdl(gmx_enerdata_t *enerd, real *lambda, t_lambda *fepvals)
             enerd->term[F_DVDL] += enerd->dvdl_lin[i] + enerd->dvdl_nonlin[i];
             if (debug)
             {
-                fprintf(debug,"dvd-%sl[%2d]: %f: non-linear %f + linear %f\n",
-                        efpt_names[0],i,enerd->term[F_DVDL],enerd->dvdl_nonlin[i],enerd->dvdl_lin[i]);
+                fprintf(debug, "dvd-%sl[%2d]: %f: non-linear %f + linear %f\n",
+                        efpt_names[0], i, enerd->term[F_DVDL], enerd->dvdl_nonlin[i], enerd->dvdl_lin[i]);
             }
         }
     }
@@ -842,7 +852,7 @@ void sum_dhdl(gmx_enerdata_t *enerd, real *lambda, t_lambda *fepvals)
      * (investigate how to overcome this post 4.6 - MRS)
      */
 
-    for(i=0; i<fepvals->n_lambda; i++)
+    for (i = 0; i < fepvals->n_lambda; i++)
     {                                         /* note we are iterating over fepvals here!
                                                  For the current lam, dlam = 0 automatically,
                                                  so we don't need to add anything to the
@@ -856,18 +866,21 @@ void sum_dhdl(gmx_enerdata_t *enerd, real *lambda, t_lambda *fepvals)
 
         enerd->enerpart_lambda[i+1] += enerd->term[F_DKDL]*dlam;
 
-        for (j=0;j<efptNR;j++)
+        for (j = 0; j < efptNR; j++)
         {
-            if (j==efptMASS) {continue;} /* no other mass term to worry about */
+            if (j == efptMASS)
+            {
+                continue;
+            }                            /* no other mass term to worry about */
 
             dlam = (fepvals->all_lambda[j][i]-lambda[j]);
             enerd->enerpart_lambda[i+1] += dlam*enerd->dvdl_lin[j];
             if (debug)
             {
-                fprintf(debug,"enerdiff lam %g: (%15s), non-linear %f linear %f*%f\n",
-                        fepvals->all_lambda[j][i],efpt_names[j],
+                fprintf(debug, "enerdiff lam %g: (%15s), non-linear %f linear %f*%f\n",
+                        fepvals->all_lambda[j][i], efpt_names[j],
                         (enerd->enerpart_lambda[i+1] - enerd->enerpart_lambda[0]),
-                        dlam,enerd->dvdl_lin[j]);
+                        dlam, enerd->dvdl_lin[j]);
             }
         }
     }
@@ -876,52 +889,57 @@ void sum_dhdl(gmx_enerdata_t *enerd, real *lambda, t_lambda *fepvals)
 
 void reset_foreign_enerdata(gmx_enerdata_t *enerd)
 {
-    int  i,j;
+    int  i, j;
 
     /* First reset all foreign energy components.  Foreign energies always called on
        neighbor search steps */
-    for(i=0; (i<egNR); i++)
+    for (i = 0; (i < egNR); i++)
     {
-        for(j=0; (j<enerd->grpp.nener); j++)
+        for (j = 0; (j < enerd->grpp.nener); j++)
         {
             enerd->foreign_grpp.ener[i][j] = 0.0;
         }
     }
 
     /* potential energy components */
-    for(i=0; (i<=F_EPOT); i++)
+    for (i = 0; (i <= F_EPOT); i++)
     {
         enerd->foreign_term[i] = 0.0;
     }
 }
 
 void reset_enerdata(t_grpopts *opts,
-                    t_forcerec *fr,gmx_bool bNS,
+                    t_forcerec *fr, gmx_bool bNS,
                     gmx_enerdata_t *enerd,
                     gmx_bool bMaster)
 {
     gmx_bool bKeepLR;
-    int  i,j;
+    int      i, j;
 
     /* First reset all energy components, except for the long range terms
      * on the master at non neighbor search steps, since the long range
      * terms have already been summed at the last neighbor search step.
      */
     bKeepLR = (fr->bTwinRange && !bNS);
-    for(i=0; (i<egNR); i++) {
-        if (!(bKeepLR && bMaster && (i == egCOULLR || i == egLJLR))) {
-            for(j=0; (j<enerd->grpp.nener); j++)
+    for (i = 0; (i < egNR); i++)
+    {
+        if (!(bKeepLR && bMaster && (i == egCOULLR || i == egLJLR)))
+        {
+            for (j = 0; (j < enerd->grpp.nener); j++)
+            {
                 enerd->grpp.ener[i][j] = 0.0;
+            }
         }
     }
-    for (i=0;i<efptNR;i++)
+    for (i = 0; i < efptNR; i++)
     {
         enerd->dvdl_lin[i]    = 0.0;
         enerd->dvdl_nonlin[i] = 0.0;
     }
 
     /* Normal potential energy components */
-    for(i=0; (i<=F_EPOT); i++) {
+    for (i = 0; (i <= F_EPOT); i++)
+    {
         enerd->term[i] = 0.0;
     }
     /* Initialize the dVdlambda term with the long range contribution */
@@ -934,7 +952,7 @@ void reset_enerdata(t_grpopts *opts,
     enerd->term[F_DKDL]            = 0.0;
     if (enerd->n_lambda > 0)
     {
-        for(i=0; i<enerd->n_lambda; i++)
+        for (i = 0; i < enerd->n_lambda; i++)
         {
             enerd->enerpart_lambda[i] = 0.0;
         }