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


View this message in rfc822 format

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Gregory Heytings <gregory <at> heytings.org>, Eli Zaretskii <eliz <at> gnu.org>
Cc: 56682 <at> debbugs.gnu.org, monnier <at> iro.umontreal.ca
Subject: bug#56682: Fix the long lines font locking related slowdowns
Date: Sat, 6 Aug 2022 23:04:03 +0300
[Message part 1 (text/plain, inline)]
On 06.08.2022 14:49, Gregory Heytings wrote:
> 
>>> Just like depicted on my latest screenshot for downloadify.js: the 
>>> beginning of the narrowed region ended up inside a string, and as a 
>>> result several screenfuls were entirely mis-fontified, with strings 
>>> and non-strings inverted.
>>
>> Yes, and IMNSHO responsiveness is more important than correctness in 
>> these cases.
>>
> 
> Especially given that the said correctness has always been relative.  So 
> it's not like we're leaving the aven of 100% correctness and falling 
> into the hell of 10% correctness.  A more reasonable view of the 
> situation is that we had 90% correctness and now have,

As someone who worked on different major mode and syntax-ppss itself a 
little, that feels moderately insulting.

No: we strive to close to 100% correctness in supporting language syntax 
and can often reach it with moderate effort (programming-wise).

> in files with 
> "loo long lines" and only in those files, 60% correctness.

Does the attached screenshot look like 60% correctness to you?

To me, it's more like -60%. Or at least, that's what the utility of such 
highlighting will be (negative).

And I didn't have to hunt for this screenshot, tweaking parameters, or 
files, or window sizes here and there.

This is just how half of dictionary.json looks with Emacs from master on 
default settings.
[Screenshot from 2022-08-06 22-55-58.png (image/png, attachment)]

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.