Enlightenment CVS committal

Author  : onefang
Project : e17
Module  : apps/e

Dir     : e17/apps/e/src/bin


Modified Files:
        e_apps_error.c e_config_dialog.c e_config_dialog.h 


Log Message:
Allocate and free the view struct as per discussion.

===================================================================
RCS file: /cvsroot/enlightenment/e17/apps/e/src/bin/e_apps_error.c,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -3 -r1.4 -r1.5
--- e_apps_error.c      22 Jan 2006 13:01:40 -0000      1.4
+++ e_apps_error.c      22 Jan 2006 14:18:20 -0000      1.5
@@ -24,18 +24,19 @@
    E_Config_Dialog *cfd;
    E_Config_Dialog_View *v;
 
-   v = E_NEW(E_Config_Dialog_View, 1);  /* FIXME: Currenly this never gets 
free'd, awaiting the outcome of mailing list discussion. */
-   /* Dialog Methods */
-   v->create_cfdata = _e_app_error_dialog_create_data;
-   v->free_cfdata = _e_app_error_dialog_free_data;
-//   v->basic.apply_cfdata = NULL;
-   v->basic.create_widgets = _e_app_error_dialog_basic_create_widgets;
-//   v->advanced.apply_cfdata = NULL;
-   v->advanced.create_widgets = _e_app_error_dialog_advanced_create_widgets;
-
-   /* Create The Dialog */
-   cfd = e_config_dialog_new(con, _("Run error, wtf?  That sux."), NULL, 0, v, 
app);
-   app->error_dialog = cfd;
+   v = E_NEW(E_Config_Dialog_View, 1);  /* This gets freed by e_config_dialog. 
*/
+   if (v)
+      {
+         /* Dialog Methods */
+         v->create_cfdata = _e_app_error_dialog_create_data;
+         v->free_cfdata = _e_app_error_dialog_free_data;
+         v->basic.create_widgets = _e_app_error_dialog_basic_create_widgets;
+         v->advanced.create_widgets = 
_e_app_error_dialog_advanced_create_widgets;
+
+         /* Create The Dialog */
+         cfd = e_config_dialog_new(con, _("Run error, wtf?  That sux."), NULL, 
0, v, app);
+         app->error_dialog = cfd;
+      }
 }
 
 static void 
===================================================================
RCS file: /cvsroot/enlightenment/e17/apps/e/src/bin/e_config_dialog.c,v
retrieving revision 1.18
retrieving revision 1.19
diff -u -3 -r1.18 -r1.19
--- e_config_dialog.c   21 Jan 2006 12:44:48 -0000      1.18
+++ e_config_dialog.c   22 Jan 2006 14:18:20 -0000      1.19
@@ -25,14 +25,7 @@
    
 
    cfd = E_OBJECT_ALLOC(E_Config_Dialog, E_CONFIG_DIALOG_TYPE, 
_e_config_dialog_free);
-   cfd->view = *view;
-   /* Seems that every user of this allocates view it on the stack and doesn't 
clear it, 
-    * so I can't rely on this being NULL.  I currently set it to NULL in 
e_config_dialog_new()
-    * and if you want to use it, set it in create_widgets();
-    * I suspect that allocating a structure on the stack that lives beyond the 
function 
-    * call is just asking for trouble.
-    */
-   cfd->view.close_cfdata = NULL;
+   cfd->view = view;
    cfd->con = con;
    cfd->title = evas_stringshare_add(title);
    if (icon)
@@ -43,7 +36,7 @@
    cfd->data = data;
    cfd->hide_buttons = 1;
 
-   if ((cfd->view.basic.apply_cfdata) || (cfd->view.advanced.apply_cfdata))
+   if ((cfd->view->basic.apply_cfdata) || (cfd->view->advanced.apply_cfdata))
       cfd->hide_buttons = 0;
 
    _e_config_dialog_go(cfd, E_CONFIG_DIALOG_CFDATA_TYPE_BASIC);
@@ -60,7 +53,7 @@
    if (cfd->icon) evas_stringshare_del(cfd->icon);
    if (cfd->cfdata)
      {
-       cfd->view.free_cfdata(cfd, cfd->cfdata);
+       cfd->view->free_cfdata(cfd, cfd->cfdata);
        cfd->cfdata = NULL;
      }
    if (cfd->dia)
@@ -69,6 +62,7 @@
        e_object_del(E_OBJECT(cfd->dia));
        cfd->dia = NULL;
      }
+   E_FREE(cfd->view);
    free(cfd);
 }
 
@@ -87,14 +81,14 @@
    e_dialog_title_set(cfd->dia, cfd->title);
    if (cfd->icon) e_dialog_icon_set(cfd->dia, cfd->icon, cfd->icon_size);
    
-   if (!cfd->cfdata) cfd->cfdata = cfd->view.create_cfdata(cfd);
+   if (!cfd->cfdata) cfd->cfdata = cfd->view->create_cfdata(cfd);
    
    if (type == E_CONFIG_DIALOG_CFDATA_TYPE_BASIC)
      {
-       if (cfd->view.advanced.create_widgets)
+       if (cfd->view->advanced.create_widgets)
          {
             o = e_widget_list_add(e_win_evas_get(cfd->dia->win), 0, 0);
-            ob = cfd->view.basic.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
+            ob = cfd->view->basic.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
             e_widget_list_object_append(o, ob, 1, 1, 0.0);
             ob = e_widget_button_add(e_win_evas_get(cfd->dia->win),
                                      _("Advanced"), "widget/new_dialog",
@@ -102,14 +96,14 @@
             e_widget_list_object_append(o, ob, 0, 0, 1.0);
          }
        else
-         o = cfd->view.basic.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
+         o = cfd->view->basic.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
      }
    else
      {
-       if (cfd->view.basic.create_widgets)
+       if (cfd->view->basic.create_widgets)
          {
             o = e_widget_list_add(e_win_evas_get(cfd->dia->win), 0, 0);
-            ob = cfd->view.advanced.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
+            ob = cfd->view->advanced.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
             e_widget_list_object_append(o, ob, 1, 1, 0.0);
             ob = e_widget_button_add(e_win_evas_get(cfd->dia->win), 
                                      _("Basic"), "widget/new_dialog",
@@ -117,7 +111,7 @@
             e_widget_list_object_append(o, ob, 0, 0, 1.0);
          }
        else
-         o = cfd->view.advanced.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
+         o = cfd->view->advanced.create_widgets(cfd, 
e_win_evas_get(cfd->dia->win), cfd->cfdata);
      }
    
    e_widget_min_size_get(o, &mw, &mh);
@@ -167,7 +161,7 @@
 {
    E_Dialog *dia;
    E_Config_Dialog *cfd;
-   
+
    dia = obj;
    cfd = dia->data;
    cfd->dia = NULL;
@@ -183,13 +177,13 @@
    cfd = dia->data;
    if (cfd->view_type == E_CONFIG_DIALOG_CFDATA_TYPE_BASIC)
       {
-         if (cfd->view.basic.apply_cfdata)
-            ok = cfd->view.basic.apply_cfdata(cfd, cfd->cfdata);
+         if (cfd->view->basic.apply_cfdata)
+            ok = cfd->view->basic.apply_cfdata(cfd, cfd->cfdata);
       }
    else
       {
-         if (cfd->view.advanced.apply_cfdata)
-            ok = cfd->view.advanced.apply_cfdata(cfd, cfd->cfdata);
+         if (cfd->view->advanced.apply_cfdata)
+            ok = cfd->view->advanced.apply_cfdata(cfd, cfd->cfdata);
       }
    if (ok) e_object_del(E_OBJECT(cfd));
 }
@@ -203,13 +197,13 @@
    cfd = dia->data;
    if (cfd->view_type == E_CONFIG_DIALOG_CFDATA_TYPE_BASIC)
       {
-         if (cfd->view.basic.apply_cfdata)
-            ok = cfd->view.basic.apply_cfdata(cfd, cfd->cfdata);
+         if (cfd->view->basic.apply_cfdata)
+            ok = cfd->view->basic.apply_cfdata(cfd, cfd->cfdata);
       }
    else
       {
-         if (cfd->view.advanced.apply_cfdata)
-            ok = cfd->view.advanced.apply_cfdata(cfd, cfd->cfdata);
+         if (cfd->view->advanced.apply_cfdata)
+            ok = cfd->view->advanced.apply_cfdata(cfd, cfd->cfdata);
       }
    if ((ok) && (!cfd->hide_buttons))
      {
@@ -256,8 +250,8 @@
    int ok = 1;
 
    cfd = dia->data;
-   if (cfd->view.close_cfdata)
-      ok = cfd->view.close_cfdata(cfd, cfd->cfdata);
+   if (cfd->view->close_cfdata)
+      ok = cfd->view->close_cfdata(cfd, cfd->cfdata);
 
    if (ok)
       e_object_del(E_OBJECT(dia));
===================================================================
RCS file: /cvsroot/enlightenment/e17/apps/e/src/bin/e_config_dialog.h,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -3 -r1.7 -r1.8
--- e_config_dialog.h   21 Jan 2006 12:44:48 -0000      1.7
+++ e_config_dialog.h   22 Jan 2006 14:18:20 -0000      1.8
@@ -23,10 +23,6 @@
 {
    void           *(*create_cfdata)     (E_Config_Dialog *cfd);
    void            (*free_cfdata)       (E_Config_Dialog *cfd, 
E_Config_Dialog_Data *cfdata);
-   /* Seems that every user of this structure allocates it on the stack and 
doesn't clear it, 
-    * so I can't rely on this being NULL.  I currently set it to NULL in 
e_config_dialog_new()
-    * and if you want to use it, set it in create_widgets();
-    */
    int             (*close_cfdata)      (E_Config_Dialog *cfd, 
E_Config_Dialog_Data *cfdata);
    struct {
       int          (*apply_cfdata)      (E_Config_Dialog *cfd, 
E_Config_Dialog_Data *cfdata);
@@ -39,7 +35,7 @@
    E_Object                     e_obj_inherit;
    
    E_Config_Dialog_CFData_Type  view_type;
-   E_Config_Dialog_View         view;
+   E_Config_Dialog_View        *view;
    E_Config_Dialog_Data        *cfdata;
    E_Container                 *con;
    char                        *title;




-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
enlightenment-cvs mailing list
enlightenment-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs

Reply via email to