[evolution-kolab/ek-wip-porting] CamelIMAPXExtdFolder: object types cleanup
- From: Christian Hilberg <chilberg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution-kolab/ek-wip-porting] CamelIMAPXExtdFolder: object types cleanup
- Date: Fri, 13 Jan 2012 19:02:31 +0000 (UTC)
commit 14925ebb44f7c938bd9680edeb33aa9b0ae589cd
Author: Christian Hilberg <hilberg kernelconcepts de>
Date: Fri Jan 13 19:37:54 2012 +0100
CamelIMAPXExtdFolder: object types cleanup
* in imapx_extd_folder_transfer_messages_to_sync(), check
for CamelFolder instead of CamelIMAPXExtdFolder
* this function seems to be more of a utility function
and may be removed from the class functions
.../providers/imapx/camel-imapx-extd-folder.c | 10 ++++++++--
1 files changed, 8 insertions(+), 2 deletions(-)
---
diff --git a/src/camel/providers/imapx/camel-imapx-extd-folder.c b/src/camel/providers/imapx/camel-imapx-extd-folder.c
index 501d5ba..4c351ec 100644
--- a/src/camel/providers/imapx/camel-imapx-extd-folder.c
+++ b/src/camel/providers/imapx/camel-imapx-extd-folder.c
@@ -278,9 +278,15 @@ imapx_extd_folder_transfer_messages_to_sync (CamelFolder *source,
{
gboolean ok = FALSE;
- g_assert (CAMEL_IS_IMAPX_EXTD_FOLDER (source));
+ /* TODO is CamelFolder always CamelIMAPXExtdFolder
+ * here, so we can assert that? looks as though
+ * this one is more of a utility function than
+ * a CamelIMAPXFolder class function...
+ */
+
+ g_assert (CAMEL_IS_FOLDER (source));
g_assert (message_uids != NULL);
- g_assert (CAMEL_IS_IMAPX_EXTD_FOLDER (destination));
+ g_assert (CAMEL_IS_FOLDER (destination));
g_assert (*transferred_uids == NULL);
/* cancellable may be NULL */
g_return_val_if_fail (err == NULL || *err == NULL, FALSE);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]