[dovecot-cvs] dovecot/src/lib-index mail-index-view-sync.c, 1.52.2.2, 1.52.2.3

[tss at dovecot.org]

Update of /var/lib/cvs/dovecot/src/lib-index
In directory talvi:/tmp/cvs-serv9349

Modified Files:
      Tag: branch_1_0
	mail-index-view-sync.c 
Log Message:
If we didn't sync everything in the view, we could have synced some
transactions multiple times causing different kinds of problems.



Index: mail-index-view-sync.c
===================================================================
RCS file: /var/lib/cvs/dovecot/src/lib-index/mail-index-view-sync.c,v
retrieving revision 1.52.2.2
retrieving revision 1.52.2.3
diff -u -d -r1.52.2.2 -r1.52.2.3
--- mail-index-view-sync.c	15 Oct 2006 17:56:06 -0000	1.52.2.2
+++ mail-index-view-sync.c	31 Dec 2006 16:08:56 -0000	1.52.2.3
@@ -377,6 +377,15 @@
 		if (view_sync_pos_find(&view->syncs_done, seq, offset))
 			continue;
 
+		if (ctx->skipped_some) {
+			/* We've been skipping some transactions, which means
+			   we'll go through these same transactions again
+			   later. Since we're syncing this one, we don't want
+			   to do it again. */
+			mail_index_view_add_synced_transaction(view, seq,
+							       offset);
+		}
+
 		/* Apply transaction to view's mapping if needed (meaning we
 		   didn't just re-map the view to head mapping). */
 		if (ctx->sync_map_update) {
@@ -400,13 +409,6 @@
 			continue;
 		break;
 	}
-
-	if (ctx->skipped_some) {
-		/* We've been skipping some transactions, which means we'll
-		   go through these same transaction again later. Since we're
-		   syncing this one, we don't want to do it again. */
-		mail_index_view_add_synced_transaction(view, seq, offset);
-	}
 	return 1;
 }