From: Tony Crisci Date: Fri, 2 May 2014 14:22:40 +0000 (-0400) Subject: Refactor binding accessor X-Git-Tag: 4.8~50 X-Git-Url: https://git.sur5r.net/?p=i3%2Fi3;a=commitdiff_plain;h=5fc1b5d02dd43ffcc2bc7d8c99dc497cd9b94e5f Refactor binding accessor Change the primary binding accessor to `get_binding_from_xcb_event`. This function gets a binding from a generic xcb event of type KeyPress, KeyRelease, ButtonPress, or ButtonRelease by determining the input type (keyboard or mouse), the modifiers pressed from the filtered event `state`, managing the proper fall back in case mode switch is enabled, and finally querying the bindings for a binding that matches the event. The logic of querying keyboard bindings is not intended to be altered by this change. The general accessor has been slightly modified to work with mouse bindings and made private because it is only used in bindings.c --- diff --git a/include/bindings.h b/include/bindings.h index d3d3aa82..52105d39 100644 --- a/include/bindings.h +++ b/include/bindings.h @@ -31,11 +31,11 @@ Binding *configure_binding(const char *bindtype, const char *modifiers, const ch void grab_all_keys(xcb_connection_t *conn, bool bind_mode_switch); /** - * Returns a pointer to the keyboard Binding with the specified modifiers and - * keycode or NULL if no such binding exists. + * Returns a pointer to the Binding that matches the given xcb event or NULL if + * no such binding exists. * */ -Binding *get_keyboard_binding(uint16_t modifiers, bool key_release, xcb_keycode_t keycode); +Binding *get_binding_from_xcb_event(xcb_generic_event_t *event); /** * Translates keysymbols to keycodes for all bindings which use keysyms. diff --git a/include/data.h b/include/data.h index 6fc7b40a..f6dc0d7c 100644 --- a/include/data.h +++ b/include/data.h @@ -91,6 +91,14 @@ typedef enum { L_SPLITH = 6 } layout_t; +/** + * Binding input types. See Binding::input_type. + */ +typedef enum { + B_KEYBOARD = 0, + B_MOUSE = 1 +} input_type_t; + /** * Stores a rectangle, for example the size of a window, the child window etc. * It needs to be packed so that the compiler will not add any padding bytes. @@ -215,12 +223,7 @@ struct regex { struct Binding { /* The type of input this binding is for. (Mouse bindings are not yet * implemented. All bindings are currently assumed to be keyboard bindings.) */ - enum { - /* Created with "bindsym", "bindcode", and "bind" */ - B_KEYBOARD = 0, - /* Created with "bindmouse" (not yet implemented). */ - B_MOUSE = 1, - } input_type; + input_type_t input_type; /** If true, the binding should be executed upon a KeyRelease event, not a * KeyPress (the default). */ diff --git a/src/bindings.c b/src/bindings.c index 3d3dbd9c..50644d91 100644 --- a/src/bindings.c +++ b/src/bindings.c @@ -123,18 +123,18 @@ void grab_all_keys(xcb_connection_t *conn, bool bind_mode_switch) { } /* - * Returns a pointer to the keyboard Binding with the specified modifiers and + * Returns a pointer to the Binding with the specified modifiers and * keycode or NULL if no such binding exists. * */ -Binding *get_keyboard_binding(uint16_t modifiers, bool key_release, xcb_keycode_t keycode) { +static Binding *get_binding(uint16_t modifiers, bool is_release, uint16_t input_code, input_type_t input_type) { Binding *bind; - if (!key_release) { - /* On a KeyPress event, we first reset all - * B_UPON_KEYRELEASE_IGNORE_MODS bindings back to B_UPON_KEYRELEASE */ + if (!is_release) { + /* On a press event, we first reset all B_UPON_KEYRELEASE_IGNORE_MODS + * bindings back to B_UPON_KEYRELEASE */ TAILQ_FOREACH(bind, bindings, bindings) { - if (bind->input_type != B_KEYBOARD) + if (bind->input_type != input_type) continue; if (bind->release == B_UPON_KEYRELEASE_IGNORE_MODS) bind->release = B_UPON_KEYRELEASE; @@ -145,37 +145,37 @@ Binding *get_keyboard_binding(uint16_t modifiers, bool key_release, xcb_keycode_ /* First compare the modifiers (unless this is a * B_UPON_KEYRELEASE_IGNORE_MODS binding and this is a KeyRelease * event) */ - if (bind->input_type != B_KEYBOARD) + if (bind->input_type != input_type) continue; if (bind->mods != modifiers && (bind->release != B_UPON_KEYRELEASE_IGNORE_MODS || - !key_release)) + !is_release)) continue; - /* If a symbol was specified by the user, we need to look in - * the array of translated keycodes for the event’s keycode */ - if (bind->symbol != NULL) { + /* For keyboard bindings where a symbol was specified by the user, we + * need to look in the array of translated keycodes for the event’s + * keycode */ + if (input_type == B_KEYBOARD && bind->symbol != NULL) { if (memmem(bind->translated_to, bind->number_keycodes * sizeof(xcb_keycode_t), - &keycode, sizeof(xcb_keycode_t)) == NULL) + &input_code, sizeof(xcb_keycode_t)) == NULL) continue; } else { /* This case is easier: The user specified a keycode */ - if (bind->keycode != keycode) + if (bind->keycode != input_code) continue; } - /* If this keybinding is a KeyRelease binding, it matches the key which - * the user pressed. We therefore mark it as - * B_UPON_KEYRELEASE_IGNORE_MODS for later, so that the user can - * release the modifiers before the actual key and the KeyRelease will - * still be matched. */ - if (bind->release == B_UPON_KEYRELEASE && !key_release) + /* If this binding is a release binding, it matches the key which the + * user pressed. We therefore mark it as B_UPON_KEYRELEASE_IGNORE_MODS + * for later, so that the user can release the modifiers before the + * actual key or button and the release event will still be matched. */ + if (bind->release == B_UPON_KEYRELEASE && !is_release) bind->release = B_UPON_KEYRELEASE_IGNORE_MODS; - /* Check if the binding is for a KeyPress or a KeyRelease event */ - if ((bind->release == B_UPON_KEYPRESS && key_release) || - (bind->release >= B_UPON_KEYRELEASE && !key_release)) + /* Check if the binding is for a press or a release event */ + if ((bind->release == B_UPON_KEYPRESS && is_release) || + (bind->release >= B_UPON_KEYRELEASE && !is_release)) continue; break; @@ -184,6 +184,59 @@ Binding *get_keyboard_binding(uint16_t modifiers, bool key_release, xcb_keycode_ return (bind == TAILQ_END(bindings) ? NULL : bind); } +/* + * Returns a pointer to the Binding that matches the given xcb button or key + * event or NULL if no such binding exists. + * + */ +Binding *get_binding_from_xcb_event(xcb_generic_event_t *event) { + bool is_release = (event->response_type == XCB_KEY_RELEASE + || event->response_type == XCB_BUTTON_RELEASE); + + input_type_t input_type = ((event->response_type == XCB_BUTTON_RELEASE + || event->response_type == XCB_BUTTON_PRESS) + ? B_MOUSE + : B_KEYBOARD); + + uint16_t event_state = ((xcb_key_press_event_t *)event)->state; + uint16_t event_detail = ((xcb_key_press_event_t *)event)->detail; + + /* Remove the numlock bit, all other bits are modifiers we can bind to */ + uint16_t state_filtered = event_state & ~(xcb_numlock_mask | XCB_MOD_MASK_LOCK); + DLOG("(removed numlock, state = %d)\n", state_filtered); + /* Only use the lower 8 bits of the state (modifier masks) so that mouse + * button masks are filtered out */ + state_filtered &= 0xFF; + DLOG("(removed upper 8 bits, state = %d)\n", state_filtered); + + if (xkb_current_group == XkbGroup2Index) + state_filtered |= BIND_MODE_SWITCH; + + DLOG("(checked mode_switch, state %d)\n", state_filtered); + + /* Find the binding */ + Binding *bind = get_binding(state_filtered, is_release, event_detail, input_type); + + /* No match? Then the user has Mode_switch enabled but does not have a + * specific keybinding. Fall back to the default keybindings (without + * Mode_switch). Makes it much more convenient for users of a hybrid + * layout (like ru). */ + if (bind == NULL) { + state_filtered &= ~(BIND_MODE_SWITCH); + DLOG("no match, new state_filtered = %d\n", state_filtered); + if ((bind = get_binding(state_filtered, is_release, event_detail, input_type)) == NULL) { + /* This is not a real error since we can have release and + * non-release bindings. On a press event for which there is only a + * !release-binding, but no release-binding, the corresponding + * release event will trigger this. No problem, though. */ + DLOG("Could not lookup key binding (modifiers %d, keycode %d)\n", + state_filtered, event_detail); + } + } + + return bind; +} + /* * Translates keysymbols to keycodes for all bindings which use keysyms. * diff --git a/src/key_press.c b/src/key_press.c index 68e2fca2..65b8e578 100644 --- a/src/key_press.c +++ b/src/key_press.c @@ -70,40 +70,11 @@ void handle_key_press(xcb_key_press_event_t *event) { DLOG("%s %d, state raw = %d\n", (key_release ? "KeyRelease" : "KeyPress"), event->detail, event->state); - /* Remove the numlock bit, all other bits are modifiers we can bind to */ - uint16_t state_filtered = event->state & ~(xcb_numlock_mask | XCB_MOD_MASK_LOCK); - DLOG("(removed numlock, state = %d)\n", state_filtered); - /* Only use the lower 8 bits of the state (modifier masks) so that mouse - * button masks are filtered out */ - state_filtered &= 0xFF; - DLOG("(removed upper 8 bits, state = %d)\n", state_filtered); - - if (xkb_current_group == XkbGroup2Index) - state_filtered |= BIND_MODE_SWITCH; - - DLOG("(checked mode_switch, state %d)\n", state_filtered); - - /* Find the binding */ - Binding *bind = get_keyboard_binding(state_filtered, key_release, event->detail); - - /* No match? Then the user has Mode_switch enabled but does not have a - * specific keybinding. Fall back to the default keybindings (without - * Mode_switch). Makes it much more convenient for users of a hybrid - * layout (like us, ru). */ - if (bind == NULL) { - state_filtered &= ~(BIND_MODE_SWITCH); - DLOG("no match, new state_filtered = %d\n", state_filtered); - if ((bind = get_keyboard_binding(state_filtered, key_release, event->detail)) == NULL) { - /* This is not a real error since we can have release and - * non-release keybindings. On a KeyPress event for which there is - * only a !release-binding, but no release-binding, the - * corresponding KeyRelease event will trigger this. No problem, - * though. */ - DLOG("Could not lookup key binding (modifiers %d, keycode %d)\n", - state_filtered, event->detail); - return; - } - } + Binding *bind = get_binding_from_xcb_event((xcb_generic_event_t *)event); + + /* if we couldn't find a binding, we are done */ + if (bind == NULL) + return; char *command_copy = sstrdup(bind->command); struct CommandResult *command_output = parse_command(command_copy);