From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 18 Aug 2011 00:25:59 +0000 (-0700)
Subject: Merge branch 'jc/maint-combined-diff-work-tree'
X-Git-Tag: v1.7.7-rc0~30
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=660578d4da9d71fbc1e553c95d18987b40af5f0f;p=git.git

Merge branch 'jc/maint-combined-diff-work-tree'

* jc/maint-combined-diff-work-tree:
  diff -c/--cc: do not mistake "resolved as deletion" as "use working tree"

Conflicts:
	combine-diff.c
---

660578d4da9d71fbc1e553c95d18987b40af5f0f
diff --cc combine-diff.c
index be67cfcd4,360b816b3..b11eb7102
--- a/combine-diff.c
+++ b/combine-diff.c
@@@ -691,84 -681,9 +691,85 @@@ static void dump_quoted_path(const cha
  	puts(buf.buf);
  }
  
 +static void show_combined_header(struct combine_diff_path *elem,
 +				 int num_parent,
 +				 int dense,
 +				 struct rev_info *rev,
 +				 int mode_differs,
 +				 int show_file_header)
 +{
 +	struct diff_options *opt = &rev->diffopt;
 +	int abbrev = DIFF_OPT_TST(opt, FULL_INDEX) ? 40 : DEFAULT_ABBREV;
 +	const char *a_prefix = opt->a_prefix ? opt->a_prefix : "a/";
 +	const char *b_prefix = opt->b_prefix ? opt->b_prefix : "b/";
 +	int use_color = DIFF_OPT_TST(opt, COLOR_DIFF);
 +	const char *c_meta = diff_get_color(use_color, DIFF_METAINFO);
 +	const char *c_reset = diff_get_color(use_color, DIFF_RESET);
 +	const char *abb;
 +	int added = 0;
 +	int deleted = 0;
 +	int i;
 +
 +	if (rev->loginfo && !rev->no_commit_id)
 +		show_log(rev);
 +
 +	dump_quoted_path(dense ? "diff --cc " : "diff --combined ",
 +			 "", elem->path, c_meta, c_reset);
 +	printf("%sindex ", c_meta);
 +	for (i = 0; i < num_parent; i++) {
 +		abb = find_unique_abbrev(elem->parent[i].sha1,
 +					 abbrev);
 +		printf("%s%s", i ? "," : "", abb);
 +	}
 +	abb = find_unique_abbrev(elem->sha1, abbrev);
 +	printf("..%s%s\n", abb, c_reset);
 +
 +	if (mode_differs) {
 +		deleted = !elem->mode;
 +
 +		/* We say it was added if nobody had it */
 +		added = !deleted;
 +		for (i = 0; added && i < num_parent; i++)
 +			if (elem->parent[i].status !=
 +			    DIFF_STATUS_ADDED)
 +				added = 0;
 +		if (added)
 +			printf("%snew file mode %06o",
 +			       c_meta, elem->mode);
 +		else {
 +			if (deleted)
 +				printf("%sdeleted file ", c_meta);
 +			printf("mode ");
 +			for (i = 0; i < num_parent; i++) {
 +				printf("%s%06o", i ? "," : "",
 +				       elem->parent[i].mode);
 +			}
 +			if (elem->mode)
 +				printf("..%06o", elem->mode);
 +		}
 +		printf("%s\n", c_reset);
 +	}
 +
 +	if (!show_file_header)
 +		return;
 +
 +	if (added)
 +		dump_quoted_path("--- ", "", "/dev/null",
 +				 c_meta, c_reset);
 +	else
 +		dump_quoted_path("--- ", a_prefix, elem->path,
 +				 c_meta, c_reset);
 +	if (deleted)
 +		dump_quoted_path("+++ ", "", "/dev/null",
 +				 c_meta, c_reset);
 +	else
 +		dump_quoted_path("+++ ", b_prefix, elem->path,
 +				 c_meta, c_reset);
 +}
 +
  static void show_patch_diff(struct combine_diff_path *elem, int num_parent,
- 			    int dense, struct rev_info *rev)
+ 			    int dense, int working_tree_file,
+ 			    struct rev_info *rev)
  {
  	struct diff_options *opt = &rev->diffopt;
  	unsigned long result_size, cnt, lno;
@@@ -777,18 -692,13 +778,17 @@@
  	struct sline *sline; /* survived lines */
  	int mode_differs = 0;
  	int i, show_hunks;
- 	int working_tree_file = is_null_sha1(elem->sha1);
 -	int abbrev = DIFF_OPT_TST(opt, FULL_INDEX) ? 40 : DEFAULT_ABBREV;
 -	const char *a_prefix, *b_prefix;
  	mmfile_t result_file;
 +	struct userdiff_driver *userdiff;
 +	struct userdiff_driver *textconv = NULL;
 +	int is_binary;
  
  	context = opt->context;
 -	a_prefix = opt->a_prefix ? opt->a_prefix : "a/";
 -	b_prefix = opt->b_prefix ? opt->b_prefix : "b/";
 +	userdiff = userdiff_find_by_path(elem->path);
 +	if (!userdiff)
 +		userdiff = userdiff_find_by_name("default");
 +	if (DIFF_OPT_TST(opt, ALLOW_TEXTCONV))
 +		textconv = userdiff_get_textconv(userdiff);
  
  	/* Read the result of merge first */
  	if (!working_tree_file)