diff options
author | Bryce Harrington <bryce@osg.samsung.com> | 2017-04-18 16:18:56 +0900 |
---|---|---|
committer | Jean-Philippe Andre <jp.andre@samsung.com> | 2017-04-18 18:42:51 +0900 |
commit | 10f1fedfec128cd319adb7053373f8ae850d9eb4 (patch) | |
tree | 6272c04753b5079154444843534bf2cfd611b4a4 /src/lib | |
parent | c4e7d611d635c79fdc4d32a33b5bc1292ed8dd33 (diff) |
ecore_evas: Unnecessary to test p before calling free(p)
Summary:
Modern, standards-compliant compilers already test p, so as per the C
spec it is superfluous to do so before the call.
Reviewers: jpeg
Reviewed By: jpeg
Subscribers: cedric, jpeg
Differential Revision: https://phab.enlightenment.org/D4791
Diffstat (limited to 'src/lib')
-rw-r--r-- | src/lib/ecore_evas/ecore_evas.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/lib/ecore_evas/ecore_evas.c b/src/lib/ecore_evas/ecore_evas.c index bc847b03eb..d34fa8e6e3 100644 --- a/src/lib/ecore_evas/ecore_evas.c +++ b/src/lib/ecore_evas/ecore_evas.c | |||
@@ -3098,19 +3098,19 @@ _ecore_evas_free(Ecore_Evas *ee) | |||
3098 | } | 3098 | } |
3099 | if (ee->data) eina_hash_free(ee->data); | 3099 | if (ee->data) eina_hash_free(ee->data); |
3100 | ee->data = NULL; | 3100 | ee->data = NULL; |
3101 | if (ee->name) free(ee->name); | 3101 | free(ee->name); |
3102 | ee->name = NULL; | 3102 | ee->name = NULL; |
3103 | if (ee->prop.title) free(ee->prop.title); | 3103 | free(ee->prop.title); |
3104 | ee->prop.title = NULL; | 3104 | ee->prop.title = NULL; |
3105 | if (ee->prop.name) free(ee->prop.name); | 3105 | free(ee->prop.name); |
3106 | ee->prop.name = NULL; | 3106 | ee->prop.name = NULL; |
3107 | if (ee->prop.clas) free(ee->prop.clas); | 3107 | free(ee->prop.clas); |
3108 | ee->prop.clas = NULL; | 3108 | ee->prop.clas = NULL; |
3109 | _ecore_evas_window_profile_free(ee); | 3109 | _ecore_evas_window_profile_free(ee); |
3110 | ee->prop.profile.name = NULL; | 3110 | ee->prop.profile.name = NULL; |
3111 | _ecore_evas_window_available_profiles_free(ee); | 3111 | _ecore_evas_window_available_profiles_free(ee); |
3112 | ee->prop.profile.available_list = NULL; | 3112 | ee->prop.profile.available_list = NULL; |
3113 | if (ee->prop.wm_rot.available_rots) free(ee->prop.wm_rot.available_rots); | 3113 | free(ee->prop.wm_rot.available_rots); |
3114 | ee->prop.wm_rot.available_rots = NULL; | 3114 | ee->prop.wm_rot.available_rots = NULL; |
3115 | if (ee->prop.wm_rot.manual_mode.timer) | 3115 | if (ee->prop.wm_rot.manual_mode.timer) |
3116 | ecore_timer_del(ee->prop.wm_rot.manual_mode.timer); | 3116 | ecore_timer_del(ee->prop.wm_rot.manual_mode.timer); |
@@ -3119,7 +3119,7 @@ _ecore_evas_free(Ecore_Evas *ee) | |||
3119 | efl_event_callback_array_del(ee->evas, _ecore_evas_device_cbs(), ee); | 3119 | efl_event_callback_array_del(ee->evas, _ecore_evas_device_cbs(), ee); |
3120 | eina_hash_free(ee->prop.cursors); | 3120 | eina_hash_free(ee->prop.cursors); |
3121 | ee->prop.cursors = NULL; | 3121 | ee->prop.cursors = NULL; |
3122 | if (ee->evas) evas_free(ee->evas); | 3122 | evas_free(ee->evas); |
3123 | ee->evas = NULL; | 3123 | ee->evas = NULL; |
3124 | ECORE_MAGIC_SET(ee, ECORE_MAGIC_NONE); | 3124 | ECORE_MAGIC_SET(ee, ECORE_MAGIC_NONE); |
3125 | ee->driver = NULL; | 3125 | ee->driver = NULL; |