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

Previous Next

Full log

Message #67 received at 57499-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Gregory Heytings <gregory <at> heytings.org>
Cc: 57499-done <at> debbugs.gnu.org
Subject: Re: bug#57499: Documentation bug in the docstring of
 set-face-attribute?
Date: Thu, 01 Sep 2022 15:08:53 +0300

> Date: Thu, 01 Sep 2022 11:56:06 +0000
> From: Gregory Heytings <gregory <at> heytings.org>
> cc: 57499-done <at> debbugs.gnu.org
> 
> >> 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.
> >
> 
> Would a warning in *Messages* be okay then?

We could try that, yes.  But it has to be only in *Messages*, not an
actual warning, since faces get merged and realized as part of
redisplay, when signaling an error is a bad idea.

> People who use nil with frame = t or nil will not get what they
> expect, right?

If they expect what you think they expect, yes.

Previous Next

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