Re: [PATCH v2] receive-pack: purge temporary data if no command is ready to run

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 5, 2022, 08:02 UTC
Message-ID: <xmqq1r0hlp12.fsf@gitster.g>
In-Reply-To: <CADuS7Apcr0r6F1UQf1WKmVuW+SM_wD1MBPA9Szvd1pF9zJjsUw@mail.gmail.com>

Bojun Chen <bojun.cbj@gmail.com> writes:

Show 9 quoted lines

>> This check can _almost_ be doable without being able to access the
>> new objects, and as a follow-on work, it might not be a bad little
>> project to see how we can move the call to check_aliased_updates()
>> before this loop we are adding in this patch (#leftoverbits).
>>
>> Thanks.
>
> Thanks for your suggestion, I agree with you. But I'm confused should
> I continue in this patch or start a new patch.

Neither.

You are under no obligation to take such a different project, which may be vaguely related to this one. This early return by itself is a worthwhile improvement, so let's concentrat on finishing this topic first.