Switched to regular expressions to check if the value is in [0-100] range.

xvitaly · xvitaly · commit ae386384f32e · 2024-10-08T10:17:03.000+02:00
diff --git a/src/lib/zswapobject/zswapobject.cpp b/src/lib/zswapobject/zswapobject.cpp
@@ -16,9 +16,7 @@
 
 bool ZSwapObject::CheckPercent(const std::string& Value) const
 {
-    if (Value.empty()) return true;
-    const int ValueInt = std::stoi(Value);
-    return (ValueInt < 0) || (ValueInt > 100);
+    return !std::regex_match(Value, std::regex("^\\d{1,2}|100$"));
 }
 
 bool ZSwapObject::CheckEnabled(const std::string& Value) const

Original file line number	Diff line number	Diff line change
`@@ -16,9 +16,7 @@`
`16`	`16`
`17`	`17`	`bool ZSwapObject::CheckPercent(const std::string& Value) const`
`18`	`18`	`{`
`19`		`- if (Value.empty()) return true;`
`20`		`- const int ValueInt = std::stoi(Value);`
`21`		`- return (ValueInt < 0) \|\| (ValueInt > 100);`
	`19`	`+ return !std::regex_match(Value, std::regex("^\\d{1,2}\|100$"));`
`22`	`20`	`}`
`23`	`21`
`24`	`22`	`bool ZSwapObject::CheckEnabled(const std::string& Value) const`