[PATCH] Kmscon wlterm: Fix stuck key repeat bug
David Herrmann
dh.herrmann at googlemail.com
Sun Jan 27 06:33:05 PST 2013
Hi Martin
On Sat, Jan 26, 2013 at 12:09 PM, Martin Minarik
<minarik11 at student.fiit.stuba.sk> wrote:
> 1. Depress Shift
> 2. Depress K
> 3. Release Shift
> 4. Release K
>
> Observed: key stuck repeat
> Expected: key not stuck repeating
> ---
This patch solves your particular problem, however, it isn't the
correct way to go. See this scenario:
- press a
- press b
- release a
You would expect 'b' to be still repeated after this, however, with
this patch it won't. kmscon does it the right way (see
uterm_input_uxkb.c), however, wlterm still uses keysyms to check for
key-repeat (which is obviously wrong) and this whole thing needs to be
converted to use keycodes for keyrepeat.
I applied your patch as it is still the better behavior than before.
However, I have to rework that thing to work exactly as expected.
Thanks!
David
More information about the wayland-devel
mailing list