dovecot-1.2: mail_index_lookup_first() didn't always find the re...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-1.2/rev/3453dbc2e42e
changeset: 8724:3453dbc2e42e
user:      Timo Sirainen <tss at iki.fi>
date:      Thu Feb 05 18:29:27 2009 -0500
description:
mail_index_lookup_first() didn't always find the result.
This caused SELECT not to return UNSEEN number always.

diffstat:

1 file changed, 8 insertions(+), 7 deletions(-)
src/lib-index/mail-index-view.c |   15 ++++++++-------

diffs (38 lines):

diff -r eaffedbc23f2 -r 3453dbc2e42e src/lib-index/mail-index-view.c
--- a/src/lib-index/mail-index-view.c	Thu Feb 05 11:54:53 2009 -0500
+++ b/src/lib-index/mail-index-view.c	Thu Feb 05 18:29:27 2009 -0500
@@ -304,26 +304,27 @@ static void view_lookup_first(struct mai
 {
 #define LOW_UPDATE(x) \
 	STMT_START { if ((x) > low_uid) low_uid = x; } STMT_END
+	const struct mail_index_header *hdr = &view->map->hdr;
 	const struct mail_index_record *rec;
-	uint32_t seq, low_uid = 1;
+	uint32_t seq, seq2, low_uid = 1;
 
 	*seq_r = 0;
 
 	if ((flags_mask & MAIL_SEEN) != 0 && (flags & MAIL_SEEN) == 0)
-		LOW_UPDATE(view->map->hdr.first_unseen_uid_lowwater);
+		LOW_UPDATE(hdr->first_unseen_uid_lowwater);
 	if ((flags_mask & MAIL_DELETED) != 0 && (flags & MAIL_DELETED) != 0)
-		LOW_UPDATE(view->map->hdr.first_deleted_uid_lowwater);
+		LOW_UPDATE(hdr->first_deleted_uid_lowwater);
 
 	if (low_uid == 1)
 		seq = 1;
 	else {
-		if (!mail_index_lookup_seq(view, low_uid, &seq))
+		if (!mail_index_lookup_seq_range(view, low_uid, hdr->next_uid,
+						 &seq, &seq2))
 			return;
 	}
 
-	i_assert(view->map->hdr.messages_count <=
-		 view->map->rec_map->records_count);
-	for (; seq <= view->map->hdr.messages_count; seq++) {
+	i_assert(hdr->messages_count <= view->map->rec_map->records_count);
+	for (; seq <= hdr->messages_count; seq++) {
 		rec = MAIL_INDEX_MAP_IDX(view->map, seq-1);
 		if ((rec->flags & flags_mask) == (uint8_t)flags) {
 			*seq_r = seq;