GNU bug report logs - #33830
27.0.50; help-xref-stack contains bad entries

Previous Next

Package: emacs;

Reported by: Alex Branham <alex.branham <at> gmail.com>

Date: Fri, 21 Dec 2018 16:57:02 UTC

Severity: normal

Tags: confirmed, moreinfo

Found in versions 27.0.50, 28.0.50

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


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

From: Federico Tedin <federicotedin <at> gmail.com>
To: Alex Branham <alex.branham <at> gmail.com>
Cc: 33830 <at> debbugs.gnu.org
Subject: Re: bug#33830: 27.0.50; help-xref-stack contains bad entries
Date: Thu, 27 Dec 2018 19:38:59 -0300
Alex Branham <alex.branham <at> gmail.com> writes:

> Sometimes `help-xref-stack' contains "bad" entries, which makes
> help-go-forward and help-go-back error in a strange way. To see what I
> mean:
>
> emacs -q
>
> C-h f car RET
> C-h k C-x t
> C-h f cdr RET
>
> Now that the *Help* buffer is displaying `cdr', if you try to go back
> (with "l" or clicking the back button), you get "user-error C-x t is
> undefined".
>
> Alex

I've noticed that the `describe-key' and `describe-function' functions both
call `help-setup-xref' with key sequences and functions respectively,
even if said key sequences or functions are not valid (i.e. when the key
sequence isn't bound to anything and the function does not exist).

An example of this:

1) M-:: (describe-function 'hello) RET
2) C-h f car RET
3) On the *Help* buffer, press 'l'

The message 'Symbol’s function definition is void: hello' will be
displayed, and the *Help* buffer will contain "hello is ". Also, for
some reason, after pressing 'r', two [back] buttons will be shown.

Maybe `help-setup-xref' should only be called when it is certain that
the key sequence/function/etc. exists?

(tested on master branch)




This bug report was last modified 3 years and 298 days ago.

Previous Next


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