bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Drew Adams]

These functions, especially `line-move', are used all over the place.
They need doc strings, with complete descriptions of their parameters.

In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2013-07-01 on LEG570
Bzr revision: 113246 lekktu@gmail.com-20130701165437-ea20s94hqwp3ttaj
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix=/c/usr --enable-checking CFLAGS='-O0 -g3'
 CPPFLAGS='-DGLYPH_DEBUG=1 -I/c/usr/include''

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Eli Zaretskii]

> Date: Thu, 11 Jul 2013 08:40:49 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
> 
> These functions, especially `line-move', are used all over the place.
> They need doc strings, with complete descriptions of their parameters.

They are internal functions.  Under what circumstances did you need to
use them directly?  Why wasn't next/previous-line enough?

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Stefan Monnier]

> They need doc strings, with complete descriptions of their parameters.

Agreed.


        Stefan "Looking forward to your corresponding commit"

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Lars Ingebrigtsen]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>> They need doc strings, with complete descriptions of their parameters.
>
> Agreed.

Fixed on trunk.

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

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Drew Adams]

> > These functions, especially `line-move', are used all over the place.
> > They need doc strings, with complete descriptions of their parameters.
> 
> They are internal functions.  Under what circumstances did you need to
> use them directly?  Why wasn't next/previous-line enough?

Read what I wrote: they "are used all over the place".  In the Emacs code
alone, not to mention in 3rd-party code.  They are not "internal" to
simple.el.  They are not internal to anything.  Reality.

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Eli Zaretskii]

> Date: Thu, 11 Jul 2013 11:50:06 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
> Cc: 14843@debbugs.gnu.org
> 
> > > These functions, especially `line-move', are used all over the place.
> > > They need doc strings, with complete descriptions of their parameters.
> > 
> > They are internal functions.  Under what circumstances did you need to
> > use them directly?  Why wasn't next/previous-line enough?
> 
> Read what I wrote: they "are used all over the place".  In the Emacs code
> alone, not to mention in 3rd-party code.  They are not "internal" to
> simple.el.  They are not internal to anything.  Reality.

That an internal function is used all over the Emacs code is a small
wonder.  It just means someone was smart enough to capture a frequent
need of many features.

As for 3rd-party code, if you have such code, or happen to know why
line-move was used instead of next/previous-line, please tell.

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Drew Adams]

> > > > These functions, especially `line-move', are used all over the place.
> > > > They need doc strings, with complete descriptions of their parameters.
> > >
> > > They are internal functions.  Under what circumstances did you need to
> > > use them directly?  Why wasn't next/previous-line enough?
> >
> > Read what I wrote: they "are used all over the place".  In the Emacs code
> > alone, not to mention in 3rd-party code.  They are not "internal" to
> > simple.el.  They are not internal to anything.  Reality.
> 
> That an internal function is used all over the Emacs code is a small
> wonder.  It just means someone was smart enough to capture a frequent
> need of many features.

And each person who wrote such Emacs code needed to figure out for herself
what the parameters mean (or not bother).

Perhaps it is also a small wonder that NONE of the uses of `line-move',
including those in simple.el itself, make ANY USE of the additional
parameters introduced in Emacs 22 (and not used even in 22).

An indication, perhaps, that even Emacs Dev could use a little Emacs
self-documenting.  Doc strings are for everyone.

Arguing not to create clear doc strings for "internal" functions made little
sense in the 1960s.  It makes no sense at all now.

In bygone days, Emacs code had comments instead of doc strings for some
functions (sometimes systematically, for a given library).  Most of those
comments were changed to doc strings once the Dark Ages were over.

Look at the large comments preceding these two function defuns.  Yet even
those elaborate comments do not describe the parameters.  Emacs can do
better.  Much better.

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Eli Zaretskii]

> Date: Thu, 11 Jul 2013 13:52:23 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
> Cc: 14843@debbugs.gnu.org
> 
> Perhaps it is also a small wonder that NONE of the uses of `line-move',
> including those in simple.el itself, make ANY USE of the additional
> parameters introduced in Emacs 22 (and not used even in 22).

That's not true.  These arguments are used by next-line and previous
line.

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Drew Adams]

> > Perhaps it is also a small wonder that NONE of the uses of `line-move',
> > including those in simple.el itself, make ANY USE of the additional
> > parameters introduced in Emacs 22 (and not used even in 22).
> 
> That's not true.  These arguments are used by next-line and previous
> line.

Yes and no.  One of them is used; the other (TO-END) is not.

But you're missing the point.  Neither is used in any of the zillion uses
of `line-move' - other than `(next|previous)-line'.  And that lends some
support to the idea that those programming those uses might not have even
understood these undocumented parameters.

In addition, it is NOT the case that optional parameter TO-END is used
anywhere.  And I do mean strictly anywhere this time.  It was introduced
in Emacs 22 but NEVER used anywhere, AFAICT.  Again, not being documented
might be one reason this vestige (vestige of nothing) has never been
used and has never been removed.

I made this clear in bug #14844 (where I mentioned only TO-END, not also
TRY-VSCROLL).

bug#14843: 24.3.50; `line-move', `line-move-visual' need doc strings

[Eli Zaretskii]

> Date: Fri, 12 Jul 2013 01:26:38 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
> Cc: 14843@debbugs.gnu.org
> 
> > > Perhaps it is also a small wonder that NONE of the uses of `line-move',
> > > including those in simple.el itself, make ANY USE of the additional
> > > parameters introduced in Emacs 22 (and not used even in 22).
> > 
> > That's not true.  These arguments are used by next-line and previous
> > line.
> 
> Yes and no.  One of them is used; the other (TO-END) is not.

It is documented to be unused.

> But you're missing the point.  Neither is used in any of the zillion uses
> of `line-move' - other than `(next|previous)-line'.  And that lends some
> support to the idea that those programming those uses might not have even
> understood these undocumented parameters.

The arguments to next/previous-line are all documented now.