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

Jani Nikula jani at nikula.org
Sat Jan 28 04:55:22 PST 2012


On Jan 28, 2012 2:41 PM, "David Bremner" <david at tethera.net> wrote:
>
> On Sat, 14 Jan 2012 10:17:18 +0100, Pieter Praet <pieter at praet.org> wrote:
> > Less code, same results, without sacrificing readability.
> >
>
> This looks OK, although the re-indenting makes these kind of changes
> painful to review (not that I'm suggesting we should re-indent, just
> some random complaining).

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?

>
> d
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120128/2e169a16/attachment.html>


More information about the notmuch mailing list