From 5fc4b971dc7c01b6d76322cfb90e254b57c717c5 Mon Sep 17 00:00:00 2001 From: Chris Michael Date: Mon, 11 Jul 2016 11:56:17 -0400 Subject: [PATCH] ecore-wl2: Fix issue of misleading 'if' statement Gcc reports that this 'if' statement is misleading due to indentation. We should only be sending the 'finish' call if the dnd version supports it, however we should always be sending 'destroy' @fix Signed-off-by: Chris Michael --- src/lib/ecore_wl2/ecore_wl2_dnd.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/lib/ecore_wl2/ecore_wl2_dnd.c b/src/lib/ecore_wl2/ecore_wl2_dnd.c index 827bc55..7e17d86 100644 --- a/src/lib/ecore_wl2/ecore_wl2_dnd.c +++ b/src/lib/ecore_wl2/ecore_wl2_dnd.c @@ -244,9 +244,11 @@ _selection_data_read(void *data, Ecore_Fd_Handler *fdh) { if (source->input->display->wl.data_device_manager_version >= WL_DATA_OFFER_FINISH_SINCE_VERSION) - wl_data_offer_finish(source->offer); - wl_data_offer_destroy(source->offer); - source->offer = NULL; + { + wl_data_offer_finish(source->offer); + } + wl_data_offer_destroy(source->offer); + source->offer = NULL; } fd = ecore_main_fd_handler_fd_get(source->fdh); -- 2.7.4