[PATCH] emacs: globally replace non-branching "(if COND (progn ..." with "(when ..."
Pieter Praet
pieter at praet.org
Wed Feb 1 05:46:19 PST 2012
On Mon, 30 Jan 2012 09:23:40 +0000, David Edmondson <dme at dme.org> wrote:
> On Sat, 28 Jan 2012 13:14:45 -0400, David Bremner <david at tethera.net> wrote:
> > 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).
>
> Each patch should be valid in the repository without any following
> patches (preceding are obviously okay). Incorrect indentation would
> disqualify a patch from being 'valid', so it shouldn't be accepted.
+1.
Indentation corrections should always be part of the same patch as
the change(s) that invalidated the indentation in the first place.
Spotting the *actual* (non-indentation) changes is a non-issue
when using `diff-refine-hunk' (or a wrapper thereof [1]).
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
Peace
--
Pieter
[1] id:"8739ax7jts.fsf at praet.org"
More information about the notmuch
mailing list