evas: drop redundant pointer check
authorBryce Harrington <bryce@osg.samsung.com>
Wed, 15 Mar 2017 17:40:01 +0000 (10:40 -0700)
committerCedric BAIL <cedric@osg.samsung.com>
Wed, 15 Mar 2017 17:49:47 +0000 (10:49 -0700)
Summary:
The function already has a precondition test for !font, so this
additional test will never trigger.

@fix CID1371525

Reviewers: cedric, jpeg

Subscribers: jpeg, cedric

Differential Revision: https://phab.enlightenment.org/D4682

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
src/lib/evas/canvas/evas_object_text.c

index 2e243be..c91e9d2 100644 (file)
@@ -432,8 +432,7 @@ _evas_text_efl_text_properties_font_set(Eo *eo_obj, Evas_Text_Data *o, const cha
 
    evas_object_async_block(obj);
    if ((size == o->cur.size) &&
-       ((!font && !o->cur.font) ||
-        (font && o->cur.font && !strcmp(font, o->cur.font)))) return;
+       (o->cur.font && !strcmp(font, o->cur.font))) return;
 
    /* We can't assume the given font is same with current fdesc by comparing string.
       Since Evas starts to supporting "auto" for language,