Re: [PATCH 7/9] config: do not use strbuf_split()

From

Eric Sunshine <sunshine@sunshineco.com>

Date

Jul 31, 2025, 20:15 UTC

Message-ID

<CAPig+cTnNeZ9gtQbKdL6P2KBn7jzGJT6BC+tPMnzBNaR0tyrvQ@mail.gmail.com>

In-Reply-To

<20250731074154.2835370-8-gitster@pobox.com>

On Thu, Jul 31, 2025 at 3:42 AM Junio C Hamano <gitster@pobox.com> wrote:

> When parsing an old-style GIT_CONFIG_PARAMETERS environment
> variable, the code parses the key=value pair by spliting them at '='
> into an array of strbuf's.  As strbuf_split() leafes the delimiter
> at the end of the split piece, the code has to manually trim it.

> If we split with string_list_split(), that becomes unnecessary.
> Retire the use of strbuf_split().
>
> Note that the max parameter of string_list_split() is of
> an ergonomically iffy design---it specifies the maximum number of
> times the function is allowed to split, which means that in order to
> split a text into up to 2 pieces, you have to pass 1, not 2.
>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>