GNU bug report logs - #3889
23.1.50; [PATCH] Increase blink-matching-paren-distance

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (Emacs bug Tracking System)
To: Ian Eure <ian <at> digg.com>
Subject: bug#3889 closed by Glenn Morris <rgm <at> gnu.org> (Re: bug#3889:
 23.1.50; [PATCH] Increase blink-matching-paren-distance)
Date: Wed, 22 Jul 2009 02:40:05 +0000

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

This is an automatic notification regarding your bug report
which was filed against the emacs package:

#3889: 23.1.50; [PATCH] Increase blink-matching-paren-distance

It has been closed by Glenn Morris <rgm <at> gnu.org>.

Their explanation is attached below along with your original report.
If this explanation is unsatisfactory and you have not received a
better one in a separate message then please contact Glenn Morris <rgm <at> gnu.org> by
replying to this email.


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

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

From: Glenn Morris <rgm <at> gnu.org>
To: 3889-done <at> debbugs.gnu.org
Subject: Re: bug#3889: 23.1.50; [PATCH] Increase blink-matching-paren-distance
Date: Tue, 21 Jul 2009 22:35:50 -0400

Stefan Monnier wrote:

>> Attached is a patch which doubles this to 50kb, as this should be fine for
>> modern machines.
>
> 100kB should be fine, indeed.

So increased.

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

From: Ian Eure <ian <at> digg.com>
To: emacs-pretest-bug <at> gnu.org
Subject: 23.1.50; [PATCH] Increase blink-matching-paren-distance
Date: Mon, 20 Jul 2009 13:49:46 -0700

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

A user came into #emacs today complaining that his paren matching was  
not working with a large file, and that font-locking broke as a result.

I found that this is controlled by blink-matching-paren-distance,  
which was increased to 25kb around 1997.

Attached is a patch which doubles this to 50kb, as this should be fine  
for modern machines. I think that we should seriously consider setting  
the default to nil, but this seems like a more conservative approach  
to the matter.

[blink-matching-paren.patch (application/octet-stream, attachment)]

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

Previous Next

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