Fix a locking problem. (One branch of an if statement was returning with

* camel-folder.c (camel_folder_copy_message_to): Fix a locking
	problem. (One branch of an if statement was returning with the
	lock still locked.) Also remove the deprecation comments, to match
	move_message_to.

svn path=/trunk/; revision=7409
This commit is contained in:
Dan Winship
2001-01-11 21:53:34 +00:00
parent 90c5d14695
commit 093c24b9a7
2 changed files with 8 additions and 6 deletions

View File

@ -1,3 +1,10 @@
2001-01-11 Dan Winship <danw@ximian.com>
* camel-folder.c (camel_folder_copy_message_to): Fix a locking
problem. (One branch of an if statement was returning with the
lock still locked.) Also remove the deprecation comments, to match
move_message_to.
2001-01-10 Jeffrey Stedfast <fejj@helixcode.com>
* providers/local/camel-local-provider.c: Changed the descriptions

View File

@ -1095,9 +1095,6 @@ copy_message_to (CamelFolder *source, const char *uid, CamelFolder *dest, CamelE
* This copies a message from one folder to another. If the @source and
* @dest folders have the same parent_store, this may be more efficient
* than a camel_folder_append_message().
*
* FIXME: This call should be deprecated, as append_message() can determine
* this information for itself.
**/
void
camel_folder_copy_message_to (CamelFolder *source, const char *uid,
@ -1107,12 +1104,10 @@ camel_folder_copy_message_to (CamelFolder *source, const char *uid,
g_return_if_fail (CAMEL_IS_FOLDER (dest));
g_return_if_fail (uid != NULL);
g_warning("CamelFolder.copy_message_to() is a deprecated api");
CAMEL_FOLDER_LOCK(source, lock);
if (source->parent_store == dest->parent_store)
return CF_CLASS (source)->copy_message_to (source, uid, dest, ex);
CF_CLASS (source)->copy_message_to (source, uid, dest, ex);
else
copy_message_to (source, uid, dest, ex);