From c983370e5e208cca4deda82184478ba445a156e6 Mon Sep 17 00:00:00 2001 From: Junio C Hamano Date: Sun, 1 May 2005 09:33:12 -0700 Subject: [PATCH] [PATCH] Rework built-in diff to make its output more dense. Linus says, The fewer lines there are that don't usually tell a human anything, the better. Dense is good. This patch makes the default diff output more dense. This removes the previous misguided attempt to be cg-patch compatible. Signed-off-by: Junio C Hamano Signed-off-by: Linus Torvalds --- diff.c | 26 +++++++++++++++----------- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/diff.c b/diff.c index 8659e4b04..65110333f 100644 --- a/diff.c +++ b/diff.c @@ -82,35 +82,32 @@ static void builtin_diff(const char *name, struct diff_tempfile *temp) { int i, next_at; - const char *diff_cmd = "diff -L'%s%s%s' -L'%s%s%s'"; + const char *diff_cmd = "diff -L'%s%s' -L'%s%s'"; const char *diff_arg = "'%s' '%s'"; const char *input_name_sq[2]; const char *path0[2]; const char *path1[2]; - char mode[2][20]; const char *name_sq = sq_expand(name); char *cmd; - /* diff_cmd and diff_arg have 8 %s in total which makes - * the sum of these strings 16 bytes larger than required. + /* diff_cmd and diff_arg have 6 %s in total which makes + * the sum of these strings 12 bytes larger than required. * we use 2 spaces around diff-opts, and we need to count - * terminating NUL, so we subtract 13 here. + * terminating NUL, so we subtract 9 here. */ int cmd_size = (strlen(diff_cmd) + strlen(diff_opts) + - strlen(diff_arg) - 13); + strlen(diff_arg) - 9); for (i = 0; i < 2; i++) { input_name_sq[i] = sq_expand(temp[i].name); if (!strcmp(temp[i].name, "/dev/null")) { path0[i] = "/dev/null"; path1[i] = ""; - mode[i][0] = 0; } else { path0[i] = i ? "l/" : "k/"; path1[i] = name_sq; - sprintf(mode[i], " (mode:%s)", temp[i].mode); } cmd_size += (strlen(path0[i]) + strlen(path1[i]) + - strlen(mode[i]) + strlen(input_name_sq[i])); + strlen(input_name_sq[i])); } cmd = xmalloc(cmd_size); @@ -118,13 +115,20 @@ static void builtin_diff(const char *name, next_at = 0; next_at += snprintf(cmd+next_at, cmd_size-next_at, diff_cmd, - path0[0], path1[0], mode[0], - path0[1], path1[1], mode[1]); + path0[0], path1[0], path0[1], path1[1]); next_at += snprintf(cmd+next_at, cmd_size-next_at, " %s ", diff_opts); next_at += snprintf(cmd+next_at, cmd_size-next_at, diff_arg, input_name_sq[0], input_name_sq[1]); + if (!path1[0][0]) + printf("Created: %s (mode:%s)\n", name, temp[1].mode); + else if (!path1[1][0]) + printf("Deleted: %s\n", name); + else if (strcmp(temp[0].mode, temp[1].mode)) + printf("Mode changed: %s (%s->%s)\n", name, + temp[0].mode, temp[1].mode); + fflush(NULL); execlp("/bin/sh","sh", "-c", cmd, NULL); } -- 2.26.2