From: shaheba.m <shaheba.m@shahebam-ubuntu.sisodomain.com>
Date: Fri, 5 Jul 2013 11:35:10 +0000 (+0530)
Subject: [quicklaunch, elm_gesture_layer, elm_plug, elm_theme] Resolved Issues reflected by... 
X-Git-Tag: submit/tizen_2.2/20130716.173137~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3ae76bddaa646d654aa8707db424c847804fce15;p=framework%2Fuifw%2Felementary.git

[quicklaunch, elm_gesture_layer, elm_plug, elm_theme] Resolved Issues reflected by Prevent tool

Change-Id: Ia30c1d42f40d1fd8a477716685b6751ad53a3e9e
---

diff --git a/src/bin/quicklaunch.c b/src/bin/quicklaunch.c
index 316918d..d6b8717 100644
--- a/src/bin/quicklaunch.c
+++ b/src/bin/quicklaunch.c
@@ -179,7 +179,13 @@ main(int argc, char **argv)
 
    if (!(disp = getenv("DISPLAY"))) disp = "unknown";
    snprintf(buf, sizeof(buf), "/tmp/elm-ql-%i", getuid());
-   if (stat(buf, &st) < 0) mkdir(buf, S_IRUSR | S_IWUSR | S_IXUSR);
+   if (stat(buf, &st) < 0)
+      if (mkdir(buf, S_IRUSR | S_IWUSR | S_IXUSR) < 0)
+        {
+           CRITICAL("cannot create directory '%s' : %s",
+                buf, strerror(errno));
+           exit(-1);
+        }
    snprintf(buf, sizeof(buf), "/tmp/elm-ql-%i/%s", getuid(), disp);
    unlink(buf);
    sock = socket(AF_UNIX, SOCK_STREAM, 0);
diff --git a/src/lib/elm_gesture_layer.c b/src/lib/elm_gesture_layer.c
index c21dad2..940b5be 100644
--- a/src/lib/elm_gesture_layer.c
+++ b/src/lib/elm_gesture_layer.c
@@ -3332,7 +3332,7 @@ _zoom_test(Evas_Object *obj,
         /* if non-continues mode and gesture NOT started, ignore MOVE */
         if ((!sd->glayer_continues_enable) &&
             (!st->zoom_st.timestamp))
-          return;
+          break;
 
       case EVAS_CALLBACK_MOUSE_DOWN:
       case EVAS_CALLBACK_MULTI_DOWN:
@@ -3349,7 +3349,7 @@ _zoom_test(Evas_Object *obj,
                  EINA_FALSE);
              _event_consume(sd, event_info, event_type, ev_flag);
 
-             return;
+             break;
           }
 
         if (!st->zoom_st.timestamp) /* Now scan touched-devices list
@@ -3362,7 +3362,7 @@ _zoom_test(Evas_Object *obj,
                }
 
              if (!p) /* Single finger on touch */
-               return;
+               break;
 
              /* Record down fingers */
              _event_consume(sd, event_info, event_type, ev_flag);
@@ -3393,7 +3393,7 @@ _zoom_test(Evas_Object *obj,
                   _event_consume(sd, event_info, event_type, ev_flag);
                }
 
-             return; /* Zoom started */
+             break; /* Zoom started */
           } /* End of ZOOM_START handling */
 
         /* if we got here, we have (exacally) two fingers on surfce */
@@ -3428,7 +3428,7 @@ _zoom_test(Evas_Object *obj,
                }
           } /* End of ZOOM_MOVE handling */
 
-        return;
+        break;
       }
 
       case EVAS_CALLBACK_MOUSE_UP:
@@ -3443,7 +3443,7 @@ _zoom_test(Evas_Object *obj,
                                   &st->info, EINA_FALSE);
              _event_consume(sd, event_info, event_type, ev_flag);
 
-             return;
+             break;
           }
 
         /* if we got here not a ZOOM */
@@ -3458,10 +3458,10 @@ _zoom_test(Evas_Object *obj,
 
         _zoom_test_reset(gesture_zoom);
 
-        return;
+        break;
 
       default:
-        return;
+        break;
      }
 }
 
@@ -3578,7 +3578,7 @@ _rotate_test(Evas_Object *obj,
         /* if non-continues mode and gesture NOT started, ignore MOVE */
         if ((!sd->glayer_continues_enable) &&
             (!st->rotate_st.timestamp))
-          return;
+          break;
 
       case EVAS_CALLBACK_MOUSE_DOWN:
       case EVAS_CALLBACK_MULTI_DOWN:
@@ -3594,7 +3594,7 @@ _rotate_test(Evas_Object *obj,
                  (gesture, ELM_GESTURE_STATE_ABORT, &st->info, EINA_FALSE);
              _event_consume(sd, event_info, event_type, ev_flag);
 
-             return;
+             break;
           }
 
         if (!st->rotate_st.timestamp) /* Now scan touched-devices list
@@ -3607,7 +3607,7 @@ _rotate_test(Evas_Object *obj,
                }
 
              if (!p)
-               return;  /* Single finger on touch */
+               break;  /* Single finger on touch */
 
              /* Record down fingers */
              _event_consume(sd, event_info, event_type, ev_flag);
@@ -3630,7 +3630,7 @@ _rotate_test(Evas_Object *obj,
                                   &st->info, EINA_FALSE);
              _event_consume(sd, event_info, event_type, ev_flag);
 
-             return; /* Rotate started */
+             break; /* Rotate started */
           } /* End of ROTATE_START handling */
 
         /* if we got here, we have (exacally) two fingers on surfce */
@@ -3665,7 +3665,7 @@ _rotate_test(Evas_Object *obj,
                }
           } /* End of ROTATE_MOVE handling */
 
-        return;
+        break;
       }
 
       case EVAS_CALLBACK_MOUSE_UP:
@@ -3679,7 +3679,7 @@ _rotate_test(Evas_Object *obj,
                                   &st->info, EINA_FALSE);
              _event_consume(sd, event_info, event_type, ev_flag);
 
-             return;
+             break;
           }
 
         /* Must be != undefined, if gesture started */
@@ -3691,10 +3691,10 @@ _rotate_test(Evas_Object *obj,
           }
 
         _rotate_test_reset(gesture);
-        return;
+        break;
 
       default:
-        return;
+        break;
      }
 }
 
diff --git a/src/lib/elm_plug.c b/src/lib/elm_plug.c
index d2dfc5f..860df87 100644
--- a/src/lib/elm_plug.c
+++ b/src/lib/elm_plug.c
@@ -187,9 +187,6 @@ _elm_plug_smart_activate(Evas_Object *obj, Elm_Activate act)
        case ELM_ACTIVATE_DOWN:
           msg_id = ELM_ACCESS_ACTION_DOWN;
           break;
-
-       default:
-          break;
      }
 
    if (msg_id == ELM_ACCESS_ACTION_FIRST) return EINA_FALSE;