git-diff-tree: clean up output
This only shows the tree headers when something actually changed. Also,
add a "silent" mode, which doesn't actually show the changes at all,
just the commit information.
diff --git a/diff-tree.c b/diff-tree.c
index 9e15b70..b9267d6 100644
--- a/diff-tree.c
+++ b/diff-tree.c
@@ -2,11 +2,13 @@
#include "cache.h"
#include "diff.h"
+static int silent = 0;
static int ignore_merges = 1;
static int recursive = 0;
static int read_stdin = 0;
static int line_termination = '\n';
static int generate_patch = 0;
+static const char *header = NULL;
// What paths are we interested in?
static int nr_paths = 0;
@@ -67,6 +69,14 @@
const char *path;
const unsigned char *sha1 = extract(tree, size, &path, &mode);
+ if (header) {
+ printf("%s", header);
+ header = NULL;
+ }
+
+ if (silent)
+ return;
+
if (recursive && S_ISDIR(mode)) {
char type[20];
unsigned long size;
@@ -138,6 +148,13 @@
return retval;
}
+ if (header) {
+ printf("%s", header);
+ header = NULL;
+ }
+ if (silent)
+ return 0;
+
if (generate_patch) {
if (!S_ISDIR(mode1))
diff_change(mode1, mode2, sha1, sha2, base, path1);
@@ -258,6 +275,7 @@
int len = strlen(line);
unsigned char commit[20], parent[20];
unsigned long size, offset;
+ static char this_header[100];
char *buf;
if (!len || line[len-1] != '\n')
@@ -266,9 +284,10 @@
if (get_sha1_hex(line, commit))
return -1;
if (isspace(line[40]) && !get_sha1_hex(line+41, parent)) {
- line[40] = ' ';
+ line[40] = 0;
line[81] = 0;
- printf("%s:\n", line);
+ sprintf(this_header, "%s (from %s)\n", line, line+41);
+ header = this_header;
return diff_tree_sha1(parent, commit, "");
}
buf = read_object_with_reference(commit, "commit", &size, NULL);
@@ -286,7 +305,8 @@
while (offset + 48 < size && !memcmp(buf + offset, "parent ", 7)) {
if (get_sha1_hex(buf + offset + 7, parent))
return -1;
- printf("%s %s:\n", line, sha1_to_hex(parent));
+ sprintf(this_header, "%s (from %s)\n", line, sha1_to_hex(parent));
+ header = this_header;
diff_tree_sha1(parent, commit, "");
offset += 48;
}
@@ -330,6 +350,10 @@
ignore_merges = 0;
continue;
}
+ if (!strcmp(arg, "-s")) {
+ silent = 1;
+ continue;
+ }
if (!strcmp(arg, "--stdin")) {
read_stdin = 1;
continue;
