desktop-shell: Make surface_touch_move take a touch instead of a seat

It never actually needs the seat, and we always verify the touch pointer
before calling it, so let's just pass a touch pointer instead of having
an assumption that the seat's touch pointer has been verified.

Reviewed-by: Jonas Ådahl <jadahl@gmail.com>
Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
dev
Derek Foreman 9 years ago committed by Bryce Harrington
parent 8ae2db5b0c
commit b7674ae4dc
  1. 12
      desktop-shell/shell.c

@ -1634,7 +1634,7 @@ static const struct weston_touch_grab_interface touch_move_grab_interface = {
}; };
static int static int
surface_touch_move(struct shell_surface *shsurf, struct weston_seat *seat) surface_touch_move(struct shell_surface *shsurf, struct weston_touch *touch)
{ {
struct weston_touch_move_grab *move; struct weston_touch_move_grab *move;
@ -1650,12 +1650,12 @@ surface_touch_move(struct shell_surface *shsurf, struct weston_seat *seat)
move->active = 1; move->active = 1;
move->dx = wl_fixed_from_double(shsurf->view->geometry.x) - move->dx = wl_fixed_from_double(shsurf->view->geometry.x) -
seat->touch->grab_x; touch->grab_x;
move->dy = wl_fixed_from_double(shsurf->view->geometry.y) - move->dy = wl_fixed_from_double(shsurf->view->geometry.y) -
seat->touch->grab_y; touch->grab_y;
shell_touch_grab_start(&move->base, &touch_move_grab_interface, shsurf, shell_touch_grab_start(&move->base, &touch_move_grab_interface, shsurf,
seat->touch); touch);
return 0; return 0;
} }
@ -1797,7 +1797,7 @@ common_surface_move(struct wl_resource *resource,
seat->touch->grab_serial == serial) { seat->touch->grab_serial == serial) {
surface = weston_surface_get_main_surface(seat->touch->focus->surface); surface = weston_surface_get_main_surface(seat->touch->focus->surface);
if ((surface == shsurf->surface) && if ((surface == shsurf->surface) &&
(surface_touch_move(shsurf, seat) < 0)) (surface_touch_move(shsurf, seat->touch) < 0))
wl_resource_post_no_memory(resource); wl_resource_post_no_memory(resource);
} }
} }
@ -4681,7 +4681,7 @@ touch_move_binding(struct weston_touch *touch, uint32_t time, void *data)
shsurf->state.maximized) shsurf->state.maximized)
return; return;
surface_touch_move(shsurf, touch->seat); surface_touch_move(shsurf, touch);
} }
static void static void

Loading…
Cancel
Save