From d8075c3a709112a50d3449603840898f3a12433d Mon Sep 17 00:00:00 2001 From: lucas Date: Thu, 26 Aug 2010 20:45:09 +0000 Subject: [PATCH] Apply double_condition_check.cocci The offending projects were: E16/e/src/backgrounds.c | 10 ++++------ PROTO/eon/src/lib/layout/eon_stack.c | 4 +--- ecore/src/lib/ecore_win32/ecore_win32.c | 3 +-- ecore/src/lib/ecore_wince/ecore_wince.c | 3 +-- edje/src/lib/edje_edit.c | 3 +-- evas/src/lib/cache/evas_cache_image.c | 2 +- exalt/src/lib/libexalt_private.c | 2 +- This patch assumes code in these places were insane and the fix is to remove one condition check. Most likely this is not true, but there's no automatic fix for that. Looking at the patch, it seems that some places should use "x" and "y" vars but used just one of them and therefore they were caught by coccinelle. git-svn-id: http://svn.enlightenment.org/svn/e/trunk/edje@51666 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33 --- src/lib/edje_edit.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/lib/edje_edit.c b/src/lib/edje_edit.c index 04b4257..849ed00 100644 --- a/src/lib/edje_edit.c +++ b/src/lib/edje_edit.c @@ -5889,8 +5889,7 @@ _edje_generate_source_of_part(Evas_Object *obj, Edje_Part *ep, Eina_Strbuf *buf) effects[edje_edit_part_effect_get(obj, part)]); //Dragable - if (edje_edit_part_drag_x_get(obj, part) || - edje_edit_part_drag_x_get(obj, part)) + if (edje_edit_part_drag_x_get(obj, part)) { BUF_APPEND(I4"dragable {\n"); BUF_APPENDF(I5"x: %d %d %d;\n", -- 2.7.4