From b66a80f71be538d0eeeeca113d61859701b1d315 Mon Sep 17 00:00:00 2001 From: Peter Williams Date: Mon, 9 Jul 2001 14:56:13 +0000 Subject: Deleted (as opposed to #if 0) (get_folder_info): Changed to assume it has 2001-07-09 Peter Williams * 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 --- mail/mail-folder-cache.c | 82 +++++++++++++++++------------------------------- 1 file changed, 29 insertions(+), 53 deletions(-) (limited to 'mail/mail-folder-cache.c') 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 -- cgit v1.2.3