[PATCH 1/4] Port Wayland clients to new xkbcommon API

Bill Spitzak spitzak at gmail.com
Mon May 7 16:30:05 PDT 2012


Kristian Høgsberg wrote:

> The original code tests that shift and control are
> pressed, but it really should test modifers == SHIFT | CONTROL, that
> is, that those and *only* those modifiers are down.

Watch out that this is only testing what the user thinks of as 
modifiers. I did this and was burned by programs that failed if 
capslock, numlock, or scrolllock was down.

Though of course Wayland (and maybe XKB) can avoid this by making sure 
things that are not what the user considers transient modifiers are just 
  not there, so zero means what you expect.


More information about the wayland-devel mailing list