unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#64797: 30.0.50; advised alias -> "function is an alias for nil" in Help
@ 2023-07-23  3:13 Michael Heerdegen
  2023-07-27 15:11 ` Mattias Engdegård
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Heerdegen @ 2023-07-23  3:13 UTC (permalink / raw)
  To: 64797; +Cc: Mattias Engdegård, Stefan Monnier


Hello,

an example: `manual-entry' is an alias for `man'.  When now the alias is
advised, for example like

  (advice-add 'manual-entry :filter-return #'identity)

and you do C-h f manual-entry RET, the first line of the Help buffer
will say:

| manual-entry is an alias for ‘nil’, which is not defined.

which is obviously nonsense.

Seems that this regression in `help-fns--analyze-function' had been
introduced by

d30fde6b0cc "Avoid infloops in help-fns--analyze-function with aliases"
(Lars Ingebrigtsen <larsi@gnus.org> 2022-01-13)

I CC Mattias and Stefan you might be able to help.


TIA,

Michael.








^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#64797: 30.0.50; advised alias -> "function is an alias for nil" in Help
  2023-07-23  3:13 bug#64797: 30.0.50; advised alias -> "function is an alias for nil" in Help Michael Heerdegen
@ 2023-07-27 15:11 ` Mattias Engdegård
  2023-07-27 23:08   ` Michael Heerdegen
  0 siblings, 1 reply; 3+ messages in thread
From: Mattias Engdegård @ 2023-07-27 15:11 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: 64797-done, Eli Zaretskii, Stefan Monnier

23 juli 2023 kl. 05.13 skrev Michael Heerdegen <michael_heerdegen@web.de>:

> Seems that this regression in `help-fns--analyze-function' had been
> introduced by
> 
> d30fde6b0cc "Avoid infloops in help-fns--analyze-function with aliases"
> (Lars Ingebrigtsen <larsi@gnus.org> 2022-01-13)

Right. This is a regression in Emacs 29.
The fix is going to master for now. Whether and when to backport it is Eli's decision to make.

Thank you for localising and reporting it!






^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#64797: 30.0.50; advised alias -> "function is an alias for nil" in Help
  2023-07-27 15:11 ` Mattias Engdegård
@ 2023-07-27 23:08   ` Michael Heerdegen
  0 siblings, 0 replies; 3+ messages in thread
From: Michael Heerdegen @ 2023-07-27 23:08 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 64797-done, Eli Zaretskii, Stefan Monnier

Mattias Engdegård <mattias.engdegard@gmail.com> writes:

> Right. This is a regression in Emacs 29.  The fix is going to master
> for now. Whether and when to backport it is Eli's decision to make.
>
> Thank you for localising and reporting it!

Thank you for fixing - works well for me.

Michael.





^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-07-27 23:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-23  3:13 bug#64797: 30.0.50; advised alias -> "function is an alias for nil" in Help Michael Heerdegen
2023-07-27 15:11 ` Mattias Engdegård
2023-07-27 23:08   ` Michael Heerdegen

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).