From cba66580cae27c3c533db818e78607e108e54048 Mon Sep 17 00:00:00 2001
From: Andreas Volz <linux@brachttal.net>
Date: Tue, 14 Apr 2009 18:54:29 +0000
Subject: [PATCH] - correct free - 'if' arround free cfdata - it's better like
 this

SVN revision: 40046
---
 src/modules/conf_dpms/e_int_config_dpms.c | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/src/modules/conf_dpms/e_int_config_dpms.c b/src/modules/conf_dpms/e_int_config_dpms.c
index 3384fd99d..eec73222e 100644
--- a/src/modules/conf_dpms/e_int_config_dpms.c
+++ b/src/modules/conf_dpms/e_int_config_dpms.c
@@ -146,20 +146,16 @@ _create_data(E_Config_Dialog *cfd)
 static void
 _free_data(E_Config_Dialog *cfd, E_Config_Dialog_Data *cfdata)
 {
-   Eina_List *l = NULL;
+  dpms_list = eina_list_free(dpms_list);
 
-   // delete the list
-   for (l = dpms_list; l; l = eina_list_next(l))
+   if (cfdata)
    {
-      dpms_list = eina_list_remove_list(dpms_list, l);
+      cfdata->standby_slider=NULL;
+      cfdata->suspend_slider=NULL;
+      cfdata->off_slider=NULL;
+      
+      E_FREE(cfdata);
    }
-
-   if (!cfdata) return;
-   cfdata->standby_slider=NULL;
-   cfdata->suspend_slider=NULL;
-   cfdata->off_slider=NULL;
-
-   E_FREE(cfdata);
 }
 
 static int