Subject: Re: [PATCH v5 10/14] trailer: execute command from 'trailer.<name>.command'

From: Junio C Hamano <[email-masked]>
Date: Thu, 06 Feb 2014 16:18:10 -0800

Christian Couder <[email-masked]> writes:

> +#define TRAILER_ARG_STRING "$ARG"

No need to support users who may want to use a string that happens
to match this substring literally as part of the command line?

>  struct trailer_item {
>  	struct trailer_item *previous;
>  	struct trailer_item *next;
> @@ -56,6 +60,13 @@ static inline int contains_only_spaces(const char *str)
>  	return !*s;
>  }
>  
> +static inline void strbuf_replace(struct strbuf *sb, const char *a, const char *b)

Same comment about "inline" for an earlier patch applies.