[EGIT] [core/elementary] master 19/22: elm_win.c: ELM_SAFE_FREE adoption.

[Daniel Juyung Seo - Enlightenment Git]

seoz pushed a commit to branch master.

commit ec504e9ae0a06b69da8562bd9537fd555edcf9ca
Author: Daniel Juyung Seo <seojuyu...@gmail.com>
Date:   Wed May 29 21:11:46 2013 +0900

    elm_win.c: ELM_SAFE_FREE adoption.
    
    I splited ELM_SAFE_FREE refactoring patches. One commit per each file as 
recommended.
    For the detail, please refer 3072dab12f12fe83fb5a628d15efd5cded11787f.
---
 src/lib/elm_win.c | 31 +++++++++----------------------
 1 file changed, 9 insertions(+), 22 deletions(-)

diff --git a/src/lib/elm_win.c b/src/lib/elm_win.c
index 39e9230..02f44b1 100644
--- a/src/lib/elm_win.c
+++ b/src/lib/elm_win.c
@@ -651,9 +651,7 @@ _elm_win_mouse_in(Ecore_Evas *ee)
 static void
 _elm_win_focus_highlight_reconfigure_job_stop(Elm_Win_Smart_Data *sd)
 {
-   if (sd->focus_highlight.reconf_job)
-     ecore_job_del(sd->focus_highlight.reconf_job);
-   sd->focus_highlight.reconf_job = NULL;
+   ELM_SAFE_FREE(sd->focus_highlight.reconf_job, ecore_job_del);
 }
 
 static void
@@ -883,22 +881,15 @@ _elm_win_available_profiles_del(Elm_Win_Smart_Data *sd)
 
    unsigned int i;
    for (i = 0; i < sd->profile.count; i++)
-     if (sd->profile.available_list[i])
-       {
-          eina_stringshare_del(sd->profile.available_list[i]);
-          sd->profile.available_list[i] = NULL;
-       }
+     ELM_SAFE_FREE(sd->profile.available_list[i], eina_stringshare_del);
    sd->profile.count = 0;
-   free(sd->profile.available_list);
-   sd->profile.available_list = NULL;
+   ELM_SAFE_FREE(sd->profile.available_list, free);
 }
 
 static void
 _elm_win_profile_del(Elm_Win_Smart_Data *sd)
 {
-   if (!sd->profile.name) return;
-   eina_stringshare_del(sd->profile.name);
-   sd->profile.name = NULL;
+   ELM_SAFE_FREE(sd->profile.name, eina_stringshare_del);
 }
 
 static Eina_Bool
@@ -1470,10 +1461,10 @@ _elm_win_smart_del(Eo *obj, void *_pd, va_list *list 
EINA_UNUSED)
         ecore_evas_callback_resize_set(sd->ee, NULL);
      }
 
-   ELM_FREE_FUNC(sd->deferred_resize_job, ecore_job_del);
-   ELM_FREE_FUNC(sd->deferred_child_eval_job, ecore_job_del);
-   ELM_FREE_FUNC(sd->shot.info, eina_stringshare_del);
-   ELM_FREE_FUNC(sd->shot.timer, ecore_timer_del);
+   if (sd->deferred_resize_job) ecore_job_del(sd->deferred_resize_job);
+   if (sd->deferred_child_eval_job) ecore_job_del(sd->deferred_child_eval_job);
+   if (sd->shot.info) eina_stringshare_del(sd->shot.info);
+   if (sd->shot.timer) ecore_timer_del(sd->shot.timer);
 
 #ifdef HAVE_ELEMENTARY_X
    if (sd->x.client_message_handler)
@@ -1883,11 +1874,7 @@ void
 _elm_win_shutdown(void)
 {
    while (_elm_win_list) evas_object_del(_elm_win_list->data);
-   if (_elm_win_state_eval_job)
-     {
-        ecore_job_del(_elm_win_state_eval_job);
-        _elm_win_state_eval_job = NULL;
-     }
+   ELM_SAFE_FREE(_elm_win_state_eval_job, ecore_job_del);
 }
 
 void

-- 

------------------------------------------------------------------------------
Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET
Get 100% visibility into your production application - at no cost.
Code-level diagnostics for performance bottlenecks with <2% overhead
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap1