From: Jonathan Nieder <jrnieder@gmail.com>
Date: Tue, 30 Oct 2012 22:07:17 GMT
Subject: Re: [PATCH v2 3/4] fast-export: don't handle uninteresting refs
Message-ID: <20121030220717.GO15167@elie.Belkin>
In-Reply-To: <CAMP44s1b+E8a0kdSgREbGzRTFy+nCw4VcjHadd3soQAXRkNzZw@mail.gmail.com>

Felipe Contreras wrote:

> Well, that's what we have now, and you want to preserve this "feature"
> (aka bug), right?

Nope.  I just don't want regressions, and found a patch description
that did nothing to explain to the reader how it avoids regressions
more than a little disturbing.

I also think the proposed behavior is wrong.

I don't think there's much benefit to be gained from continuing to
discuss this.  Consider it a single data point: I would be deeply
worried if this patch were applied without at least a clearer
description of the change in behavior.  Maybe I'm the only one!

Hope that helps,
Jonathan