*
* Copyright (c) 1991-2000, University of Groningen, The Netherlands.
* Copyright (c) 2001-2004, The GROMACS development team.
- * Copyright (c) 2013, by the GROMACS development team, led by
+ * Copyright (c) 2013,2014, by the GROMACS development team, led by
* Mark Abraham, David van der Spoel, Berk Hess, and Erik Lindahl,
* and including many others, as listed in the AUTHORS file in the
* top-level source directory and at http://www.gromacs.org.
* To help us fund GROMACS development, we humbly ask that you cite
* the research papers on the package. Check out http://www.gromacs.org.
*/
-#ifdef HAVE_CONFIG_H
-#include <config.h>
-#endif
+#include "gmxpre.h"
+
+#include "vsite_parm.h"
-#include <stdio.h>
-#include <math.h>
#include <assert.h>
+#include <math.h>
+#include <stdio.h>
#include <string.h>
-#include "vsite_parm.h"
-#include "smalloc.h"
-#include "resall.h"
-#include "add_par.h"
-#include "vec.h"
-#include "toputil.h"
-#include "physics.h"
-#include "index.h"
-#include "names.h"
-#include "gmx_fatal.h"
-#include "string2.h"
-#include "physics.h"
-#include "macros.h"
+
+#include "gromacs/gmxpreprocess/add_par.h"
+#include "gromacs/gmxpreprocess/resall.h"
+#include "gromacs/gmxpreprocess/toputil.h"
+#include "gromacs/legacyheaders/macros.h"
+#include "gromacs/legacyheaders/names.h"
+#include "gromacs/math/units.h"
+#include "gromacs/math/vec.h"
+#include "gromacs/utility/cstringutil.h"
+#include "gromacs/utility/fatalerror.h"
+#include "gromacs/utility/smalloc.h"
typedef struct {
t_iatom a[4];
static void clean_vsite_dihs(t_params *plist, t_pindex pindex[],
int cftype, int vsite_type[])
{
- int ftype, i, parnr, k, l, m, n, nvsite, kept_i, vsnral;
- atom_id atom, constr;
- atom_id vsiteatoms[4];
- gmx_bool bKeep, bUsed, bPresent;
- t_params *ps;
+ int i, kept_i;
+ t_params *ps;
ps = &(plist[cftype]);
- vsnral = 0;
kept_i = 0;
for (i = 0; (i < ps->nr); i++) /* for all dihedrals in the plist */
{
+ int ftype, parnr, k, l, m, n, nvsite;
+ int vsnral = 0; /* keep the compiler happy */
+ atom_id atom, constr;
+ atom_id vsiteatoms[4] = { 0 }; /* init to zero to make gcc4.8 happy */
+ gmx_bool bKeep, bUsed, bPresent;
+
+
bKeep = FALSE;
/* check if all virtual sites are constructed from the same atoms */
nvsite = 0;
atom = ps->param[i].a[k];
if (vsite_type[atom] != NOTSET)
{
- nvsite++;
- if (nvsite == 1)
+ if (nvsite == 0)
{
/* store construction atoms of first vsite */
vsnral = NRAL(pindex[atom].ftype)-1;
}
}
else
- /* check if this vsite is constructed from the same atoms */
- if (vsnral == NRAL(pindex[atom].ftype)-1)
{
- for (m = 0; (m < vsnral) && !bKeep; m++)
+ /* check if this vsite is constructed from the same atoms */
+ if (vsnral == NRAL(pindex[atom].ftype)-1)
{
- bPresent = FALSE;
- constr =
- plist[pindex[atom].ftype].param[pindex[atom].parnr].a[m+1];
- for (n = 0; (n < vsnral) && !bPresent; n++)
+ for (m = 0; (m < vsnral) && !bKeep; m++)
{
- if (constr == vsiteatoms[n])
+ bPresent = FALSE;
+ constr =
+ plist[pindex[atom].ftype].param[pindex[atom].parnr].a[m+1];
+ for (n = 0; (n < vsnral) && !bPresent; n++)
{
- bPresent = TRUE;
+ if (constr == vsiteatoms[n])
+ {
+ bPresent = TRUE;
+ }
+ }
+ if (!bPresent)
+ {
+ bKeep = TRUE;
}
- }
- if (!bPresent)
- {
- bKeep = TRUE;
}
}
}
+ nvsite++;
}
}
atom = ps->param[i].a[k];
if (vsite_type[atom] == NOTSET)
{
+ /* vsnral will be set here, we don't get here with nvsite==0 */
bUsed = FALSE;
for (m = 0; (m < vsnral) && !bUsed; m++)
{