Bug#751476: mutt: 1.5.22 major performance regression with color regexp matching

2015-04-25 Thread Kevin J. McCarthy
This may be fixed by http://dev.mutt.org/hg/mutt/rev/755a18da99bc just
committed upstream.  ~b has a pretty big performance regression fixed by
that changeset.

-Kevin


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#751476: mutt: 1.5.22 major performance regression with color regexp matching

2014-06-13 Thread Martin Pitt
Package: mutt
Version: 1.5.22-1

Hello,

I recently upgraded mutt from 1.5.21-6.4 to 1.5.23-1. Before,
switching folders with 'c' took much less than a second (I see
Mailbox unchanged for a split second, then the new mail folder
appeared. After the upgrade it is annoyingly slow, I see sorting
mailbox.. for about 5 seconds during which mutt uses a lot of CPU.

I installed the intermediate 1.5.22-1 version and this is also slow,
so this is the version that introduced the regression.

I'm using Maildir format, have some 10 folders, with  50 mails in
each folder, so it shouldn't really take a noticeable time.

As a reproducer, I attach a folder with a couple of Debian bugs, so
it's all public:

  cd /tmp
  tar xf testfolder.tar.gz
  mutt -f /tmp/testfolder

This will sit there for several seconds sorting through the 18 mails
for my mutt configuration. That doesn't happen without any ~/.muttrc,
so I bisected what's causing this and found that it's the regular
expression matching:

- .muttrc ---
color index default yellow '~b Status: .+ Incomplete'
color index yellow  default '~b This bug is a duplicate of 
bug'
color index default green '~b Status: .+ Fix Released'
color index red green '~b Status: .+ Invalid'
color index default red '~b Status: .+Fix Released .'
color index default red '~b Status: .+Invalid .'
color index default red '~b Status: .+ Expired'


I use that for coloring (Launchpad) bug mail based on their state
changes, but I don't suppose it's specific to the particular text.

Thanks,

Martin

-- 
Martin Pitt| http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)


testfolder.tar.gz
Description: application/gzip


signature.asc
Description: Digital signature