Re: [PATCH v2 1/2] trailer: append trailers in-process and drop the fork to `interpret-trailers`

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 11, 2025, 15:34 UTC
Message-ID: <xmqqa56ejnrk.fsf@gitster.g>
In-Reply-To: <407631ab-1184-41b1-8043-fd91d860e845@gmail.com>

phillip.wood123@gmail.com writes:

Show 8 quoted lines

> Exactly - I was expecting to see a refactoring of interpret_trailers()
> in builtin/interpret-trailers.c that moved most of the function body
> into a new function in trailer.c that added the trailers to an
> strbuf. This seems to be a parallel implementation which doesn't sound
> like the best plan.
>
> I'm going to be off the list for a couple of weeks, I'll take a more
> detailed look at this series when I'm back

Thanks.