From: Timo Sirainen <timo.sirainen@dovecot.fi>
Date: Thu, 5 Oct 2017 10:17:59 +0000 (+0300)
Subject: mbox: Remove unnecessary check from assert - move_diff is always negative here
X-Git-Tag: 2.3.0.rc1~902
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=85cf1c27bcb9b38831a3cb4267f9fc814396f7be;p=thirdparty%2Fdovecot%2Fcore.git

mbox: Remove unnecessary check from assert - move_diff is always negative here
---

diff --git a/src/lib-storage/index/mbox/mbox-sync.c b/src/lib-storage/index/mbox/mbox-sync.c
index e5c8fef3db..fd09d2133c 100644
--- a/src/lib-storage/index/mbox/mbox-sync.c
+++ b/src/lib-storage/index/mbox/mbox-sync.c
@@ -673,8 +673,7 @@ static int mbox_sync_handle_header(struct mbox_sync_mail_context *mail_ctx)
 		if (ret > 0) {
 			/* rewrite successful, write From-line to
 			   new location */
-			i_assert(move_diff > 0 ||
-				 (off_t)mail_ctx->mail.from_offset >=
+			i_assert((off_t)mail_ctx->mail.from_offset >=
 				 -move_diff);
 			mail_ctx->mail.from_offset += move_diff;
 			mail_ctx->mail.offset += move_diff;