undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix twoway_merge that passed d/f conflict marker to merged_entry().

Browse Source 
When switching from one tree to another, we should not send a
marker that says "this file does not exist in the new tree -- I
am a placeholder to tell you that, and not a real blob" down to
merged_entry() as the result of the merge.
This commit is contained in:
Junio C Hamano 2007-04-02 16:29:56 -07:00
parent 2960a1d9ee
commit b8ba1535bd
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
unpack-trees.c
View File

@ -711,12 +711,18 @@ int twoway_merge(struct cache_entry **src,
struct unpack_trees_options *o) struct unpack_trees_options *o)
{ {
struct cache_entry *current = src[0]; struct cache_entry *current = src[0];
struct cache_entry *oldtree = src[1], *newtree = src[2]; struct cache_entry *oldtree = src[1];
struct cache_entry *newtree = src[2];
if (o->merge_size != 2) if (o->merge_size != 2)
return error("Cannot do a twoway merge of %d trees", return error("Cannot do a twoway merge of %d trees",
o->merge_size); o->merge_size);
if (oldtree == o->df_conflict_entry)
oldtree = NULL;
if (newtree == o->df_conflict_entry)
newtree = NULL;
if (current) { if (current) {
if ((!oldtree && !newtree) || /* 4 and 5 */ if ((!oldtree && !newtree) || /* 4 and 5 */
(!oldtree && newtree && (!oldtree && newtree &&
@ -724,7 +730,7 @@ int twoway_merge(struct cache_entry **src,
(oldtree && newtree && (oldtree && newtree &&
same(oldtree, newtree)) || /* 14 and 15 */ same(oldtree, newtree)) || /* 14 and 15 */
(oldtree && newtree && (oldtree && newtree &&
!same(oldtree, newtree) && /* 18 and 19*/ !same(oldtree, newtree) && /* 18 and 19 */
same(current, newtree))) { same(current, newtree))) {
return keep_entry(current, o); return keep_entry(current, o);
} }