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