* Re: trunk r115768: * lisp/electric.el: Move all electric-pair-* to elec-pair.el.
[not found] <E1VwWvR-0005Uo-5Q@vcs.savannah.gnu.org>
@ 2013-12-28 2:48 ` Leo Liu
2013-12-28 3:38 ` João Távora
2013-12-28 13:31 ` Stefan Monnier
0 siblings, 2 replies; 4+ messages in thread
From: Leo Liu @ 2013-12-28 2:48 UTC (permalink / raw)
To: Stefan Monnier; +Cc: Joao Tavora, emacs-devel
On 2013-12-27 20:54 +0800, Stefan Monnier wrote:
> +2013-12-27 Stefan Monnier <monnier@iro.umontreal.ca>
> +
> + * electric.el: Move all electric-pair-* to elec-pair.el.
> + * elec-pair.el: New file, split from electric.el.
elec-pair looks bizarre. How about give it back the name autopair.el?
Leo
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: trunk r115768: * lisp/electric.el: Move all electric-pair-* to elec-pair.el.
2013-12-28 2:48 ` trunk r115768: * lisp/electric.el: Move all electric-pair-* to elec-pair.el Leo Liu
@ 2013-12-28 3:38 ` João Távora
2013-12-28 13:31 ` Stefan Monnier
1 sibling, 0 replies; 4+ messages in thread
From: João Távora @ 2013-12-28 3:38 UTC (permalink / raw)
To: Leo Liu; +Cc: Stefan Monnier, emacs-devel
Leo Liu <sdl.web@gmail.com> writes:
> elec-pair looks bizarre. How about give it back the name autopair.el?
Probably not my call, but I wouldn't object and FWIW I'll be deprecating
the original autopair.el soon in favour of the new feature.
João
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: trunk r115768: * lisp/electric.el: Move all electric-pair-* to elec-pair.el.
2013-12-28 2:48 ` trunk r115768: * lisp/electric.el: Move all electric-pair-* to elec-pair.el Leo Liu
2013-12-28 3:38 ` João Távora
@ 2013-12-28 13:31 ` Stefan Monnier
2013-12-28 14:43 ` João Távora
1 sibling, 1 reply; 4+ messages in thread
From: Stefan Monnier @ 2013-12-28 13:31 UTC (permalink / raw)
To: Leo Liu; +Cc: Joao Tavora, emacs-devel
>> + * electric.el: Move all electric-pair-* to elec-pair.el.
>> + * elec-pair.el: New file, split from electric.el.
> elec-pair looks bizarre. How about give it back the name autopair.el?
I'm not opposed to it, but:
1- the name of the file isn't particularly important. 99.9% of the
users won't ever know about it.
2- the only case where it's important is in terms of "namespace
conflicts", so giving the name of an existing file could introduce
problems (e.g. someone expecting (require 'autopair) to define
specific functions/variables).
So I think it's better to stay with elec-pair.el and move on.
Stefan
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: trunk r115768: * lisp/electric.el: Move all electric-pair-* to elec-pair.el.
2013-12-28 13:31 ` Stefan Monnier
@ 2013-12-28 14:43 ` João Távora
0 siblings, 0 replies; 4+ messages in thread
From: João Távora @ 2013-12-28 14:43 UTC (permalink / raw)
To: Stefan Monnier; +Cc: Leo Liu, emacs-devel
Stefan Monnier <monnier@IRO.UMontreal.CA> writes:
> giving the name of an existing file could introduce problems
> (e.g. someone expecting (require 'autopair) to define specific
> functions/variables).
Yes, it would probably only make some sense if additionally we
defaliased `autopair-global-mode' to point to `electric-pair-mode'.
> So I think it's better to stay with elec-pair.el and move on.
Yeah, most of the names under lisp/ sound bizarre anyway...
(minibuf-eldef.el? just discovered it, actually might be useful...)
João
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-12-28 14:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <E1VwWvR-0005Uo-5Q@vcs.savannah.gnu.org>
2013-12-28 2:48 ` trunk r115768: * lisp/electric.el: Move all electric-pair-* to elec-pair.el Leo Liu
2013-12-28 3:38 ` João Távora
2013-12-28 13:31 ` Stefan Monnier
2013-12-28 14:43 ` João Távora
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).