GNU bug report logs - #73862
[PATCH] Add `header-line-active` and `header-line-inactive` faces.

Previous Next

Full log

View this message in rfc822 format

From: Aaron Jensen <aaronjensen <at> gmail.com>
To: Eshel Yaron <me <at> eshelyaron.com>
Cc: trevor.m.murphy <at> gmail.com, Eli Zaretskii <eliz <at> gnu.org>, monnier <at> iro.umontreal.ca, 73862 <at> debbugs.gnu.org
Subject: bug#73862: [PATCH] Add `header-line-active` and `header-line-inactive` faces.
Date: Sun, 8 Dec 2024 12:26:29 -0500

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

On Sun, Dec 08, 2024 at 8:29 AM, Eshel Yaron <me <at> eshelyaron.com> wrote:

> Eli Zaretskii <eliz <at> gnu.org> writes:
>
> From: Eshel Yaron <me <at> eshelyaron.com>
> Cc: Aaron Jensen <aaronjensen <at> gmail.com>, monnier <at> iro.umontreal.ca,
> trevor.m.murphy <at> gmail.com, 73862 <at> debbugs.gnu.org
> Date: Sun, 08 Dec 2024 15:11:07 +0100
>
> Eli Zaretskii <eliz <at> gnu.org> writes:
>
> We also haven't yet heard from Eshel.
>
> I don't have any concrete suggestions at this point. It's surely a
> complicated situation.
>
> Did you try the patch I posted, and if you did, did it resolve your
> original problems?
>
> I did now, and it doesn't entirely solve the problem: with this patch,
> after remapping header-line in one buffer, the remapping now works for that
> buffer as expected, but it still also "spreads" to other buffers, which is
> unexpected.
>

I see the leak now as well. When I tested it at first, I did not see it
perhaps because I did not switch back to the original window and trigger a
computation of the basic faces. That's what causes the leak.

Steps:

(setq header-line-format "Some header")
(face-remap-set-base 'header-line 'highlight)
(switch-to-buffer-other-window "new")
;; In new buffer/window:
(setq header-line-format "Some header")
(other-window)
;; In original buffer/window:
(set-face-attribute 'default nil :height (+ (face-attribute 'default
:height) 10))

Aaron

[Message part 2 (text/html, inline)]

Previous Next

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