GNU bug report logs - #57176
29.0.50; Commit 744ed0ce applies outline-minor-mode-highlight-buffer repeatedly

Previous Next

Package: emacs;

Reported by: daanturo <daanturo <at> gmail.com>

Date: Sat, 13 Aug 2022 14:13:02 UTC

Severity: normal

Merged with 57175

Fixed in version 29.0.50

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Stephen Berman <stephen.berman <at> gmx.net>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: daanturo <daanturo <at> gmail.com>, 57176 <at> debbugs.gnu.org
Subject: bug#57176: 29.0.50; Commit 744ed0ce applies outline-minor-mode-highlight-buffer repeatedly
Date: Wed, 17 Aug 2022 09:57:59 +0200
On Mon, 15 Aug 2022 09:42:33 +0200 Lars Ingebrigtsen <larsi <at> gnus.org> wrote:

> daanturo <daanturo <at> gmail.com> writes:
>
>> I think we should clear old outline overlays before applying new ones,
>> or adjust the conditions like the old one.
>
> I guess this might be the result of this change?
>
> -          (if (and global-font-lock-mode (font-lock-specified-p major-mode))
> -              (progn
> -                (font-lock-add-keywords nil outline-font-lock-keywords t)
> -                (font-lock-flush))
> -            (outline-minor-mode-highlight-buffer)))
> +          (when (and global-font-lock-mode (font-lock-specified-p major-mode))
> +            (font-lock-add-keywords nil outline-font-lock-keywords t)
> +            (font-lock-flush))
> +          (outline-minor-mode-highlight-buffer))
>
> I've added Stephen to the CCs; perhaps he has some comments.

Sorry for the late response.  Juri Linkov already noted a probably
related problem with my patch, see <86sfm4q6nu.fsf <at> mail.linkov.net>.
I'm afraid my reply to his message still holds:

>> I'm not familiar enough with the intricacies of font-lock to whip
>> up a quick fix and can't afford to spend a lot of time on it, so I hope
>> someone better qualified than me will do it.

Juri replied to that:

> I guess what is needed here is to find the right place to call
> outline--fix-up-all-buttons on first visiting the NEWS buffer.

but I haven't had time to look into that, nor whether that would also
work for bug#57176.

Steve Berman




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

Previous Next


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