Don't ignore inactive type entries
authorDaniel Stone <daniel@fooishbar.org>
Fri, 22 Jun 2012 15:03:17 +0000 (16:03 +0100)
committerDaniel Stone <daniel@fooishbar.org>
Fri, 22 Jun 2012 15:05:45 +0000 (16:05 +0100)
An entry for a type will only get marked as active if a virtual modifier
can be directly mapped to it, and not if an action indirectly leads to
it (e.g. LevelThree).  We don't really need this test since entries which
can never be triggered ... won't be triggered.

The entire map->active thing should probably just go away.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>
src/map.c

index d9afc1a..90048cc 100644 (file)
--- a/src/map.c
+++ b/src/map.c
@@ -253,9 +253,6 @@ xkb_key_get_level(struct xkb_state *state, xkb_keycode_t key,
     active_mods &= type->mods.mask;
 
     darray_foreach(entry, type->map) {
-        if (!entry->active)
-            continue;
-
         if (entry->mods.mask == active_mods)
             return entry->level;
     }