* Fixes NB#85740, save separator position as float in order to enhace placement...

[modest] / src / modest-main.c

diff --git a/src/modest-main.c b/src/modest-main.c
index 563fe29..b330ca2 100644 (file)
--- a/src/modest-main.c
+++ b/src/modest-main.c
@@ -37,12 +37,26 @@
 #include <widgets/modest-main-window.h>
 #include <string.h>
 
- static gboolean
+typedef struct {
+       gulong queue_handler;
+       gulong window_list_handler;
+} MainSignalHandlers;
+
+static gboolean
 on_idle_exit_modest (gpointer data)
 {
+       MainSignalHandlers *handlers;
+
        /* Protect the Gtk calls */
        gdk_threads_enter ();
 
+       /* Disconnect signals. Will be freed by the destroy notify */
+       handlers = (MainSignalHandlers *) data;
+       g_signal_handler_disconnect (modest_runtime_get_mail_operation_queue (), 
+                                    handlers->queue_handler);
+       g_signal_handler_disconnect (modest_runtime_get_window_mgr (), 
+                                    handlers->window_list_handler);
+
        /* Wait for remaining tasks */
        while (gtk_events_pending ())
                gtk_main_iteration ();
@@ -60,9 +74,11 @@ on_queue_empty (ModestMailOperationQueue *queue,
 {
        ModestWindowMgr *mgr = modest_runtime_get_window_mgr ();
 
-       /* Exit if the queue is empty and there are no more windows */
+       /* Exit if the queue is empty and there are no more
+          windows. We can exit as well if the main window is hidden
+          and it's the only one */
        if (modest_window_mgr_num_windows (mgr) == 0)
-               g_idle_add_full (G_PRIORITY_LOW, on_idle_exit_modest, NULL, NULL);
+               g_idle_add_full (G_PRIORITY_LOW, on_idle_exit_modest, user_data, g_free);
 }
 
 static void
@@ -73,7 +89,7 @@ on_window_list_empty (ModestWindowMgr *window_mgr,
 
        /* Exit if there are no more windows and the queue is empty */
        if (modest_mail_operation_queue_num_elements (queue) == 0)
-               g_idle_add_full (G_PRIORITY_LOW, on_idle_exit_modest, NULL, NULL);
+               g_idle_add_full (G_PRIORITY_LOW, on_idle_exit_modest, user_data, g_free);
 }
 
 int
@@ -85,6 +101,7 @@ main (int argc, char *argv[])
         * command line.: */
        gboolean show_ui_without_top_application_method = FALSE;
        int retval  = 0;
+       MainSignalHandlers *handlers;
 
        if (argc >= 2) {
                if (strcmp (argv[1], "showui") == 0)
@@ -120,15 +137,18 @@ main (int argc, char *argv[])
        }
 
        /* Connect to the queue-emtpy signal */
-       g_signal_connect (modest_runtime_get_mail_operation_queue (),
-                         "queue-empty",
-                         G_CALLBACK (on_queue_empty),
-                         NULL);
-
-       g_signal_connect (modest_runtime_get_window_mgr (),
-                         "window-list-empty",
-                         G_CALLBACK (on_window_list_empty),
-                         NULL);
+       handlers = g_malloc0 (sizeof (MainSignalHandlers));
+       handlers->queue_handler = 
+               g_signal_connect (modest_runtime_get_mail_operation_queue (),
+                                 "queue-empty",
+                                 G_CALLBACK (on_queue_empty),
+                                 handlers);
+
+       handlers->window_list_handler = 
+               g_signal_connect (modest_runtime_get_window_mgr (),
+                                 "window-list-empty",
+                                 G_CALLBACK (on_window_list_empty),
+                                 handlers);
 
        /* Usually, we only show the UI when we get the "top_application" D-Bus method.
         * This allows modest to start via D-Bus activation to provide a service,