Initialize force vector in PotentialPointData constructor.
authorM. Eric Irrgang <ericirrgang@gmail.com>
Fri, 12 Oct 2018 12:11:38 +0000 (15:11 +0300)
committerM. Eric Irrgang <ericirrgang@gmail.com>
Fri, 12 Oct 2018 12:11:38 +0000 (15:11 +0300)
A default-constructed PotentialPointData could have uninitialized
data, but it is sometimes helpful to be able to default construct an
object and not worry about whether it is valid. Objects will usually
be constructed with initializing values, but this change updates the
default constructor to avoid surprises.

Change-Id: I9ae87d8bc361da5b3134f05ca8761f25d6ca7646

src/gromacs/restraint/restraintpotential.cpp

index 56c5b7bd9143253f89a1c6af11380303834d2e6a..6c5cbd767ec0b2cda0e126dc4960b551524022ad 100644 (file)
@@ -41,7 +41,7 @@ namespace gmx
 {
 
 PotentialPointData::PotentialPointData() :
-    PotentialPointData {Vector(), real(0.0)}
+    PotentialPointData {Vector(0., 0., 0.), real(0.0)}
 {}
 
 PotentialPointData::PotentialPointData(const Vector &f, const real e) :