list: added comments about an unused function.
authorDaniel Juyung Seo <seojuyung2@gmail.com>
Sat, 10 May 2014 14:41:10 +0000 (23:41 +0900)
committerDaniel Juyung Seo <seojuyung2@gmail.com>
Sat, 10 May 2014 14:44:28 +0000 (23:44 +0900)
Fixing this issue is not trivial and will bring another issues like crashing.
So it is better to fix this in a development phase by refactoring list.

To fix this issue, the following are needed:
1. it->walking concept should be adopted instead of using just sd->walking.
   sd->walking was introduced in beb418d6
2. elm_widget_item_del() should be called instead of the combination of
   _elm_list_item_free() + elm_widget_item_free()
   This was introduced in f343011d

src/lib/elm_list.c

index dedc4a1..0098d1c 100644 (file)
@@ -1869,6 +1869,12 @@ _item_text_get_hook(const Elm_Object_Item *it,
    return ((Elm_List_Item *)it)->label;
 }
 
+/* FIXME: this _item_del_pre_hook is never been called at all!
+ To fix this,
+ 1. it->walking concept should be adopted.
+ 2. elm_widget_item_del() should be called instead of the combination of
+ _elm_list_item_free() + elm_widget_item_free()
+ */
 static Eina_Bool
 _item_del_pre_hook(Elm_Object_Item *it)
 {