Subject: Re: [PATCH v2 01/10] trailer: prepare to expose functions as part of API

From: Josh Steadmon <[email-masked]>
Date: Mon, 29 Jan 2024 16:44:01 -0800

On 2024.01.26 22:38, Linus Arver via GitGitGadget wrote:
> From: Linus Arver <[email-masked]>
> 
> In the next patch, we will move "process_trailers" from trailer.c to
> builtin/interpret-trailers.c. That move will necessitate the growth of
> the trailer.h API, forcing us to expose some additional functions in
> trailer.h.
> 
> Rename relevant functions so that they include the term "trailer" in
> their name, so that clients of the API will be able to easily identify
> them by their "trailer" moniker, just like all the other functions
> already exposed by trailer.h.
> 
> The the opportunity to start putting trailer processions options (opts)

Nitpick: typo in the commit message.
s/The the opportunity/Take the opportunity/ ?

> as the first parameter. This will be the pattern going forward in this
> series.
> 
> Helped-by: Junio C Hamano <[email-masked]>
> Signed-off-by: Linus Arver <[email-masked]>
> ---
>  builtin/interpret-trailers.c |  4 ++--
>  trailer.c                    | 26 +++++++++++++-------------
>  trailer.h                    |  6 +++---
>  3 files changed, 18 insertions(+), 18 deletions(-)