Mercurial > dovecot > original-hg > dovecot-1.2

changeset 6094:b9f7eb84d4be HEAD

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fixes to MAIL_INDEX_TRANSACTION_FLAG_AVOID_FLAG_UPDATES handling.
author Timo Sirainen <tss@iki.fi>
date Thu, 19 Jul 2007 03:11:21 +0300
parents 76da3b271bcf
children b6a477859a3c
files src/lib-index/mail-index-transaction.c
diffstat 1 files changed, 24 insertions(+), 16 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib-index/mail-index-transaction.c	Thu Jul 19 02:56:40 2007 +0300
+++ b/src/lib-index/mail-index-transaction.c	Thu Jul 19 03:11:21 2007 +0300
@@ -673,10 +673,12 @@
 			i_assert(tmp_update.uid1 <= tmp_update.uid2);
 			i_assert(updates[idx].uid1 <= updates[idx].uid2);
 
-			if (mail_transaction_update_want_add(t, &tmp_update))
+			if (mail_transaction_update_want_add(t, &tmp_update)) {
 				array_insert(&t->updates, idx, &tmp_update, 1);
-			updates = array_get_modifiable(&t->updates, &count);
-			idx += move;
+				updates = array_get_modifiable(&t->updates,
+							       &count);
+				idx += move;
+			}
 		} else if (u.uid1 < updates[idx].uid1) {
 			updates[idx].uid1 = u.uid1;
 		}
@@ -724,10 +726,12 @@
 	if (u.uid1 <= u.uid2) {
 		i_assert(idx == 0 || updates[idx-1].uid2 < u.uid1);
 		i_assert(idx == count || updates[idx].uid1 > u.uid2);
-		if (mail_transaction_update_want_add(t, &u))
+		if (mail_transaction_update_want_add(t, &u)) {
 			array_insert(&t->updates, idx, &u, 1);
+			count++;
+		}
 	}
-	t->last_update_idx = idx;
+	t->last_update_idx = idx == count ? count-1 : idx;
 }
 
 static void mail_index_record_modify_flags(struct mail_index_record *rec,
@@ -754,7 +758,7 @@
 {
 	struct mail_index_record *rec;
 	struct mail_transaction_flag_update u, *last_update;
-	unsigned int count;
+	unsigned int first_idx, count;
 
 	if (seq2 >= t->first_new_seq) {
 		/* updates for appended messages, modify them directly */
@@ -822,17 +826,21 @@
 	if (t->last_update_idx == count) {
 		if (mail_transaction_update_want_add(t, &u))
 			array_append(&t->updates, &u, 1);
-		return;
-	}
+		else if (t->last_update_idx > 0)
+			t->last_update_idx--;
+	} else {
+		i_assert(t->last_update_idx < count);
 
-	/* slow path */
-	if (seq1 > last_update->uid2) {
-		/* added after this */
-		mail_index_insert_flag_update(t, u, t->last_update_idx + 1,
-					      count);
-	} else {
-		/* added before this or on top of this */
-		mail_index_insert_flag_update(t, u, 0, t->last_update_idx + 1);
+		/* slow path */
+		if (seq1 > last_update->uid2) {
+			/* added after this */
+			first_idx = t->last_update_idx + 1;
+		} else {
+			/* added before this or on top of this */
+			first_idx = 0;
+			count = t->last_update_idx + 1;
+		}
+		mail_index_insert_flag_update(t, u, first_idx, count);
 	}
 }