GNU bug report logs - #38675
27.0.50; define-obsolete-face-alias definition does not match documentation

Previous Next

Full log

View this message in rfc822 format

From: James Felix Black <jfb <at> homonculus.net>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 38675 <at> debbugs.gnu.org
Subject: bug#38675: 27.0.50; define-obsolete-face-alias definition does not match documentation
Date: Sat, 21 Dec 2019 17:47:45 -0500

I’m easy either way. Do the usage and docs match the other “define-obsolete-“ macros?

> On Dec 21, 2019, at 04:38, Eli Zaretskii <eliz <at> gnu.org> wrote:
> 
> 
>> 
>> From: James Felix Black <jfb <at> homonculus.net>
>> Date: Thu, 19 Dec 2019 12:49:01 -0500
>> 
>> This was discovered in `haskell-mode` [1]. The documentation of  define-obsolete-face-alias claims that `when` is optional, but the macro definition does not follow the docs. This patch fixes the problem.
> 
> I'm not sure this isn't a confusion caused by ambiguous wording of the
> doc string.  I think by "if provided" it means that WHEN can be nil.
> The ELisp manual, btw, doesn't have this problem.
> 
> Would it be okay to modify the doc string instead to say WHEN is
> required, but can be nil?
> 
> Thanks.

Previous Next

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