GNU bug report logs - #36712
27.0.50; nnimap unconditionally setting marks in a loop

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Eric Abrahamsen <eric <at> ericabrahamsen.net>
Subject: bug#36712: closed (Re: bug#36712: Acknowledgement (27.0.50;
 nnimap unconditionally setting marks in a loop))
Date: Thu, 25 Jul 2019 19:27:03 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#36712: 27.0.50; nnimap unconditionally setting marks in a loop

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 36712 <at> debbugs.gnu.org.

-- 
36712: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=36712
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Eric Abrahamsen <eric <at> ericabrahamsen.net>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 36712 <at> debbugs.gnu.org, 36712-done <at> debbugs.gnu.org
Subject: Re: bug#36712: Acknowledgement (27.0.50; nnimap unconditionally
 setting marks in a loop)
Date: Thu, 25 Jul 2019 12:26:34 -0700

On 07/25/19 19:03 PM, Lars Ingebrigtsen wrote:
> Eric Abrahamsen <eric <at> ericabrahamsen.net> writes:
>
>> Well, I'm fairly sure now this is the right thing to do. I'll leave this
>> open for a few more days, then push a fix.
>
> I think your patch looks correct...

Okay, I went ahead and pushed it.

Thanks!

[Message part 3 (message/rfc822, inline)]

From: Eric Abrahamsen <eric <at> ericabrahamsen.net>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50; nnimap unconditionally setting marks in a loop
Date: Wed, 17 Jul 2019 18:09:45 -0700

[Message part 4 (text/plain, inline)]

I started doing some very basic profiling of Gnus, and noticed that
getting new mail was resulting in a couple thousand calls to
`gnus-info-set-entry', as well as some of the range manipulation
functions.

It looks like updating IMAP group marks using qresync would loop over
all possible flag/mark pairs and unconditionally set everything, even
when there was nothing to set. With fewer than 100 IMAP groups in total,
that was a couple thousand extra function calls. Most of them with nil
arguments, but still.

I'm not super familiar with the code in question, but the attached diff
looks like it will pull the qresync behavior in line with the
non-qresync behavior, so I'm hoping it's correct. It cuts way down on
function calls, anyway!




In GNU Emacs 27.0.50 (build 4, x86_64-pc-linux-gnu, GTK+ Version 3.24.10)
 of 2019-07-15 built on slip
Repository revision: c687f25ee24bd29d623a23819eef930937252b25
Windowing system distributor 'The X.Org Foundation', version 11.0.12005000
System Description:	Arch Linux

[nnimap-dont-add-marks.diff (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.