[PATCH] xkb: don't setMods locking modifiers (#27903)

Daniel Stone daniel at fooishbar.org
Sun Apr 22 09:25:35 PDT 2012


Hi,

On 28 July 2011 05:36, Peter Hutterer <peter.hutterer at who-t.net> wrote:
> @@ -348,7 +348,11 @@ _XkbFilterLockState(       XkbSrvInfoPtr   xkbi,
>        filter->filter = _XkbFilterLockState;
>        filter->upAction = *pAction;
>        xkbi->state.locked_mods^= pAction->mods.mask;
> -       xkbi->setMods = pAction->mods.mask;
> +       /* Don't actually change setMods here. We're locking modifiers, so
> +        * the combined results of base + locked is the same regardless. By
> +        * not setting them, we get to release locked modifiers on the
> +        * second key _press_, not release (Bug 27903)
> +        */

This mostly seems OK to me, but breaks the LockNoLock state; I guess
you'd want to directly change base_mods in the other branch.  With
that fixed:
Acked-by: Daniel Stone <daniel at fooishbar.org>

Cheers,
Daniel


More information about the xorg-devel mailing list