GNU bug report logs - #57499
Documentation bug in the docstring of set-face-attribute?

Previous Next

Package: emacs;

Reported by: Gregory Heytings <gregory <at> heytings.org>

Date: Wed, 31 Aug 2022 08:15:02 UTC

Severity: minor

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

Bug is archived. No further changes may be made.

Full log


Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Gregory Heytings <gregory <at> heytings.org>
To: bug-gnu-emacs <at> gnu.org
Subject: Documentation bug in the docstring of set-face-attribute?
Date: Wed, 31 Aug 2022 08:14:00 +0000
The docstring of set-face-attribute says:

"As an exception, to reset the value of some attribute to `unspecified' in 
a way that overrides the non-`unspecified' value defined by the face's 
spec in `defface', for new frames, you must explicitly call this function 
with FRAME set to t and the attribute's value set to `unspecified'; just 
using FRAME of nil will not affect new frames in this case."

Not only is that sentence hard to parse, it also seems wrong.

Can someone come up with a scenario in which a call

(set-face-attribute 'some-face nil :some-attribute 'unspecified)

only affects existing frames?  In my testing it affects all frames 
(existing and future ones), and that's also what the code seems to do: 
set-face-attribute sets where to 0 when frame is nil, and calls 
internal-set-face-attribute




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

Previous Next


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