i3lock-color

Index Commits Files Refs README LICENSE

commit 1da637c41964180a71904e4870cbb777a11b88e0
parent dd02dff44a582fe95d2d4662a7ca537d6adebffb
Author: Michael Stapelberg <michael@stapelberg.de>
Date:   Wed, 30 May 2012 16:53:16 +0200

skip shift when getting the modifier mask (Thanks SardemFF7)

Diffstat:
M
xcb.c
 | 
16
+++++++++++++++-

1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/xcb.c b/xcb.c
@@ -159,7 +159,20 @@ uint32_t get_mod_mask(xcb_connection_t *conn, xcb_key_symbols_t *symbols, uint32
     modmap_r = xcb_get_modifier_mapping_reply(conn, xcb_get_modifier_mapping(conn), NULL);
     modmap = xcb_get_modifier_mapping_keycodes(modmap_r);
 
-    for (int i = 0; i < 8; i++)
+    for (int i = 0; i < 8; i++) {
+        /* We skip i == 0 (Shift) because Shift is always Shift. Handling it
+         * would make i3lock believe that Shift is the modifier for CapsLock in
+         * case CapsLock is activated using the shift keysym:
+         *
+         * $ xmodmap
+         * shift       Shift_L (0x32),  Shift_R (0x3e)
+         * lock        Shift_L (0x32)
+         *
+         * The X11 documentation states that CapsLock and ShiftLock can only be
+         * on the lock modifier.
+         */
+        if (i == 0)
+            continue;
         for (int j = 0; j < modmap_r->keycodes_per_modifier; j++) {
             kc = modmap[i * modmap_r->keycodes_per_modifier + j];
             for (xcb_keycode_t *ktest = modeswitchcodes; *ktest; ktest++) {
@@ -171,6 +184,7 @@ uint32_t get_mod_mask(xcb_connection_t *conn, xcb_key_symbols_t *symbols, uint32
                 return (1 << i);
             }
         }
+    }
 
     free(modeswitchcodes);
     free(modmap_r);