Re: [PATCH] Remove redundant 'function's around lambda

[Kyle Meyer]

Stefan Kangas writes:

> I've been working on removing redundant `function' around `lambda' in
> Emacs core, so here is a patch which does the same for Org-mode.

Thanks.

> Subject: [PATCH] Remove redundant 'function's around lambda
[...]
> diff --git a/lisp/ox-odt.el b/lisp/ox-odt.el
> index 4a0cca612..da351ef82 100644
> --- a/lisp/ox-odt.el
> +++ b/lisp/ox-odt.el
> @@ -2200,10 +2200,10 @@ SHORT-CAPTION are strings."
>  (defun org-odt--image-size
>    (file info &optional user-width user-height scale dpi embed-as)
>    (let* ((--pixels-to-cms
> -          (lambda (pixels dpi)
> -            (let ((cms-per-inch 2.54)
> -                  (inches (/ pixels dpi)))
> -              (* cms-per-inch inches))))
> +       (lambda (pixels dpi)
> +         (let ((cms-per-inch 2.54)
> +               (inches (/ pixels dpi)))
> +           (* cms-per-inch inches))))

All these look good to me except this unrelated whitespace change, which
actually touches the change ported from your 61dca6e92a (Don't quote
lambdas in several places, 2020-11-14) in the Emacs repo.

Org files don't use a consistent style, despite Org's .dir-locals.el
setting indent-tabs-mode to t, which should probably be changed to match
Emacs's nil value.  At any rate, I've dropped this hunk since it's not
actually making the advertised change, and pushed the rest with
1a480e01a.

Thanks again.