[PATCH 06/11] xfree86/dga: DGA2 events are missing the dx/dy information

Fernando Carrijo fcarrijo at freedesktop.org
Sun Jan 23 15:30:55 PST 2011


Ville Syrjala <syrjala at sci.fi> wrote:

> Copy dx/dy from the internal event to the DGA2 Motion/Button events.
> Do the same for Key events for the sake of keeping the code consistent.
> 
> Signed-off-by: Ville Syrjala <syrjala at sci.fi>
> ---
>  hw/xfree86/common/xf86DGA.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/hw/xfree86/common/xf86DGA.c b/hw/xfree86/common/xf86DGA.c
> index 9d0c621..c11b6c3 100644
> --- a/hw/xfree86/common/xf86DGA.c
> +++ b/hw/xfree86/common/xf86DGA.c
> @@ -1064,8 +1064,8 @@ DGAProcessKeyboardEvent (ScreenPtr pScreen, DGAEvent *event, DeviceIntPtr keybd)
>          de.u.u.type = *XDGAEventBase + GetCoreType((InternalEvent*)&ev);
>          de.u.u.detail = event->detail;
>          de.u.event.time = event->time;
> -        de.u.event.dx = 0;
> -        de.u.event.dy = 0;
> +        de.u.event.dx = event->dx;
> +        de.u.event.dy = event->dx;

The above assignment looks pretty suspicious to me.

>          de.u.event.screen = pScreen->myNum;
>          de.u.event.state = ev.corestate;
>  
> @@ -1120,8 +1120,8 @@ DGAProcessPointerEvent (ScreenPtr pScreen, DGAEvent *event, DeviceIntPtr mouse)
>          de.u.u.type = *XDGAEventBase + coreEquiv;
>          de.u.u.detail = event->detail;
>          de.u.event.time = event->time;
> -        de.u.event.dx = 0;
> -        de.u.event.dy = 0;
> +        de.u.event.dx = event->dx;
> +        de.u.event.dy = event->dx;
>          de.u.event.screen = pScreen->myNum;
>          de.u.event.state = ev.corestate;
>  
> -- 
> 1.7.3.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