Remove unused variable and don't create icons if there's no icon set
authorIván Briano <sachieru@gmail.com>
Mon, 25 Jan 2010 19:56:45 +0000 (19:56 +0000)
committerIván Briano <sachieru@gmail.com>
Mon, 25 Jan 2010 19:56:45 +0000 (19:56 +0000)
SVN revision: 45564

src/edje_externals/elm.c
src/edje_externals/elm_button.c

index cc26583..df06c61 100644 (file)
@@ -35,7 +35,7 @@ external_common_icon_param_parse(Evas_Object **icon, Evas_Object *obj, const Ein
    Edje_External_Param *p;
 
    p = edje_external_param_find(params, "icon");
-   if (p)
+   if (p && p->s)
      {
        Evas_Object *parent = evas_object_smart_parent_get(obj);
        const char *file;
@@ -48,8 +48,8 @@ external_common_icon_param_parse(Evas_Object **icon, Evas_Object *obj, const Ein
          return;
 
        evas_object_del(*icon);
-       *icon = NULL;
      }
+   *icon = NULL;
 }
 
 void
index ad3514f..1edacc8 100644 (file)
@@ -29,7 +29,6 @@ static void *
 external_button_params_parse(void *data, Evas_Object *obj, const Eina_List *params)
 {
    Elm_Params_Button *mem;
-   Edje_External_Param *param;
 
    mem = external_common_params_parse(Elm_Params_Button, data, obj, params);
    if (!mem)