edje: Edje_Edit - add missed iterator and object deletion in _delete_play_actions...
authorKateryna Fesyna <k.fesyna@samsung.com>
Mon, 23 Jun 2014 09:18:59 +0000 (11:18 +0200)
committerCedric BAIL <c.bail@partner.samsung.com>
Mon, 23 Jun 2014 09:33:24 +0000 (11:33 +0200)
Summary:
this commit adds missed eina_iterator_free() and evas_object_del() calls
in _delete_play_actions() functions.

@feature

Reviewers: cedric, Hermet, seoz, raster, reutskiy.v.v, tasn

Reviewed By: reutskiy.v.v

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D1059

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>
src/lib/edje/edje_edit.c

index aa84165..a908b32 100644 (file)
@@ -971,11 +971,17 @@ _delete_play_actions(Evas_Object *obj, const char* name, int action_type, Eet_Fi
           return EINA_FALSE;
 
         if (!edje_object_file_set(eeo, ed->file->path, pce->entry))
-          continue;
+          {
+             evas_object_del(eeo);
+             continue;
+          }
 
         programs_list = edje_edit_programs_list_get(eeo);
         if (!programs_list)
-          continue;
+          {
+             evas_object_del(eeo);
+             continue;
+          }
 
         eed = eo_data_scope_get(eeo, EDJE_CLASS);
         for (i = 0; i < eed->collection->patterns.table_programs_size; i++)
@@ -1008,7 +1014,10 @@ _delete_play_actions(Evas_Object *obj, const char* name, int action_type, Eet_Fi
           }
         if (is_collection_changed)
           _edje_edit_collection_save(eetf, eed->collection);
+        evas_object_del(eeo);
      }
+
+   eina_iterator_free(it);
    return EINA_TRUE;
 }