Re: [PATCH v2 3/4] fast-export: don't handle uninteresting refs
- From
Felipe Contreras <felipe.contreras@gmail.com>
- Date
- Oct 30, 2012, 22:22 UTC
- Message-ID
- <CAMP44s3ArAQXH+-EbH4MHYaV6fTAWdwGzBdZwzn_qtCABHyonQ@mail.gmail.com>
- In-Reply-To
- <20121030220717.GO15167@elie.Belkin>
On Tue, Oct 30, 2012 at 11:07 PM, Jonathan Nieder <jrnieder@gmail.com> wrote:
Show 8 quoted lines
> 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 see, so you don't have any specific case where this could cause regressions, you are just saying it _might_ (like all patches).
> I also think the proposed behavior is wrong.
That's a different matter, lets see what others think.
-- Felipe Contreras