From: Bence Ferdinandy <bence@ferdinandy.com>
Date: Wed, 09 Oct 2024 19:47:21 GMT
Subject: Re: [PATCH v5 2/6] set-head: add new variable for readability
Message-ID: <D4RJ9EZMHPHG.3BMTRPU3B2PMD@ferdinandy.com>
In-Reply-To: <xmqqttdlqdr2.fsf@gitster.g>


On Wed Oct 09, 2024 at 21:26, Junio C Hamano <gitster@pobox.com> wrote:
> Bence Ferdinandy <bence@ferdinandy.com> writes:
>
>> Instead of calling get_main_ref_store(the_repository) multiple times,
>> call it once and store in a new refs variable. Although this change
>> probably offers some performance benefits, the main purpose is to
>> shorten the line lengths of function calls using this variable for
>> better readability.
>> ---
>>
>> Notes:
>>     v5: new patch (split from the next patch as a preparatory step)
>
> It is a good idea to help making it obvious that we are accessing
> the same ref store.

Indeed, although it was your idea ;)

>
> The patch needs to be signed-off, though.

Damn :/ Sorry, another oversight ... Should I send a v6 for the series with
this fixed now or wait a bit to see if there are other comments?

Thanks again for the patience!

Best,
Bence