[PATCH] [xserver] XInput extended button press/release report regression
Daniel Stone
daniel at fooishbar.org
Fri Nov 30 10:37:40 PST 2007
On Sun, Nov 25, 2007 at 02:49:50AM +0200, Daniel Stone wrote:
> On Sat, Nov 24, 2007 at 10:59:12PM +0100, Magnus Vigerlöf wrote:
> > We (linuxwacom) had some strange reports about button presses that got out of
> > sync with the actual button mapping for the InputDevice. This has shown up
> > starting with xserver 1.4.
> >
> > I've tracked it down to a button conversion made twice only for extended input
> > events, which was introduced with xserver 1.4/input rework.
> >
> > The two places where the conversion is made today are here:
> > ./dix/getevents.c:656 in GetPointerEvents
> > ./Xi/exevents.c:237 in ProcessOtherEvent
> > ---
> > Remove duplicate button press/release translation for extended events.
>
> Thanks, but the patch should be the other one: core events should get
> the button map from the extended mouse, and ProcessOtherInput shouldn't
> modify the detail field at all.
Hi,
I've pushed this to master and 1.4 branch. Thanks for the heads up:
good to know someone's using all this stuff.
Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg/attachments/20071130/4d380482/attachment.pgp>
More information about the xorg
mailing list