cedric pushed a commit to branch master. http://git.enlightenment.org/core/efl.git/commit/?id=e15b8c27ac4b34dda12c50753ead3580b5c8cd2e
commit e15b8c27ac4b34dda12c50753ead3580b5c8cd2e Author: Igor Gala <[email protected]> Date: Tue Jun 10 17:19:40 2014 +0200 remove unnecessary second checks on validity Summary: Some checks were removed, becouse they are useless, due macross GET_PD_OR_RETURN(NULL) does all this work. @fix Reviewers: seoz, cedric, raster, Hermet CC: reutskiy.v.v, cedric Differential Revision: https://phab.enlightenment.org/D980 --- src/lib/edje/edje_edit.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/lib/edje/edje_edit.c b/src/lib/edje/edje_edit.c index 2ae2bbf..7e4e2e4 100644 --- a/src/lib/edje/edje_edit.c +++ b/src/lib/edje/edje_edit.c @@ -4881,7 +4881,8 @@ EAPI Eina_Bool edje_edit_state_text_set(Evas_Object *obj, const char *part, const char *state, double value, const char *text) { Edje_Part_Description_Text *txt; - if ((!obj) || (!part) || (!state) || (!text)) + + if (!text) return EINA_FALSE; GET_PD_OR_RETURN(EINA_FALSE); @@ -4920,8 +4921,6 @@ edje_edit_state_text_size_set(Evas_Object *obj, const char *part, const char *st { Edje_Part_Description_Text *txt; - if ((!obj) || (!part) || (!state)) - return EINA_FALSE; if (size < 0) return EINA_FALSE; GET_PD_OR_RETURN(EINA_FALSE); @@ -5243,8 +5242,6 @@ edje_edit_state_font_set(Evas_Object *obj, const char *part, const char *state, { Edje_Part_Description_Text *txt; - if ((!obj) || (!part) || (!state)) - return EINA_FALSE; GET_PD_OR_RETURN(EINA_FALSE); if ((rp->part->type != EDJE_PART_TYPE_TEXT) && --
