From: Junio C Hamano Date: Wed, 11 Jun 2025 15:34:07 GMT Subject: Re: [PATCH v2 1/2] trailer: append trailers in-process and drop the fork to `interpret-trailers` Message-ID: In-Reply-To: <407631ab-1184-41b1-8043-fd91d860e845@gmail.com> phillip.wood123@gmail.com writes: > 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.