Browse Source

combine-diff: remove misguided --show-empty hack.

Now --always flag is available in diff-tree, there is no reason
to have that hack in the diffcore side.

Signed-off-by: Junio C Hamano <junkio@cox.net>
maint
Junio C Hamano 19 years ago
parent
commit
e3c3a550d4
  1. 10
      combine-diff.c
  2. 2
      diff-files.c
  3. 3
      diff-tree.c
  4. 4
      diff.h

10
combine-diff.c

@ -608,7 +608,7 @@ static void reuse_combine_diff(struct sline *sline, unsigned long cnt,
} }


int show_combined_diff(struct combine_diff_path *elem, int num_parent, int show_combined_diff(struct combine_diff_path *elem, int num_parent,
int dense, const char *header, int show_empty) int dense, const char *header)
{ {
unsigned long size, cnt, lno; unsigned long size, cnt, lno;
char *result, *cp, *ep; char *result, *cp, *ep;
@ -742,8 +742,7 @@ int show_combined_diff(struct combine_diff_path *elem, int num_parent,
} }


int diff_tree_combined_merge(const unsigned char *sha1, int diff_tree_combined_merge(const unsigned char *sha1,
const char *header, const char *header, int dense)
int show_empty_merge, int dense)
{ {
struct commit *commit = lookup_commit(sha1); struct commit *commit = lookup_commit(sha1);
struct diff_options diffopts; struct diff_options diffopts;
@ -777,12 +776,11 @@ int diff_tree_combined_merge(const unsigned char *sha1,
if (p->len) if (p->len)
num_paths++; num_paths++;
} }
if (num_paths || show_empty_merge) { if (num_paths) {
for (p = paths; p; p = p->next) { for (p = paths; p; p = p->next) {
if (!p->len) if (!p->len)
continue; continue;
if (show_combined_diff(p, num_parent, dense, header, if (show_combined_diff(p, num_parent, dense, header))
show_empty_merge))
header = NULL; header = NULL;
} }
} }

2
diff-files.c

@ -160,7 +160,7 @@ int main(int argc, const char **argv)
if (combine_merges && num_compare_stages == 2) { if (combine_merges && num_compare_stages == 2) {
show_combined_diff(&combine.p, 2, show_combined_diff(&combine.p, 2,
dense_combined_merges, dense_combined_merges,
NULL, 0); NULL);
free(combine.p.path); free(combine.p.path);
continue; continue;
} }

3
diff-tree.c

@ -6,7 +6,6 @@ static int show_root_diff = 0;
static int no_commit_id = 0; static int no_commit_id = 0;
static int verbose_header = 0; static int verbose_header = 0;
static int ignore_merges = 1; static int ignore_merges = 1;
static int show_empty_combined = 0;
static int combine_merges = 0; static int combine_merges = 0;
static int dense_combined_merges = 0; static int dense_combined_merges = 0;
static int read_stdin = 0; static int read_stdin = 0;
@ -127,7 +126,6 @@ static int diff_tree_commit(const unsigned char *commit_sha1)
else if (combine_merges) { else if (combine_merges) {
header = generate_header(sha1, sha1, commit); header = generate_header(sha1, sha1, commit);
return diff_tree_combined_merge(sha1, header, return diff_tree_combined_merge(sha1, header,
show_empty_combined,
dense_combined_merges); dense_combined_merges);
} }
} }
@ -278,7 +276,6 @@ int main(int argc, const char **argv)


if (combine_merges) { if (combine_merges) {
diff_options.output_format = DIFF_FORMAT_PATCH; diff_options.output_format = DIFF_FORMAT_PATCH;
show_empty_combined = !ignore_merges;
ignore_merges = 0; ignore_merges = 0;
} }



4
diff.h

@ -68,9 +68,9 @@ struct combine_diff_path {
}; };


int show_combined_diff(struct combine_diff_path *elem, int num_parent, int show_combined_diff(struct combine_diff_path *elem, int num_parent,
int dense, const char *header, int show_empty); int dense, const char *header);


extern int diff_tree_combined_merge(const unsigned char *sha1, const char *, int, int); extern int diff_tree_combined_merge(const unsigned char *sha1, const char *, int);


extern void diff_addremove(struct diff_options *, extern void diff_addremove(struct diff_options *,
int addremove, int addremove,

Loading…
Cancel
Save