bug#57499: Documentation bug in the docstring of set-face-attribute?

[Eli Zaretskii]

> Date: Thu, 01 Sep 2022 09:02:50 +0000
> From: Gregory Heytings <gregory@heytings.org>
> cc: 57499-done@debbugs.gnu.org
> 
> >> So this call is already included in the previous one.  Why should we 
> >> tell users to add such a redundant call in their code?
> >
> > The new text doesn't say the call with FRAME = t should be an additional 
> > call.
> >
> 
> It doesn't say so explicitly indeed, but without reading the code of 
> set-face-attribute everyone understands that it should be an additional 
> call.

I disagree with your certainty that everyone will understand that.
The text deliberately doesn't say that.

> Would the patch below be okay?  This would be another way to clarify the 
> matter.

No.  I don't want to lose this compatibility feature because lots of
people still use nil.