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

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 38675 in the body.
You can then email your comments to 38675 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: James Felix Black <jfb <at> homonculus.net>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50; define-obsolete-face-alias definition does not match
 documentation
Date: Thu, 19 Dec 2019 12:49:01 -0500

[Message part 1 (text/plain, inline)]

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.

In GNU Emacs 27.0.50 (build 1, x86_64-apple-darwin19.2.0, NS appkit-1894.20 Version 10.15.2 (Build 19C57))
 of 2019-12-19 built on moomin.local
Repository revision: f38e8a86650e8a7f72293d81a7392289c298ae5a
Repository branch: fix_define-obsolete-face-alias_optional
Windowing system distributor 'Apple', version 10.3.1894
System Description:  Mac OS X 10.15.2

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.
user-error: The mark is not set now, so there is no region
Making completion list...

Configured features:
NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES
THREADS PDUMPER LCMS2 GMP

Important settings:
  value of $LANG: en_CA.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc puny dired dired-loaddefs
format-spec rfc822 mml easymenu mml-sec password-cache epa derived epg
epg-config gnus-util rmail rmail-loaddefs text-property-search time-date
subr-x seq byte-opt gv bytecomp byte-compile cconv mm-decode mm-bodies
mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader cl-loaddefs
cl-lib sendmail rfc2047 rfc2045 ietf-drums mm-util mail-prsvr mail-utils
tooltip eldoc electric uniquify ediff-hook vc-hooks lisp-float-type
mwheel term/ns-win ns-win ucs-normalize mule-util term/common-win
tool-bar dnd fontset image regexp-opt fringe tabulated-list replace
newcomment text-mode elisp-mode lisp-mode prog-mode register page
tab-bar menu-bar rfn-eshadow isearch timer select scroll-bar mouse
jit-lock font-lock syntax facemenu font-core term/tty-colors frame
minibuffer cl-generic cham georgian utf-8-lang misc-lang vietnamese
tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek
romanian slovak czech european ethiopic indian cyrillic chinese
composite charscript charprop case-table epa-hook jka-cmpr-hook help
simple abbrev obarray cl-preloaded nadvice loaddefs button faces
cus-face macroexp files text-properties overlay sha1 md5 base64 format
env code-pages mule custom widget hashtable-print-readable backquote
threads kqueue cocoa ns lcms2 multi-tty make-network-process emacs)

Memory information:
((conses 16 43939 6393)
 (symbols 48 5909 1)
 (strings 32 15245 1247)
 (string-bytes 1 502913)
 (vectors 16 9297)
 (vector-slots 8 118962 11572)
 (floats 8 20 25)
 (intervals 56 185 0)
 (buffers 1000 12))



[1] https://github.com/haskell/haskell-mode/issues/1691

[0001-Fix-macro-definition.patch (application/octet-stream, attachment)]

Message #8 received at 38675 <at> debbugs.gnu.org (full text, mbox):

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

> 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.

Message #11 received at 38675 <at> debbugs.gnu.org (full text, mbox):

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

ced10a4c9f changed the arg spec but not the doc (or code).
The intent seems clear: a proper "when" should always be provided.

Message #14 received at 38675 <at> debbugs.gnu.org (full text, mbox):

From: James Felix Black <jfb <at> homonculus.net>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 38675 <at> debbugs.gnu.org
Subject: Re: 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.

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: James Felix Black <jfb <at> homonculus.net>
Cc: 38675-done <at> debbugs.gnu.org
Subject: Re: bug#38675: 27.0.50;
 define-obsolete-face-alias definition does not match documentation
Date: Sat, 28 Dec 2019 11:15:09 +0200

> From: James Felix Black <jfb <at> homonculus.net>
> Date: Sat, 21 Dec 2019 17:47:45 -0500
> Cc: 38675 <at> debbugs.gnu.org
> 
> I’m easy either way. Do the usage and docs match the other “define-obsolete-“ macros?

I fixed the documentation to fit the advertised calling convention,
and I'm therefore closing this bug report.

Thanks.

Previous Next

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