From 060bb076625dba3aad86485014d49a348cf13671 Mon Sep 17 00:00:00 2001 From: Matthew Barnes Date: Mon, 15 Jul 2013 15:05:08 -0400 Subject: mail_folder_cache_get_folder_info_flags: Change parameters. Take a CamelStore and folder name instead of a CamelFolder. CamelStore and folder name can easily be obtained from either a folder URI or a CamelFolder instance, and the function is more efficient with separate parameters. --- mail/e-mail-backend.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) (limited to 'mail/e-mail-backend.c') diff --git a/mail/e-mail-backend.c b/mail/e-mail-backend.c index c2952daffd..b7fb84735e 100644 --- a/mail/e-mail-backend.c +++ b/mail/e-mail-backend.c @@ -673,24 +673,14 @@ mail_backend_folder_changed_cb (MailFolderCache *folder_cache, EMEvent *event = em_event_peek (); EMEventTargetFolder *target; EMFolderTreeModel *model; - CamelFolder *folder; gchar *folder_uri; gint folder_type; CamelFolderInfoFlags flags = 0; folder_uri = e_mail_folder_uri_build (store, folder_name); - folder = mail_folder_cache_ref_folder ( - folder_cache, store, folder_name); - if (folder != NULL) { - /* XXX Need to rethink this API. Why should we - * need the CamelFolder instance just to get - * folder flags? The flags are more readily - * available than the CamelFolder instance. */ - mail_folder_cache_get_folder_info_flags ( - folder_cache, folder, &flags); - g_object_unref (folder); - } + mail_folder_cache_get_folder_info_flags ( + folder_cache, store, folder_name, &flags); target = em_event_target_new_folder ( event, store, folder_uri, new_messages, -- cgit