AUDIT: r17508 - gnucash/trunk/src/report/report-gnome - Bug #551038: Don't destroy the options database before destroying the options dialog that depends on it.

[Charles Day]

Author: cedayiv
Date: 2008-09-14 13:48:58 -0400 (Sun, 14 Sep 2008)
New Revision: 17508
Trac: http://svn.gnucash.org/trac/changeset/17508

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


Modified: gnucash/trunk/src/report/report-gnome/dialog-style-sheet.c
===================================================================
--- gnucash/trunk/src/report/report-gnome/dialog-style-sheet.c	2008-09-14 16:52:34 UTC (rev 17507)
+++ gnucash/trunk/src/report/report-gnome/dialog-style-sheet.c	2008-09-14 17:48:58 UTC (rev 17508)
@@ -117,8 +117,8 @@
     gtk_tree_path_free(path);
   }
   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/trunk/src/report/report-gnome/window-report.c
===================================================================
--- gnucash/trunk/src/report/report-gnome/window-report.c	2008-09-14 16:52:34 UTC (rev 17507)
+++ gnucash/trunk/src/report/report-gnome/window-report.c	2008-09-14 17:48:58 UTC (rev 17508)
@@ -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);
 }