seoz pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=5ef676dc9880811a8278405fcd69d6230ce05120

commit 5ef676dc9880811a8278405fcd69d6230ce05120
Author: Daniel Juyung Seo <seojuyu...@gmail.com>
Date:   Fri Jan 16 14:13:30 2015 +0900

    colorselector: Remove unnecessary pointer redirection.
    
    Elm_Colorselector_Data pointer is already there.
---
 src/lib/elm_colorselector.c | 22 +++++++++-------------
 1 file changed, 9 insertions(+), 13 deletions(-)

diff --git a/src/lib/elm_colorselector.c b/src/lib/elm_colorselector.c
index ca49b93..4a29693 100644
--- a/src/lib/elm_colorselector.c
+++ b/src/lib/elm_colorselector.c
@@ -1170,13 +1170,11 @@ _item_sizing_eval(Elm_Color_Item_Data *item)
 
 /* fix size hints of color palette items, so that the box gets it */
 static void
-_palette_sizing_eval(Evas_Object *obj)
+_palette_sizing_eval(Elm_Colorselector_Data *sd)
 {
    Eina_List *elist;
    Elm_Object_Item *eo_item;
 
-   ELM_COLORSELECTOR_DATA_GET(obj, sd);
-
    EINA_LIST_FOREACH(sd->items, elist, eo_item)
      {
         ELM_COLOR_ITEM_DATA_GET(eo_item, item);
@@ -1185,13 +1183,11 @@ _palette_sizing_eval(Evas_Object *obj)
 }
 
 static void
-_component_sizing_eval(Evas_Object *obj)
+_component_sizing_eval(Elm_Colorselector_Data *sd)
 {
    Evas_Coord minw = -1, minh = -1;
    int i;
 
-   ELM_COLORSELECTOR_DATA_GET(obj, sd);
-
    for (i = 0; i < 4; i++)
      {
         if (sd->cb_data[i])
@@ -1212,10 +1208,10 @@ _component_sizing_eval(Evas_Object *obj)
 }
 
 static void
-_full_sizing_eval(Evas_Object *obj)
+_full_sizing_eval(Elm_Colorselector_Data *sd)
 {
-   _palette_sizing_eval(obj);
-   _component_sizing_eval(obj);
+   _palette_sizing_eval(sd);
+   _component_sizing_eval(sd);
 }
 
 static void
@@ -1241,15 +1237,15 @@ _elm_colorselector_elm_layout_sizing_eval(Eo *obj, 
Elm_Colorselector_Data *sd)
    switch (sd->mode)
      {
       case ELM_COLORSELECTOR_PALETTE:
-        _palette_sizing_eval(obj);
+        _palette_sizing_eval(sd);
         break;
 
       case ELM_COLORSELECTOR_COMPONENTS:
-        _component_sizing_eval(obj);
+        _component_sizing_eval(sd);
         break;
 
       case ELM_COLORSELECTOR_BOTH:
-        _full_sizing_eval(obj);
+        _full_sizing_eval(sd);
         break;
 
       case ELM_COLORSELECTOR_PICKER:
@@ -1257,7 +1253,7 @@ _elm_colorselector_elm_layout_sizing_eval(Eo *obj, 
Elm_Colorselector_Data *sd)
         break;
 
       case ELM_COLORSELECTOR_ALL:
-        _full_sizing_eval(obj);
+        _full_sizing_eval(sd);
         break;
 
       default:

-- 


Reply via email to