Am 22.03.2013 19:17, schrieb Nicolas Goaziou:
You might want to use the (function (lambda ...)) form.

Why? (function (lambda ...)) is equivalent to (lambda ...). In fact,
`lambda' uses `function' as a subroutine.

http://www.gnu.org/software/emacs/manual/html_node/elisp/Anonymous-Functions.html

http://stackoverflow.com/questions/1852844/emacs-lisp-difference-between-function-lambda-and-lambda

With Emacs 24.3 and eager macro expoansion the need to do this has diminished, but for the benefit of older Emacsen we should probably continue the practise.

That's why I also suggest to drop the (mapcar 'org-trim ...) wrapper.
There's no reason to remove blanks if the user inserted them in the
first place.

Oh, OK - I did not realise that.

Since that's not costly to achieve, there no real reason not to support
it.

I'll trust you on the "not costly" part, I can't do any experiment right now.

So, any objection (or further comment) for that change?

Please go ahead.


Regards,
--
Achim.

(on the road :-)


Reply via email to