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

Previous Next

Package: emacs;

Reported by: Eric Abrahamsen <eric <at> ericabrahamsen.net>

Date: Thu, 18 Jul 2019 01:11:01 UTC

Severity: normal

Found in version 27.0.50

Done: Eric Abrahamsen <eric <at> ericabrahamsen.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eric Abrahamsen <eric <at> ericabrahamsen.net>
To: 36712 <at> debbugs.gnu.org
Subject: bug#36712: 27.0.50; nnimap unconditionally setting marks in a loop
Date: Wed, 17 Jul 2019 18:09:45 -0700
[Message part 1 (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)]

This bug report was last modified 5 years and 296 days ago.

Previous Next


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