* bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions
@ 2015-02-22 0:53 Drew Adams
2016-04-30 17:09 ` Lars Ingebrigtsen
0 siblings, 1 reply; 6+ messages in thread
From: Drew Adams @ 2015-02-22 0:53 UTC (permalink / raw)
To: 19918
In my setup, i.e., with Bookmark+ loaded, function `bookmark-bmenu-list'
is defined in file bookmark+-bmu.el. In the debugger, there is a line
like this:
* bookmark-bmenu-list(nil)
with the function name highlighted with a button. `C-u C-x =' on the
name shows this:
Here is a `help-function-def' button labeled `bookmark-bmenu-list'.
There are text properties here:
button (t)
category help-function-def-button
help-args [Show]
Clicking [Show] shows this:
(bookmark-bmenu-list
"d:/Emacs-25-2014-10-20/share/emacs/25.0.50/lisp/bookmark.el")
That's wrong. And clicking mouse-2 on that button takes me to that
wrong file. And so does `C-h f' followed by the name and clicking on
the library, which is shown as bookmark.el.
The debugger correctly traverses the code of the version of the function
that is defined in bookmark+-bmu.el, however.
IOW, the function is defined correctly. It is only the links in Help
and the debugger that are wrong, as well as the file name in Help. But
the doc string in Help is correct - it is the doc string for the
Bookmark+ version of the function.
The redefinition of the function in bookmark+-bmu.el is a
straightforward defun.
In GNU Emacs 25.0.50.1 (i686-pc-mingw32)
of 2014-10-20 on LEG570
Bzr revision: 118168 rgm@gnu.org-20141020195941-icp42t8ttcnud09g
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
`configure --enable-checking=yes,glyphs CPPFLAGS=-DGLYPH_DEBUG=1'
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions
2015-02-22 0:53 bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions Drew Adams
@ 2016-04-30 17:09 ` Lars Ingebrigtsen
2016-04-30 18:03 ` Drew Adams
0 siblings, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-04-30 17:09 UTC (permalink / raw)
To: Drew Adams; +Cc: 19918
Drew Adams <drew.adams@oracle.com> writes:
> In my setup, i.e., with Bookmark+ loaded, function `bookmark-bmenu-list'
> is defined in file bookmark+-bmu.el. In the debugger, there is a line
> like this:
>
> * bookmark-bmenu-list(nil)
>
> with the function name highlighted with a button. `C-u C-x =' on the
> name shows this:
>
> Here is a `help-function-def' button labeled `bookmark-bmenu-list'.
>
> There are text properties here:
> button (t)
> category help-function-def-button
> help-args [Show]
>
> Clicking [Show] shows this:
>
> (bookmark-bmenu-list
> "d:/Emacs-25-2014-10-20/share/emacs/25.0.50/lisp/bookmark.el")
Do you have a recipe not involving your own code?
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions
2016-04-30 17:09 ` Lars Ingebrigtsen
@ 2016-04-30 18:03 ` Drew Adams
2019-12-01 8:18 ` Stefan Kangas
0 siblings, 1 reply; 6+ messages in thread
From: Drew Adams @ 2016-04-30 18:03 UTC (permalink / raw)
To: Lars Ingebrigtsen; +Cc: 19918
> Do you have a recipe not involving your own code?
The point is that the help button does not reflect the fact
that the command was redefined.
The redefining code (in bookmark+-bmu.el, at top level) is
simply this:
(defun bookmark-bmenu-list (&optional filteredp msg-p)
"..."
(interactive "i\np")
...)
The problem is with the button link - it has apparently
hard-coded the address of the original definition of
this command, in standard library bookmark.el.
That's the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions
2016-04-30 18:03 ` Drew Adams
@ 2019-12-01 8:18 ` Stefan Kangas
2019-12-01 17:44 ` Drew Adams
0 siblings, 1 reply; 6+ messages in thread
From: Stefan Kangas @ 2019-12-01 8:18 UTC (permalink / raw)
To: Drew Adams; +Cc: Lars Ingebrigtsen, 19918
Drew Adams <drew.adams@oracle.com> writes:
>> Do you have a recipe not involving your own code?
>
> The point is that the help button does not reflect the fact
> that the command was redefined.
>
> The redefining code (in bookmark+-bmu.el, at top level) is
> simply this:
>
> (defun bookmark-bmenu-list (&optional filteredp msg-p)
> "..."
> (interactive "i\np")
> ...)
>
> The problem is with the button link - it has apparently
> hard-coded the address of the original definition of
> this command, in standard library bookmark.el.
Could you please give a recipe for how to reproduce this bug using
vanilla Emacs?
Best regards,
Stefan Kangas
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions
2019-12-01 8:18 ` Stefan Kangas
@ 2019-12-01 17:44 ` Drew Adams
2019-12-01 21:57 ` Stefan Kangas
0 siblings, 1 reply; 6+ messages in thread
From: Drew Adams @ 2019-12-01 17:44 UTC (permalink / raw)
To: Stefan Kangas; +Cc: Lars Ingebrigtsen, 19918
> Could you please give a recipe for how to reproduce
> this bug using vanilla Emacs?
That bug was reported 5 years ago, for Emacs 25.
It seems that it's been fixed, in Emacs 26.3 at
least. So I guess the bug can be closed. Thx.
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions
2019-12-01 17:44 ` Drew Adams
@ 2019-12-01 21:57 ` Stefan Kangas
0 siblings, 0 replies; 6+ messages in thread
From: Stefan Kangas @ 2019-12-01 21:57 UTC (permalink / raw)
To: Drew Adams; +Cc: Lars Ingebrigtsen, 19918-done
Drew Adams <drew.adams@oracle.com> writes:
>> Could you please give a recipe for how to reproduce
>> this bug using vanilla Emacs?
>
> That bug was reported 5 years ago, for Emacs 25.
> It seems that it's been fixed, in Emacs 26.3 at
> least. So I guess the bug can be closed. Thx.
Thanks for that, and I'm therefore closing this bug.
Best regards,
Stefan Kangas
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-12-01 21:57 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-02-22 0:53 bug#19918: 25.0.50; links in *Backtrace* & *Help* go to the wrong function definitions Drew Adams
2016-04-30 17:09 ` Lars Ingebrigtsen
2016-04-30 18:03 ` Drew Adams
2019-12-01 8:18 ` Stefan Kangas
2019-12-01 17:44 ` Drew Adams
2019-12-01 21:57 ` Stefan Kangas
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).