Open-code a radio group SimpleLookup

Because, joke's on you, it wasn't actually looking up radio groups.
Just checking to see if it was a string that was "none", or an integer.

Lord give me strength.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

diff --git a/src/xkbcomp/symbols.c b/src/xkbcomp/symbols.c
index 08b9bd7d06b6c9415e5acc8a976d9708cf1c47e7..0b225cc7c973b2ad530c786e519a84340e3f807c 100644 (file)
--- a/src/xkbcomp/symbols.c
+++ b/src/xkbcomp/symbols.c
@@ -1100,11 +1100,6 @@ static LookupEntry repeatEntries[] = {
     {NULL, 0}
 };
 
-static LookupEntry rgEntries[] = {
-    {"none", 0},
-    {NULL, 0}
-};
-
 static Bool
 SetSymbolsField(KeyInfo * key,
                 struct xkb_desc * xkb,
@@ -1189,12 +1184,19 @@ SetSymbolsField(KeyInfo * key,
         Bool permanent = False;
         if (uStrCaseCmp(field, "permanentradiogroup") == 0)
             permanent = True;
-        ok = ExprResolveInteger(value, &tmp, SimpleLookup,
-                                (char *) rgEntries);
+        if (ExprResolveString(value, &tmp)) {
+            ok = (strcmp(tmp.str, "none") == 0);
+            free(tmp.str);
+            if (ok)
+                tmp.uval = 0;
+        }
+        else {
+            ok = ExprResolveInteger(value, &tmp, NULL, 0);
+        }
         if (!ok)
         {
             ERROR("Illegal radio group specification for %s\n",
-                   longText(key->name));
+                  longText(key->name));
             ACTION("Non-integer radio group ignored\n");
             return False;
         }