diff --git a/include/xwayland/selection.h b/include/xwayland/selection.h index 3f00fd54..43d59054 100644 --- a/include/xwayland/selection.h +++ b/include/xwayland/selection.h @@ -77,6 +77,6 @@ bool primary_selection_source_is_xwayland( void xwm_seat_handle_start_drag(struct wlr_xwm *xwm, struct wlr_drag *drag); void xwm_selection_init(struct wlr_xwm *xwm); -void xwm_selection_finish(struct wlr_xwm *xwm); +void xwm_selection_finish(struct wlr_xwm_selection *selection); #endif diff --git a/xwayland/selection/selection.c b/xwayland/selection/selection.c index a6042582..64c957b7 100644 --- a/xwayland/selection/selection.c +++ b/xwayland/selection/selection.c @@ -5,7 +5,6 @@ #include #include #include -#include #include #include #include "xwayland/selection.h" @@ -245,53 +244,18 @@ void xwm_selection_init(struct wlr_xwm *xwm) { selection_init(xwm, &xwm->dnd_selection, xwm->atoms[DND_SELECTION]); } -void xwm_selection_finish(struct wlr_xwm *xwm) { - if (!xwm) { +void xwm_selection_finish(struct wlr_xwm_selection *selection) { + if (!selection) { return; } - struct wlr_xwm_selection *selections[] = { - &xwm->clipboard_selection, - &xwm->primary_selection, - &xwm->dnd_selection, - }; - - for (size_t i = 0; i < sizeof(selections)/sizeof(selections[0]); i++) { - struct wlr_xwm_selection *selection = selections[i]; - - struct wlr_xwm_selection_transfer *outgoing, *tmp; - wl_list_for_each_safe(outgoing, tmp, &selection->outgoing, outgoing_link) { - wlr_log(WLR_INFO, "destroyed pending transfer %ld/%p", i, outgoing); - xwm_selection_transfer_destroy_outgoing(outgoing); - } - - xwm_selection_transfer_finish(&selection->incoming); + struct wlr_xwm_selection_transfer *outgoing, *tmp; + wl_list_for_each_safe(outgoing, tmp, &selection->outgoing, outgoing_link) { + wlr_log(WLR_INFO, "destroyed pending transfer %p", outgoing); + xwm_selection_transfer_destroy_outgoing(outgoing); } - if (xwm->selection_window) { - xcb_destroy_window(xwm->xcb_conn, xwm->selection_window); - } - - if (xwm->dnd_window) { - xcb_destroy_window(xwm->xcb_conn, xwm->dnd_window); - } - - if (xwm->seat) { - if (xwm->seat->selection_source && - data_source_is_xwayland(xwm->seat->selection_source)) { - wlr_seat_set_selection(xwm->seat, NULL, - wl_display_next_serial(xwm->xwayland->wl_display)); - } - - if (xwm->seat->primary_selection_source && - primary_selection_source_is_xwayland( - xwm->seat->primary_selection_source)) { - wlr_seat_set_primary_selection(xwm->seat, NULL, - wl_display_next_serial(xwm->xwayland->wl_display)); - } - - wlr_xwayland_set_seat(xwm->xwayland, NULL); - } + xwm_selection_transfer_finish(&selection->incoming); } static void xwm_selection_set_owner(struct wlr_xwm_selection *selection, diff --git a/xwayland/xwm.c b/xwayland/xwm.c index 732882f7..99417d16 100644 --- a/xwayland/xwm.c +++ b/xwayland/xwm.c @@ -5,6 +5,8 @@ #include #include #include +#include +#include #include #include #include @@ -1581,7 +1583,36 @@ void xwm_destroy(struct wlr_xwm *xwm) { if (!xwm) { return; } - xwm_selection_finish(xwm); + + xwm_selection_finish(&xwm->clipboard_selection); + xwm_selection_finish(&xwm->primary_selection); + xwm_selection_finish(&xwm->dnd_selection); + + if (xwm->selection_window) { + xcb_destroy_window(xwm->xcb_conn, xwm->selection_window); + } + + if (xwm->dnd_window) { + xcb_destroy_window(xwm->xcb_conn, xwm->dnd_window); + } + + if (xwm->seat) { + if (xwm->seat->selection_source && + data_source_is_xwayland(xwm->seat->selection_source)) { + wlr_seat_set_selection(xwm->seat, NULL, + wl_display_next_serial(xwm->xwayland->wl_display)); + } + + if (xwm->seat->primary_selection_source && + primary_selection_source_is_xwayland( + xwm->seat->primary_selection_source)) { + wlr_seat_set_primary_selection(xwm->seat, NULL, + wl_display_next_serial(xwm->xwayland->wl_display)); + } + + wlr_xwayland_set_seat(xwm->xwayland, NULL); + } + if (xwm->cursor) { xcb_free_cursor(xwm->xcb_conn, xwm->cursor); }