changeset 6878:0c3ab0aef33c HEAD

maildir_uidlist_sync_init() was called wrong, although it just happened to work.
author Timo Sirainen <tss@iki.fi>
date Sat, 01 Dec 2007 09:09:21 +0200
parents a5a7d38b6df7
children fed2843b4ab8
files src/lib-storage/index/maildir/maildir-save.c src/lib-storage/index/maildir/maildir-sync.c
diffstat 2 files changed, 5 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib-storage/index/maildir/maildir-save.c	Thu Nov 29 19:38:06 2007 +0200
+++ b/src/lib-storage/index/maildir/maildir-save.c	Sat Dec 01 09:09:21 2007 +0200
@@ -598,7 +598,8 @@
 	}
 
 	if (ctx->locked) {
-		ret = maildir_uidlist_sync_init(ctx->mbox->uidlist, TRUE,
+		ret = maildir_uidlist_sync_init(ctx->mbox->uidlist,
+						MAILDIR_UIDLIST_SYNC_PARTIAL,
 						&ctx->uidlist_sync_ctx);
 		i_assert(ret > 0); /* already locked, shouldn't fail */
 
--- a/src/lib-storage/index/maildir/maildir-sync.c	Thu Nov 29 19:38:06 2007 +0200
+++ b/src/lib-storage/index/maildir/maildir-sync.c	Sat Dec 01 09:09:21 2007 +0200
@@ -695,6 +695,7 @@
 static int maildir_sync_context(struct maildir_sync_context *ctx, bool forced,
 				bool *lost_files_r)
 {
+	enum maildir_uidlist_sync_flags sync_flags;
 	bool new_changed, cur_changed;
 	int ret;
 
@@ -754,7 +755,8 @@
 	*/
 
 	ctx->partial = !cur_changed;
-	ret = maildir_uidlist_sync_init(ctx->mbox->uidlist, ctx->partial,
+	sync_flags = ctx->partial ? MAILDIR_UIDLIST_SYNC_PARTIAL : 0;
+	ret = maildir_uidlist_sync_init(ctx->mbox->uidlist, sync_flags,
 					&ctx->uidlist_sync_ctx);
 	if (ret <= 0) {
 		/* failure / timeout. if forced is TRUE, we could still go