From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 13 Jan 2010 06:56:24 GMT
Subject: Re: [PATCH 16/18] blob.c: remove unused function
Message-ID: <7veilusdbb.fsf@alter.siamese.dyndns.org>
In-Reply-To: <alpine.LNX.2.00.1001121128410.14365@iabervon.org>

Daniel Barkalow <barkalow@iabervon.org> writes:

> On Mon, 11 Jan 2010, Junio C Hamano wrote:
>
>> parse_blob() is not used anywhere since a510bfa (Mark blobs as parsed when
>> they're actually parsed, 2005-04-28).
>
> Perhaps it should be replaced with a comment that blobs are never parsed, 
> because they don't need to be? We don't need the actual function, but I 
> think it's worth having a note where the function would be.

I tend to agree.

Care to come up with a patch, with a real description of the reason _why_
they don't need to be parsed, please?