[PATCH] dix: don't update the slave coordinates from the VCK.

Keith Packard keithp at keithp.com
Mon Jan 4 20:12:22 PST 2010


On Tue, 5 Jan 2010 13:46:07 +1000, Peter Hutterer <peter.hutterer at who-t.net> wrote:

> @@ -630,9 +630,12 @@ updateFromMaster(EventListPtr events, DeviceIntPtr dev, int type, int *num_event
>      if (master && master->last.slave != dev)
>      {
>          CreateClassesChangedEvent(events, master, dev, type);
> -        updateSlaveDeviceCoords(master, dev);
> -        master->last.slave = dev;
> -        master->last.numValuators = dev->last.numValuators;
> +        if (IsPointerDevice(master))
> +        {
> +            updateSlaveDeviceCoords(master, dev);
> +            master->last.slave = dev;
> +            master->last.numValuators = dev->last.numValuators;
> +        }

Are you sure you want to leave the last.slave entry unchanged for
non-pointer devices here? If so, you'll generate a stream of DCCE
events, right?

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-devel/attachments/20100104/323e4993/attachment.pgp 


More information about the xorg-devel mailing list