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

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#78813: closed (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 message dated Mon, 23 Jun 2025 14:15:15 +0300
with message-id <86pleug14s.fsf <at> gnu.org>
and subject line Re: bug#78813: feature/igc: [PATCH] Make the array face_attr_sym readonly
has caused the debbugs.gnu.org bug report #78813,
regarding feature/igc: [PATCH] Make the array face_attr_sym readonly
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> 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: 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 3 (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)]

[Message part 5 (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.

Previous Next

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