Skip to content

Conversation

@gahr
Copy link
Member

@gahr gahr commented May 9, 2023

We only want to change msg_new if we're gathering stats. If we are only checking whether the mailbox has any new messages, we'll stop at the first new message we encounter, so msg_new would always be 1.

We've been touching this code lately, so I'm unsure if I'm breaking something else..

Fixes #3843

We only want to change msg_new if we're gathering stats. If we are only
checking whether the mailbox has any new messages, we'll stop at the
first new message we encounter, so msg_new would always be 1.

Fixes #3843
@gahr gahr requested a review from a team as a code owner May 9, 2023 13:06
@lucilanga
Copy link
Contributor

It does work for me in the scope of #3843. Thank you!
I do not see any immediate breakage but I need to test this more.

Possibly related: #3648, although not directly as it mentioned different set of expandos.

@gahr
Copy link
Member Author

gahr commented May 9, 2023

Possibly related: #3648, although not directly as it mentioned different set of expandos.

Thanks!

@flatcap flatcap merged commit b5a30f0 into main May 9, 2023
@flatcap flatcap deleted the devel/issue-3843 branch May 9, 2023 14:29
@flatcap flatcap added the type:bug Bug label May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sidebar %Z expando is wrong for maildir

4 participants