r17574 - gnucash/branches/2.2/src/report/report-gnome - [r17508] Bug #551038: Don't destroy the options database before destroying the options dialog that depends on it.

Andreas Köhler andi5 at cvs.gnucash.org
Fri Sep 19 13:47:04 EDT 2008


Author: andi5
Date: 2008-09-19 13:47:03 -0400 (Fri, 19 Sep 2008)
New Revision: 17574
Trac: http://svn.gnucash.org/trac/changeset/17574

Modified:
   gnucash/branches/2.2/src/report/report-gnome/dialog-style-sheet.c
   gnucash/branches/2.2/src/report/report-gnome/window-report.c
Log:
[r17508] Bug #551038: Don't destroy the options database before destroying the options dialog that depends on it.

Committed by cedayiv.

Modified: gnucash/branches/2.2/src/report/report-gnome/dialog-style-sheet.c
===================================================================
--- gnucash/branches/2.2/src/report/report-gnome/dialog-style-sheet.c	2008-09-19 17:46:53 UTC (rev 17573)
+++ gnucash/branches/2.2/src/report/report-gnome/dialog-style-sheet.c	2008-09-19 17:47:03 UTC (rev 17574)
@@ -117,8 +117,8 @@
     
   }
   gtk_tree_row_reference_free (ssi->row_ref);
+  gnc_options_dialog_destroy(ssi->odialog);
   gnc_option_db_destroy(ssi->odb);
-  gnc_options_dialog_destroy(ssi->odialog);
   scm_gc_unprotect_object(ssi->stylesheet);
   g_free(ssi);
 }

Modified: gnucash/branches/2.2/src/report/report-gnome/window-report.c
===================================================================
--- gnucash/branches/2.2/src/report/report-gnome/window-report.c	2008-09-19 17:46:53 UTC (rev 17573)
+++ gnucash/branches/2.2/src/report/report-gnome/window-report.c	2008-09-19 17:47:03 UTC (rev 17574)
@@ -112,9 +112,9 @@
   SCM    set_editor = scm_c_eval_string("gnc:report-set-editor-widget!");
   
   scm_call_2(set_editor, win->cur_report, SCM_BOOL_F);
+  gnc_options_dialog_destroy(win->win);
   gnc_option_db_destroy(win->db);
   scm_gc_unprotect_object(win->scm_options);
-  gnc_options_dialog_destroy(win->win);
   g_free(win);
 }
 



More information about the gnucash-changes mailing list