Messages: merge macros with and without message code
[platform/upstream/libxkbcommon.git] / src / xkbcomp / keycodes.c
index f870f00..91471ea 100644 (file)
  *
  ********************************************************/
 
+#include "config.h"
+
 #include "xkbcomp-priv.h"
 #include "text.h"
 #include "expr.h"
-#include "keycodes.h"
 #include "include.h"
 
-/*
- * The xkb_keycodes section
- * ========================
- *
- * This is the simplest section type, and is the first one to be
- * compiled. The purpose of this is mostly to map between the
- * hardware/evdev scancodes and xkb keycodes. Each key is given a name
- * of up to 4 letters, by which it can be referred to later, e.g. in the
- * symbols section.
- *
- * Minimum/Maximum keycode
- * -----------------------
- * Statements of the form:
- *      minimum = 8;
- *      maximum = 255;
- *
- * The file may explicitly declare the minimum and/or maximum keycode
- * contained therein (traditionally 8-255, inherited from old xfree86
- * keycodes). If these are stated explicitly, they are enforced. If
- * they are not stated, they are computed automatically.
- *
- * Keycode statements
- * ------------------
- * Statements of the form:
- *      <TLDE> = 49;
- *      <AE01> = 10;
- *
- * The above would let 49 and 10 be valid keycodes in the keymap, and
- * assign them the names TLDE and AE01 respectively. The format <WXYZ> is
- * always used to refer to a key by name.
- *
- * [ The naming convention <AE01> just denoted the position of the key
- * in the main alphanumric section of the keyboard, with the top left key
- * (usually "~") acting as origin. <AE01> is the key in the first row
- * second column (which is usually "1"). ]
- *
- * In the common case this just maps to the evdev scancodes from
- * /usr/include/linux/input.h, e.g. the following definitions:
- *      #define KEY_GRAVE            41
- *      #define KEY_1                2
- * Similar definitions appear in the xf86-input-keyboard driver. Note
- * that in all current keymaps there's a constant offset of 8 (for
- * historical reasons).
- *
- * If there's a conflict, like the same name given to different keycodes,
- * or same keycode given different names, it is resolved according to the
- * merge mode which applies to the definitions.
- *
- * The reason for the 4 characters limit is that the name is sometimes
- * converted to an unsigned long (in a direct mapping), instead of a char
- * array (see KeyNameToLong, LongToKeyName).
- *
- * Alias statements
- * ----------------
- * Statements of the form:
- *      alias <MENU> = <COMP>;
- *
- * Allows to refer to a previously defined key (here <COMP>) by another
- * name (here <MENU>). Conflicts are handled similarly.
- *
- * Indicator name statements
- * -------------------------
- * Statements of the form:
- *      indicator 1 = "Caps Lock";
- *      indicator 2 = "Num Lock";
- *      indicator 3 = "Scroll Lock";
- *
- * Assigns a name the indicator (i.e. keyboard LED) with the given index.
- * The amount of possible indicators is predetermined (XKB_NUM_INDICATORS).
- * The indicator may be referred by this name later in the compat section
- * and by the user.
- *
- * Effect on the keymap
- * --------------------
- * After all of the xkb_keycodes sections have been compiled, the
- * following members of struct xkb_keymap are finalized:
- *      xkb_keycode_t min_key_code;
- *      xkb_keycode_t max_key_code;
- *      darray(struct xkb_key_alias) key_aliases;
- *      char *keycodes_section_name;
- * The 'name' field of indicators declared in xkb_keycodes:
- *      struct xkb_indicator_map indicators[XKB_NUM_INDICATORS];
- * Further, the array of keys:
- *      darray(struct xkb_key) keys;
- * had been resized to its final size (i.e. all of the xkb_key objects are
- * referable by their keycode). However the objects themselves do not
- * contain any useful information besides the key name at this point.
- */
-
-typedef struct _AliasInfo {
+typedef struct {
     enum merge_mode merge;
-    unsigned file_id;
 
-    unsigned long alias;
-    unsigned long real;
+    xkb_atom_t alias;
+    xkb_atom_t real;
 } AliasInfo;
 
-typedef struct _IndicatorNameInfo {
+typedef struct {
     enum merge_mode merge;
-    unsigned file_id;
 
     xkb_atom_t name;
-} IndicatorNameInfo;
+} LedNameInfo;
 
-typedef struct _KeyNamesInfo {
-    char *name;     /* e.g. evdev+aliases(qwerty) */
+typedef struct {
+    char *name;
     int errorCount;
-    unsigned file_id;
-    enum merge_mode merge;
 
-    xkb_keycode_t computedMin; /* lowest keycode stored */
-    xkb_keycode_t computedMax; /* highest keycode stored */
-    xkb_keycode_t explicitMin;
-    xkb_keycode_t explicitMax;
-    darray(unsigned long) names;
-    darray(unsigned int) files;
-    IndicatorNameInfo indicator_names[XKB_NUM_INDICATORS];
+    xkb_keycode_t min_key_code;
+    xkb_keycode_t max_key_code;
+    darray(xkb_atom_t) key_names;
+    LedNameInfo led_names[XKB_MAX_LEDS];
+    unsigned int num_led_names;
     darray(AliasInfo) aliases;
 
     struct xkb_context *ctx;
 } KeyNamesInfo;
 
-static void
-ResizeKeyNameArrays(KeyNamesInfo *info, int newMax)
-{
-    if (newMax < darray_size(info->names))
-        return;
-
-    darray_resize0(info->names, newMax + 1);
-    darray_resize0(info->files, newMax + 1);
-}
+/***====================================================================***/
 
 static void
-InitAliasInfo(AliasInfo *info, enum merge_mode merge, unsigned file_id,
-              char alias[XKB_KEY_NAME_LENGTH], char real[XKB_KEY_NAME_LENGTH])
+InitAliasInfo(AliasInfo *info, enum merge_mode merge,
+              xkb_atom_t alias, xkb_atom_t real)
 {
     memset(info, 0, sizeof(*info));
     info->merge = merge;
-    info->file_id = file_id;
-    info->alias = KeyNameToLong(alias);
-    info->real = KeyNameToLong(real);
+    info->alias = alias;
+    info->real = real;
 }
 
-static IndicatorNameInfo *
-FindIndicatorByName(KeyNamesInfo *info, xkb_atom_t name,
-                    xkb_led_index_t *idx_out)
+static LedNameInfo *
+FindLedByName(KeyNamesInfo *info, xkb_atom_t name,
+              xkb_led_index_t *idx_out)
 {
-    xkb_led_index_t idx;
-
-    for (idx = 0; idx < XKB_NUM_INDICATORS; idx++) {
-        if (info->indicator_names[idx].name == name) {
+    for (xkb_led_index_t idx = 0; idx < info->num_led_names; idx++) {
+        LedNameInfo *ledi = &info->led_names[idx];
+        if (ledi->name == name) {
             *idx_out = idx;
-            return &info->indicator_names[idx];
+            return ledi;
         }
     }
 
@@ -190,74 +86,63 @@ FindIndicatorByName(KeyNamesInfo *info, xkb_atom_t name,
 }
 
 static bool
-AddIndicatorName(KeyNamesInfo *info, enum merge_mode merge,
-                 IndicatorNameInfo *new, xkb_led_index_t new_idx)
+AddLedName(KeyNamesInfo *info, enum merge_mode merge, bool same_file,
+           LedNameInfo *new, xkb_led_index_t new_idx)
 {
     xkb_led_index_t old_idx;
-    IndicatorNameInfo *old;
-    bool replace, report;
-    int verbosity = xkb_get_log_verbosity(info->ctx);
-
-    replace = (merge == MERGE_REPLACE) || (merge == MERGE_OVERRIDE);
+    LedNameInfo *old;
+    const int verbosity = xkb_context_get_log_verbosity(info->ctx);
+    const bool report = (same_file && verbosity > 0) || verbosity > 9;
+    const bool replace = (merge == MERGE_REPLACE || merge == MERGE_OVERRIDE);
 
-    old = FindIndicatorByName(info, new->name, &old_idx);
+    /* LED with the same name already exists. */
+    old = FindLedByName(info, new->name, &old_idx);
     if (old) {
-        report = ((old->file_id == new->file_id && verbosity > 0) ||
-                  verbosity > 9);
-
         if (old_idx == new_idx) {
-            if (report)
-                log_warn(info->ctx, "Multiple indicators named %s; "
-                         "Identical definitions ignored\n",
-                         xkb_atom_text(info->ctx, new->name));
+            log_warn(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                     "Multiple indicators named \"%s\"; "
+                     "Identical definitions ignored\n",
+                     xkb_atom_text(info->ctx, new->name));
             return true;
         }
 
-        if (report)
-            log_warn(info->ctx, "Multiple indicators named %s; "
-                     "Using %d, ignoring %d\n",
-                     xkb_atom_text(info->ctx, new->name),
-                     (replace ? old_idx + 1 : new_idx + 1),
-                     (replace ? new_idx + 1 : old_idx + 1));
-
-        /*
-         * XXX: If in the next check we ignore new, than we will have
-         * deleted this old for nothing!
-         */
+        if (report) {
+            xkb_led_index_t use = (replace ? new_idx + 1 : old_idx + 1);
+            xkb_led_index_t ignore = (replace ? old_idx + 1 : new_idx + 1);
+            log_warn(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                     "Multiple indicators named %s; Using %d, ignoring %d\n",
+                     xkb_atom_text(info->ctx, new->name), use, ignore);
+        }
+
         if (replace)
-            memset(old, 0, sizeof(*old));
+            *old = *new;
+
+        return true;
     }
 
-    old = &info->indicator_names[new_idx];
-    if (old->name != XKB_ATOM_NONE) {
-        report = ((old->file_id == new->file_id && verbosity > 0) ||
-                  verbosity > 9);
+    if (new_idx >= info->num_led_names)
+        info->num_led_names = new_idx + 1;
 
-        if (old->name == new->name) {
-            if (report)
-                log_warn(info->ctx, "Multiple names for indicator %d; "
-                         "Identical definitions ignored\n", new_idx + 1);
-        }
-        else if (replace) {
-            if (report)
-                log_warn(info->ctx, "Multiple names for indicator %d; "
-                         "Using %s, ignoring %s\n", new_idx + 1,
-                         xkb_atom_text(info->ctx, new->name),
-                         xkb_atom_text(info->ctx, old->name));
-            old->name = new->name;
-        }
-        else {
-            if (report)
-                log_warn(info->ctx, "Multiple names for indicator %d; "
-                         "Using %s, ignoring %s\n", new_idx + 1,
-                         xkb_atom_text(info->ctx, old->name),
-                         xkb_atom_text(info->ctx, new->name));
+    /* LED with the same index already exists. */
+    old = &info->led_names[new_idx];
+    if (old->name != XKB_ATOM_NONE) {
+        if (report) {
+            const xkb_atom_t use = (replace ? new->name : old->name);
+            const xkb_atom_t ignore = (replace ? old->name : new->name);
+            log_warn(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                     "Multiple names for indicator %d; "
+                     "Using %s, ignoring %s\n", new_idx + 1,
+                     xkb_atom_text(info->ctx, use),
+                     xkb_atom_text(info->ctx, ignore));
         }
 
+        if (replace)
+            *old = *new;
+
         return true;
     }
 
-    info->indicator_names[new_idx] = *new;
+    *old = *new;
     return true;
 }
 
@@ -265,149 +150,115 @@ static void
 ClearKeyNamesInfo(KeyNamesInfo *info)
 {
     free(info->name);
-    darray_free(info->names);
-    darray_free(info->files);
+    darray_free(info->key_names);
     darray_free(info->aliases);
 }
 
 static void
-InitKeyNamesInfo(KeyNamesInfo *info, struct xkb_context *ctx,
-                 unsigned file_id)
+InitKeyNamesInfo(KeyNamesInfo *info, struct xkb_context *ctx)
 {
     memset(info, 0, sizeof(*info));
     info->ctx = ctx;
-    info->merge = MERGE_DEFAULT;
-    info->file_id = file_id;
-    info->computedMin = XKB_KEYCODE_MAX;
+    info->min_key_code = XKB_KEYCODE_INVALID;
+#if XKB_KEYCODE_INVALID < XKB_KEYCODE_MAX
+#error "Hey, you can't be changing stuff like that."
+#endif
 }
 
-static int
-FindKeyByLong(KeyNamesInfo * info, unsigned long name)
+static xkb_keycode_t
+FindKeyByName(KeyNamesInfo *info, xkb_atom_t name)
 {
     xkb_keycode_t i;
 
-    for (i = info->computedMin; i <= info->computedMax; i++)
-        if (darray_item(info->names, i) == name)
+    for (i = info->min_key_code; i <= info->max_key_code; i++)
+        if (darray_item(info->key_names, i) == name)
             return i;
 
-    return 0;
+    return XKB_KEYCODE_INVALID;
 }
 
-/**
- * Store the name of the key as a long in the info struct under the given
- * keycode. If the same keys is referred to twice, print a warning.
- * Note that the key's name is stored as a long, the keycode is the index.
- */
 static bool
-AddKeyName(KeyNamesInfo *info, xkb_keycode_t kc, unsigned long name,
-           enum merge_mode merge, unsigned file_id, bool reportCollisions)
+AddKeyName(KeyNamesInfo *info, xkb_keycode_t kc, xkb_atom_t name,
+           enum merge_mode merge, bool same_file, bool report)
 {
-    xkb_keycode_t old;
-    int verbosity = xkb_get_log_verbosity(info->ctx);
+    xkb_atom_t old_name;
+    xkb_keycode_t old_kc;
+    const int verbosity = xkb_context_get_log_verbosity(info->ctx);
 
-    ResizeKeyNameArrays(info, kc);
+    report = report && ((same_file && verbosity > 0) || verbosity > 7);
 
-    if (kc < info->computedMin)
-        info->computedMin = kc;
-    if (kc > info->computedMax)
-        info->computedMax = kc;
+    if (kc >= darray_size(info->key_names))
+        darray_resize0(info->key_names, kc + 1);
 
-    if (reportCollisions)
-        reportCollisions = (verbosity > 7 ||
-                            (verbosity > 0 &&
-                             file_id == darray_item(info->files, kc)));
+    info->min_key_code = MIN(info->min_key_code, kc);
+    info->max_key_code = MAX(info->max_key_code, kc);
 
-    if (darray_item(info->names, kc) != 0) {
-        const char *lname = LongKeyNameText(darray_item(info->names, kc));
-        const char *kname = LongKeyNameText(name);
+    /* There's already a key with this keycode. */
+    old_name = darray_item(info->key_names, kc);
+    if (old_name != XKB_ATOM_NONE) {
+        const char *lname = KeyNameText(info->ctx, old_name);
+        const char *kname = KeyNameText(info->ctx, name);
 
-        if (darray_item(info->names, kc) == name && reportCollisions) {
-            log_warn(info->ctx, "Multiple identical key name definitions; "
-                     "Later occurences of \"%s = %d\" ignored\n", lname, kc);
+        if (old_name == name) {
+            if (report)
+                log_warn(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                         "Multiple identical key name definitions; "
+                         "Later occurrences of \"%s = %d\" ignored\n",
+                         lname, kc);
             return true;
         }
-
-        if (merge == MERGE_AUGMENT) {
-            if (reportCollisions)
-                log_warn(info->ctx, "Multiple names for keycode %d; "
+        else if (merge == MERGE_AUGMENT) {
+            if (report)
+                log_warn(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                         "Multiple names for keycode %d; "
                          "Using %s, ignoring %s\n", kc, lname, kname);
             return true;
         }
         else {
-            if (reportCollisions)
-                log_warn(info->ctx, "Multiple names for keycode %d; "
+            if (report)
+                log_warn(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                         "Multiple names for keycode %d; "
                          "Using %s, ignoring %s\n", kc, kname, lname);
-            darray_item(info->names, kc) = 0;
-            darray_item(info->files, kc) = 0;
+            darray_item(info->key_names, kc) = XKB_ATOM_NONE;
         }
     }
 
-    old = FindKeyByLong(info, name);
-    if (old != 0 && old != kc) {
-        const char *kname = LongKeyNameText(name);
+    /* There's already a key with this name. */
+    old_kc = FindKeyByName(info, name);
+    if (old_kc != XKB_KEYCODE_INVALID && old_kc != kc) {
+        const char *kname = KeyNameText(info->ctx, name);
 
         if (merge == MERGE_OVERRIDE) {
-            darray_item(info->names, old) = 0;
-            darray_item(info->files, old) = 0;
-            if (reportCollisions)
-                log_warn(info->ctx, "Key name %s assigned to multiple keys; "
-                         "Using %d, ignoring %d\n", kname, kc, old);
+            darray_item(info->key_names, old_kc) = XKB_ATOM_NONE;
+            if (report)
+                log_warn(info->ctx,
+                         XKB_WARNING_CONFLICTING_KEY_NAME,
+                         "Key name %s assigned to multiple keys; "
+                         "Using %d, ignoring %d\n", kname, kc, old_kc);
         }
         else {
-            if (reportCollisions && verbosity > 3)
-                log_warn(info->ctx, "Key name %s assigned to multiple keys; "
-                         "Using %d, ignoring %d\n", kname, old, kc);
+            if (report)
+                log_vrb(info->ctx, 3,
+                        XKB_WARNING_CONFLICTING_KEY_NAME,
+                        "Key name %s assigned to multiple keys; "
+                        "Using %d, ignoring %d\n", kname, old_kc, kc);
             return true;
         }
     }
 
-    darray_item(info->names, kc) = name;
-    darray_item(info->files, kc) = file_id;
+    darray_item(info->key_names, kc) = name;
     return true;
 }
 
 /***====================================================================***/
 
-static int
-HandleAliasDef(KeyNamesInfo *info, KeyAliasDef *def, enum merge_mode merge,
-               unsigned file_id);
-
 static bool
-MergeAliases(KeyNamesInfo *into, KeyNamesInfo *from, enum merge_mode merge)
-{
-    AliasInfo *alias;
-    KeyAliasDef def;
-
-    if (darray_empty(from->aliases))
-        return true;
-
-    if (darray_empty(into->aliases)) {
-        into->aliases = from->aliases;
-        darray_init(from->aliases);
-        return true;
-    }
-
-    memset(&def, 0, sizeof(def));
-
-    darray_foreach(alias, from->aliases) {
-        def.merge = (merge == MERGE_DEFAULT) ? alias->merge : merge;
-        LongToKeyName(alias->alias, def.alias);
-        LongToKeyName(alias->real, def.real);
-
-        if (!HandleAliasDef(into, &def, def.merge, alias->file_id))
-            return false;
-    }
-
-    return true;
-}
+HandleAliasDef(KeyNamesInfo *info, KeyAliasDef *def, enum merge_mode merge);
 
 static void
 MergeIncludedKeycodes(KeyNamesInfo *into, KeyNamesInfo *from,
                       enum merge_mode merge)
 {
-    xkb_keycode_t i;
-    xkb_led_index_t idx;
-
     if (from->errorCount > 0) {
         into->errorCount += from->errorCount;
         return;
@@ -418,92 +269,110 @@ MergeIncludedKeycodes(KeyNamesInfo *into, KeyNamesInfo *from,
         from->name = NULL;
     }
 
-    ResizeKeyNameArrays(into, from->computedMax);
+    /* Merge key names. */
+    if (darray_empty(into->key_names)) {
+        into->key_names = from->key_names;
+        darray_init(from->key_names);
+        into->min_key_code = from->min_key_code;
+        into->max_key_code = from->max_key_code;
+    }
+    else {
+        if (darray_size(into->key_names) < darray_size(from->key_names))
+            darray_resize0(into->key_names, darray_size(from->key_names));
 
-    for (i = from->computedMin; i <= from->computedMax; i++) {
-        unsigned long name = darray_item(from->names, i);
-        if (name == 0)
-            continue;
+        for (unsigned i = from->min_key_code; i <= from->max_key_code; i++) {
+            xkb_atom_t name = darray_item(from->key_names, i);
+            if (name == XKB_ATOM_NONE)
+                continue;
 
-        if (!AddKeyName(into, i, name, merge, from->file_id, false))
-            into->errorCount++;
+            if (!AddKeyName(into, i, name, merge, true, false))
+                into->errorCount++;
+        }
     }
 
-    for (idx = 0; idx < XKB_NUM_INDICATORS; idx++) {
-        IndicatorNameInfo *led = &from->indicator_names[idx];
-        if (led->name == XKB_ATOM_NONE)
-            continue;
+    /* Merge key aliases. */
+    if (darray_empty(into->aliases)) {
+        into->aliases = from->aliases;
+        darray_init(from->aliases);
+    }
+    else {
+        AliasInfo *alias;
+
+        darray_foreach(alias, from->aliases) {
+            KeyAliasDef def;
 
-        led->merge = (merge == MERGE_DEFAULT ? led->merge : merge);
-        if (!AddIndicatorName(into, led->merge, led, idx))
-            into->errorCount++;
+            def.merge = (merge == MERGE_DEFAULT ? alias->merge : merge);
+            def.alias = alias->alias;
+            def.real = alias->real;
+
+            if (!HandleAliasDef(into, &def, def.merge))
+                into->errorCount++;
+        }
     }
 
-    if (!MergeAliases(into, from, merge))
-        into->errorCount++;
+    /* Merge LED names. */
+    if (into->num_led_names == 0) {
+        memcpy(into->led_names, from->led_names,
+               sizeof(*from->led_names) * from->num_led_names);
+        into->num_led_names = from->num_led_names;
+        from->num_led_names = 0;
+    }
+    else {
+        for (xkb_led_index_t idx = 0; idx < from->num_led_names; idx++) {
+            LedNameInfo *ledi = &from->led_names[idx];
 
-    if (from->explicitMin != 0)
-        if (into->explicitMin == 0 || into->explicitMin > from->explicitMin)
-            into->explicitMin = from->explicitMin;
+            if (ledi->name == XKB_ATOM_NONE)
+                continue;
 
-    if (from->explicitMax > 0)
-        if (into->explicitMax == 0 || into->explicitMax < from->explicitMax)
-            into->explicitMax = from->explicitMax;
+            ledi->merge = (merge == MERGE_DEFAULT ? ledi->merge : merge);
+            if (!AddLedName(into, ledi->merge, false, ledi, idx))
+                into->errorCount++;
+        }
+    }
 }
 
 static void
 HandleKeycodesFile(KeyNamesInfo *info, XkbFile *file, enum merge_mode merge);
 
 static bool
-HandleIncludeKeycodes(KeyNamesInfo *info, IncludeStmt *stmt)
+HandleIncludeKeycodes(KeyNamesInfo *info, IncludeStmt *include)
 {
-    enum merge_mode merge = MERGE_DEFAULT;
-    XkbFile *rtrn;
-    KeyNamesInfo included, next_incl;
-
-    InitKeyNamesInfo(&included, info->ctx, info->file_id);
-    if (stmt->stmt) {
-        free(included.name);
-        included.name = stmt->stmt;
-        stmt->stmt = NULL;
-    }
+    KeyNamesInfo included;
+
+    InitKeyNamesInfo(&included, info->ctx);
+    included.name = include->stmt;
+    include->stmt = NULL;
 
-    for (; stmt; stmt = stmt->next_incl) {
-        if (!ProcessIncludeFile(info->ctx, stmt, FILE_TYPE_KEYCODES,
-                                &rtrn, &merge)) {
+    for (IncludeStmt *stmt = include; stmt; stmt = stmt->next_incl) {
+        KeyNamesInfo next_incl;
+        XkbFile *file;
+
+        file = ProcessIncludeFile(info->ctx, stmt, FILE_TYPE_KEYCODES);
+        if (!file) {
             info->errorCount += 10;
             ClearKeyNamesInfo(&included);
             return false;
         }
 
-        InitKeyNamesInfo(&next_incl, info->ctx, rtrn->id);
+        InitKeyNamesInfo(&next_incl, info->ctx);
 
-        HandleKeycodesFile(&next_incl, rtrn, MERGE_OVERRIDE);
+        HandleKeycodesFile(&next_incl, file, MERGE_OVERRIDE);
 
-        MergeIncludedKeycodes(&included, &next_incl, merge);
+        MergeIncludedKeycodes(&included, &next_incl, stmt->merge);
 
         ClearKeyNamesInfo(&next_incl);
-        FreeXkbFile(rtrn);
+        FreeXkbFile(file);
     }
 
-    MergeIncludedKeycodes(info, &included, merge);
+    MergeIncludedKeycodes(info, &included, include->merge);
     ClearKeyNamesInfo(&included);
 
     return (info->errorCount == 0);
 }
 
-static int
+static bool
 HandleKeycodeDef(KeyNamesInfo *info, KeycodeDef *stmt, enum merge_mode merge)
 {
-    if ((info->explicitMin != 0 && stmt->value < info->explicitMin) ||
-        (info->explicitMax != 0 && stmt->value > info->explicitMax)) {
-        log_err(info->ctx, "Illegal keycode %lu for name %s; "
-                "Must be in the range %d-%d inclusive\n",
-                stmt->value, KeyNameText(stmt->name), info->explicitMin,
-                info->explicitMax ? info->explicitMax : XKB_KEYCODE_MAX);
-        return 0;
-    }
-
     if (stmt->merge != MERGE_DEFAULT) {
         if (stmt->merge == MERGE_REPLACE)
             merge = MERGE_OVERRIDE;
@@ -511,194 +380,121 @@ HandleKeycodeDef(KeyNamesInfo *info, KeycodeDef *stmt, enum merge_mode merge)
             merge = stmt->merge;
     }
 
-    return AddKeyName(info, stmt->value, KeyNameToLong(stmt->name), merge,
-                      info->file_id, true);
-}
-
-static void
-HandleAliasCollision(KeyNamesInfo *info, AliasInfo *old, AliasInfo *new)
-{
-    int verbosity = xkb_get_log_verbosity(info->ctx);
-    bool report = ((new->file_id == old->file_id && verbosity > 0) ||
-                   verbosity > 9);
-
-    if (new->real == old->real) {
-        if (report)
-            log_warn(info->ctx, "Alias of %s for %s declared more than once; "
-                     "First definition ignored\n",
-                     LongKeyNameText(new->alias), LongKeyNameText(new->real));
-    }
-    else {
-        unsigned long use, ignore;
-
-        use = (new->merge == MERGE_AUGMENT ? old->real : new->real);
-        ignore = (new->merge == MERGE_AUGMENT ? new->real : old->real);
-
-        if (report)
-            log_warn(info->ctx, "Multiple definitions for alias %s; "
-                     "Using %s, ignoring %s\n",
-                     LongKeyNameText(old->alias), LongKeyNameText(use),
-                     LongKeyNameText(ignore));
-
-        old->real = use;
+    if (stmt->value < 0 || stmt->value > XKB_KEYCODE_MAX) {
+        log_err(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                "Illegal keycode %lld: must be between 0..%u; "
+                "Key ignored\n", (long long) stmt->value, XKB_KEYCODE_MAX);
+        return false;
     }
 
-    old->file_id = new->file_id;
-    old->merge = new->merge;
+    return AddKeyName(info, (xkb_keycode_t) stmt->value,
+                      stmt->name, merge, false, true);
 }
 
-static int
-HandleAliasDef(KeyNamesInfo *info, KeyAliasDef *def, enum merge_mode merge,
-               unsigned file_id)
+static bool
+HandleAliasDef(KeyNamesInfo *info, KeyAliasDef *def, enum merge_mode merge)
 {
-    AliasInfo *alias, new;
-
-    darray_foreach(alias, info->aliases) {
-        if (alias->alias == KeyNameToLong(def->alias)) {
-            InitAliasInfo(&new, merge, file_id, def->alias, def->real);
-            HandleAliasCollision(info, alias, &new);
+    AliasInfo *old, new;
+
+    darray_foreach(old, info->aliases) {
+        if (old->alias == def->alias) {
+            if (def->real == old->real) {
+                log_vrb(info->ctx, 1,
+                        XKB_WARNING_CONFLICTING_KEY_NAME,
+                        "Alias of %s for %s declared more than once; "
+                        "First definition ignored\n",
+                        KeyNameText(info->ctx, def->alias),
+                        KeyNameText(info->ctx, def->real));
+            }
+            else {
+                xkb_atom_t use, ignore;
+
+                use = (merge == MERGE_AUGMENT ? old->real : def->real);
+                ignore = (merge == MERGE_AUGMENT ? def->real : old->real);
+
+                log_warn(info->ctx,
+                         XKB_WARNING_CONFLICTING_KEY_NAME,
+                         "Multiple definitions for alias %s; "
+                         "Using %s, ignoring %s\n",
+                         KeyNameText(info->ctx, old->alias),
+                         KeyNameText(info->ctx, use),
+                         KeyNameText(info->ctx, ignore));
+
+                old->real = use;
+            }
+
+            old->merge = merge;
             return true;
         }
     }
 
-    InitAliasInfo(&new, merge, file_id, def->alias, def->real);
+    InitAliasInfo(&new, merge, def->alias, def->real);
     darray_append(info->aliases, new);
     return true;
 }
 
-#define MIN_KEYCODE_DEF 0
-#define MAX_KEYCODE_DEF 1
-
-static int
+static bool
 HandleKeyNameVar(KeyNamesInfo *info, VarDef *stmt)
 {
     const char *elem, *field;
-    xkb_keycode_t kc;
     ExprDef *arrayNdx;
-    int which;
 
     if (!ExprResolveLhs(info->ctx, stmt->name, &elem, &field, &arrayNdx))
         return false;
 
     if (elem) {
-        log_err(info->ctx, "Unknown element %s encountered; "
+        log_err(info->ctx, XKB_LOG_MESSAGE_NO_ID, "Unknown element %s encountered; "
                 "Default for field %s ignored\n", elem, field);
         return false;
     }
 
-    if (istreq(field, "minimum")) {
-        which = MIN_KEYCODE_DEF;
-    }
-    else if (istreq(field, "maximum")) {
-        which = MAX_KEYCODE_DEF;
-    }
-    else {
-        log_err(info->ctx, "Unknown field encountered; "
-                "Assigment to field %s ignored\n", field);
-        return false;
-    }
-
-    if (arrayNdx != NULL) {
-        log_err(info->ctx, "The %s setting is not an array; "
-                "Illegal array reference ignored\n", field);
-        return false;
-    }
-
-    if (!ExprResolveKeyCode(info->ctx, stmt->value, &kc)) {
-        log_err(info->ctx, "Illegal keycode encountered; "
+    if (!istreq(field, "minimum") && !istreq(field, "maximum")) {
+        log_err(info->ctx, XKB_LOG_MESSAGE_NO_ID, "Unknown field encountered; "
                 "Assignment to field %s ignored\n", field);
         return false;
     }
 
-    if (kc > XKB_KEYCODE_MAX) {
-        log_err(info->ctx,
-                "Illegal keycode %d (must be in the range %d-%d inclusive); "
-                "Value of \"%s\" not changed\n",
-                kc, 0, XKB_KEYCODE_MAX, field);
-        return false;
-    }
-
-    if (which == MIN_KEYCODE_DEF) {
-        if (info->explicitMax > 0 && info->explicitMax < kc) {
-            log_err(info->ctx,
-                    "Minimum key code (%d) must be <= maximum key code (%d); "
-                    "Minimum key code value not changed\n",
-                    kc, info->explicitMax);
-            return false;
-        }
-
-        if (info->computedMax > 0 && info->computedMin < kc) {
-            log_err(info->ctx,
-                    "Minimum key code (%d) must be <= lowest defined key (%d); "
-                    "Minimum key code value not changed\n",
-                    kc, info->computedMin);
-            return false;
-        }
-
-        info->explicitMin = kc;
-    }
-    else if (which == MAX_KEYCODE_DEF) {
-        if (info->explicitMin > 0 && info->explicitMin > kc) {
-            log_err(info->ctx,
-                    "Maximum code (%d) must be >= minimum key code (%d); "
-                    "Maximum code value not changed\n",
-                    kc, info->explicitMin);
-            return false;
-        }
-
-        if (info->computedMax > 0 && info->computedMax > kc) {
-            log_err(info->ctx,
-                    "Maximum code (%d) must be >= highest defined key (%d); "
-                    "Maximum code value not changed\n",
-                    kc, info->computedMax);
-            return false;
-        }
-
-        info->explicitMax = kc;
-    }
-
+    /* We ignore explicit min/max statements, we always use computed. */
     return true;
 }
 
-static int
-HandleIndicatorNameDef(KeyNamesInfo *info, IndicatorNameDef *def,
-                       enum merge_mode merge)
+static bool
+HandleLedNameDef(KeyNamesInfo *info, LedNameDef *def,
+                 enum merge_mode merge)
 {
-    IndicatorNameInfo ii;
+    LedNameInfo ledi;
     xkb_atom_t name;
 
-    if (def->ndx < 1 || def->ndx > XKB_NUM_INDICATORS) {
+    if (def->ndx < 1 || def->ndx > XKB_MAX_LEDS) {
         info->errorCount++;
-        log_err(info->ctx,
-                "Name specified for illegal indicator index %d\n; Ignored\n",
-                def->ndx);
+        log_err(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                "Illegal indicator index (%d) specified; must be between 1 .. %d; "
+                "Ignored\n", def->ndx, XKB_MAX_LEDS);
         return false;
     }
 
     if (!ExprResolveString(info->ctx, def->name, &name)) {
         char buf[20];
-        snprintf(buf, sizeof(buf), "%d", def->ndx);
+        snprintf(buf, sizeof(buf), "%u", def->ndx);
         info->errorCount++;
-        return ReportBadType(info->ctx, "indicator", "name", buf,
-                             "string");
+        return ReportBadType(info->ctx, XKB_ERROR_WRONG_FIELD_TYPE,
+                             "indicator", "name", buf, "string");
     }
 
-    ii.merge = info->merge;
-    ii.file_id = info->file_id;
-    ii.name = name;
-    return AddIndicatorName(info, merge, &ii, def->ndx - 1);
+    ledi.merge = merge;
+    ledi.name = name;
+    return AddLedName(info, merge, true, &ledi, def->ndx - 1);
 }
 
 static void
 HandleKeycodesFile(KeyNamesInfo *info, XkbFile *file, enum merge_mode merge)
 {
-    ParseCommon *stmt;
     bool ok;
 
     free(info->name);
     info->name = strdup_safe(file->name);
 
-    for (stmt = file->defs; stmt; stmt = stmt->next) {
+    for (ParseCommon *stmt = file->defs; stmt; stmt = stmt->next) {
         switch (stmt->type) {
         case STMT_INCLUDE:
             ok = HandleIncludeKeycodes(info, (IncludeStmt *) stmt);
@@ -707,18 +503,16 @@ HandleKeycodesFile(KeyNamesInfo *info, XkbFile *file, enum merge_mode merge)
             ok = HandleKeycodeDef(info, (KeycodeDef *) stmt, merge);
             break;
         case STMT_ALIAS:
-            ok = HandleAliasDef(info, (KeyAliasDef *) stmt, merge,
-                                info->file_id);
+            ok = HandleAliasDef(info, (KeyAliasDef *) stmt, merge);
             break;
         case STMT_VAR:
             ok = HandleKeyNameVar(info, (VarDef *) stmt);
             break;
-        case STMT_INDICATOR_NAME:
-            ok = HandleIndicatorNameDef(info, (IndicatorNameDef *) stmt,
-                                        merge);
+        case STMT_LED_NAME:
+            ok = HandleLedNameDef(info, (LedNameDef *) stmt, merge);
             break;
         default:
-            log_err(info->ctx,
+            log_err(info->ctx, XKB_LOG_MESSAGE_NO_ID,
                     "Keycode files may define key and indicator names only; "
                     "Ignoring %s\n", stmt_type_to_string(stmt->type));
             ok = false;
@@ -729,116 +523,152 @@ HandleKeycodesFile(KeyNamesInfo *info, XkbFile *file, enum merge_mode merge)
             info->errorCount++;
 
         if (info->errorCount > 10) {
-            log_err(info->ctx, "Abandoning keycodes file \"%s\"\n",
-                    file->topName);
+            log_err(info->ctx, XKB_LOG_MESSAGE_NO_ID,
+                    "Abandoning keycodes file \"%s\"\n",
+                    file->name);
             break;
         }
     }
 }
 
-static void
-ApplyAliases(KeyNamesInfo *info, struct xkb_keymap *keymap)
+/***====================================================================***/
+
+static bool
+CopyKeyNamesToKeymap(struct xkb_keymap *keymap, KeyNamesInfo *info)
 {
-    struct xkb_key *key;
-    struct xkb_key_alias *a, new;
-    AliasInfo *alias;
+    struct xkb_key *keys;
+    xkb_keycode_t min_key_code, max_key_code, kc;
+
+    min_key_code = info->min_key_code;
+    max_key_code = info->max_key_code;
+    /* If the keymap has no keys, let's just use the safest pair we know. */
+    if (min_key_code == XKB_KEYCODE_INVALID) {
+        min_key_code = 8;
+        max_key_code = 255;
+    }
+
+    keys = calloc(max_key_code + 1, sizeof(*keys));
+    if (!keys)
+        return false;
 
+    for (kc = min_key_code; kc <= max_key_code; kc++)
+        keys[kc].keycode = kc;
+
+    for (kc = info->min_key_code; kc <= info->max_key_code; kc++)
+        keys[kc].name = darray_item(info->key_names, kc);
+
+    keymap->min_key_code = min_key_code;
+    keymap->max_key_code = max_key_code;
+    keymap->keys = keys;
+    return true;
+}
+
+static bool
+CopyKeyAliasesToKeymap(struct xkb_keymap *keymap, KeyNamesInfo *info)
+{
+    AliasInfo *alias;
+    unsigned i, num_key_aliases;
+    struct xkb_key_alias *key_aliases;
+
+    /*
+     * Do some sanity checking on the aliases. We can't do it before
+     * because keys and their aliases may be added out-of-order.
+     */
+    num_key_aliases = 0;
     darray_foreach(alias, info->aliases) {
         /* Check that ->real is a key. */
-        key = FindNamedKey(keymap, alias->real, false);
-        if (!key) {
+        if (!XkbKeyByName(keymap, alias->real, false)) {
             log_vrb(info->ctx, 5,
+                    XKB_WARNING_UNDEFINED_KEYCODE,
                     "Attempt to alias %s to non-existent key %s; Ignored\n",
-                    LongKeyNameText(alias->alias),
-                    LongKeyNameText(alias->real));
+                    KeyNameText(info->ctx, alias->alias),
+                    KeyNameText(info->ctx, alias->real));
+            alias->real = XKB_ATOM_NONE;
             continue;
         }
 
         /* Check that ->alias is not a key. */
-        key = FindNamedKey(keymap, alias->alias, false);
-        if (key) {
+        if (XkbKeyByName(keymap, alias->alias, false)) {
             log_vrb(info->ctx, 5,
+                    XKB_WARNING_ILLEGAL_KEYCODE_ALIAS,
                     "Attempt to create alias with the name of a real key; "
                     "Alias \"%s = %s\" ignored\n",
-                    LongKeyNameText(alias->alias),
-                    LongKeyNameText(alias->real));
+                    KeyNameText(info->ctx, alias->alias),
+                    KeyNameText(info->ctx, alias->real));
+            alias->real = XKB_ATOM_NONE;
             continue;
         }
 
-        /* Check that ->alias in not already an alias, and if so handle it. */
-        darray_foreach(a, keymap->key_aliases) {
-            AliasInfo old_alias;
+        num_key_aliases++;
+    }
 
-            if (KeyNameToLong(a->alias) != alias->alias)
-                continue;
+    /* Copy key aliases. */
+    key_aliases = NULL;
+    if (num_key_aliases > 0) {
+        key_aliases = calloc(num_key_aliases, sizeof(*key_aliases));
+        if (!key_aliases)
+            return false;
 
-            InitAliasInfo(&old_alias, MERGE_AUGMENT, 0, a->alias, a->real);
-            HandleAliasCollision(info, &old_alias, alias);
-            LongToKeyName(old_alias.alias, a->alias);
-            LongToKeyName(old_alias.real, a->real);
-            alias->alias = 0;
+        i = 0;
+        darray_foreach(alias, info->aliases) {
+            if (alias->real != XKB_ATOM_NONE) {
+                key_aliases[i].alias = alias->alias;
+                key_aliases[i].real = alias->real;
+                i++;
+            }
         }
-        if (alias->alias == 0)
-            continue;
-
-        /* Add the alias. */
-        LongToKeyName(alias->alias, new.alias);
-        LongToKeyName(alias->real, new.real);
-        darray_append(keymap->key_aliases, new);
     }
 
-    darray_free(info->aliases);
+    keymap->num_key_aliases = num_key_aliases;
+    keymap->key_aliases = key_aliases;
+    return true;
 }
 
 static bool
-CopyKeyNamesToKeymap(struct xkb_keymap *keymap, KeyNamesInfo *info)
+CopyLedNamesToKeymap(struct xkb_keymap *keymap, KeyNamesInfo *info)
 {
-    xkb_keycode_t kc;
-    xkb_led_index_t idx;
-
-    if (info->explicitMin > 0)
-        keymap->min_key_code = info->explicitMin;
-    else
-        keymap->min_key_code = info->computedMin;
-
-    if (info->explicitMax > 0)
-        keymap->max_key_code = info->explicitMax;
-    else
-        keymap->max_key_code = info->computedMax;
+    keymap->num_leds = info->num_led_names;
+    for (xkb_led_index_t idx = 0; idx < info->num_led_names; idx++) {
+        LedNameInfo *ledi = &info->led_names[idx];
 
-    darray_resize0(keymap->keys, keymap->max_key_code + 1);
-    for (kc = info->computedMin; kc <= info->computedMax; kc++)
-        LongToKeyName(darray_item(info->names, kc),
-                      XkbKey(keymap, kc)->name);
-
-    keymap->keycodes_section_name = strdup_safe(info->name);
-
-    for (idx = 0; idx < XKB_NUM_INDICATORS; idx++) {
-        IndicatorNameInfo *led = &info->indicator_names[idx];
-        if (led->name == XKB_ATOM_NONE)
+        if (ledi->name == XKB_ATOM_NONE)
             continue;
 
-        keymap->indicators[idx].name = led->name;
+        keymap->leds[idx].name = ledi->name;
     }
 
-    ApplyAliases(info, keymap);
+    return true;
+}
+
+static bool
+CopyKeyNamesInfoToKeymap(struct xkb_keymap *keymap, KeyNamesInfo *info)
+{
+    /* This function trashes keymap on error, but that's OK. */
+    if (!CopyKeyNamesToKeymap(keymap, info) ||
+        !CopyKeyAliasesToKeymap(keymap, info) ||
+        !CopyLedNamesToKeymap(keymap, info))
+        return false;
 
+    keymap->keycodes_section_name = strdup_safe(info->name);
+    XkbEscapeMapName(keymap->keycodes_section_name);
     return true;
 }
 
+/***====================================================================***/
+
 bool
 CompileKeycodes(XkbFile *file, struct xkb_keymap *keymap,
                 enum merge_mode merge)
 {
     KeyNamesInfo info;
 
-    InitKeyNamesInfo(&info, keymap->ctx, file->id);
+    InitKeyNamesInfo(&info, keymap->ctx);
 
     HandleKeycodesFile(&info, file, merge);
     if (info.errorCount != 0)
         goto err_info;
 
-    if (!CopyKeyNamesToKeymap(keymap, &info))
+    if (!CopyKeyNamesInfoToKeymap(keymap, &info))
         goto err_info;
 
     ClearKeyNamesInfo(&info);
@@ -848,39 +678,3 @@ err_info:
     ClearKeyNamesInfo(&info);
     return false;
 }
-
-struct xkb_key *
-FindNamedKey(struct xkb_keymap *keymap, unsigned long name, bool use_aliases)
-{
-    struct xkb_key *key;
-
-    xkb_foreach_key(key, keymap)
-        if (KeyNameToLong(key->name) == name)
-            return key;
-
-    if (use_aliases) {
-        unsigned long new_name;
-        if (FindKeyNameForAlias(keymap, name, &new_name))
-            return FindNamedKey(keymap, new_name, false);
-    }
-
-    return NULL;
-}
-
-bool
-FindKeyNameForAlias(struct xkb_keymap *keymap, unsigned long lname,
-                    unsigned long *real_name)
-{
-    char name[XKB_KEY_NAME_LENGTH];
-    struct xkb_key_alias *a;
-
-    LongToKeyName(lname, name);
-    darray_foreach(a, keymap->key_aliases) {
-        if (strncmp(name, a->alias, XKB_KEY_NAME_LENGTH) == 0) {
-            *real_name = KeyNameToLong(a->real);
-            return true;
-        }
-    }
-
-    return false;
-}