diff options
author | Sergio Gómez <sergio.g.delreal@gmail.com> | 2023-03-08 17:36:34 -0500 |
---|---|---|
committer | Marius Vlad <marius.vlad@collabora.com> | 2023-05-16 11:25:25 +0300 |
commit | d3a636dfb8991b194a3ae1df99179032173d3883 (patch) | |
tree | 7d56e5e9f2acd9029c6db242991a2f42ac182943 | |
parent | 6b539a7c78a66df117e233ccda86242e2cd4533d (diff) | |
download | weston-d3a636dfb8991b194a3ae1df99179032173d3883.tar.gz |
libweston/input: Remove redundant surface destroy listener in constraints
Currently, the surface destroy listener in pointer constraints is redundant,
since surface destruction already handles pointer constraints destruction (see
libweston/compositor.c:weston_surface_unref()).
Signed-off-by: Sergio Gómez <sergio.g.delreal@gmail.com>
(cherry picked from commit 64da736d37a7df8b3bd6fd43746ac513bae72748)
-rw-r--r-- | include/libweston/libweston.h | 1 | ||||
-rw-r--r-- | libweston/input.c | 17 |
2 files changed, 0 insertions, 18 deletions
diff --git a/include/libweston/libweston.h b/include/libweston/libweston.h index d99dc763..998cf9a7 100644 --- a/include/libweston/libweston.h +++ b/include/libweston/libweston.h @@ -1441,7 +1441,6 @@ struct weston_pointer_constraint { bool hint_is_pending; struct wl_listener pointer_destroy_listener; - struct wl_listener surface_destroy_listener; struct wl_listener surface_commit_listener; struct wl_listener surface_activate_listener; }; diff --git a/libweston/input.c b/libweston/input.c index 6fb4bed3..eb9bea44 100644 --- a/libweston/input.c +++ b/libweston/input.c @@ -3655,8 +3655,6 @@ enable_pointer_constraint(struct weston_pointer_constraint *constraint, constraint->view = view; pointer_constraint_notify_activated(constraint); weston_pointer_start_grab(constraint->pointer, &constraint->grab); - wl_list_remove(&constraint->surface_destroy_listener.link); - wl_list_init(&constraint->surface_destroy_listener.link); } static bool @@ -3680,7 +3678,6 @@ weston_pointer_constraint_destroy(struct weston_pointer_constraint *constraint) weston_pointer_constraint_disable(constraint); wl_list_remove(&constraint->pointer_destroy_listener.link); - wl_list_remove(&constraint->surface_destroy_listener.link); wl_list_remove(&constraint->surface_commit_listener.link); wl_list_remove(&constraint->surface_activate_listener.link); @@ -3877,16 +3874,6 @@ pointer_constraint_pointer_destroyed(struct wl_listener *listener, void *data) } static void -pointer_constraint_surface_destroyed(struct wl_listener *listener, void *data) -{ - struct weston_pointer_constraint *constraint = - container_of(listener, struct weston_pointer_constraint, - surface_destroy_listener); - - weston_pointer_constraint_destroy(constraint); -} - -static void pointer_constraint_surface_committed(struct wl_listener *listener, void *data) { struct weston_pointer_constraint *constraint = @@ -3947,8 +3934,6 @@ weston_pointer_constraint_create(struct weston_surface *surface, constraint->surface_activate_listener.notify = pointer_constraint_surface_activate; - constraint->surface_destroy_listener.notify = - pointer_constraint_surface_destroyed; constraint->surface_commit_listener.notify = pointer_constraint_surface_committed; constraint->pointer_destroy_listener.notify = @@ -3958,8 +3943,6 @@ weston_pointer_constraint_create(struct weston_surface *surface, &constraint->surface_activate_listener); wl_signal_add(&pointer->destroy_signal, &constraint->pointer_destroy_listener); - wl_signal_add(&surface->destroy_signal, - &constraint->surface_destroy_listener); wl_signal_add(&surface->commit_signal, &constraint->surface_commit_listener); |