From 2583bd0d41a6ef0bcf91ebc8788157aad57a312e Mon Sep 17 00:00:00 2001 From: Larry Ewing Date: Fri, 6 Jul 2001 00:50:03 +0000 Subject: i2001-07-05 Larry Ewing * e-msg-composer.c: add ask argment. (autosave_manager_query_load_orphans): move the load loop out of the dir reading loop. (autosave_manager_new): move initialization out or register. (autosave_manager_register): protect against recursion with ask. (init): call new, this still needs to be moved. svn path=/trunk/; revision=10837 --- composer/ChangeLog | 9 +++++ composer/e-msg-composer.c | 97 +++++++++++++++++++++++++++++------------------ 2 files changed, 69 insertions(+), 37 deletions(-) diff --git a/composer/ChangeLog b/composer/ChangeLog index 364a4cd093..d7e8368d43 100644 --- a/composer/ChangeLog +++ b/composer/ChangeLog @@ -1,3 +1,12 @@ +2001-07-05 Larry Ewing + + * e-msg-composer.c: add ask argment. + (autosave_manager_query_load_orphans): move the load loop out of + the dir reading loop. + (autosave_manager_new): move initialization out or register. + (autosave_manager_register): protect against recursion with ask. + (init): call new, this still needs to be moved. + 2001-07-05 Jeffrey Stedfast * e-msg-composer.c (save): Play the "File exists, overwrite?" diff --git a/composer/e-msg-composer.c b/composer/e-msg-composer.c index df955c383e..4996b9fa3c 100644 --- a/composer/e-msg-composer.c +++ b/composer/e-msg-composer.c @@ -1018,6 +1018,7 @@ typedef struct _AutosaveManager AutosaveManager; struct _AutosaveManager { GHashTable *table; guint id; + gboolean ask; }; static AutosaveManager *am = NULL; @@ -1115,23 +1116,23 @@ autosave_query_cb (gint reply, gpointer data) *ok = !reply; } -static GList * -autosave_query_load_orphans (AutosaveManager *am, EMsgComposer *composer) +static void +autosave_manager_query_load_orphans (AutosaveManager *am, EMsgComposer *composer) { GtkWidget *dialog; DIR *dir; struct dirent *d; - GList *matches = NULL; + GSList *match = NULL; gint len = strlen (AUTOSAVE_SEED); gint pre_len; gint ok; - + /* length of the seed minus the XXXXXX */ pre_len = len - 6; - + dir = opendir (g_get_home_dir()); if (!dir) { - return NULL; + return; } while ((d = readdir (dir))) { @@ -1145,16 +1146,25 @@ autosave_query_load_orphans (AutosaveManager *am, EMsgComposer *composer) gnome_dialog_run_and_close (GNOME_DIALOG (dialog)); if (ok) { - char *filename = g_strdup_printf ("%s/%s", g_get_home_dir (), d->d_name); - EMsgComposer *composer; - - composer = autosave_load_draft (filename); - - g_free (filename); + match = g_slist_prepend (match, + g_strdup_printf ("%s/%s", + g_get_home_dir(), + d->d_name)); } } } - return matches; + + while (match != NULL) { + GSList *next = match->next; + char *filename = match->data; + EMsgComposer *composer; + + composer = autosave_load_draft (filename); + + g_free (filename); + g_slist_free_1 (match); + match = next; + } } static void @@ -1175,63 +1185,73 @@ autosave_run (gpointer data) return TRUE; } +static gboolean +autosave_init_file (EMsgComposer *composer) +{ + if (composer->autosave_file == NULL) { + composer->autosave_file = g_strdup_printf ("%s/%s", g_get_home_dir(), AUTOSAVE_SEED); + composer->autosave_fd = mkstemp (composer->autosave_file); + return TRUE; + } + return FALSE; +} static void -autosave_start (AutosaveManager *am) +autosave_manager_start (AutosaveManager *am) { if (am->id == 0) am->id = gtk_timeout_add (AUTOSAVE_INTERVAL, autosave_run, am); } static void -autosave_stop (AutosaveManager *am) +autosave_manager_stop (AutosaveManager *am) { if (am->id) gtk_timeout_remove (am->id); } -static gboolean -autosave_init_file (EMsgComposer *composer) +AutosaveManager * +autosave_manager_new () { - if (composer->autosave_file == NULL) { - composer->autosave_file = g_strdup_printf ("%s/%s", g_get_home_dir(), AUTOSAVE_SEED); - composer->autosave_fd = mkstemp (composer->autosave_file); - return TRUE; - } - return FALSE; + AutosaveManager *am; + + am = g_new (AutosaveManager, 1); + am->table = g_hash_table_new (g_str_hash, g_str_equal); + am->id = 0; + am->ask = TRUE; + + return am; } static void -autosave_register (EMsgComposer *composer) +autosave_manager_register (AutosaveManager *am, EMsgComposer *composer) { char *key; g_return_if_fail (composer != NULL); - if (am == NULL) { - am = g_new (AutosaveManager, 1); - am->table = g_hash_table_new (g_str_hash, g_str_equal); - am->id = 0; - } - if (autosave_init_file (composer)) { key = g_basename (composer->autosave_file); g_hash_table_insert (am->table, key, composer); - autosave_query_load_orphans (am, composer); + if (am->ask) { + am->ask = FALSE; + autosave_manager_query_load_orphans (am, composer); + } + } - autosave_start (am); + autosave_manager_start (am); } static void -autosave_unregister (EMsgComposer *composer) +autosave_manager_unregister (AutosaveManager *am, EMsgComposer *composer) { g_hash_table_remove (am->table, g_basename (composer->autosave_file)); close (composer->autosave_fd); unlink (composer->autosave_file); g_free (composer->autosave_file); - + if (g_hash_table_size (am->table) == 0) - autosave_stop (am); + autosave_manager_stop (am); } static void @@ -1875,7 +1895,7 @@ destroy (GtkObject *object) g_free (composer->charset); - autosave_unregister (composer); + autosave_manager_unregister (am, composer); CORBA_exception_init (&ev); @@ -2066,7 +2086,10 @@ init (EMsgComposer *composer) composer->autosave_file = NULL; composer->autosave_fd = -1; - autosave_register (composer); + if (am == NULL) { + am = autosave_manager_new (); + } + autosave_manager_register (am, composer); } -- cgit