Re: [PATCH] rev-parse --show-prefix: add in trailing newline
From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:53:32
(cc-ing Duy) Hi, Ross Lagerwall wrote:
quoted hunk ↗ jump to hunk
--- a/builtin/rev-parse.c +++ b/builtin/rev-parse.c@@ -634,6 +634,8 @@ int cmd_rev_parse(int argc, const char **argv, const char *prefix) if (!strcmp(arg, "--show-prefix")) { if (prefix) puts(prefix); + else + putchar('\n'); continue;
This makes the output more consistent but it is a little puzzling how it does that. Why is prefix NULL instead of "" when we are at the toplevel of the worktree? Thanks, Jonathan