Deleted (as opposed to #if 0) (get_folder_info): Changed to assume it has

2001-07-09  Peter Williams  <peterw@ximian.com>

	* mail-folder-cache.c (mail_folder_cache_set_update_shellview): Deleted
	(as opposed to #if 0)
	(get_folder_info): Changed to assume it has the folder lock.
	(make_folder_name): Same.
	(make_folder_status): Same.
	(update_idle): Don't unlock around the make_folder_ functions.
	(*): Changed behavior wrt. get_folder_info. Lock before calling, but
	also move preconditions before -- get_folder_info can only return NULL
	if uri is NULL. Also add preconditions for other arguments where
	necessary.

svn path=/trunk/; revision=10910

1 files changed, 29 insertions, 53 deletions

diff --git a/mail/mail-folder-cache.c b/mail/mail-folder-cache.c
index d00f3c02d9..d31f5efeef 100644
--- a/mail/mail-folder-cache.c
+++ b/mail/mail-folder-cache.c
@@ -90,6 +90,7 @@ static FolderBrowser *folder_browser = NULL;
 
 /* Private functions */
 
+/* call this with the folders locked */
 static mail_folder_info *
 get_folder_info (const gchar *uri)
 {
@@ -97,8 +98,6 @@ get_folder_info (const gchar *uri)
 
 	g_return_val_if_fail (uri, NULL);
 
-	LOCK_FOLDERS ();
-
 	if (folders == NULL) {
 		dm("Initializing");
 		folders = g_hash_table_new (g_str_hash, g_str_equal);
@@ -123,31 +122,28 @@ get_folder_info (const gchar *uri)
 	} else
 		dm("Hit cache for uri %s", uri);
 
-	UNLOCK_FOLDERS ();
-
 	return mfi;
 }
 
+/* call with the folders locked */
+
 static gchar *
 make_folder_name (mail_folder_info *mfi)
 {
 	GString *work;
 	gchar *ret;
 
-	LOCK_FOLDERS ();
-
 	work = g_string_new (mfi->name);
 
 	if (mfi->flags & MAIL_FIF_UNREAD_VALID && mfi->unread)
 		g_string_sprintfa (work, " (%d)", mfi->unread);
 
-	UNLOCK_FOLDERS ();
-
 	ret = work->str;
 	g_string_free (work, FALSE);
 	return ret;
 }
 
+/* call with the folders locked */
 
 static gchar *
 make_folder_status (mail_folder_info *mfi)
@@ -156,8 +152,6 @@ make_folder_status (mail_folder_info *mfi)
 	GString *work;
 	gchar *ret;
 
-	LOCK_FOLDERS ();
-
 	/* Build the display string */
 
 	work = g_string_new ("");
@@ -180,8 +174,6 @@ make_folder_status (mail_folder_info *mfi)
 		g_string_sprintfa (work, _("%d total"), mfi->total);
 	}
 
-	UNLOCK_FOLDERS ();
-
 	ret = work->str;
 	g_string_free (work, FALSE);
 	return ret;
@@ -221,10 +213,8 @@ update_idle (gpointer user_data)
 
 	/* Get the display string */
 
-	UNLOCK_FOLDERS ();
 	f_name = make_folder_name (mfi);
 	f_status = make_folder_status (mfi);
-	LOCK_FOLDERS ();
 
 	/* bold? */
 
@@ -554,11 +544,12 @@ mail_folder_cache_set_update_estorage (const gchar *uri, EvolutionStorage *estor
 {
 	mail_folder_info *mfi;
 
-	mfi = get_folder_info (uri);
-	g_return_if_fail (mfi);
+	g_return_if_fail (uri);
 
 	LOCK_FOLDERS ();
 
+	mfi = get_folder_info (uri);
+
 	if (mfi->update_mode != MAIL_FIUM_UNKNOWN) {
 		/* we could check to see that update_mode = ESTORAGE */
 		/*g_warning ("folder cache: update mode already set??");*/
@@ -578,11 +569,12 @@ mail_folder_cache_set_update_lstorage (const gchar *uri, GNOME_Evolution_LocalSt
 {
 	mail_folder_info *mfi;
 
-	mfi = get_folder_info (uri);
-	g_return_if_fail (mfi);
+	g_return_if_fail (uri);
 
 	LOCK_FOLDERS ();
 
+	mfi = get_folder_info (uri);
+
 	if (mfi->update_mode != MAIL_FIUM_UNKNOWN) {
 		/*we could check to see that update_mode = lstorage */
 		/*g_warning ("folder cache: update mode already set??");*/
@@ -601,41 +593,18 @@ mail_folder_cache_set_update_lstorage (const gchar *uri, GNOME_Evolution_LocalSt
 	UNLOCK_FOLDERS ();
 }
 
-#if 0
-void 
-mail_folder_cache_set_update_shellview (const gchar *uri)
-{
-	mail_folder_info *mfi;
-
-	mfi = get_folder_info (uri);
-	g_return_if_fail (mfi);
-
-	LOCK_FOLDERS ();
-
-	if (mfi->update_mode != MAIL_FIUM_UNKNOWN) {
-		/*we could check to see that update_mode = shellview */
-		/*g_warning ("folder cache: update mode already set??");*/
-		UNLOCK_FOLDERS ();
-		return;
-	}
-
-	dm ("Uri %s updates with SHELL VIEW", uri);
-	mfi->update_mode = MAIL_FIUM_SHELL_VIEW;
-
-	UNLOCK_FOLDERS ();
-}
-#endif
-
 void
 mail_folder_cache_note_folder (const gchar *uri, CamelFolder *folder)
 {
 	mail_folder_info *mfi;
 
-	mfi = get_folder_info (uri);
-	g_return_if_fail (mfi);
+	g_return_if_fail (uri);
+	g_return_if_fail (CAMEL_IS_FOLDER (folder));
 
 	LOCK_FOLDERS ();
 
+	mfi = get_folder_info (uri);
+
 	if (mfi->flags & MAIL_FIF_FOLDER_VALID) {
 		if (mfi->folder != folder)
 			g_warning ("folder cache: CamelFolder being changed for %s??? I refuse.", uri);
@@ -665,11 +634,13 @@ mail_folder_cache_note_fb (const gchar *uri, FolderBrowser *fb)
 {
 	mail_folder_info *mfi;
 
-	mfi = get_folder_info (uri);
-	g_return_if_fail (mfi);
+	g_return_if_fail (uri);
+	g_return_if_fail (IS_FOLDER_BROWSER (fb));
 
 	LOCK_FOLDERS ();
 
+	mfi = get_folder_info (uri);
+
 	if (!(mfi->flags & MAIL_FIF_FOLDER_VALID)) {
 		dm ("No folder specified so ignoring NOTE_FB at %s", uri);
 		UNLOCK_FOLDERS ();
@@ -695,11 +666,13 @@ mail_folder_cache_note_folderinfo (const gchar *uri, CamelFolderInfo *fi)
 {
 	mail_folder_info *mfi;
 
-	mfi = get_folder_info (uri);
-	g_return_if_fail (mfi);
+	g_return_if_fail (uri);
+	g_return_if_fail (fi);
 
 	LOCK_FOLDERS ();
 
+	mfi = get_folder_info (uri);
+
 	dm ("Noting folderinfo %p for %s", fi, uri);
 
 	if (fi->unread_message_count != -1) {
@@ -723,11 +696,13 @@ mail_folder_cache_note_name (const gchar *uri, const gchar *name)
 {
 	mail_folder_info *mfi;
 
-	mfi = get_folder_info (uri);
-	g_return_if_fail (mfi);
+	g_return_if_fail (uri);
+	g_return_if_fail (name);
 
 	LOCK_FOLDERS ();
 
+	mfi = get_folder_info (uri);
+
 	dm ("Noting name %s for %s", name, uri);
 
 	if (mfi->flags & MAIL_FIF_NAME_VALID) {
@@ -751,11 +726,12 @@ mail_folder_cache_try_folder (const gchar *uri)
 	mail_folder_info *mfi;
 	CamelFolder *ret;
 
-	mfi = get_folder_info (uri);
-	g_return_val_if_fail (mfi, NULL);
+	g_return_val_if_fail (uri, NULL);
 
 	LOCK_FOLDERS ();
 
+	mfi = get_folder_info (uri);
+
 	if (mfi->flags & MAIL_FIF_FOLDER_VALID)
 		ret = mfi->folder;
 	else