On Thu, 11 Aug 2011 11:13:28 +1000, Peter Hutterer <[email protected]> wrote:
This sequence looks a *lot* nicer. Someday we'll be able to understand
the XI2 code.x
> + if ((type = GetXIType(event)) != 0)
> + {
> + filter = GetEventFilterMask(dev, type);
> +
> + /* Check for XI mask */
> + if (type && inputMasks &&
Don't need to recheck 'type' here.
> + (inputMasks->deliverableEvents[dev->id] & filter) &&
> + (inputMasks->inputEvents[dev->id] & filter))
> + rc |= EVENT_XI1_MASK;
> +
> + /* Check for XI DontPropagate mask */
> + if (type && inputMasks &&
Or here.
> + (inputMasks->dontPropagateMask[dev->id] & filter))
> + rc |= EVENT_DONT_PROPAGATE_MASK;
> +
> + }
> +
> + if ((type = GetCoreType(event)) != 0)
> + {
> + filter = GetEventFilterMask(dev, type);
> +
> + /* Check for core mask */
> + if (type && (win->deliverableEvents & filter) &&
Not even here.
> + ((wOtherEventMasks(win) | win->eventMask) & filter))
> + rc |= EVENT_CORE_MASK;
> +
> + /* Check for core DontPropagate mask */
> + if (type && (filter & wDontPropagateMask(win)))
And, not here.
> + rc |= EVENT_DONT_PROPAGATE_MASK;
> + }
>
> return rc;
> }
> --
> 1.7.6
--
[email protected]
pgpfSdR6R3Yq4.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
