[PATCH] xkb: fix turbo-repeat of RedirectKey-ed keysyms

Daniel Stone daniels at collabora.com
Wed Sep 14 10:47:15 UTC 2016


From: Mihail Konev <k.mvc at ya.ru>

RedirectKey() action had been broken by commit 2e6190.
A dropped check caused over-intense autorepeat of keysyms enriched
with the action.

Previous to this commit, the check wrapped the entire switch() block,
which was dropped with the move to a separate function.

Restore the checking.

Signed-off-by: Mihail Konev <k.mvc at ya.ru>
Reviewed-by: Daniel Stone <daniels at collabora.com>
---
 xkb/xkbActions.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/xkb/xkbActions.c b/xkb/xkbActions.c
index 048ed44..2ffd3fa 100644
--- a/xkb/xkbActions.c
+++ b/xkb/xkbActions.c
@@ -1337,7 +1337,8 @@ XkbHandleActions(DeviceIntPtr dev, DeviceIntPtr kbd, DeviceEvent *event)
         }
 
         sendEvent = _XkbApplyFilters(xkbi, key, &act);
-        XkbActionGetFilter(dev, event, key, &act, &sendEvent);
+        if (sendEvent)
+            XkbActionGetFilter(dev, event, key, &act, &sendEvent);
     }
     else {
         if (!keyEvent)
-- 
2.9.3



More information about the xorg-devel mailing list