[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#24842: [External] : Re: bug#24842: bug#9445: 24.0.50; `help-make-xre
From: |
Drew Adams |
Subject: |
bug#24842: [External] : Re: bug#24842: bug#9445: 24.0.50; `help-make-xrefs' has bad logic |
Date: |
Tue, 7 Sep 2021 20:13:02 +0000 |
> > (defun foo1 ())
> > (defvar foo2 ())
> >
> > (defvar foo nil
> > "
> > `foo1' something.
> > `foo1'\t- units and their movement functions
> > `foo2' something.
> > `foo2'\t- units and their movement functions
> > ")
> >
> But the recipe Lars gave doesn't quite match the description Drew gave,
> IIUC: if in the recipe you change the first occurrence of `functions'
> to
> `function', then eval and do `C-h v foo', then the first occurrence of
> `foo2' in the doc string is not turned into a link. Isn't that what
> the
> problem was, and still is?
Hm. You're right, Stephen. But not with just
that change, I think. And it looks like the
problem is maybe for defun doc strings but not
for defvar doc strings? (But see below.)
(defun foo1 ())
(defvar foo2 ())
(defvar foo nil
"
`foo2' something.
`foo2'\t- units and their movement function
`foo1' something.
`foo1'\t- units and their movement function
")
(defun bar ()
"
`foo2' something.
`foo2'\t- units and their movement function
`foo' something.
`foo'\t- units and their movement function
" 42)
`C-h v foo' shows links everywhere, for both
var foo2 and function foo1.
But `C-h f bar' does not show a link for the
first occurrence of `foo'.
___
What's odd, and the reason I thought this was
fixed, is that I checked (using `emacs -Q')
the doc string in my code that I was reporting
about. That doc string is for a function, not
a variable, but it (correctly) has links for
all of the variables mentioned in it.
The recipe for the original report is this:
1. Download library isearch+.el from here:
https://www.emacswiki.org/emacs/download/isearch%2b.el
2. Load that source code.
3. `C-h f isearch-forward'
4. This variable (user option) in the doc was
not linked, but should have been:
`isearchp-prompt-for-filter-name'.
But now that option has a link. (With the above
recipe all have links except those with `[*]',
which the doc tells you are defined in another
library - so no links is correct without that
library loaded.)