Peter Hutterer <[email protected]> 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 <[email protected]> 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.

-- 
-keith

Attachment: signature.asc
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to