UI: RuleTextParser: Simplify parseValue()

src/ui/util/conf/ruletextparser.cpp

@@ -365,19 +365,8 @@ bool RuleTextParser::parseValue(bool expectValueEnd)
 {
     const QChar *value = parsedCharPtr();
 
-    for (;;) {
+    if (!parseValueChars(expectValueEnd))
-        const char *extraChars = expectValueEnd ? extraValueEndChars : extraValueChars;
+        return false;
-
-        if (!parseChars(CharLetter | CharValue, extraChars))
-            return false;
-
-        if (!checkValueEnd(expectValueEnd)) {
-            if (hasError())
-                return false;
-
-            break;
-        }
-    }
 
     const QStringView valueView(value, currentCharPtr() - value);
 
@@ -386,6 +375,21 @@ bool RuleTextParser::parseValue(bool expectValueEnd)
     return true;
 }
 
+bool RuleTextParser::parseValueChars(bool &expectValueEnd)
+{
+    for (;;) {
+        const char *extraChars = expectValueEnd ? extraValueEndChars : extraValueChars;
+
+        if (!parseChars(CharLetter | CharValue, extraChars))
+            return false;
+
+        if (!checkValueEnd(expectValueEnd))
+            break;
+    }
+
+    return !hasError();
+}
+
 bool RuleTextParser::checkValueEnd(bool &expectValueEnd)
 {
     const bool isValueBegin = (m_charType == CharValueBegin);

src/ui/util/conf/ruletextparser.h

@@ -112,6 +112,7 @@ private:
     bool checkBracketValuesSeparator(RuleCharTypes expectedSeparator);
     bool checkBracketValueEnd();
     bool parseValue(bool expectValueEnd);
+    bool parseValueChars(bool &expectValueEnd);
     bool checkValueEnd(bool &expectValueEnd);
 
     bool checkAddFilter();