On Sun, Dec 11, 2011 at 09:53:24PM -0800, Chase Douglas wrote: > On 12/08/2011 07:36 PM, Peter Hutterer wrote: > > No functional changes > > > > Signed-off-by: Peter Hutterer <[email protected]> > > --- > > Xi/exevents.c | 37 +++++++++++++++++++++++++------------ > > 1 files changed, 25 insertions(+), 12 deletions(-) > > > > diff --git a/Xi/exevents.c b/Xi/exevents.c > > index b2e82ec..1140554 100644 > > --- a/Xi/exevents.c > > +++ b/Xi/exevents.c > > @@ -704,6 +704,29 @@ ChangeMasterDeviceClasses(DeviceIntPtr device, > > DeviceChangedEvent *dce) > > } > > > > /** > > + * Add state and motionMask to the filter for this event. The protocol > > + * supports some extra masks for moton when a button is down: > > + * ButtonXMotionMask and the DeviceButtonMotionMask to trigger only when at > > + * least one button (or that specific button is down). These masks need to > > + * be added to the filters for core/XI motion events. > > I don't comprehend this comment ^^.
I've fixed the typo, but without explaining what the event_filter does exactly (and that should be explained elsewhere) I'm not quite sure what to write here. It's an improvement over the previous non-existing documentation though :) > Functionally this is fine though. I'll assume you can clean up the > comment without further review. > > Reviewed-by: Chase Douglas <[email protected]> Cheers, Peter _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
