Christian Couder <chrisc...@tuxfamily.org> 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.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to