From 34547da64387960e32e9afa302dfb703784c0c71 Mon Sep 17 00:00:00 2001 From: Matthew Barnes Date: Fri, 23 Aug 2013 10:07:36 -0400 Subject: EMailReader cleanups. --- mail/e-mail-reader.c | 24 +++++++++++++----------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/mail/e-mail-reader.c b/mail/e-mail-reader.c index ecf07c50f9..0ebf5f175e 100644 --- a/mail/e-mail-reader.c +++ b/mail/e-mail-reader.c @@ -1324,30 +1324,32 @@ action_mail_reply_sender_check (CamelFolder *folder, gboolean munged_list_message; gboolean active; const gchar *key; - GError *error = NULL; + GError *local_error = NULL; alert_sink = e_activity_get_alert_sink (closure->activity); - message = camel_folder_get_message_finish (folder, result, &error); + message = camel_folder_get_message_finish ( + folder, result, &local_error); - if (e_activity_handle_cancellation (closure->activity, error)) { - g_warn_if_fail (message == NULL); + /* Sanity check. */ + g_return_if_fail ( + ((message != NULL) && (local_error == NULL)) || + ((message == NULL) && (local_error != NULL))); + + if (e_activity_handle_cancellation (closure->activity, local_error)) { mail_reader_closure_free (closure); - g_error_free (error); + g_error_free (local_error); return; - } else if (error != NULL) { - g_warn_if_fail (message == NULL); + } else if (local_error != NULL) { e_alert_submit ( alert_sink, "mail:no-retrieve-message", - error->message, NULL); + local_error->message, NULL); mail_reader_closure_free (closure); - g_error_free (error); + g_error_free (local_error); return; } - g_return_if_fail (CAMEL_IS_MIME_MESSAGE (message)); - settings = g_settings_new ("org.gnome.evolution.mail"); key = "composer-ignore-list-reply-to"; -- cgit v1.2.3