all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* 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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.