From a69dc01a6e585a5346323022c6d5e920d800979d Mon Sep 17 00:00:00 2001 From: Neil Roberts Date: Wed, 23 Apr 2014 18:02:47 +0100 Subject: [PATCH] clients/window: Don't remove the touch listener on a frame event It looks like the handler for frame events from the wl_touch interface for widgets may have been erroneously copied from the cancel handler so that it removes all handlers as they are processed. I don't think this makes much sense for the frame event. This was stopping the panel icons from being pushable with touch events when using libinput since commit 1679f232e541489e. All that commit does it make it start sending the frame events. --- clients/window.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/clients/window.c b/clients/window.c index e770a040..e2f70107 100644 --- a/clients/window.c +++ b/clients/window.c @@ -3065,9 +3065,6 @@ touch_handle_frame(void *data, struct wl_touch *wl_touch) if (tp->widget->touch_frame_handler) (*tp->widget->touch_frame_handler)(tp->widget, input, tp->widget->user_data); - - wl_list_remove(&tp->link); - free(tp); } }