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

2020-12-22 Thread Kyle Meyer
Bastien writes: > Hi Kyle, > > Kyle Meyer writes: > >> 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. > > FWIW, I'm all for setting `indent-tabs-mode' to nil in Org's

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

2020-12-13 Thread Bastien
Hi Kyle, Kyle Meyer writes: > 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. FWIW, I'm all for setting `indent-tabs-mode' to nil in Org's .dir-locals.el. Your call, of course.

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

2020-11-19 Thread Neil Jerram
On Thu, 19 Nov 2020 at 14:51, Stefan Kangas wrote: > Neil Jerram writes: > > >> I've been working on removing redundant `function' around `lambda' in > >> Emacs core, > > > > I'm slightly curious about the history and reasoning around this. If I > > understand correctly, (lambda ...) on its

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

2020-11-19 Thread Stefan Kangas
Neil Jerram writes: >> I've been working on removing redundant `function' around `lambda' in >> Emacs core, > > I'm slightly curious about the history and reasoning around this. If I > understand correctly, (lambda ...) on its own has always worked, and it's > never been strictly necessary to

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

2020-11-19 Thread Neil Jerram
On Tue, 17 Nov 2020 at 17:08, Stefan Kangas wrote: > I've been working on removing redundant `function' around `lambda' in > Emacs core, I'm slightly curious about the history and reasoning around this. If I understand correctly, (lambda ...) on its own has always worked, and it's never been

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

2020-11-18 Thread Stefan Kangas
Kyle Meyer writes: > 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. I must have included that part by mistake, sorry about that. > Org

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

2020-11-18 Thread 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-od

[PATCH] Remove redundant 'function's around lambda

2020-11-17 Thread Stefan Kangas
From: Stefan Kangas Date: Tue, 17 Nov 2020 17:35:49 +0100 Subject: [PATCH] Remove redundant 'function's around lambda * contrib/lisp/org-secretary.el (join): * contrib/lisp/ox-taskjuggler.el (org-taskjuggler-project-plan): * lisp/org-clock.el (org-resolve-clocks): * lisp/org-element.el (org-element