Set some default permanent_flags. This is the other cause of bug #28038.
2002-08-08 Jeffrey Stedfast <fejj@ximian.com> * providers/imap/camel-imap-folder.c (camel_imap_folder_init): Set some default permanent_flags. This is the other cause of bug #28038. (imap_refresh_info): After calling camel_imap_command() with a NULL format value (this is a shortcut to change the current_folder by SELECTing it), call camel_imap_folder_selected(). This may have been why permanent_flags never got set on the folder even after going online in bug #28038. (imap_update_summary): Same here. Also NULL-guard camel_imap_response_free(). svn path=/trunk/; revision=17743
This commit is contained in:
committed by
Jeffrey Stedfast
parent
d5d4d529b2
commit
e0a73ef707
@ -3,6 +3,13 @@
|
||||
* providers/imap/camel-imap-folder.c (camel_imap_folder_init): Set
|
||||
some default permanent_flags. This is the other cause of bug
|
||||
#28038.
|
||||
(imap_refresh_info): After calling camel_imap_command() with a
|
||||
NULL format value (this is a shortcut to change the current_folder
|
||||
by SELECTing it), call camel_imap_folder_selected(). This may have
|
||||
been why permanent_flags never got set on the folder even after
|
||||
going online in bug #28038.
|
||||
(imap_update_summary): Same here. Also NULL-guard
|
||||
camel_imap_response_free().
|
||||
|
||||
2002-08-07 Jeffrey Stedfast <fejj@ximian.com>
|
||||
|
||||
|
||||
@ -473,13 +473,16 @@ imap_refresh_info (CamelFolder *folder, CamelException *ex)
|
||||
}
|
||||
|
||||
/* If the folder isn't selected, select it (which will force
|
||||
* a rescan if one is needed.
|
||||
* a rescan if one is needed).
|
||||
*/
|
||||
CAMEL_IMAP_STORE_LOCK (imap_store, command_lock);
|
||||
if (imap_store->current_folder != folder) {
|
||||
CAMEL_IMAP_STORE_UNLOCK (imap_store, command_lock);
|
||||
response = camel_imap_command (imap_store, folder, ex, NULL);
|
||||
camel_imap_response_free (imap_store, response);
|
||||
if (response) {
|
||||
camel_imap_folder_selected (folder, response, NULL);
|
||||
camel_imap_response_free (imap_store, response);
|
||||
}
|
||||
return;
|
||||
}
|
||||
CAMEL_IMAP_STORE_UNLOCK (imap_store, command_lock);
|
||||
@ -2264,7 +2267,10 @@ imap_update_summary (CamelFolder *folder, int exists,
|
||||
|
||||
/* now re-select it and process the EXISTS response */
|
||||
response = camel_imap_command (imap_store, folder, ex, NULL);
|
||||
camel_imap_response_free (imap_store, response);
|
||||
if (response) {
|
||||
camel_imap_folder_selected (folder, response, NULL);
|
||||
camel_imap_response_free (imap_store, response);
|
||||
}
|
||||
}
|
||||
|
||||
return;
|
||||
|
||||
Reference in New Issue
Block a user