[PATCH] emacs: globally replace non-branching "(if COND (progn ..." with "(when ..."

David Bremner david at tethera.net
Sat Jan 28 09:14:45 PST 2012


On Sat, 28 Jan 2012 14:55:22 +0200, Jani Nikula <jani at nikula.org> wrote:
> On Jan 28, 2012 2:41 PM, "David Bremner" <david at tethera.net> wrote:
> 
> Sometimes someone (Dmitry?) sent patches that separated a small functional
> change, and the big non-functional indentation change it caused,
> separately. Would you prefer (or tolerate ;) that style?

Hmm, that might be nicer, I'm not 100% sure.

I wouldn't say it's mandatory for a patch like this (and I'd say other
peoples views on what's easy to review are at least as important as mine
here).

d




More information about the notmuch mailing list