From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Wed, 29 May 2013 05:41:22 GMT
Subject: Re: [RFC/PATCH v2 3/8] rebase: cherry-pick: fix sequence continuation
Message-ID: <CAMP44s0Zy4KpPN1n6HOVXWyCuevenbSFnH589YngMB9NVWcamQ@mail.gmail.com>
In-Reply-To: <CANiSa6ivOnRfOVMTsgDygi=2dvxmMOqqWdqs7CBYohThOVzt7Q@mail.gmail.com>

On Wed, May 29, 2013 at 12:33 AM, Martin von Zweigbergk
<martinvonz@gmail.com> wrote:
>  As Junio asked in the previous iteration, shouldn't this have been in
> the first patch?

No, the first patch is splitting the code without introducing any
functional changes.

This is fixing a bug that already exists, we could fix it before the
split, or after, but mixing the split and the fix at the same time is
a no-no.

One change splits, the other change fixes, what's wrong with that?

This way it's easy to see what each patch does.

-- 
Felipe Contreras