Subject: Re: bug#52493: 29.0.50; Setting Inconsolata up in init.el makes default face rendered wrong

From: Eli Zaretskii <[email-masked]>
Date: Mon, 26 Dec 2022 14:25:32 +0200

> Date: Mon, 26 Dec 2022 00:46:30 +0000
> From: Gregory Heytings <[email-masked]>
> cc: [email-masked], Eli Zaretskii <[email-masked]>, [email-masked], 
>     Lars Ingebrigtsen <[email-masked]>, [email-masked]
> 
> --- a/lisp/faces.el
> +++ b/lisp/faces.el
> @@ -690,6 +690,17 @@ set-face-attribute
>  what the FACE's face spec says, call this function with FRAME set to
>  t and the ATTRIBUTE's value set to `unspecified'.
>  
> +Note that the ATTRIBUTE VALUE pairs are evaluated in the order
> +they are specified, except the `:family' and `:foundry'
> +attributes which are evaluated first.  This means both that only
> +the last VALUE of a given ATTRIBUTE will be used, and that in
> +some cases a different order will give different results.  For
> +example, when `:weight' is placed before `:font', the weight
> +value is applied to the current font of the face and might be
> +rounded to the closest available weight of that font, whereas
> +when `:font' is placed before `:weight' the weight value is
> +applied to the specified font.

The text is OK, but please put this in the manual, not in the doc
string.  If we want something to this effect in the doc string, let's
just have the first sentence there, and then a reference to the
manual.

Thanks.