IMAPX: Don't pass GError when not checking for error.
authorMatthew Barnes <mbarnes@redhat.com>
Wed, 14 Aug 2013 17:10:51 +0000 (13:10 -0400)
committerMatthew Barnes <mbarnes@redhat.com>
Thu, 15 Aug 2013 09:47:55 +0000 (05:47 -0400)
Error checking around CamelDB and CamelFolderSummary calls are virtually
non-existent, but that's a mess for another day.  In the meantime, avoid
blindly passing GErrors where we don't bother checking for errors, since
that leads to GError overwrite warnings.

camel/camel-imapx-server.c

index 4e01729..39cba6b 100644 (file)
@@ -5906,7 +5906,7 @@ imapx_command_expunge_done (CamelIMAPXServer *is,
                parent_store = camel_folder_get_parent_store (folder);
 
                camel_folder_summary_save_to_db (folder->summary, NULL);
-               uids = camel_db_get_folder_deleted_uids (parent_store->cdb_r, full_name, error);
+               uids = camel_db_get_folder_deleted_uids (parent_store->cdb_r, full_name, NULL);
 
                if (uids && uids->len)  {
                        CamelFolderChangeInfo *changes;
@@ -6654,7 +6654,7 @@ imapx_command_sync_changes_done (CamelIMAPXServer *is,
                        }
                }
 
-               camel_folder_summary_save_to_db (folder->summary, error);
+               camel_folder_summary_save_to_db (folder->summary, NULL);
                camel_store_summary_save ((CamelStoreSummary *)((CamelIMAPXStore *) parent_store)->summary);
 
                imapx_unregister_job (is, job);