AUDIT: r20693 - gnucash/trunk/src/gnome-utils - Fix up the Mac shutdown process to give Gnucash a chance to clean up without spewing critical errors about the main loop already running (because of calling gnc_shutdown in an event handler).

John Ralls jralls at code.gnucash.org
Thu May 26 19:14:14 EDT 2011


Author: jralls
Date: 2011-05-26 19:14:13 -0400 (Thu, 26 May 2011)
New Revision: 20693
Trac: http://svn.gnucash.org/trac/changeset/20693

Modified:
   gnucash/trunk/src/gnome-utils/gnc-main-window.c
Log:
Fix up the Mac shutdown process to give Gnucash a chance to clean up without spewing critical errors about the main loop already running (because of calling gnc_shutdown in an event handler).

BP

Modified: gnucash/trunk/src/gnome-utils/gnc-main-window.c
===================================================================
--- gnucash/trunk/src/gnome-utils/gnc-main-window.c	2011-05-26 23:14:05 UTC (rev 20692)
+++ gnucash/trunk/src/gnome-utils/gnc-main-window.c	2011-05-26 23:14:13 UTC (rev 20693)
@@ -3401,12 +3401,25 @@
 }
 
 #ifdef MAC_INTEGRATION
+/* Event handlers for the shutdown process.  Gnc_quartz_shutdown is
+ * connected to NSApplicationWillTerminate, the last chance to do
+ * anything before quitting. The problem is that it's launched from a
+ * CFRunLoop, not a g_main_loop, and if we call anything that would
+ * affect the main_loop we get an assert that we're in a subidiary
+ * loop.
+ */
 static void
 gtk_quartz_shutdown (GtkOSXApplication *theApp, gpointer data)
 {
-    gnc_shutdown(0);
+/* Do Nothing. It's too late. */
 }
-
+/* Should quit responds to NSApplicationBlockTermination; returning
+ * TRUE means "don't terminate", FALSE means "do terminate". If we
+ * decide that it's OK to terminate, then we queue a gnc_shutdown for
+ * the next idle time (because we're not running in the main loop) and
+ * then tell the OS not to terminate. That gives the gnc_shutdown an
+ * opportunity to shut down.
+ */
 static gboolean
 gtk_quartz_should_quit (GtkOSXApplication *theApp, GncMainWindow *window)
 {
@@ -3421,8 +3434,9 @@
                  !gnc_file_save_in_progress();
     if (needs_save && gnc_main_window_prompt_for_save(GTK_WIDGET(window)))
         return TRUE;
-    gnc_shutdown(0);
-    return FALSE;
+
+    g_idle_add((GSourceFunc)gnc_shutdown, 0);
+    return TRUE;
 }
 
 static void



More information about the gnucash-changes mailing list