diff options
author | Linus Arver <linusa@google.com> | 2024-03-15 06:55:02 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2024-03-15 10:10:24 -0700 |
commit | 41ea0a900221897c5ba36afb9f0b31bf543cea7e (patch) | |
tree | 678ec2aa50784d86b0330b23218249bb108567c0 /trailer.c | |
parent | 65b4ad82b81e1a1f4afbb7f4974384d7db479c0a (diff) | |
download | git-41ea0a900221897c5ba36afb9f0b31bf543cea7e.tar.gz |
format_trailer_info(): drop redundant unfold_value()
This is another preparatory refactor to unify the trailer formatters.
In the last patch we made format_trailer_info() use trailer_item objects
instead of the "trailers" string array. This means that the call to
unfold_value() here is redundant because the trailer_item objects are
already unfolded in parse_trailers() which is a dependency of our
caller, format_trailers_from_commit().
Remove the redundant call.
Signed-off-by: Linus Arver <linusa@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'trailer.c')
-rw-r--r-- | trailer.c | 3 |
1 files changed, 0 insertions, 3 deletions
@@ -1101,9 +1101,6 @@ static void format_trailer_info(const struct process_trailer_options *opts, strbuf_addstr(&val, item->value); if (!opts->filter || opts->filter(&tok, opts->filter_data)) { - if (opts->unfold) - unfold_value(&val); - if (opts->separator && out->len != origlen) strbuf_addbuf(out, opts->separator); if (!opts->value_only) |