unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#24386: 24.5; `read--expression' --> `read-expression', please
@ 2016-09-07  2:37 Drew Adams
  2019-07-27 14:45 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 5+ messages in thread
From: Drew Adams @ 2016-09-07  2:37 UTC (permalink / raw)
  To: 24386

Please consider renaming `read--expression' to `read-expression' and
taking away its "internal" designation.  What it does is equally useful
to ordinary Emacs user-programmers.


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=/c/usr --host=i686-pc-mingw32'





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

* bug#24386: 24.5; `read--expression' --> `read-expression', please
  2016-09-07  2:37 bug#24386: 24.5; `read--expression' --> `read-expression', please Drew Adams
@ 2019-07-27 14:45 ` Lars Ingebrigtsen
  2019-07-27 16:39   ` Drew Adams
  0 siblings, 1 reply; 5+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-27 14:45 UTC (permalink / raw)
  To: Drew Adams; +Cc: 24386

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

> Please consider renaming `read--expression' to `read-expression' and
> taking away its "internal" designation.  What it does is equally useful
> to ordinary Emacs user-programmers.

It does seem like a rather internal function (setting up eldoc stuff and
the like), so I think it's probably well-named as is; closing.

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





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

* bug#24386: 24.5; `read--expression' --> `read-expression', please
  2019-07-27 14:45 ` Lars Ingebrigtsen
@ 2019-07-27 16:39   ` Drew Adams
  2019-07-27 22:44     ` Michael Heerdegen
  0 siblings, 1 reply; 5+ messages in thread
From: Drew Adams @ 2019-07-27 16:39 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 24386

> > Please consider renaming `read--expression' to `read-expression' and
> > taking away its "internal" designation.  What it does is equally
> > useful to ordinary Emacs user-programmers.
> 
> It does seem like a rather internal function (setting up eldoc stuff
> and the like), so I think it's probably well-named as is; closing.

There's _absolutely nothing_ internal about the
behavior this function offers.  You can use it
_any_ place where you want to read a sexp.

In fact, users should be _encouraged_ to use
this, as opposed to more primitive, less
sexp-oriented ways of reading text when what
they want is a Lisp sexp.

That's akin to telling users to use file-name
manipulation functions instead of just concat,
when dealing with file names.

The name should probably be changed to have
"Lisp" in it, however, as it reads only Lisp
sexps (currently).

And this function needs a doc string (badly).

As for it using `eldoc-mode': That presumably
makes the function more useful for specifically
Lisp-sexp reading.  If we think otherwise then
that could be removed or optional.  In any case,
its presence has _nothing_ whatsoever to do
with whether the function is useful generally
or should be considered internal or local to
only some uses.

This is a bad judgment call, IMHO.





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

* bug#24386: 24.5; `read--expression' --> `read-expression', please
  2019-07-27 16:39   ` Drew Adams
@ 2019-07-27 22:44     ` Michael Heerdegen
  2019-07-28  0:34       ` Drew Adams
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Heerdegen @ 2019-07-27 22:44 UTC (permalink / raw)
  To: Drew Adams; +Cc: Lars Ingebrigtsen, 24386

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

> > > Please consider renaming `read--expression' to `read-expression' and
> > > taking away its "internal" designation.  What it does is equally
> > > useful to ordinary Emacs user-programmers.
> >
> > It does seem like a rather internal function (setting up eldoc stuff
> > and the like), so I think it's probably well-named as is; closing.
>
> There's _absolutely nothing_ internal about the
> behavior this function offers.  You can use it
> _any_ place where you want to read a sexp.

IMO we should offer an official way to read expression input.  That
would likely look very much like read--expression.

> As for it using `eldoc-mode': That presumably makes the function more
> useful for specifically Lisp-sexp reading.  If we think otherwise then
> that could be removed or optional.

FWIW it wasn't there from the beginning - it was added while fixing
Bug#27202.  If we speak of Lisp, the name "expression" implies something
to be intended for evaluation, so having eldoc support is totally
legitimate.

Why not also add a `read-sexp'?

Michael.





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

* bug#24386: 24.5; `read--expression' --> `read-expression', please
  2019-07-27 22:44     ` Michael Heerdegen
@ 2019-07-28  0:34       ` Drew Adams
  0 siblings, 0 replies; 5+ messages in thread
From: Drew Adams @ 2019-07-28  0:34 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: Lars Ingebrigtsen, 24386

> > > > Please consider renaming `read--expression' to `read-expression'
> and
> > > > taking away its "internal" designation.  What it does is equally
> > > > useful to ordinary Emacs user-programmers.
> > >
> > > It does seem like a rather internal function (setting up eldoc
> stuff
> > > and the like), so I think it's probably well-named as is; closing.
> >
> > There's _absolutely nothing_ internal about the
> > behavior this function offers.  You can use it
> > _any_ place where you want to read a sexp.
> 
> IMO we should offer an official way to read expression input.  That
> would likely look very much like read--expression.
> 
> > As for it using `eldoc-mode': That presumably makes the function more
> > useful for specifically Lisp-sexp reading.  If we think otherwise
> > then that could be removed or optional.
> 
> FWIW it wasn't there from the beginning - it was added while fixing
> Bug#27202.  If we speak of Lisp, the name "expression" implies
> something to be intended for evaluation, so having eldoc support
> is totally legitimate.
> 
> Why not also add a `read-sexp'?

+1 to all you said.

`read--expression' is a good beginning for such
a function.  If we don't do as you suggest then it
would also be a good end: just remove the extra `-'.

(But I don't think the name "expression" implies
something to be intended for evaluation.  It often
means that, but it's also used just for "sexp".)





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

end of thread, other threads:[~2019-07-28  0:34 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-07  2:37 bug#24386: 24.5; `read--expression' --> `read-expression', please Drew Adams
2019-07-27 14:45 ` Lars Ingebrigtsen
2019-07-27 16:39   ` Drew Adams
2019-07-27 22:44     ` Michael Heerdegen
2019-07-28  0:34       ` Drew Adams

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