Hi Paul,
On 2015-05-18 17:06, Paul Tan wrote:
> diff --git a/builtin/pull.c b/builtin/pull.c
> index ba2ff01..81e31a1 100644
> --- a/builtin/pull.c
> +++ b/builtin/pull.c
> @@ -212,6 +212,26 @@ static void argv_push_force(struct argv_array *arr)
> }
>
> /**
> + * Sets the GIT_REFLOG_ACTION environment variable to the concatenation of argv
> + */
> +static void set_reflog_message(int argc, const char **argv)
> +{
> + int i;
> + struct strbuf msg = STRBUF_INIT;
> +
> + for (i = 0; i < argc; i++) {
> + strbuf_addstr(&msg, argv[i]);
> + strbuf_addch(&msg, ' ');
> + }
> +
> + strbuf_rtrim(&msg);
Since this is not performance critical code, I would be slightly in favor of simplifying thusly:
/* Join the argument list, separated by spaces */
for (i = 0; i < argc; i++)
strbuf_addf(&msg, "%s%s", i ? " " : "", argv[i]);
Just a suggestion,
Dscho
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html