summaryrefslogtreecommitdiff
path: root/src/lib/edje/edje_edit.c
diff options
context:
space:
mode:
authorVorobiov Vitalii <vi.vorobiov@samsung.com>2013-11-01 11:01:37 +0900
committerCedric Bail <cedric.bail@samsung.com>2013-11-01 11:01:42 +0900
commit848cc1d1ea1fc99bf5c8181a6735a414c178076f (patch)
tree245593c1a9a6e6c2f833507d921e2fe171b28c47 /src/lib/edje/edje_edit.c
parentcf452cc419b5c4449d0538a9638f79b898f10025 (diff)
edje: make functions in edje_edit API return Eina_Bool.
Make the following functions return Eina_Bool so the caller can detect errors: edje_edit_state_visible_set edje_edit_state_color_class_set Removed unnecessary check in edje_edit_state_image_border_fill_set. Reviewers: cedric, seoz Reviewed By: cedric CC: reutskiy.v.v Differential Revision: https://phab.enlightenment.org/D305 Signed-off-by: Cedric Bail <cedric.bail@samsung.com>
Diffstat (limited to '')
-rw-r--r--src/lib/edje/edje_edit.c24
1 files changed, 16 insertions, 8 deletions
diff --git a/src/lib/edje/edje_edit.c b/src/lib/edje/edje_edit.c
index 34dfb45cc2..2699604d1b 100644
--- a/src/lib/edje/edje_edit.c
+++ b/src/lib/edje/edje_edit.c
@@ -3533,14 +3533,18 @@ edje_edit_state_visible_get(Evas_Object *obj, const char *part, const char *stat
3533 return pd->visible; 3533 return pd->visible;
3534} 3534}
3535 3535
3536EAPI void 3536EAPI Eina_Bool
3537edje_edit_state_visible_set(Evas_Object *obj, const char *part, const char *state, double value, Eina_Bool visible) 3537edje_edit_state_visible_set(Evas_Object *obj, const char *part, const char *state, double value, Eina_Bool visible)
3538{ 3538{
3539 GET_PD_OR_RETURN(); 3539 if ((!obj) || (!part) || (!state))
3540 //printf("Set state visible flag of part: %s state: %s to: %d\n", part, state, visible); 3540 return EINA_FALSE;
3541 GET_PD_OR_RETURN(EINA_FALSE);
3542
3541 if (visible) pd->visible = 1; 3543 if (visible) pd->visible = 1;
3542 else pd->visible = 0; 3544 else pd->visible = 0;
3545
3543 edje_object_calc_force(obj); 3546 edje_object_calc_force(obj);
3547 return EINA_TRUE;
3544} 3548}
3545 3549
3546EAPI unsigned char 3550EAPI unsigned char
@@ -3569,13 +3573,17 @@ edje_edit_state_color_class_get(Evas_Object *obj, const char *part, const char *
3569 return eina_stringshare_add(pd->color_class); 3573 return eina_stringshare_add(pd->color_class);
3570} 3574}
3571 3575
3572EAPI void 3576EAPI Eina_Bool
3573edje_edit_state_color_class_set(Evas_Object *obj, const char *part, const char *state, double value, const char *color_class) 3577edje_edit_state_color_class_set(Evas_Object *obj, const char *part, const char *state, double value, const char *color_class)
3574{ 3578{
3575 GET_PD_OR_RETURN(); 3579 if ((!obj) || (!part) || (!state))
3576 //printf("Set ColorClass of part: %s state: %s [to: %s]\n", part, state, color_class); 3580 return EINA_FALSE;
3581 GET_PD_OR_RETURN(EINA_FALSE);
3582
3577 _edje_if_string_free(ed, pd->color_class); 3583 _edje_if_string_free(ed, pd->color_class);
3578 pd->color_class = (char*)eina_stringshare_add(color_class); 3584
3585 pd->color_class = (char *)eina_stringshare_add(color_class);
3586 return EINA_TRUE;
3579} 3587}
3580 3588
3581EAPI const Eina_List * 3589EAPI const Eina_List *
@@ -4798,7 +4806,7 @@ edje_edit_state_image_border_fill_set(Evas_Object *obj, const char *part, const
4798 4806
4799 if ((!obj) || (!part) || (!state)) 4807 if ((!obj) || (!part) || (!state))
4800 return EINA_FALSE; 4808 return EINA_FALSE;
4801 if ((fill < 0) || (fill > 2)) 4809 if (fill > 2)
4802 return EINA_FALSE; 4810 return EINA_FALSE;
4803 4811
4804 eina_error_set(0); 4812 eina_error_set(0);