Instead of using the length of the shortcuts list as the index for the

2001-08-24  Peter Williams  <peterw@ximian.com>

	* mail-config.c (add_shortcut_entry): Instead of using the length
	of the shortcuts list as the index for the shortcut, use -1, which
	means "last".

	* mail-config-druid.c (druid_finish): Remove the account adding
	stuff since that happens in wizard_finish now.

	* mail-config.c (add_new_storage): New function. Add a
	MailConfigAccount to the shell as a storage.
	(maybe_add_shortcut): Renamed to new_source_created.
	(new_source_created): Call add_new_storage here.

	* component-factory.c (mail_remove_storage_by_uri): Don't warn
	if the storage isn't remote... no point in making the caller
	do extra work.

svn path=/trunk/; revision=12445

1 files changed, 5 insertions, 2 deletions

diff --git a/mail/component-factory.c b/mail/component-factory.c
index c197827fc1..3a6732f987 100644
--- a/mail/component-factory.c
+++ b/mail/component-factory.c
@@ -1106,8 +1106,11 @@ mail_remove_storage_by_uri (const char *uri)
 			mail_remove_storage (CAMEL_STORE (store));
 			camel_object_unref (CAMEL_OBJECT (store));
 		}
-	} else
-		g_warning ("%s is not a remote storage.", uri);
+	} else {
+		/* why make the caller redundantly check this? */
+		/*g_warning ("%s is not a remote storage.", uri);*/
+	}
+
 	camel_exception_clear (&ex);
 }