include/nutconf.hpp: BoolInt: avoid "if (b)" constructs in favor of e…

…xplicit call to operator via "if (b == true)" [networkupstools#2294] Signed-off-by: Jim Klimov <[email protected]>
jimklimov · Apr 29, 2024 · 60cc7a2 · 60cc7a2
1 parent e7af979
commit 60cc7a2
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/include/nutconf.hpp b/include/nutconf.hpp
@@ -417,7 +417,10 @@ class BoolInt
 		if (i.set()) return i;
 		if (bool01.set() && bool01 == true) {
 			if (b.set()) {
-				if (b) return 1;
+				/** Cause use of operator to avoid warnings like
+				 * "may be used uninitialized in this function"
+				 */
+				if (b == true) return 1;
 				return 0;
 			}
 		} else {
@@ -447,7 +450,7 @@ class BoolInt
 
 	inline std::string toString()const {
 		if (b.set()) {
-			if (b) return "yes";
+			if (b == true) return "yes";
 			return "no";
 		}