Re: [PATCH] Avoid warning when bisecting a merge
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 5, 2008, 08:31 UTC
- Message-ID
- <7v3akfvvsp.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <48C0DD32.5010008@isy.liu.se>
Gustaf Hendeby <hendeby@isy.liu.se> writes:
Show 7 quoted lines
> On 09/05/2008 08:29 AM, Junio C Hamano wrote: > ... >> Yup, that might make more sense. In the meantime, I suspect this makes >> more sense than introducing a new state "0". > > I can't argue with that. Junio, your suggestion makes more sense. I > should have paid more attention to what I was doing. Sorry about the noise.
That's Ok. Spotting mistakes is an important part of the review process; there is no need to be afraid of or apologetic about it.