all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables
@ 2013-11-20 22:08 Drew Adams
  2014-02-08  3:13 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Drew Adams @ 2013-11-20 22:08 UTC (permalink / raw)
  To: 15940


emacs -Q

C-h f lwarn

The doc string says:

"(but see `warning-minimum-level' and `warning-minimum-log-level')"

That's impossible, before warning.el has been loaded - the variables are
undefined.



In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2013-11-12 on LEG570
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'





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

* bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables
  2013-11-20 22:08 bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables Drew Adams
@ 2014-02-08  3:13 ` Lars Ingebrigtsen
  2014-02-08  9:11   ` Eli Zaretskii
  2014-02-08 23:05   ` Drew Adams
  0 siblings, 2 replies; 6+ messages in thread
From: Lars Ingebrigtsen @ 2014-02-08  3:13 UTC (permalink / raw)
  To: Drew Adams; +Cc: 15940

Drew Adams <drew.adams@oracle.com> writes:

> C-h f lwarn
>
> The doc string says:
>
> "(but see `warning-minimum-level' and `warning-minimum-log-level')"
>
> That's impossible, before warning.el has been loaded - the variables are
> undefined.

Yeah, it's unfortunate that autoloaded functions can't really refer to
other variables from its "home package" in the help system.

Would it make sense for the help system to always load the package an
autoloaded function is defined in?  Especially if the doc string refers
to other `...' items?

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/





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

* bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables
  2014-02-08  3:13 ` Lars Ingebrigtsen
@ 2014-02-08  9:11   ` Eli Zaretskii
  2014-02-08 20:06     ` Glenn Morris
  2014-02-08 23:05   ` Drew Adams
  1 sibling, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2014-02-08  9:11 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 15940

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Date: Fri, 07 Feb 2014 19:13:29 -0800
> Cc: 15940@debbugs.gnu.org
> 
> Drew Adams <drew.adams@oracle.com> writes:
> 
> > C-h f lwarn
> >
> > The doc string says:
> >
> > "(but see `warning-minimum-level' and `warning-minimum-log-level')"
> >
> > That's impossible, before warning.el has been loaded - the variables are
> > undefined.
> 
> Yeah, it's unfortunate that autoloaded functions can't really refer to
> other variables from its "home package" in the help system.
> 
> Would it make sense for the help system to always load the package an
> autoloaded function is defined in?  Especially if the doc string refers
> to other `...' items?

Why not simply auto-load those variables, especially if the doc string
refers to them?





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

* bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables
  2014-02-08  9:11   ` Eli Zaretskii
@ 2014-02-08 20:06     ` Glenn Morris
  2014-02-09  0:20       ` Glenn Morris
  0 siblings, 1 reply; 6+ messages in thread
From: Glenn Morris @ 2014-02-08 20:06 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 15940, Lars Ingebrigtsen

Eli Zaretskii wrote:

>> Would it make sense for the help system to always load the package an
>> autoloaded function is defined in?  Especially if the doc string refers
>> to other `...' items?
>
> Why not simply auto-load those variables, especially if the doc string
> refers to them?

Because this is a general problem that comes up again and again,
and papering over it with manual autoloads every time seems like bad
practice.

A change was added for this in 24.3:

    *** `C-h f' (`describe-function') can now perform autoloading.
    When this command is called for an autoloaded function whose docstring
    contains a key substitution construct, that function's library is
    automatically loaded, so that the documentation can be shown
    correctly.  To disable this, set `help-enable-auto-load' to nil.

It needs to be generalized somehow.

I'm pretty sure this has been discussed multiple times in the past,
so there's probably an idea/implementation out there somewhere in the
archives.





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

* bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables
  2014-02-08  3:13 ` Lars Ingebrigtsen
  2014-02-08  9:11   ` Eli Zaretskii
@ 2014-02-08 23:05   ` Drew Adams
  1 sibling, 0 replies; 6+ messages in thread
From: Drew Adams @ 2014-02-08 23:05 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 15940

> Yeah, it's unfortunate that autoloaded functions can't really refer
> to other variables from its "home package" in the help system.
> 
> Would it make sense for the help system to always load the package
> an autoloaded function is defined in?  Especially if the doc string
> refers to other `...' items?

Sorry, I can't help you here.  But I'm sure Stefan has an idea about it.





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

* bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables
  2014-02-08 20:06     ` Glenn Morris
@ 2014-02-09  0:20       ` Glenn Morris
  0 siblings, 0 replies; 6+ messages in thread
From: Glenn Morris @ 2014-02-09  0:20 UTC (permalink / raw)
  To: 15940-done

Version: 24.4

Fixed.






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

end of thread, other threads:[~2014-02-09  0:20 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-20 22:08 bug#15940: 24.3.50; doc string of `lwarn' refers to undefined variables Drew Adams
2014-02-08  3:13 ` Lars Ingebrigtsen
2014-02-08  9:11   ` Eli Zaretskii
2014-02-08 20:06     ` Glenn Morris
2014-02-09  0:20       ` Glenn Morris
2014-02-08 23:05   ` Drew Adams

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.