From: Victoria Dye <vdye@github.com>
Date: Wed, 17 May 2023 18:47:03 GMT
Subject: Re: [RFC][PATCH V1] diff-tree: integrate with sparse index
Message-ID: <2d99fbed-3074-be22-2b8c-a75dd22bda65@github.com>
In-Reply-To: <xmqqsfbvswci.fsf@gitster.g>

Junio C Hamano wrote:
> Victoria Dye <vdye@github.com> writes:
> 
>> Note that, if you want to make
>> that change in this series, it should be done in a separate patch _before_
>> this one (since dropping the deprecated setup infrastructure isn't really
>> part of the sparse index integration).
> 
> Or after this one, perhaps?  I agree that the clean-up opportunity
> you found is very much unrelated to the work to teach diff-tree to
> take advantage of the sparse index.

You're right, it doesn't need to come before this patch (or belong to this
series). *If* the cleanup was done in this series, my thought was that it
would be (subjectively) better to end the series on the sparse index
integration. However, it doesn't really make a practical difference whether
the cleanup is done before or after this patch, since it's functionally
unrelated to the sparse index work. 

Thanks for the clarification!