Re: [PATCH weston] compositor: Support notifying with absolute position too

2013-04-01 Thread Daniel Stone
-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org > [mailto:wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org] > On Behalf Of Bill Spitzak > Sent: Wednesday, March 27, 2013 12:24 PM > To: Bradford, Robert > Cc: wayland-devel@lists.freedesktop.org > Subject: Re: [PAT

RE: [PATCH weston] compositor: Support notifying with absolute position too

2013-04-01 Thread Eoff, Ullysses A
esktop.org [mailto:wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Bill Spitzak Sent: Wednesday, March 27, 2013 12:24 PM To: Bradford, Robert Cc: wayland-devel@lists.freedesktop.org Subject: Re: [PATCH weston] compositor: Support notifying with absolute position too

Re: [PATCH weston] compositor: Support notifying with absolute position too

2013-03-27 Thread Bill Spitzak
Looks like there is a lot of duplicated code between notify_motion and notify_motion_absolute. It might be better to have notify_motion just add pointer->x and pointer->y and call notify_motion_absolute, and put all the shared logic, and possibly all of move_pointer(), into that. Rob Bradfor