index, list, menu: Optimize eo item usage.
authorDaniel Juyung Seo <seojuyung2@gmail.com>
Tue, 25 Nov 2014 12:55:37 +0000 (21:55 +0900)
committerDaniel Juyung Seo <seojuyung2@gmail.com>
Tue, 25 Nov 2014 12:55:39 +0000 (21:55 +0900)
Remove unnecessary pointer reference.

src/lib/elm_index.c
src/lib/elm_list.c
src/lib/elm_menu.c

index 3c9bb24acc7499cf6caa00bc7871aa623e16c40f..139a903690c824c586d5d541a35cd1de85a8eb67 100644 (file)
@@ -477,7 +477,7 @@ _item_find(Evas_Object *obj,
    EINA_LIST_FOREACH(sd->items, l, eo_item)
      {
         ELM_INDEX_ITEM_DATA_GET(eo_item, it);
-        if (WIDGET_ITEM_DATA_GET(EO_OBJ(it)) == data) return it;
+        if (WIDGET_ITEM_DATA_GET(eo_item) == data) return it;
      }
    return NULL;
 }
index b196f808e4bb8bccab59cbc4e1862c7224abbf2b..fbe49c289a6be2e6ddd99555033356e4b9686582 100644 (file)
@@ -604,7 +604,7 @@ static Eina_Bool _key_action_select(Evas_Object *obj, const char *params EINA_UN
    if (eo_it)
      {
         ELM_LIST_ITEM_DATA_GET(eo_it, it);
-        evas_object_smart_callback_call(WIDGET(it), SIG_ACTIVATED, EO_OBJ(it));
+        evas_object_smart_callback_call(WIDGET(it), SIG_ACTIVATED, eo_it);
      }
 
    return EINA_TRUE;
@@ -1365,6 +1365,7 @@ _item_select(Elm_List_Item_Data *it)
    ELM_LIST_ITEM_CHECK_OR_RETURN(it);
    obj = WIDGET(it);
    ELM_LIST_DATA_GET(obj, sd);
+   Elm_Object_Item *eo_it = EO_OBJ(it);
 
    if (it->base->disabled || _is_no_select(it))
      return;
@@ -1392,15 +1393,15 @@ _item_select(Elm_List_Item_Data *it)
    _elm_list_item_content_focus_set(it, ELM_FOCUS_PREVIOUS, sd->h_mode);
 
    it->selected = EINA_TRUE;
-   sd->selected = eina_list_append(sd->selected, EO_OBJ(it));
+   sd->selected = eina_list_append(sd->selected, eo_it);
 
 call:
    evas_object_ref(obj);
    _elm_list_walk(sd);
 
-   if (it->func) it->func((void *)WIDGET_ITEM_DATA_GET(EO_OBJ(it)), WIDGET(it), EO_OBJ(it));
-   evas_object_smart_callback_call(obj, SIG_SELECTED, EO_OBJ(it));
-   sd->last_selected_item = EO_OBJ(it);
+   if (it->func) it->func((void *)WIDGET_ITEM_DATA_GET(eo_it), WIDGET(it), eo_it);
+   evas_object_smart_callback_call(obj, SIG_SELECTED, eo_it);
+   sd->last_selected_item = eo_it;
 
    _elm_list_unwalk(obj, sd);
    evas_object_unref(obj);
index 7bcf352ac79c60ad6c621e8122ab67df8af589c4..b696ecebfede34b578135e2d592ccb79b1b0e477 100644 (file)
@@ -1005,7 +1005,7 @@ _elm_menu_item_add(Eo *obj, Elm_Menu_Data *sd, Elm_Object_Item *parent, const ch
 
    ELM_MENU_ITEM_DATA_GET(eo_item, it);
 
-   WIDGET_ITEM_DATA_SET(EO_OBJ(it), data);
+   WIDGET_ITEM_DATA_SET(eo_item, data);
    it->func = func;
    it->parent = eo_data_scope_get(parent, ELM_MENU_ITEM_CLASS);
    it->content = icon_obj;