On Wed, Sep 14, 2016 at 11:47:15AM +0100, Daniel Stone wrote:
> From: Mihail Konev
>
> 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
> Reviewed-by: Daniel Stone
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