[PATCH libinput 3/4] evdev: Release still pressed keys/buttons when removing device

Bill Spitzak spitzak at gmail.com
Mon Aug 11 14:11:55 PDT 2014


On 08/10/2014 10:39 PM, Peter Hutterer wrote:
> On Sun, Aug 10, 2014 at 12:27:07PM +0200, Jonas Ådahl wrote:

>> We could also drop the mask and generate 'pressed' events when adding a
>> device; can we do that reliably without risking a race condition?
>
> technically yes, but I'm not sure it is a good idea. I think it's better to
> ignore any keys down when the device is added and wait for the next press.
> the side-effects of out-of-order key events are greater than the benefits
> here (e.g. was it Backspace, then Ctrl + Alt on resume or Ctrl + Alt, then
> Backspace?)

I don't think that would be a problem if the pressed event also included 
the fact that it was due to the key already being down.


More information about the wayland-devel mailing list