[PATCH 2/6] dix: drop unused argument from XISendDeviceChangedEvent

Jeremy Huddleston jeremyhu at apple.com
Fri Oct 21 16:18:54 PDT 2011


I'm confused by the comment, "and just using master" ... because the prototype changes:

> -XISendDeviceChangedEvent(DeviceIntPtr device, DeviceIntPtr master, DeviceChangedEvent *dce)
> +XISendDeviceChangedEvent(DeviceIntPtr device, DeviceChangedEvent *dce)

which seems like you were just using device...

On Oct 19, 2011, at 11:25 PM, Peter Hutterer wrote:

> Instead of device and master (and just using master), drop the master
> argument and let the callers pass in the device the event is to be sent for.
> 
> No effective functional changes.
> 
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
> Xi/exevents.c      |    6 +++---
> dix/devices.c      |    2 +-
> include/exevents.h |    3 +--
> 3 files changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/Xi/exevents.c b/Xi/exevents.c
> index 74a78ec..2ae5a62 100644
> --- a/Xi/exevents.c
> +++ b/Xi/exevents.c
> @@ -651,7 +651,7 @@ DeepCopyDeviceClasses(DeviceIntPtr from, DeviceIntPtr to, DeviceChangedEvent *dc
>  * Send an XI2 DeviceChangedEvent to all interested clients.
>  */
> void
> -XISendDeviceChangedEvent(DeviceIntPtr device, DeviceIntPtr master, DeviceChangedEvent *dce)
> +XISendDeviceChangedEvent(DeviceIntPtr device, DeviceChangedEvent *dce)
> {
>     xXIDeviceChangedEvent *dcce;
>     int rc;
> @@ -665,7 +665,7 @@ XISendDeviceChangedEvent(DeviceIntPtr device, DeviceIntPtr master, DeviceChanged
> 
>     /* we don't actually swap if there's a NullClient, swapping is done
>      * later when event is delivered. */
> -    SendEventToAllWindows(master, XI_DeviceChangedMask, (xEvent*)dcce, 1);
> +    SendEventToAllWindows(device, XI_DeviceChangedMask, (xEvent*)dcce, 1);
>     free(dcce);
> }
> 
> @@ -699,7 +699,7 @@ ChangeMasterDeviceClasses(DeviceIntPtr device, DeviceChangedEvent *dce)
> 
>     /* FIXME: the classes may have changed since we generated the event. */
>     DeepCopyDeviceClasses(slave, device, dce);
> -    XISendDeviceChangedEvent(slave, device, dce);
> +    XISendDeviceChangedEvent(device, dce);
> }
> 
> /**
> diff --git a/dix/devices.c b/dix/devices.c
> index 64557aa..7c196e0 100644
> --- a/dix/devices.c
> +++ b/dix/devices.c
> @@ -2360,7 +2360,7 @@ RecalculateMasterButtons(DeviceIntPtr slave)
>             event.keys.max_keycode = master->key->xkbInfo->desc->max_key_code;
>         }
> 
> -        XISendDeviceChangedEvent(master, master, &event);
> +        XISendDeviceChangedEvent(master, &event);
>     }
> }
> 
> diff --git a/include/exevents.h b/include/exevents.h
> index 4fe6c61..720fb2e 100644
> --- a/include/exevents.h
> +++ b/include/exevents.h
> @@ -322,8 +322,7 @@ extern int
> XIShouldNotify(ClientPtr client, DeviceIntPtr dev);
> 
> extern void
> -XISendDeviceChangedEvent(DeviceIntPtr device, DeviceIntPtr master,
> -                         DeviceChangedEvent *dce);
> +XISendDeviceChangedEvent(DeviceIntPtr device, DeviceChangedEvent *dce);
> 
> extern int
> XISetEventMask(DeviceIntPtr dev, WindowPtr win, ClientPtr client,
> -- 
> 1.7.6.4
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 



More information about the xorg-devel mailing list