From 451a93d9d2a03aebc4d6ae9ac4a31ae6432913b7 Mon Sep 17 00:00:00 2001 From: Mike Blumenkrantz Date: Mon, 24 Jul 2017 08:28:39 -0400 Subject: [PATCH] Revert "ecore wl2 - dnd - handle NULL drags... shouldnt happen but does" This reverts commit 85e585846600742a7f395751816f348b4152269f. please do not randomly add null checks for the purpose of quickly closing tickets. --- src/lib/ecore_wl2/ecore_wl2_dnd.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/src/lib/ecore_wl2/ecore_wl2_dnd.c b/src/lib/ecore_wl2/ecore_wl2_dnd.c index ae6ee5743b..cd04a12883 100644 --- a/src/lib/ecore_wl2/ecore_wl2_dnd.c +++ b/src/lib/ecore_wl2/ecore_wl2_dnd.c @@ -316,16 +316,14 @@ _ecore_wl2_dnd_leave(Ecore_Wl2_Input *input) if (input->focus.keyboard) ev->source = input->focus.keyboard->id; - if (input->drag) - ev->win = input->drag->window_id; + ev->win = input->drag->window_id; ev->offer = input->drag; ev->offer->ref++; ev->seat = input->id; ev->display = input->display; ev->display->refs++; - if (input->drag) - input->drag->window_id = 0; + input->drag->window_id = 0; ecore_event_add(ECORE_WL2_EVENT_DND_LEAVE, ev, _delay_offer_destroy, ev->offer); input->drag = NULL; } @@ -341,14 +339,12 @@ _ecore_wl2_dnd_motion(Ecore_Wl2_Input *input, int x, int y, uint32_t serial) ev = calloc(1, sizeof(Ecore_Wl2_Event_Dnd_Motion)); if (!ev) return; - if (input->drag) - input->drag->serial = serial; + input->drag->serial = serial; if (input->focus.keyboard) ev->source = input->focus.keyboard->id; - if (input->drag) - ev->win = input->drag->window_id; + ev->win = input->drag->window_id; ev->x = x; ev->y = y; ev->offer = input->drag; @@ -370,8 +366,7 @@ _ecore_wl2_dnd_drop(Ecore_Wl2_Input *input) if (input->focus.keyboard) ev->source = input->focus.keyboard->id; - if (input->drag) - ev->win = input->drag->window_id; + ev->win = input->drag->window_id; ev->x = input->pointer.sx; ev->y = input->pointer.sy; ev->offer = input->drag;