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: http://svn.enlightenment.org/svn/e/trunk/ecore@51666 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33

src/lib/ecore_win32/ecore_win32.c
src/lib/ecore_wince/ecore_wince.c

index fc3686a..4eccf38 100644 (file)
@@ -362,8 +362,7 @@ _ecore_win32_window_procedure(HWND   window,
        _ecore_win32_event_handle_destroy_notify(data);
        return 0;
      case WM_SHOWWINDOW:
-       if ((data->data_param == SW_OTHERUNZOOM) ||
-           (data->data_param == SW_OTHERUNZOOM))
+       if ((data->data_param == SW_OTHERUNZOOM))
          return 0;
 
        if (data->window_param)
index a663823..6a21450 100644 (file)
@@ -311,8 +311,7 @@ _ecore_wince_window_procedure(HWND   window,
        _ecore_wince_event_handle_destroy_notify(data);
        break;
      case WM_SHOWWINDOW:
-       if ((data->data_param == SW_OTHERUNZOOM) ||
-           (data->data_param == SW_OTHERUNZOOM))
+       if ((data->data_param == SW_OTHERUNZOOM))
          break;
 
        if (data->window_param)