GNU bug report logs -
#72949
[PATCH] Wrong unread count in the Group buffer in nnimap
Previous Next
Reported by: James Thomas <jimjoe <at> gmx.net>
Date: Mon, 2 Sep 2024 06:01:02 UTC
Severity: normal
Tags: fixed, patch
Fixed in version 31.1
Done: Robert Pluim <rpluim <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
On Sep 12, 2024, James Thomas <jimjoe <at> gmx.net> wrote:
>> Sorry! I got confused again... Please try the other patch in that bug
>> report (the one that swaps the car and cdr). I find that it works with
>> your use case!
Yes, that patch fixes the problem I've had for ages. But I don't think
that it's correct. Here's what is done in nnml.el:
(nnheader-insert "211 %d %d %d %s\n"
(max (1+ (- (cdr active) (car active))) 0)
(car active) (cdr active) group)))))))
Notice that the car comes before the cdr, and that you have to add 1
to the difference between the cdr and the car, since if the active
range is 2 . 3, then there are 2 active articles, which is one more
than 3 - 2. So I suspect that the patch attached below might be
correct. Someone who knows more about what the "211" line should
contain should review this.
My patch also fixes the problem I've had. With either patch, after
moving or copying an article to a new group, the new group has (active 1
. 1) as expected, with no unexist entry.
But the two patches would behave differently when active was already set.
Dan
[nnimap.patch (text/x-diff, attachment)]
This bug report was last modified 275 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.