Re: [PATCH] rev-list: restore the NUL commit separator in --header mode
From: Jacob Keller <hidden>
Date: 2016-10-19 22:16:03
Hi, On Wed, Oct 19, 2016 at 2:04 PM, Dennis Kaarsemaker [off-list ref] wrote:
Commit 660e113 (graph: add support for --line-prefix on all graph-aware output) changed the way commits were shown. Unfortunately this dropped the NUL between commits in --header mode. Restore the NUL and add a test for this feature.
Oops! Thanks for the bug fix.
quoted hunk ↗ jump to hunk
Signed-off-by: Dennis Kaarsemaker <redacted> --- builtin/rev-list.c | 4 ++++ t/t6000-rev-list-misc.sh | 7 +++++++ 2 files changed, 11 insertions(+)diff --git a/builtin/rev-list.c b/builtin/rev-list.c index 8479f6e..cfa6a7d 100644 --- a/builtin/rev-list.c +++ b/builtin/rev-list.c@@ -157,6 +157,10 @@ static void show_commit(struct commit *commit, void *data) if (revs->commit_format == CMIT_FMT_ONELINE) putchar('\n'); } + if (revs->commit_format == CMIT_FMT_RAW) { + putchar(info->hdr_termination); + } +
This seems right to me. My one concern is that we make sure we restore it for every case (in case it needs to be there for other formats?) I'm not entirely sure about whether other non-raw modes need this or not? Thanks, Jake