keyboard: avoid stuck keys due to keybindings (alternate approach)

Before commit e77330bc3f, there were issues with keys becoming "stuck"
if other keys were pressed at the time a keybinding was matched, because
those other keys were included in the "bound" set and the release events
were incorrectly eaten by labwc.

Commit e77330bc3f solved that issue with the "big hammer" approach of
preventing keybindings from working at all if other keys were pressed:

        if (key_state_nr_pressed_keys() > 1) {
                return false;
        }

This is an alternate approach to solving the original problem, by (1)
not including those other keys in the "bound" set and (2) making sure we
always forward release events for un-bound keys to clients (even if a
menu or OSD is displayed).

Details:

- Since we only ever want to store the single matched keycode as bound,
  key_state_store_pressed_keys_as_bound() doesn't really make sense in
  the plural, so rename it to key_state_store_pressed_key_as_bound() and
  pass in the keycode.

- The calls to key_state_store_pressed_keys_as_bound() within
  handle_keybinding() appear to be redundant since it is also called
  from the parent function (handle_compositor_keybindings()). So remove
  these calls.

- Finally, rework the logic for handling key-release events so that we
  always forward release events for keys not in the "bound" set.

This PR does not remove the "key_state_nr_pressed_keys() > 1" check, and
because of that should not result in any functional change. It should
however make it possible to relax or remove that check in future.
This commit is contained in:
John Lindgren 2023-11-04 01:23:43 -04:00 committed by Johan Malm
parent 0bd265600e
commit 7571c4bed3
3 changed files with 41 additions and 33 deletions

View file

@ -79,10 +79,9 @@ key_state_set_pressed(uint32_t keycode, bool ispressed)
}
void
key_state_store_pressed_keys_as_bound(void)
key_state_store_pressed_key_as_bound(uint32_t keycode)
{
memcpy(bound.keys, pressed.keys, MAX_PRESSED_KEYS * sizeof(uint32_t));
bound.nr_keys = pressed.nr_keys;
add_key(&bound, keycode);
}
bool