diff options
| author | Linus Arver <linusa@google.com> | 2024-03-01 00:14:41 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-03-01 10:35:42 -0800 |
| commit | ae0ec2e0e0bb26474f395504c6ed6fef3f59091f (patch) | |
| tree | 737f05614fb0b160319e26e4d1f7bd8e7738a896 /commit.c | |
| parent | trailer: reorder format_trailers_from_commit() parameters (diff) | |
| download | git-ae0ec2e0e0bb26474f395504c6ed6fef3f59091f.tar.gz git-ae0ec2e0e0bb26474f395504c6ed6fef3f59091f.zip | |
trailer: move interpret_trailers() to interpret-trailers.c
The interpret-trailers.c builtin is the only place we need to call
interpret_trailers(), so move its definition there (together with a few
helper functions called only by it) and remove its external declaration
from <trailer.h>.
Several helper functions that are called by interpret_trailers() remain
in trailer.c because other callers in the same file still call them.
Declare them in <trailer.h> so that interpret_trailers() (now in
builtin/interpret-trailers.c) can continue calling them as a trailer API
user.
This enriches <trailer.h> with a more granular API, which can then be
unit-tested in the future (because interpret_trailers() by itself does
too many things to be able to be easily unit-tested).
Take this opportunity to demote some file-handling functions out of the
trailer API implementation, as these have nothing to do with trailers.
Helped-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Linus Arver <linusa@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit.c')
0 files changed, 0 insertions, 0 deletions