GNU bug report logs - #56682
Fix the long lines font locking related slowdowns

Previous Next

Package: emacs;

Reported by: Gregory Heytings <gregory <at> heytings.org>

Date: Thu, 21 Jul 2022 18:01:01 UTC

Severity: normal

Done: Gregory Heytings <gregory <at> heytings.org>

Bug is archived. No further changes may be made.

Full log


Message #1357 received at 56682 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Gregory Heytings <gregory <at> heytings.org>
Cc: 56682 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
 Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#56682: Fix the long lines font locking related slowdowns
Date: Sat, 13 Aug 2022 22:10:20 +0300
On 13.08.2022 20:16, Gregory Heytings wrote:
> 
>>> As I just said to Dmitry: the same multi-second delays we saw in 
>>> smaller JSON files.  And those who have a slower CPU will see these 
>>> multi-second delays with smaller files.
>>
>> No exactly the same ones, though, since the previous tests we did were 
>> done with the older version of JSON support (not the current 
>> js-json-mode), with somewhat different performance characteristics.
>>
> 
> Okay, not "the same ones", but delays of the same order of magnitude 
> nonetheless.  We had that discussion a month ago, and concluded that 
> local fixes in every major (and minor) modes were not the way to go.  
> Having that discussion again does not seem fruitful.

Did you have that discussion with other people who have not authored a 
single major mode? That's ridiculous.

> You improved 
> js-mode and created json-mode, that's fine, but it doesn't change 
                      ^
                       js-json-mode

> anything to the way all other major (and minor) modes behave.

It showed specific pitfalls we can take care of in other major modes as 
well.




This bug report was last modified 2 years and 8 days ago.

Previous Next


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