GNU bug report logs - #77945
[PATCH] Add optional inherit argument to 'face-all-attributes'

Previous Next

Full log

View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ship Mints <shipmints <at> gmail.com>
Cc: 77945 <at> debbugs.gnu.org
Subject: bug#77945: [PATCH] Add optional inherit argument to 'face-all-attributes'
Date: Sat, 03 May 2025 11:31:05 +0300

> From: Ship Mints <shipmints <at> gmail.com>
> Date: Sun, 20 Apr 2025 17:45:40 -0400
> Cc: 77945 <at> debbugs.gnu.org
> 
> * lisp/faces.el (face-all-attributes): Add inherit argument and
                                             ^^^^^^^
Please quote "inherit 'like this'.

> +@code{set-face-attribute}; see below.  The optional argument
> +@var{inherit} has the same definition as in @code{face-attribute}, which
                              ^^^^^^^^^^
"Meaning", not "definition".  Also, "which see" is redundant here,
since face-attribute was described just above this function in the
manual.

> +see.  This can aid understanding face definitions.

I'd either drop the last sentence, or say instead something like

  This is useful when you want the face attributes to be absolute and
  not @code{unspecified}.

> ++++
> +*** 'face-all-attributes' now accepts an optional inherit argument.

Please quote "inherit" here.

> +This will resolve all attributes.

This is inaccurate.  I'd suggest to remove this sentence, since the
rest of the NEWS entry describes what it does more accurately.

>                                 'inherit' has the same meaning as
> +'face-attribute', which already takes this argument for a single

"same meaning as the same argument to 'face-attribute'"

Thanks.

Previous Next

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