elm_icon: avoid useless assignment
authorJean Guyomarc'h <jean@guyomarch.bzh>
Fri, 28 Oct 2016 21:23:15 +0000 (23:23 +0200)
committerJean Guyomarc'h <jean@guyomarch.bzh>
Fri, 28 Oct 2016 21:23:15 +0000 (23:23 +0200)
Assigning id->edje to EINA_TRUE in a code path triggered when id->edje
is EINA_FALSE instead of unconditionnaly setting it to EINA_TRUE avoids
to assign id->edje to itself.

src/lib/elementary/elm_icon.c

index 68c9e64..303608b 100644 (file)
@@ -344,9 +344,9 @@ _elm_icon_efl_file_file_set(Eo *obj, Elm_Icon_Data *sd, const char *file, const
         if (id->show)
           evas_object_show(id->img);
         evas_object_clip_set(id->img, pclip);
+        id->edje = EINA_TRUE;
      }
 
-   id->edje = EINA_TRUE;
    if (!edje_object_file_set(id->img, file, key))
      {
         ERR("failed to set edje file '%s', group '%s': %s", file, key,