Re: [PATCH] rev-parse --show-prefix: add in trailing newline

Jonathan Nieder [off-list ref] writes:

(cc-ing Duy)
Hi,

Ross Lagerwall wrote:

quoted

--- 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?

Interesting point ;-)