On Thu, Sep 22, 2011 at 11:25:33AM +0100, Daniel Stone wrote:
> On Fri, Sep 09, 2011 at 02:20:47PM +1000, Peter Hutterer wrote:
> > Switching screens relies on rootx/y to be set to the correct value. Note:
> > though we technically take a mask for GetKeyboardEvents we don't actually
> > handle it properly to move the pointer as required (and generate motion
> > events if needed).
> > 
> > Signed-off-by: Peter Hutterer <[email protected]>
> 
> Reviewed-by: Daniel Stone <[email protected]>
> 
> but ...
> 
> > @@ -953,6 +962,11 @@ GetKeyboardEvents(InternalEvent *events, DeviceIntPtr 
> > pDev, int type,
> >  
> >      set_valuators(pDev, event, &mask);
> >  
> > +    if (!IsFloating(pDev)) {
> > +            DeviceIntPtr master = GetMaster(pDev, MASTER_POINTER);
> > +            event_set_root_coordinates(event, master->last.valuators[0], 
> > master->last.valuators[1]);
> > +    }
> > +
> 
> ... a very long line again though.

amended locally, thanks.

Cheers,
  Peter
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to