Disabling RECORD by default

[Peter Hutterer]

On 2/12/2015 18:12 , Keith Packard wrote:
> Peter Hutterer <peter.hutterer at who-t.net> writes:
>
>> disable _the_ XTEST extension? in at least two places. One comment, just to
>> be annoying: I'd prefer this split in two patches, with the second one just
>> toggling the true/false switch for RECORD and having the git subject line of
>> "Disable RECORD by default". Makes it much easier to find in the log than
>> hiding it in the first paragraph.
>> Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net> otherwise
>
> Thanks for the suggestions; I'll get new patches out with these
> changes.
>
>> but note that syndaemon is a common user of RECORD, if it is disabled we
>> fall back to regular querying of the keyboard state with all its
>> drawbacks (unnecessary wakeups, missing of some key events, etc.)
>> with the two other patches you're breaking the backup option, so anyone not
>> using libinput will lose the disable-while-typing feature.
>
> Sounds like we need a sync timer for keyboard idle time then, and then
> updates to syndaemon.

we have device idle timers, a client could just look at the idle timers 
of each device with a XIKeyClass and subscribe to those. Unfortunately 
the most common invocation of syndaemon is with the flag that ignores 
modifier keys so you can ctrl+click without having the touchpad disabled 
after pressing ctrl. You can't get that behaviour with idletimers and 
I'm a bit scared of any idletimer implementation that does that :)

Really, the only sensible solution is what we did with libinput, but 
that's not going to find its way back into synaptics.

Cheers,
   Peter