From 0ef4bba69846b14b805e219ea3a967e151b2686e Mon Sep 17 00:00:00 2001 From: Dan Winship Date: Mon, 10 Sep 2001 17:03:42 +0000 Subject: Fix obvious dumb mistake in previous commit that made it *always* complain * mail-callbacks.c (composer_get_message): Fix obvious dumb mistake in previous commit that made it *always* complain you had invalid recipients. svn path=/trunk/; revision=12733 --- mail/ChangeLog | 6 ++++++ mail/mail-callbacks.c | 34 ++++++++++++++++++---------------- 2 files changed, 24 insertions(+), 16 deletions(-) (limited to 'mail') diff --git a/mail/ChangeLog b/mail/ChangeLog index 9ada2111ec..51df80a6c2 100644 --- a/mail/ChangeLog +++ b/mail/ChangeLog @@ -1,3 +1,9 @@ +2001-09-10 Dan Winship + + * mail-callbacks.c (composer_get_message): Fix obvious dumb + mistake in previous commit that made it *always* complain you had + invalid recipients. + 2001-09-09 Jon Trowbridge * mail-callbacks.c (composer_get_message): Complain if we are diff --git a/mail/mail-callbacks.c b/mail/mail-callbacks.c index 7f725f7f5f..f216410546 100644 --- a/mail/mail-callbacks.c +++ b/mail/mail-callbacks.c @@ -410,26 +410,28 @@ composer_get_message (EMsgComposer *composer) have_invalid = TRUE; } - msg = _("This message contains invalid recipients:"); - for (i = 0; recipients[i]; ++i) { - if (! e_destination_is_valid (recipients[i])) { - new_msg = g_strdup_printf ("%s\n %s", msg, e_destination_get_address (recipients[i])); - g_free (msg); - msg = new_msg; + if (have_invalid) { + msg = _("This message contains invalid recipients:"); + for (i = 0; recipients[i]; ++i) { + if (! e_destination_is_valid (recipients[i])) { + new_msg = g_strdup_printf ("%s\n %s", msg, e_destination_get_address (recipients[i])); + g_free (msg); + msg = new_msg; + } } - } - - new_msg = e_utf8_from_locale_string (msg); - g_free (msg); - msg = new_msg; + + new_msg = e_utf8_from_locale_string (msg); + g_free (msg); + msg = new_msg; - message_box = gnome_message_box_new (msg, GNOME_MESSAGE_BOX_WARNING, GNOME_STOCK_BUTTON_OK, NULL); + message_box = gnome_message_box_new (msg, GNOME_MESSAGE_BOX_WARNING, GNOME_STOCK_BUTTON_OK, NULL); - gnome_dialog_run_and_close (GNOME_DIALOG (message_box)); + gnome_dialog_run_and_close (GNOME_DIALOG (message_box)); - camel_object_unref (CAMEL_OBJECT (message)); - message = NULL; - goto finished; + camel_object_unref (CAMEL_OBJECT (message)); + message = NULL; + goto finished; + } } /* Check for recipients */ -- cgit v1.2.3