Anders Waldenborg writes:
> AFAICU strbuf_expand doesn't suffer from the worst things that printf(3)
> suffers from wrt untrusted format string (i.e no printf style %n which
> can write to memory, and no vaargs on stack which allows leaking random
> stuff).
>
> The separator option is part of
Junio C Hamano writes:
> Anders Waldenborg writes:
>
>> @@ -1352,6 +1353,17 @@ static size_t format_commit_one(struct strbuf *sb, /*
>> in UTF-8 */
>> arg++;
>>
>> opts.only_trailers = 1;
>> +
Anders Waldenborg writes:
> + if (opts->separator && first_printed)
> + strbuf_addbuf(out, opts->separator);
> if (opts->no_key)
> - strbuf_addf(out, "%s\n",
Anders Waldenborg writes:
> @@ -1352,6 +1353,17 @@ static size_t format_commit_one(struct strbuf *sb, /*
> in UTF-8 */
> arg++;
>
> opts.only_trailers = 1;
> + } else if
By default trailer lines are terminated by linebreaks ('\n'). By
specifying the new 'separator' option they will instead be separated by
user provided string and have separator semantics rather than terminator
semantics. The separator string can contain the fundamental formatting
codes %n and %xNN
5 matches
Mail list logo
