On Tue, Dec 20, 2011 at 11:46:27AM -0500, Adam Jackson wrote:
> On 12/8/11 10:36 PM, Peter Hutterer wrote:
>
> >@@ -414,7 +414,7 @@ GetEventFilterMask(DeviceIntPtr dev, int evtype)
> > return filters[dev ? dev->id : 0][evtype];
> > }
> >
> >-static inline Mask
> >+inline Mask
> > GetXI2EventFilterMask(int evtype)
> > {
> > return (1<< (evtype % 8));
>
> This reads strangely to me. I don't remember whether saying
> 'inline' like that is legal and/or works, but it's certainly
> unnecessary. I'd take it out for clarity.
The inline was removed in adf8a0d62c0643f4c44a3b6d2e3c6c9b02532915 when
those functions were moved to inpututils and renamed to
event_get_filter_from_xi2type. That's already on master, so I'll skip
trying to slot in a patch here :)
Cheers,
Peter
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
