emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

helpers.el [was: ... lisp/emacs-lisp/helpers.el...]


From: Drew Adams
Subject: helpers.el [was: ... lisp/emacs-lisp/helpers.el...]
Date: Fri, 29 Nov 2013 15:50:30 -0800 (PST)

FWIW -

`helpers.el' is an *un*helpful file name.

It is bad enough that we already have a file `helper.el' in
the same directory.  Even for that file the name is not so
useful, but at least that is about providing "help in electric
modes".  (Something like `elec-help.el' would have been better.)

Please consider coming up with something better than "helpers".

Especially since `helpers.el' is purportedly "Some non-essential
library extensions."  Library extensions?  What library is
extended?  If not a library, what is extended by this code?

Non-essential is right, however.  This file has 7 one-liner
defsubsts in it - nothing more.  Now maybe big things are
expected for this little file in the future, but even then I'd
suggest that, at least for now, these functions be put
somewhere else.

Where to put such things, if not in a dedicated trifles bag?
Put similar things together (and not just similar by being tiny).

And if the intention is to progressively pick up other such
functions from other files and toss them in `helpers.el', so
that it becomes a growing catch-all, then I'd recommend to
think twice about that project.

But if you really cannot do any better than create a grab bag,
then please at least give it a name that reflects that failure
and does not lead to more confusion.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]