diff options
author | Marcel Hollerbach <mail@marcel-hollerbach.de> | 2020-04-17 15:15:41 +0200 |
---|---|---|
committer | Stefan Schmidt <s.schmidt@samsung.com> | 2020-04-17 17:14:57 +0200 |
commit | 04567a16db062449ac5ab02dc24984c063bcde73 (patch) | |
tree | d525afe70c59068fbc1941cf73b7c3a08da56160 | |
parent | 66adfc97f6ff8b03b813fa59207de410ec34d600 (diff) | |
download | efl-04567a16db062449ac5ab02dc24984c063bcde73.tar.gz |
exactness-inspect: unit1 can never be NULL
checkinf for it beeing NULL means that we would have to equip every
usage of unit1 with a check, but that is useless.
CID 1419859
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11721
-rw-r--r-- | src/bin/exactness/inspect.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/bin/exactness/inspect.c b/src/bin/exactness/inspect.c index 115eab90cb..33bcd23533 100644 --- a/src/bin/exactness/inspect.c +++ b/src/bin/exactness/inspect.c @@ -1002,9 +1002,9 @@ _gui_unit_display(Exactness_Unit *unit1, Exactness_Unit *unit2) } _itc_init(); - if (unit1->fonts_path || (unit2 && unit2->fonts_path)) + if ((unit1->fonts_path) || (unit2 && unit2->fonts_path)) { - if (!_show_only_diffs || !unit1 || !unit2 || + if (!_show_only_diffs || !unit2 || !unit1->fonts_path || !unit2->fonts_path || strcmp(unit1->fonts_path, unit2->fonts_path)) { @@ -1013,7 +1013,7 @@ _gui_unit_display(Exactness_Unit *unit1, Exactness_Unit *unit2) elm_genlist_item_append(glc, _grp_itc, (void *)EX_FONTS_DIR, NULL, ELM_GENLIST_ITEM_GROUP, NULL, NULL); } } - itr1 = unit1 ? unit1->actions : NULL; + itr1 = unit1->actions; itr2 = unit2 ? unit2->actions : NULL; if (itr1 || itr2) @@ -1047,7 +1047,7 @@ _gui_unit_display(Exactness_Unit *unit1, Exactness_Unit *unit2) if (itr2) itr2 = eina_list_next(itr2); } - itr1 = unit1 ? unit1->imgs : NULL; + itr1 = unit1->imgs; itr2 = unit2 ? unit2->imgs : NULL; if (itr1 || itr2) @@ -1085,7 +1085,7 @@ _gui_unit_display(Exactness_Unit *unit1, Exactness_Unit *unit2) if (itr2) itr2 = eina_list_next(itr2); } - itr1 = unit1 ? unit1->objs : NULL; + itr1 = unit1->objs; itr2 = unit2 ? unit2->objs : NULL; if (itr1 || itr2) |