From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 20 Feb 2009 00:09:45 GMT
Subject: Re: [PATCH] Make repack less likely to corrupt repository
Message-ID: <7v7i3mklyu.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <alpine.DEB.1.00.0902192337040.6223@intel-tinevez-2-302>

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

>> +for name in $names
>> +do
>> +	rm -f "$PACKDIR/old-pack-$name.idx"
>> +	rm -f "$PACKDIR/old-pack-$name.pack"
>> +done
>> +
>> +# End of pack replacement.
>
> Is this comment really needed here?

Is that hunk still needed?  I think the removal part is already there all
the way down to maint.

Other than that, I think your comments are all sane.