color class lookup speedups by using stringshare properties.

with stringshare, we can just compare pointers instead of
strcmp. Since we'll need the stringshare later, this is a good
optimization.

git-svn-id: svn+ssh://svn.enlightenment.org/var/svn/e/trunk/edje@46925 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33

diff --git a/src/lib/edje_util.c b/src/lib/edje_util.c
index f6528e9..a85b548 100644 (file)
--- a/src/lib/edje_util.c
+++ b/src/lib/edje_util.c
@@ -661,10 +661,14 @@ edje_object_color_class_set(Evas_Object *obj, const char *color_class, int r, in
    else if (b > 255) b = 255;
    if (a < 0)        a = 0;
    else if (a > 255) a = 255;
+   color_class = eina_stringshare_add(color_class);
+   if (!color_class) return EINA_FALSE;
    EINA_LIST_FOREACH(ed->color_classes, l, cc)
      {
-       if ((cc->name) && (!strcmp(cc->name, color_class)))
+       if (cc->name == color_class)
          {
+            eina_stringshare_del(color_class);
+
             if ((cc->r == r) && (cc->g == g) &&
                 (cc->b == b) && (cc->a == a) &&
                 (cc->r2 == r2) && (cc->g2 == g2) &&
@@ -693,13 +697,12 @@ edje_object_color_class_set(Evas_Object *obj, const char *color_class, int r, in
          }
      }
    cc = malloc(sizeof(Edje_Color_Class));
-   if (!cc) return EINA_FALSE;
-   cc->name = eina_stringshare_add(color_class);
-   if (!cc->name)
+   if (!cc)
      {
-       free(cc);
+       eina_stringshare_del(color_class);
        return EINA_FALSE;
      }
+   cc->name = color_class;
    cc->r = r;
    cc->g = g;
    cc->b = b;