[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#56682: locked narrowing
From: |
Stefan Monnier |
Subject: |
bug#56682: locked narrowing |
Date: |
Tue, 16 Aug 2022 17:50:39 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
Stefan Monnier [2022-08-16 17:42:45] wrote:
> Gregory Heytings [2022-08-16 20:23:46] wrote:
>> Another comment:
>>> diff --git a/nlinum.el b/nlinum.el
>>> index 4f0e02fef1..3feaaca5c3 100644
>>> --- a/nlinum.el
>>> +++ b/nlinum.el
>>> @@ -312,7 +312,7 @@ Only works right if point is at BOL."
>>> (if nlinum-widen
>>> (save-excursion
>>> (save-restriction
>>> - (widen)
>>> + (REALLY-widen)
>>> (forward-line 0) ;In case (point-min) was not at BOL.
>>> (let ((nlinum-widen nil))
>>> (nlinum--line-number-at-pos))))
>> FWIW, that's not a good example, because it's a bad idea to calculate line
>> numbers in buffers with long lines or with too many lines: it takes too
>> much time.
> In my tests, it works fast enough for pretty large buffers and pretty
> long lines,
And actually, it doesn't seem to be slowed by long lines, only by the
buffer size. So its `widen` should definitely override the
line-length-induced narrowing (tho maybe not the buffer-size-induced
narrowing if/when we add such a thing).
Stefan
- bug#56682: Fix the long lines font locking related slowdowns, (continued)
- bug#56682: Fix the long lines font locking related slowdowns, Stefan Monnier, 2022/08/16
- bug#56682: Fix the long lines font locking related slowdowns, Eli Zaretskii, 2022/08/16
- bug#56682: Fix the long lines font locking related slowdowns, Stefan Monnier, 2022/08/16
- bug#56682: Fix the long lines font locking related slowdowns, Eli Zaretskii, 2022/08/16
- bug#56682: locked narrowing (was: bug#56682: Fix the long lines font locking related slowdowns), Stefan Monnier, 2022/08/16
- bug#56682: locked narrowing (was: bug#56682: Fix the long lines font locking related slowdowns), Gregory Heytings, 2022/08/16
- bug#56682: locked narrowing, Stefan Monnier, 2022/08/16
- bug#56682: locked narrowing, Gregory Heytings, 2022/08/16
- bug#56682: locked narrowing (was: bug#56682: Fix the long lines font locking related slowdowns), Gregory Heytings, 2022/08/16
- bug#56682: locked narrowing, Stefan Monnier, 2022/08/16
- bug#56682: locked narrowing,
Stefan Monnier <=
- bug#56682: locked narrowing, dick, 2022/08/17
- bug#56682: locked narrowing (was: bug#56682: Fix the long lines font locking related slowdowns), Eli Zaretskii, 2022/08/17
- bug#56682: locked narrowing, Stefan Monnier, 2022/08/17
- bug#56682: locked narrowing, Eli Zaretskii, 2022/08/17
- bug#56682: locked narrowing, Stefan Monnier, 2022/08/17
- bug#56682: locked narrowing, Eli Zaretskii, 2022/08/17
- bug#56682: locked narrowing, Stefan Monnier, 2022/08/17
- bug#56682: locked narrowing, Eli Zaretskii, 2022/08/17
- bug#56682: locked narrowing, Stefan Monnier, 2022/08/17
- bug#56682: locked narrowing, Eli Zaretskii, 2022/08/17