unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#16837: 24.3.50; Allow a HISTORY arg for `read-file-name', `read-file-name-default'
@ 2014-02-21 18:34 Drew Adams
  2016-04-29 17:39 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 3+ messages in thread
From: Drew Adams @ 2014-02-21 18:34 UTC (permalink / raw)
  To: 16837

An enhancement request -

Please consider allowing an optional HISTORY arg to these functions
(with the default value being of course `file-name-history').

It can happen that someone wants the behavior of one of these functions,
but with a different file-name history, such as `shell-command-history'.


In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2014-02-11 on ODIEONE
Bzr revision: 116410 lekktu@gmail.com-20140211204823-l9l2s6tktfitq266
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix=/c/Devel/emacs/binary --enable-checking=yes,glyphs
 'CFLAGS=-O0 -g3' LDFLAGS=-Lc:/Devel/emacs/lib
 CPPFLAGS=-Ic:/Devel/emacs/include'





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

* bug#16837: 24.3.50; Allow a HISTORY arg for `read-file-name', `read-file-name-default'
  2014-02-21 18:34 bug#16837: 24.3.50; Allow a HISTORY arg for `read-file-name', `read-file-name-default' Drew Adams
@ 2016-04-29 17:39 ` Lars Ingebrigtsen
  2021-05-30  4:45   ` Lars Ingebrigtsen
  0 siblings, 1 reply; 3+ messages in thread
From: Lars Ingebrigtsen @ 2016-04-29 17:39 UTC (permalink / raw)
  To: Drew Adams; +Cc: 16837

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

> Please consider allowing an optional HISTORY arg to these functions
> (with the default value being of course `file-name-history').
>
> It can happen that someone wants the behavior of one of these functions,
> but with a different file-name history, such as `shell-command-history'.

Yes, that seems potentially useful.  It currently uses the hard-coded
`file-name-history' history.

Anybody have any objections to adding this?  I see one possible
complication -- it'll be Yet Another Optional Parameter that has to be
passed on to `read-file-name-function', and if somebody has defined that
to take fewer parameters, it'll bomb...

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





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

* bug#16837: 24.3.50; Allow a HISTORY arg for `read-file-name', `read-file-name-default'
  2016-04-29 17:39 ` Lars Ingebrigtsen
@ 2021-05-30  4:45   ` Lars Ingebrigtsen
  0 siblings, 0 replies; 3+ messages in thread
From: Lars Ingebrigtsen @ 2021-05-30  4:45 UTC (permalink / raw)
  To: Drew Adams; +Cc: 16837

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Yes, that seems potentially useful.  It currently uses the hard-coded
> `file-name-history' history.
>
> Anybody have any objections to adding this?  I see one possible
> complication -- it'll be Yet Another Optional Parameter that has to be
> passed on to `read-file-name-function', and if somebody has defined that
> to take fewer parameters, it'll bomb...

And thinking about this for five more years, I think that's a pretty
serious objection -- the argument list is part of the API here, so we
can't really just add another parameter here.

So people that want a different history variable here will have to make
their own `read-file-name-function' function.

Closing.

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





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

end of thread, other threads:[~2021-05-30  4:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-21 18:34 bug#16837: 24.3.50; Allow a HISTORY arg for `read-file-name', `read-file-name-default' Drew Adams
2016-04-29 17:39 ` Lars Ingebrigtsen
2021-05-30  4:45   ` Lars Ingebrigtsen

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