[PATCH weston] compositor-x11: send pointer frame after axis event

Bryce Harrington bryce at osg.samsung.com
Tue Feb 9 21:03:24 UTC 2016


On Mon, Feb 08, 2016 at 09:42:23AM +1000, Peter Hutterer wrote:
> On Wed, Feb 03, 2016 at 02:03:00PM +0100, Marek Chalupa wrote:
> > clients that implement pointer interface of version 5
> > wait for the frame event, so without it the scrolling
> > does not work (GTK+ clients do not scroll now for example).
> > Xcb axis events are discrete, so it's fine to send
> > frame after every single axis event
> > 
> > Signed-off-by: Marek Chalupa <mchqwerty at gmail.com>
> 
> Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Thanks, pushed for the rc:

To ssh://git.freedesktop.org/git/wayland/weston
   4e18448..345b4f5  master -> master

 
> Cheers,
>    Peter
> 
> > ---
> >  src/compositor-x11.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/src/compositor-x11.c b/src/compositor-x11.c
> > index b70c119..13a5d73 100644
> > --- a/src/compositor-x11.c
> > +++ b/src/compositor-x11.c
> > @@ -1092,6 +1092,7 @@ x11_backend_deliver_button_event(struct x11_backend *b,
> >  			notify_axis(&b->core_seat,
> >  				    weston_compositor_get_time(),
> >  				    &weston_event);
> > +			notify_pointer_frame(&b->core_seat);
> >  		}
> >  		return;
> >  	case 5:
> > @@ -1104,6 +1105,7 @@ x11_backend_deliver_button_event(struct x11_backend *b,
> >  			notify_axis(&b->core_seat,
> >  				    weston_compositor_get_time(),
> >  				    &weston_event);
> > +			notify_pointer_frame(&b->core_seat);
> >  		}
> >  		return;
> >  	case 6:
> > @@ -1116,6 +1118,7 @@ x11_backend_deliver_button_event(struct x11_backend *b,
> >  			notify_axis(&b->core_seat,
> >  				    weston_compositor_get_time(),
> >  				    &weston_event);
> > +			notify_pointer_frame(&b->core_seat);
> >  		}
> >  		return;
> >  	case 7:
> > @@ -1128,6 +1131,7 @@ x11_backend_deliver_button_event(struct x11_backend *b,
> >  			notify_axis(&b->core_seat,
> >  				    weston_compositor_get_time(),
> >  				    &weston_event);
> > +			notify_pointer_frame(&b->core_seat);
> >  		}
> >  		return;
> >  	default:
> > -- 
> > 2.5.0
>  
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list