# HG changeset patch
# User Timo Sirainen <timo.sirainen@dovecot.fi>
# Date 1487341912 -7200
# Node ID e76e742bfbab3eede4e4bd30af52947ae99d2c4b
# Parent  47ab3e3483be3944eafa4e3e3e4983f9e47c368e
lib-storage: Change vsize calculation to use the new mail_stream_opened

This avoids calling mail_get_virtual_size() twice and is a bit cleaner
approach.

diff -r 47ab3e3483be -r e76e742bfbab src/lib-storage/index/index-mailbox-size.c
--- a/src/lib-storage/index/index-mailbox-size.c	Fri Feb 17 16:20:06 2017 +0200
+++ b/src/lib-storage/index/index-mailbox-size.c	Fri Feb 17 16:31:52 2017 +0200
@@ -323,30 +323,30 @@
 		mails_left = update->box->storage->set->mail_vsize_bg_after_count;
 
 	while (mailbox_search_next(search_ctx, &mail)) {
-		if (mails_left == UINT_MAX) {
-			/* we want to build the full vsize here */
-			ret = mail_get_virtual_size(mail, &vsize);
-		} else {
-			/* if vsize building wants to open too many mails from
-			   storage, return temporary failure and finish up the
-			   calculation in background. */
+		if (mails_left == 0) {
+			/* if there are any more mails whose vsize can't be
+			   looked up from cache, abort and finish on
+			   background. */
 			mail->lookup_abort = MAIL_LOOKUP_ABORT_NOT_IN_CACHE;
-			ret = mail_get_virtual_size(mail, &vsize);
-			mail->lookup_abort = MAIL_LOOKUP_ABORT_NEVER;
-			if (ret < 0 &&
-			    mailbox_get_last_mail_error(update->box) == MAIL_ERROR_NOTPOSSIBLE) {
-				/* size isn't in cache. */
-				if (mails_left == 0) {
-					mail_storage_set_error(update->box->storage, MAIL_ERROR_INUSE,
-						"Finishing vsize calculation on background");
-					if (require_result)
-						update->finish_in_background = TRUE;
-					break;
-				}
-				mails_left--;
-				ret = mail_get_virtual_size(mail, &vsize);
-			}
 		}
+		ret = mail_get_virtual_size(mail, &vsize);
+		if (ret < 0 &&
+		    mailbox_get_last_mail_error(update->box) == MAIL_ERROR_NOTPOSSIBLE) {
+			/* abort and finish on background */
+			i_assert(mails_left == 0);
+
+			mail_storage_set_error(update->box->storage, MAIL_ERROR_INUSE,
+				"Finishing vsize calculation on background");
+			if (require_result)
+				update->finish_in_background = TRUE;
+			break;
+		}
+		if (mail->mail_stream_opened || mail->mail_metadata_accessed) {
+			/* slow vsize lookup */
+			i_assert(mails_left > 0);
+			mails_left--;
+		}
+
 		if (ret < 0) {
 			if (mail->expunged)
 				continue;