aboutsummaryrefslogtreecommitdiffstats
path: root/src/edje_external
diff options
context:
space:
mode:
authorJean Guyomarc'h <jean@guyomarch.bzh>2016-06-26 19:42:21 +0200
committerJean Guyomarc'h <jean@guyomarch.bzh>2016-06-26 23:11:55 +0200
commit7352bcff98fc65a08edcd505b872403af8d821a7 (patch)
tree98e30d0ea5a9ed9e79f99aedc4dd5c1c1e98d9a3 /src/edje_external
parenteina: fix typos in eina_rectangle doc (diff)
downloadefl-7352bcff98fc65a08edcd505b872403af8d821a7.tar.gz
edje_external: fix external icon handling
External icons were handled with a global variable, which reason to exist seem quite weird. It seems to me it was used to retain some states of the actual parameters to get them later without having access to the real parameters. I don't really get how this would have worked with more than one icon. When I tried to animate an external icon, with two states, there were leaks and memory corruption... Now both are gone. @fix
Diffstat (limited to 'src/edje_external')
-rw-r--r--src/edje_external/elementary/elm_icon.c59
1 files changed, 21 insertions, 38 deletions
diff --git a/src/edje_external/elementary/elm_icon.c b/src/edje_external/elementary/elm_icon.c
index 979cc0b40e..9e9f6dd1d5 100644
--- a/src/edje_external/elementary/elm_icon.c
+++ b/src/edje_external/elementary/elm_icon.c
@@ -20,7 +20,6 @@ typedef struct _Elm_Params_Icon
const char *icon;
} Elm_Params_Icon;
-static Elm_Params_Icon *param_icon;
static void
external_icon_state_set(void *data EINA_UNUSED, Evas_Object *obj,
@@ -30,6 +29,7 @@ external_icon_state_set(void *data EINA_UNUSED, Evas_Object *obj,
const Elm_Params_Icon *p;
Evas_Object *edje;
const char *file;
+ Eina_Bool param;
if (to_params) p = to_params;
else if (from_params) p = from_params;
@@ -38,46 +38,39 @@ external_icon_state_set(void *data EINA_UNUSED, Evas_Object *obj,
if (p->file)
{
elm_image_file_set(obj, p->file, NULL);
- param_icon->file = p->file;
}
if (p->smooth_exists)
{
elm_image_smooth_set(obj, p->smooth);
- param_icon->smooth = p->smooth;
}
if (p->no_scale_exists)
{
elm_image_no_scale_set(obj, p->no_scale);
- param_icon->no_scale = p->no_scale;
}
if (p->scale_up_exists && p->scale_down_exists)
{
elm_image_resizable_set(obj, p->scale_up, p->scale_down);
- param_icon->scale_up = p->scale_up;
- param_icon->scale_down = p->scale_down;
}
else if (p->scale_up_exists || p->scale_down_exists)
{
if (p->scale_up_exists)
{
- elm_image_resizable_set(obj, p->scale_up, param_icon->scale_down);
- param_icon->scale_up = p->scale_up;
+ elm_image_resizable_get(obj, NULL, &param);
+ elm_image_resizable_set(obj, p->scale_up, param);
}
else
{
- elm_image_resizable_set(obj, param_icon->scale_up, p->scale_down);
- param_icon->scale_down = p->scale_down;
+ elm_image_resizable_get(obj, &param, NULL);
+ elm_image_resizable_set(obj, param, p->scale_down);
}
}
if (p->fill_outside_exists)
{
elm_image_fill_outside_set(obj, p->fill_outside);
- param_icon->fill_outside = p->fill_outside;
}
if (p->prescale_size_exists)
{
elm_image_prescale_set(obj, p->prescale_size);
- param_icon->prescale_size = p->prescale_size;
}
if (p->icon)
{
@@ -103,55 +96,49 @@ external_icon_param_set(void *data EINA_UNUSED, Evas_Object *obj,
{
Evas_Object *edje;
const char *file;
+ Eina_Bool p;
if (!strcmp(param->name, "file")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_STRING)
{
- Eina_Bool ret = elm_image_file_set(obj, param->s, NULL);
- if (ret)
- param_icon->file = param->s;
- return ret;
+ return elm_image_file_set(obj, param->s, NULL);
}
else if (!strcmp(param->name, "smooth")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
elm_image_smooth_set(obj, param->i);
- param_icon->smooth = param->i;
return EINA_TRUE;
}
else if (!strcmp(param->name, "no scale")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
elm_image_no_scale_set(obj, param->i);
- param_icon->no_scale = param->i;
return EINA_TRUE;
}
else if (!strcmp(param->name, "scale up")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
- elm_image_resizable_set(obj, param->i, param_icon->scale_down);
- param_icon->scale_up = param->i;
+ elm_image_resizable_get(obj, NULL, &p);
+ elm_image_resizable_set(obj, param->i, p);
return EINA_TRUE;
}
else if (!strcmp(param->name, "scale down")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
- elm_image_resizable_set(obj, param_icon->scale_up, param->i);
- param_icon->scale_down = param->i;
+ elm_image_resizable_get(obj, &p, NULL);
+ elm_image_resizable_set(obj, p, param->i);
return EINA_TRUE;
}
else if (!strcmp(param->name, "fill outside")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
elm_image_fill_outside_set(obj, param->i);
- param_icon->fill_outside = param->i;
return EINA_TRUE;
}
else if (!strcmp(param->name, "prescale")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_INT)
{
elm_image_prescale_set(obj, param->i);
- param_icon->prescale_size = param->i;
return EINA_TRUE;
}
else if (!strcmp(param->name, "icon"))
@@ -183,49 +170,50 @@ external_icon_param_set(void *data EINA_UNUSED, Evas_Object *obj,
static Eina_Bool
external_icon_param_get(void *data EINA_UNUSED,
- const Evas_Object *obj EINA_UNUSED,
+ const Evas_Object *obj,
Edje_External_Param *param)
{
+
if (!strcmp(param->name, "file")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_STRING)
{
- param->s = param_icon->file;
+ elm_image_file_get(obj, &param->s, NULL);
return EINA_TRUE;
}
else if (!strcmp(param->name, "smooth")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
- param->i = param_icon->smooth;
+ param->i = elm_image_smooth_get(obj);
return EINA_TRUE;
}
else if (!strcmp(param->name, "no scale")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
- param->i = param_icon->no_scale;
+ param->i = elm_image_no_scale_get(obj);
return EINA_TRUE;
}
else if (!strcmp(param->name, "scale up")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
- param->i = param_icon->scale_up;
+ elm_image_resizable_get(obj, NULL, (Eina_Bool *)(&param->i));
return EINA_TRUE;
}
else if (!strcmp(param->name, "scale down")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
- param->i = param_icon->scale_down;
+ elm_image_resizable_get(obj, (Eina_Bool *)(&param->i), NULL);
return EINA_TRUE;
}
else if (!strcmp(param->name, "fill outside")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_BOOL)
{
- param->i = param_icon->fill_outside;
+ param->i = elm_image_fill_outside_get(obj);
return EINA_TRUE;
}
else if (!strcmp(param->name, "prescale")
&& param->type == EDJE_EXTERNAL_PARAM_TYPE_INT)
{
- param->i = param_icon->prescale_size;
+ param->i = elm_image_prescale_get(obj);
return EINA_TRUE;
}
else if (!strcmp(param->name, "icon"))
@@ -247,9 +235,8 @@ external_icon_params_parse(void *data EINA_UNUSED, Evas_Object *obj EINA_UNUSED,
Elm_Params_Icon *mem;
Edje_External_Param *param;
const Eina_List *l;
- param_icon = calloc(1, sizeof(Elm_Params_Icon));
mem = ELM_NEW(Elm_Params_Icon);
- if (!mem)
+ if (EINA_UNLIKELY(!mem))
return NULL;
EINA_LIST_FOREACH(params, l, param)
@@ -312,10 +299,6 @@ external_icon_params_free(void *params)
if (mem->file)
eina_stringshare_del(mem->file);
- if (param_icon->file)
- eina_stringshare_del(param_icon->file);
- free(param_icon);
-
if (mem->icon)
eina_stringshare_del(mem->icon);
free(mem);