Fixing the code for handling failed bool test cases

JeffersonLab · Oct 2, 2023 · 3de3221 · 3de3221
1 parent 60197c8
commit 3de3221
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/libraries/JANA/Services/JParameterManager.h b/src/libraries/JANA/Services/JParameterManager.h
@@ -325,8 +325,8 @@ inline void JParameterManager::Parse(const std::string& value, std::string& out)
 template <>
 inline void JParameterManager::Parse(const std::string& value, bool& val) {
     if (value == "0" || value =="false" || value == "off") val = false;
-    if (value == "1" || value == "true" || value == "on") val = true;
-    throw JException("'%s' not parseable as bool", value.c_str());
+    else if (value == "1" || value == "true" || value == "on") val = true;
+    else throw JException("'%s' not parseable as bool", value.c_str());
 }
 
 // @brief Template to parse a string and return in an array