Attachment 21890 Details for Bug 19297 – 0001-dix-fix-WarpPointer-calls-for-devices-with-custom-v.patch

[patch] 0001-dix-fix-WarpPointer-calls-for-devices-with-custom-v.patch

0001-dix-fix-WarpPointer-calls-for-devices-with-custom-v.patch (text/plain), 4.67 KB, created by Peter Hutterer on 2009-01-11 21:39:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Peter Hutterer

Created: 2009-01-11 21:39:14 UTC

Size: 4.67 KB

patch

obsolete

>From e3c35ebed2b4d97d0ee53f25ea210c6208943033 Mon Sep 17 00:00:00 2001
>From: Peter Hutterer <peter.hutterer@who-t.net>
>Date: Mon, 12 Jan 2009 15:29:36 +1000
>Subject: [PATCH] dix: fix WarpPointer calls for devices with custom valuator ranges (#19297)
>
>If the MD's lastSlave was a devices with custom axes ranges, then a
>WarpPointer would position the cursor at the wrong location. A WarpPointer
>request provides screen coordinates and these coordinates were scaled to the
>device range before warping.
>
>This patch consists of two parts:
>1) in the WarpPointer handling, get the lastSlave and post the event through
>   this device.
>2) assume that WarpPointer coordinates are always in screen coordinates and
>   scale them to device coordinates in GPE before continuing. Note that this
>   breaks device-coordinate based XWarpDevicePointer calls (for which the spec
>   isn't nailed down yet anyway) until a better solution is found.
>
>X.Org Bug 19297 <http://bugs.freedesktop.org/show_bug.cgi?id=19297>
>
>Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
>---
> dix/events.c    |   19 ++++++++++++-------
> dix/getevents.c |   14 +++++++++++++-
> include/input.h |    1 +
> mi/mipointer.c  |    2 +-
> 4 files changed, 27 insertions(+), 9 deletions(-)
>
>diff --git a/dix/events.c b/dix/events.c
>index ba90ff1..3468a74 100644
>--- a/dix/events.c
>+++ b/dix/events.c
>@@ -3139,8 +3139,8 @@ ProcWarpPointer(ClientPtr client)
>     WindowPtr	dest = NULL;
>     int		x, y, rc;
>     ScreenPtr	newScreen;
>-    DeviceIntPtr dev = PickPointer(client);
>-    SpritePtr   pSprite = dev->spriteInfo->sprite;
>+    DeviceIntPtr dev;
>+    SpritePtr   pSprite;
> 
>     REQUEST(xWarpPointerReq);
>     REQUEST_SIZE_MATCH(xWarpPointerReq);
>@@ -3153,6 +3153,12 @@ ProcWarpPointer(ClientPtr client)
> 		return rc;
> 	}
>     }
>+
>+    dev = PickPointer(client);
>+    if (dev->u.lastSlave)
>+        dev = dev->u.lastSlave;
>+    pSprite = dev->spriteInfo->sprite;
>+
> #ifdef PANORAMIX
>     if(!noPanoramiXExtension)
> 	return XineramaWarpPointer(client);
>@@ -3219,13 +3225,12 @@ ProcWarpPointer(ClientPtr client)
> 	else if (y >= pSprite->physLimits.y2)
> 	    y = pSprite->physLimits.y2 - 1;
> 	if (pSprite->hotShape)
>-	    ConfineToShape(PickPointer(client), pSprite->hotShape, &x, &y);
>-        (*newScreen->SetCursorPosition)(PickPointer(client), newScreen, x, y,
>-                                        TRUE);
>+	    ConfineToShape(dev, pSprite->hotShape, &x, &y);
>+        (*newScreen->SetCursorPosition)(dev, newScreen, x, y, TRUE);
>     }
>-    else if (!PointerConfinedToScreen(PickPointer(client)))
>+    else if (!PointerConfinedToScreen(dev))
>     {
>-	NewCurrentScreen(PickPointer(client), newScreen, x, y);
>+	NewCurrentScreen(dev, newScreen, x, y);
>     }
>     return Success;
> }
>diff --git a/dix/getevents.c b/dix/getevents.c
>index acc8a4f..672ff7d 100644
>--- a/dix/getevents.c
>+++ b/dix/getevents.c
>@@ -999,8 +999,20 @@ GetPointerEvents(EventList *events, DeviceIntPtr pDev, int type, int buttons,
>     events = updateFromMaster(events, pDev, &num_events);
> 
>     if (flags & POINTER_ABSOLUTE)
>+    {
>+        if (flags & POINTER_SCREEN) /* valuators are in screen coords */
>+        {
>+
>+            valuators[0] = rescaleValuatorAxis(valuators[0], NULL,
>+                                               pDev->valuator->axes + 0,
>+                                               scr->width);
>+            valuators[1] = rescaleValuatorAxis(valuators[1], NULL,
>+                                               pDev->valuator->axes + 1,
>+                                               scr->height);
>+        }
>+
>         moveAbsolute(pDev, &x, &y, first_valuator, num_valuators, valuators);
>-    else {
>+    } else {
>         if (flags & POINTER_ACCELERATE)
>             accelPointer(pDev, first_valuator, num_valuators, valuators, ms);
>         moveRelative(pDev, &x, &y, first_valuator, num_valuators, valuators);
>diff --git a/include/input.h b/include/input.h
>index 3a9bfa2..2dd29f8 100644
>--- a/include/input.h
>+++ b/include/input.h
>@@ -62,6 +62,7 @@ SOFTWARE.
> #define POINTER_RELATIVE (1 << 1)
> #define POINTER_ABSOLUTE (1 << 2)
> #define POINTER_ACCELERATE (1 << 3)
>+#define POINTER_SCREEN (1 << 4) /* Data in screen coordinates */
> 
> /*int constants for pointer acceleration schemes*/
> #define PtrAccelNoOp            0
>diff --git a/mi/mipointer.c b/mi/mipointer.c
>index 83a355e..567790c 100644
>--- a/mi/mipointer.c
>+++ b/mi/mipointer.c
>@@ -576,7 +576,7 @@ miPointerMove (DeviceIntPtr pDev, ScreenPtr pScreen, int x, int y)
>         }
>     }
> 
>-    nevents = GetPointerEvents(events, pDev, MotionNotify, 0, POINTER_ABSOLUTE, 0, 2, valuators);
>+    nevents = GetPointerEvents(events, pDev, MotionNotify, 0, POINTER_SCREEN | POINTER_ABSOLUTE, 0, 2, valuators);
> 
>     OsBlockSignals();
> #ifdef XQUARTZ
>-- 
>1.6.0.6
>

Actions: View | Splinter Review

Attachments on bug 19297: 21496 | 21784 | 21835 | 21890

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.