unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#24845: 24.5; out-of-date text in (elisp) `Autoload'?
@ 2016-11-01 15:09 Drew Adams
  2016-11-02  1:13 ` npostavs
  0 siblings, 1 reply; 4+ messages in thread
From: Drew Adams @ 2016-11-01 15:09 UTC (permalink / raw)
  To: 24845

I see this in that node:

  Other definition types:
     defcustom, defgroup, defclass (*note EIEIO: (eieio)Top.), and
     define-skeleton (see the commentary in skeleton.el).
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

But I do not see anything appropriate in the commentary in
`skeleton.el'.  What is this all about?


In GNU Emacs 24.5.1 (i686-pc-mingw32)
 of 2015-04-11 on LEG570
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix=3D/c/usr --host=3Di686-pc-mingw32'





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

* bug#24845: 24.5; out-of-date text in (elisp) `Autoload'?
  2016-11-01 15:09 bug#24845: 24.5; out-of-date text in (elisp) `Autoload'? Drew Adams
@ 2016-11-02  1:13 ` npostavs
  2016-11-02 15:51   ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: npostavs @ 2016-11-02  1:13 UTC (permalink / raw)
  To: Drew Adams; +Cc: 24845

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

> I see this in that node:
>
>   Other definition types:
>      defcustom, defgroup, defclass (*note EIEIO: (eieio)Top.), and
>      define-skeleton (see the commentary in skeleton.el).
>                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> But I do not see anything appropriate in the commentary in
> `skeleton.el'.  What is this all about?

The reference to skeleton.el's commentary was introduced in bc44be50 "*
loading.texi (Autoload): Explicitly state which forms are processed
specially.", there is no additional commentary in skeleton.el regarding
autoloading in that revision either.  I think it's just intended to be
an explanation of what `define-skeleton' is.

Perhaps a better link would be `(autotype) Skeleton Language', similar
to how defclass has a link to eieio.

diff --git i/doc/lispref/loading.texi w/doc/lispref/loading.texi
index 81a1922..82bdde0 100644
--- i/doc/lispref/loading.texi
+++ w/doc/lispref/loading.texi
@@ -610,8 +610,8 @@ Autoload
 
 @item Other definition types:
 @code{defcustom}, @code{defgroup}, @code{defclass}
-(@pxref{Top,EIEIO,,eieio,EIEIO}), and @code{define-skeleton} (see the
-commentary in @file{skeleton.el}).
+(@pxref{Top,EIEIO,,eieio,EIEIO}), and @code{define-skeleton}
+(@pxref{Skeleton Language,,,autotype,Autotyping}).
 @end table
 
   You can also use a magic comment to execute a form at build time






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

* bug#24845: 24.5; out-of-date text in (elisp) `Autoload'?
  2016-11-02  1:13 ` npostavs
@ 2016-11-02 15:51   ` Eli Zaretskii
  2016-11-05 14:13     ` npostavs
  0 siblings, 1 reply; 4+ messages in thread
From: Eli Zaretskii @ 2016-11-02 15:51 UTC (permalink / raw)
  To: npostavs; +Cc: 24845

> From: npostavs@users.sourceforge.net
> Date: Tue, 01 Nov 2016 21:13:48 -0400
> Cc: 24845@debbugs.gnu.org
> 
> Drew Adams <drew.adams@oracle.com> writes:
> 
> > I see this in that node:
> >
> >   Other definition types:
> >      defcustom, defgroup, defclass (*note EIEIO: (eieio)Top.), and
> >      define-skeleton (see the commentary in skeleton.el).
> >                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> >
> > But I do not see anything appropriate in the commentary in
> > `skeleton.el'.  What is this all about?
> 
> The reference to skeleton.el's commentary was introduced in bc44be50 "*
> loading.texi (Autoload): Explicitly state which forms are processed
> specially.", there is no additional commentary in skeleton.el regarding
> autoloading in that revision either.  I think it's just intended to be
> an explanation of what `define-skeleton' is.

Indeed, that's also my impression.

> Perhaps a better link would be `(autotype) Skeleton Language', similar
> to how defclass has a link to eieio.

I think we should link to "Using Skeletons", or maybe even to "Top" in
that manual, since the cross-reference is meant for those readers who
don't know what skeletons are.

Thanks.





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

* bug#24845: 24.5; out-of-date text in (elisp) `Autoload'?
  2016-11-02 15:51   ` Eli Zaretskii
@ 2016-11-05 14:13     ` npostavs
  0 siblings, 0 replies; 4+ messages in thread
From: npostavs @ 2016-11-05 14:13 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 24845

tags 24845 fixed
close 24845 25.2
quit

Eli Zaretskii <eliz@gnu.org> writes:
>
> I think we should link to "Using Skeletons", or maybe even to "Top" in
> that manual, since the cross-reference is meant for those readers who
> don't know what skeletons are.

I chose Top in bb61e505.





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

end of thread, other threads:[~2016-11-05 14:13 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-01 15:09 bug#24845: 24.5; out-of-date text in (elisp) `Autoload'? Drew Adams
2016-11-02  1:13 ` npostavs
2016-11-02 15:51   ` Eli Zaretskii
2016-11-05 14:13     ` npostavs

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).