Subject: Re: [GSoC PATCH v3 2/5] pack-write: add helper to fill promisor file after repack

From: Lorenzo Pegorari <[email-masked]>
Date: Tue, 7 Apr 2026 23:46:30 +0200

On Mon, Apr 06, 2026 at 02:17:02PM -0700, Junio C Hamano wrote:
> Also strbuf_split*() is a bad API.  Unless you need all the parts[]
> strbuf instances all editable at the same time, an array of strbuf
> is a data structure that is way overkill.  Splitting into string-list
> may make it more palatable, I think.
> 
> We even went through a series of patches (and follow-up effort by
> other contributors) [*] to rewrite callers that unnecessarily call
> strbuf_split*().
> 
> [References]
>   https://lore.kernel.org/git/[email-masked]/
>   https://lore.kernel.org/git/[email-masked]/

Mhm makes perfect sense. I will rewrite it using `string_list`. Thanks!

Lorenzo