dovecot-2.2: virtual: Recent flags dropping wasn't working as in...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-2.2/rev/abf049058125
changeset: 17575:abf049058125
user:      Timo Sirainen <tss at iki.fi>
date:      Thu Jul 03 14:54:43 2014 +0300
description:
virtual: Recent flags dropping wasn't working as intended.
In the old code '+' meant that \Recent flags were dropped also when the
virtual mailbox was EXAMINEd. SELECTing a mailbox always dropped \Recent
flags regardless of the '+' flag.

What should have happened (and does in new code) is that the \Recent flags
are dropped only on SELECT and only if '+' flag is set.

diffstat:

 src/plugins/virtual/virtual-storage.c |  4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diffs (14 lines):

diff -r 9c6643daae98 -r abf049058125 src/plugins/virtual/virtual-storage.c
--- a/src/plugins/virtual/virtual-storage.c	Thu Jul 03 14:37:08 2014 +0300
+++ b/src/plugins/virtual/virtual-storage.c	Thu Jul 03 14:54:43 2014 +0300
@@ -156,8 +156,8 @@
 
 	i_assert(bbox->box == NULL);
 
-	if (bbox->clear_recent)
-		flags |= MAILBOX_FLAG_DROP_RECENT;
+	if (!bbox->clear_recent)
+		flags &= ~MAILBOX_FLAG_DROP_RECENT;
 
 	mailbox = bbox->name;
 	ns = mail_namespace_find(user->namespaces, mailbox);