From cbdf3bccf6858264bb75500c695e35040a114910 Mon Sep 17 00:00:00 2001 From: Not Zed Date: Tue, 26 Nov 2002 11:28:29 +0000 Subject: Duh idiot, "un-fixed" the memleak i added. 2002-11-26 Not Zed * message-list.c (get_normalised_string): Duh idiot, "un-fixed" the memleak i added. * mail-callbacks.c (composer_get_message): Set parent window in message dialogue & DESTROY_WITH_PARENT flag. (composer_save_draft_cb): (configure_mail): (check_send_configuration): Add DESTROY_WITH_PARENT flag to gtk_message_dialog's (local_configure_done): remove some unecessary/wrong casts. (empty_trash_expunged_cb): " (do_mail_print): use gtk_window_set_transient_for instead of e_dialog_set_parent. svn path=/trunk/; revision=18923 --- mail/ChangeLog | 16 ++++++++++++++++ mail/mail-callbacks.c | 24 +++++++++++------------- mail/message-list.c | 1 - 3 files changed, 27 insertions(+), 14 deletions(-) (limited to 'mail') diff --git a/mail/ChangeLog b/mail/ChangeLog index 88d87be4ce..dcd9067a99 100644 --- a/mail/ChangeLog +++ b/mail/ChangeLog @@ -1,3 +1,19 @@ +2002-11-26 Not Zed + + * message-list.c (get_normalised_string): Duh idiot, "un-fixed" + the memleak i added. + + * mail-callbacks.c (composer_get_message): Set parent window in + message dialogue & DESTROY_WITH_PARENT flag. + (composer_save_draft_cb): + (configure_mail): + (check_send_configuration): Add DESTROY_WITH_PARENT flag to + gtk_message_dialog's + (local_configure_done): remove some unecessary/wrong casts. + (empty_trash_expunged_cb): " + (do_mail_print): use gtk_window_set_transient_for instead of + e_dialog_set_parent. + 2002-11-26 Not Zed * mail-account-editor.c (mail_account_editor_new): diff --git a/mail/mail-callbacks.c b/mail/mail-callbacks.c index 7183bcf017..1058ec83e6 100644 --- a/mail/mail-callbacks.c +++ b/mail/mail-callbacks.c @@ -151,7 +151,7 @@ configure_mail (FolderBrowser *fb) GtkWidget *dialog; int resp; - dialog = gtk_message_dialog_new (FB_WINDOW (fb), GTK_DIALOG_MODAL, + dialog = gtk_message_dialog_new (FB_WINDOW (fb), GTK_DIALOG_MODAL|GTK_DIALOG_DESTROY_WITH_PARENT, GTK_MESSAGE_QUESTION, GTK_BUTTONS_YES_NO, "%s", _("You have not configured the mail client.\n" "You need to do this before you can send,\n" @@ -194,7 +194,7 @@ check_send_configuration (FolderBrowser *fb) /* Check for an identity */ if (!account) { - dialog = gtk_message_dialog_new (FB_WINDOW (fb), GTK_DIALOG_MODAL, + dialog = gtk_message_dialog_new (FB_WINDOW (fb), GTK_DIALOG_MODAL|GTK_DIALOG_DESTROY_WITH_PARENT, GTK_MESSAGE_WARNING, GTK_BUTTONS_CLOSE, "%s", _("You need to configure an identity\n" "before you can compose mail.")); @@ -207,7 +207,7 @@ check_send_configuration (FolderBrowser *fb) /* Check for a transport */ if (!account->transport || !account->transport->url) { - dialog = gtk_message_dialog_new (FB_WINDOW (fb), GTK_DIALOG_MODAL, + dialog = gtk_message_dialog_new (FB_WINDOW (fb), GTK_DIALOG_MODAL|GTK_DIALOG_DESTROY_WITH_PARENT, GTK_MESSAGE_WARNING, GTK_BUTTONS_CLOSE, "%s", _("You need to configure a mail transport\n" "before you can compose mail.")); @@ -457,11 +457,9 @@ composer_get_message (EMsgComposer *composer, gboolean post, gboolean save_html_ if (num == 0 && !post) { GtkWidget *dialog; - dialog = gtk_message_dialog_new (NULL, GTK_DIALOG_MODAL, GTK_MESSAGE_WARNING, - GTK_BUTTONS_CLOSE, "%s", - _("You must specify recipients in order to " - "send this message.")); - + dialog = gtk_message_dialog_new ((GtkWindow *)composer, GTK_DIALOG_MODAL|GTK_DIALOG_DESTROY_WITH_PARENT, + GTK_MESSAGE_WARNING, GTK_BUTTONS_CLOSE, "%s", + _("You must specify recipients in order to send this message.")); gtk_dialog_run (GTK_DIALOG (dialog)); gtk_widget_destroy (dialog); goto finished; @@ -720,7 +718,7 @@ composer_save_draft_cb (EMsgComposer *composer, int quit, gpointer user_data) GtkWidget *dialog; int response; - dialog = gtk_message_dialog_new (GTK_WINDOW (composer), GTK_DIALOG_MODAL, + dialog = gtk_message_dialog_new (GTK_WINDOW (composer), GTK_DIALOG_MODAL|GTK_DIALOG_DESTROY_WITH_PARENT, GTK_MESSAGE_QUESTION, GTK_BUTTONS_YES_NO, "%s", _("Unable to open the drafts folder for this account.\n" "Would you like to use the default drafts folder?")); @@ -3034,7 +3032,7 @@ do_mail_print (FolderBrowser *fb, gboolean preview) dialog = GNOME_PRINT_DIALOG (gnome_print_dialog_new (_("Print Message"), GNOME_PRINT_DIALOG_COPIES)); gtk_dialog_set_default_response((GtkDialog *)dialog, GNOME_PRINT_DIALOG_RESPONSE_PRINT); - e_gnome_dialog_set_parent (GNOME_DIALOG (dialog), FB_WINDOW (fb)); + gtk_window_set_transient_for((GtkWindow *)dialog, (GtkWindow *)fb); switch(gtk_dialog_run((GtkDialog *)dialog)) { case GNOME_PRINT_DIALOG_RESPONSE_PRINT: @@ -3224,7 +3222,7 @@ local_configure_done(const char *uri, CamelFolder *folder, void *data) FolderBrowser *fb = data; if (FOLDER_BROWSER_IS_DESTROYED (fb)) { - g_object_unref((GtkObject *)fb); + g_object_unref(fb); return; } @@ -3232,7 +3230,7 @@ local_configure_done(const char *uri, CamelFolder *folder, void *data) folder = fb->folder; message_list_set_folder(fb->message_list, folder, FALSE); - g_object_unref((GtkObject *)fb); + g_object_unref(fb); } void @@ -3331,7 +3329,7 @@ stop_threads (BonoboUIComponent *uih, void *user_data, const char *path) static void empty_trash_expunged_cb (CamelFolder *folder, void *data) { - camel_object_unref (CAMEL_OBJECT (folder)); + camel_object_unref (folder); } void diff --git a/mail/message-list.c b/mail/message-list.c index 24139fa9c1..f6de16b362 100644 --- a/mail/message-list.c +++ b/mail/message-list.c @@ -413,7 +413,6 @@ get_normalised_string (MessageList *message_list, CamelMessageInfo *info, int co normalised = g_utf8_collate_key (string, -1); e_poolv_set (poolv, index, normalised, TRUE); - g_free(normalised); return e_poolv_get (poolv, index); } -- cgit