projects
/
platform
/
upstream
/
efl.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
ff6bb81
)
elm_icon: slightly clean-up code
author
Jean Guyomarc'h
<jean@guyomarch.bzh>
Fri, 28 Oct 2016 21:20:14 +0000
(23:20 +0200)
committer
Jean Guyomarc'h
<jean@guyomarch.bzh>
Fri, 28 Oct 2016 21:20:14 +0000
(23:20 +0200)
It is unnecessary to use an initialized variable on the stack to hold a
new value and then immediately return it.
src/lib/elementary/elm_icon.c
patch
|
blob
|
history
diff --git
a/src/lib/elementary/elm_icon.c
b/src/lib/elementary/elm_icon.c
index
3e0904b
..
68c9e64
100644
(file)
--- a/
src/lib/elementary/elm_icon.c
+++ b/
src/lib/elementary/elm_icon.c
@@
-327,11
+327,7
@@
_elm_icon_efl_file_file_set(Eo *obj, Elm_Icon_Data *sd, const char *file, const
ELM_SAFE_FREE(sd->stdicon, eina_stringshare_del);
if (!sd->is_video)
- {
- Eina_Bool int_ret = EINA_FALSE;
- int_ret = efl_file_set(efl_super(obj, MY_CLASS), file, key);
- return int_ret;
- }
+ return efl_file_set(efl_super(obj, MY_CLASS), file, key);
/* parent's edje file setting path replicated here (we got .eet
* extension, so bypassing it) */