GNU bug report logs - #78813
feature/igc: [PATCH] Make the array face_attr_sym readonly

Previous Next

Package: emacs;

Reported by: Helmut Eller <eller.helmut <at> gmail.com>

Date: Tue, 17 Jun 2025 08:50:02 UTC

Severity: normal

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Helmut Eller <eller.helmut <at> gmail.com>
Subject: bug#78813: closed (Re: bug#78813: feature/igc: [PATCH] Make the
 array face_attr_sym readonly)
Date: Mon, 23 Jun 2025 11:16:04 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#78813: feature/igc: [PATCH] Make the array face_attr_sym readonly

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 78813 <at> debbugs.gnu.org.

-- 
78813: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=78813
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Eli Zaretskii <eliz <at> gnu.org>
To: Helmut Eller <eller.helmut <at> gmail.com>
Cc: 78813-done <at> debbugs.gnu.org, pipcet <at> protonmail.com
Subject: Re: bug#78813: feature/igc: [PATCH] Make the array face_attr_sym
 readonly
Date: Mon, 23 Jun 2025 14:15:15 +0300
> From: Helmut Eller <eller.helmut <at> gmail.com>
> Cc: pipcet <at> protonmail.com,  78813 <at> debbugs.gnu.org
> Date: Sun, 22 Jun 2025 20:25:44 +0200
> 
> On Sat, Jun 21 2025, Eli Zaretskii wrote:
> 
> > Initialization using C initializers is used for simple values like
> > scalars.  The LISP_INITIALLY stuff should be used only very sparingly
> > and if strictly necessary, because it is very confusing at first
> > sight.  So I'd very much prefer that we kept the current conventions
> > for initializing complex variables, especially variables with Lisp
> > values.
> 
> This doesn't make much sense to me.  But your project, your rules.
> 
> So you can close this bug.

Done, thanks.

[Message part 3 (message/rfc822, inline)]
From: Helmut Eller <eller.helmut <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: feature/igc: [PATCH] Make the array face_attr_sym readonly
Date: Tue, 17 Jun 2025 10:49:10 +0200
[Message part 4 (text/plain, inline)]
Tags: patch

This is a proposal to make the variable face_attr_sym const.  Mostly so
that it's more obvious that it doesn't need to be traced by the GC.

[0001-Make-the-array-face_attr_sym-readonly.patch (text/x-diff, attachment)]

This bug report was last modified 25 days ago.

Previous Next


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