diff options
| author | Derek Foreman <derekf@osg.samsung.com> | 2015-07-15 13:00:38 -0500 |
|---|---|---|
| committer | Bryce Harrington <bryce@osg.samsung.com> | 2015-07-16 19:04:04 -0700 |
| commit | 794fa0e0311d7983fb4d2128e942df3c974adfe7 (patch) | |
| tree | 1110bfd29ad229f3c423978fab8b8168e26e35a4 /desktop-shell | |
| parent | b7674ae4dcd35ec68d1a66730e487059d0091a7f (diff) | |
desktop-shell: Make surface_move take a pointer instead of a seat
It doesn't actually need the seat and we have to validate that the seat
has a pointer before making the call, so it's safer just to pass
the validated pointer.
Reviewed-by: Jonas Ådahl <jadahl@gmail.com>
Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Diffstat (limited to 'desktop-shell')
| -rw-r--r-- | desktop-shell/shell.c | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/desktop-shell/shell.c b/desktop-shell/shell.c index 5ae395da..30aa5604 100644 --- a/desktop-shell/shell.c +++ b/desktop-shell/shell.c @@ -1748,7 +1748,7 @@ static const struct weston_pointer_grab_interface move_grab_interface = { }; static int -surface_move(struct shell_surface *shsurf, struct weston_seat *seat, +surface_move(struct shell_surface *shsurf, struct weston_pointer *pointer, bool client_initiated) { struct weston_move_grab *move; @@ -1765,13 +1765,13 @@ surface_move(struct shell_surface *shsurf, struct weston_seat *seat, return -1; move->dx = wl_fixed_from_double(shsurf->view->geometry.x) - - seat->pointer->grab_x; + pointer->grab_x; move->dy = wl_fixed_from_double(shsurf->view->geometry.y) - - seat->pointer->grab_y; + pointer->grab_y; move->client_initiated = client_initiated; shell_grab_start(&move->base, &move_grab_interface, shsurf, - seat->pointer, DESKTOP_SHELL_CURSOR_MOVE); + pointer, DESKTOP_SHELL_CURSOR_MOVE); return 0; } @@ -1790,7 +1790,7 @@ common_surface_move(struct wl_resource *resource, seat->pointer->grab_serial == serial) { surface = weston_surface_get_main_surface(seat->pointer->focus->surface); if ((surface == shsurf->surface) && - (surface_move(shsurf, seat, true) < 0)) + (surface_move(shsurf, seat->pointer, true) < 0)) wl_resource_post_no_memory(resource); } else if (seat->touch && seat->touch->focus && @@ -2041,11 +2041,12 @@ busy_cursor_grab_button(struct weston_pointer_grab *base, { struct shell_grab *grab = (struct shell_grab *) base; struct shell_surface *shsurf = grab->shsurf; - struct weston_seat *seat = grab->grab.pointer->seat; + struct weston_pointer *pointer = grab->grab.pointer; + struct weston_seat *seat = pointer->seat; if (shsurf && button == BTN_LEFT && state) { activate(shsurf->shell, shsurf->surface, seat, true); - surface_move(shsurf, seat, false); + surface_move(shsurf, pointer, false); } else if (shsurf && button == BTN_RIGHT && state) { activate(shsurf->shell, shsurf->surface, seat, true); surface_rotate(shsurf, seat); @@ -3059,7 +3060,7 @@ shell_interface_set_maximized(struct shell_surface *shsurf) static int shell_interface_move(struct shell_surface *shsurf, struct weston_seat *ws) { - return surface_move(shsurf, ws, true); + return surface_move(shsurf, ws->pointer, true); } static const struct weston_pointer_grab_interface popup_grab_interface; @@ -4609,7 +4610,7 @@ move_binding(struct weston_pointer *pointer, uint32_t time, shsurf->state.maximized) return; - surface_move(shsurf, pointer->seat, false); + surface_move(shsurf, pointer, false); } static void |