From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 15 Nov 2024 05:50:26 GMT
Subject: Re: [PATCH v12 3/8] remote set-head: refactor for readability
Message-ID: <xmqqjzd511wd.fsf@gitster.g>
In-Reply-To: <20241023153736.257733-4-bence@ferdinandy.com>

Bence Ferdinandy <bence@ferdinandy.com> writes:

> Rename buf and buf2 to something more explanatory.
>
> 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.

Also two strbufs are renamed, which is in line with the objective of
the other change---to make the resulting code easier to follow.

Looking good.

>     v9: - further improve readability by renaming buf, and buf2 consistently
>           with how patch 6 was already done