[PATCH evdev, v2] Support smooth scrolling on wheel emulation
Peter Hutterer
peter.hutterer at who-t.net
Wed Oct 12 22:55:45 PDT 2011
On Thu, Oct 06, 2011 at 11:45:58PM +0200, Max Schwarz wrote:
> Hi Peter,
>
> > you could make the support compiled in, but not compiled _out_. so even if
> > you HAVE_SMOOTH_SCROLLING, the old bits are ready to go when enabled.
> > if no smooth scrolling axis is otherwise present on the device, just post
> > button events as previously.
> Well, the proper thing to do would be to add new valuators and report the
> changed axes via a XIDeviceChangedEvent. But since there's no infrastructure
> for that yet (or is it?), I'll do it that way.
correct, we don't have that infrastructure yet. one day...
> > > #define TestBit(bit, array) ((array[(bit) / LONG_BITS]) & (1L <<
> > > ((bit) %>
> > > LONG_BITS)))
> > > +#define evdev_SetBit(bit, array) ((array[(bit) / LONG_BITS]) |= (1L <<
> > > ((bit)
> > Can we use the server's BitIsOn (inputstr.h), SetBit, etc here instead of
> > relying on our own macros with name conflicts?
> That wouldn't be the same on Big-Endian machines, since BitIsOn() from
> inputstr.h expects an char array, while the evdev bitmasks are kept in longs.
> I agree that this bit looks bad, maybe we could rename TestBit to
> evdev_TestBit to match evdev_SetBit?
evdev_set_bit, evdev_bit_isset then please. preferably inline functions to
get some type safety and extra compiler warnings.
Cheers,
Peter
More information about the xorg-devel
mailing list