On Wed, Sep 14, 2016 at 11:47:15AM +0100, Daniel Stone wrote:
> From: Mihail Konev <k....@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....@ya.ru>
> Reviewed-by: Daniel Stone <dani...@collabora.com>


merged and pushed, thanks

Cheers,
   Peter

> ---
>  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
> 
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to