Apply double_condition_check.cocci
authorlucas <lucas@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Thu, 26 Aug 2010 20:45:09 +0000 (20:45 +0000)
committerlucas <lucas@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Thu, 26 Aug 2010 20:45:09 +0000 (20:45 +0000)
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: svn+ssh://svn.enlightenment.org/var/svn/e/trunk/edje@51666 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33

src/lib/edje_edit.c

index 04b4257..849ed00 100644 (file)
@@ -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",