On Tue, May 28, 2013 at 10:41 PM, Felipe Contreras
<felipe.contre...@gmail.com> wrote:
> On Wed, May 29, 2013 at 12:33 AM, Martin von Zweigbergk
> <martinv...@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.

Oh, now I remember. I ran into that bug once.

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

I didn't say there was anything wrong. I was asking if the bug was
there before (and I didn't see an answer when Junio asked).

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to