[evolution-kolab/ek-wip-porting] CamelIMAPXExtdFolder: object types cleanup



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]