Moving the checkArgumentType error to its caller

plumed · Dec 18, 2024 · 9c5cadd · 9c5cadd
1 parent 3cc05d9
commit 9c5cadd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/core/ActionWithArguments.cpp b/src/core/ActionWithArguments.cpp
@@ -192,7 +192,9 @@ void ActionWithArguments::interpretArgumentList(const std::vector<std::string>&
     }
   }
   for(unsigned i=0; i<arg.size(); ++i) {
-    if( !readact->keywords.checkArgumentType( arg[i]->getRank(), arg[i]->hasDerivatives() ) ) readact->warning("documentation for input type is not provided in " + readact->getName() );
+    if( !readact->keywords.checkArgumentType( arg[i]->getRank(), arg[i]->hasDerivatives() ) ) {
+      readact->error("documentation for input type is not provided in " + readact->getName() );
+    }
   }
 }
 

diff --git a/src/tools/Keywords.cpp b/src/tools/Keywords.cpp
@@ -697,7 +697,6 @@ bool Keywords::checkArgumentType( const std::size_t& rank, const bool& hasderiv
     if( rank==1 && x.second.find("vector")!=std::string::npos ) return true;
     if( rank==2 && x.second.find("matrix")!=std::string::npos ) return true;
   }
-  plumed_merror("WARNING: type for input argument has not been specified");
   return false;
 }