Small fixes associated to last commit

Now that cppcheck can access header files for optional modules, a few more inconsistencies were spotted out.
plumed · Nov 8, 2022 · ccc3077 · ccc3077
1 parent 80210b8
commit ccc3077
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/src/dimred/OutputPCAProjections.cpp b/src/dimred/OutputPCAProjections.cpp
@@ -94,7 +94,7 @@ void OutputPCAProjection::performAnalysis() {
   mypdb.setAtomPositions( (mypca->myref)->getReferencePositions() );
   for(unsigned j=0; j<mypca->getArguments().size(); ++j) mypdb.setArgumentValue( (mypca->getArguments()[j])->getName(), (mypca->myref)->getReferenceArgument(j) );
   // And output the first frame
-  afile.open( filename.c_str() ); afile.printf("REMARK TYPE=%s \n", mypca->mtype.c_str() );
+  afile.open( filename ); afile.printf("REMARK TYPE=%s \n", mypca->mtype.c_str() );
   if( plumed.getAtoms().usingNaturalUnits() ) mypdb.print( 1.0, mymoldat, afile, fmt );
   else mypdb.print( atoms.getUnits().getLength()/0.1, mymoldat, afile, fmt );
   // And now output the eigenvectors

diff --git a/src/sasa/sasa_HASEL.cpp b/src/sasa/sasa_HASEL.cpp
@@ -137,6 +137,7 @@ class SASA_HASEL : public Colvar {
   unsigned nl_update;
   int firstStepFlag;
   double Ti;
+  // cppcheck-suppress duplInheritedMember
   std::vector<AtomNumber> atoms;
   vector < vector < std::string > > AtomResidueName;
   vector < vector < double > > SASAparam;

diff --git a/src/sasa/sasa_LCPO.cpp b/src/sasa/sasa_LCPO.cpp
@@ -136,6 +136,7 @@ class SASA_LCPO : public Colvar {
   unsigned nl_update;
   int firstStepFlag;
   double Ti;
+  // cppcheck-suppress duplInheritedMember
   std::vector<AtomNumber> atoms;
   vector < vector < std::string > > AtomResidueName;
   vector < vector < double > > LCPOparam;