[PATCH] Revert "dix: use the event mask of the grab for TryClientEvents."
daniel at fooishbar.org
Thu Jun 24 17:09:41 PDT 2010
On Fri, Jun 25, 2010 at 09:48:10AM +1000, Peter Hutterer wrote:
> Behaviour of earlier X servers was to deliver the ButtonPress event
> unconditionally, regardless of the actual event mask being set. Thus, a
> GrabButton event will always deliver the button press event, a GrabKey
> always the key press event, etc. Same goes for XI and XI2.
> Reproducible with a simple client requesting a button grab in the form of:
> XGrabButton(dpy, AnyButton, AnyModifier, win, True, ButtonReleaseMask,
> GrabModeAsync, GrabModeAsync, None, None);
> On servers before MPX/XI2, the client will receive a button press and
> release event. On current servers, the client receives only the release.
> Clients that expect the press event to be delivered unconditionally may
> leave the user with a stuck grab.
> XTS test results for XGrabButton are identical with and without this patch.
> This reverts commit 48585bd1e3e98db0f3df1ecc68022510216e00cc.
The reverted commit was meant to fix some kind of Motif bug (possibly
mouse getting confined to a widget) or something along those lines,
right? The explanation for this commit seems fine to me, but I vaguely
remember the original being some kind of bugfix ... is it no longer
necessary, or just didn't work out, or?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the xorg-devel