Skip to content

Commit e44b385

Browse files
jrngitster
authored andcommitted
merge_file(): add comment explaining behavior wrt conflict style
The merge_file() function is a helper for ‘git read-tree’, which does not respect the merge.conflictstyle option, so there is no need to worry about what ancestor_name it should pass to ll_merge(). Add a comment to this effect. Signed-off-by: Jonathan Nieder <jrnieder@mgila.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent f0531a2 commit e44b385

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

merge-file.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,12 @@ static void *three_way_filemerge(const char *path, mmfile_t *base, mmfile_t *our
3030
int merge_status;
3131
mmbuffer_t res;
3232

33+
/*
34+
* This function is only used by cmd_merge_tree, which
35+
* does not respect the merge.conflictstyle option.
36+
* There is no need to worry about a label for the
37+
* common ancestor.
38+
*/
3339
merge_status = ll_merge(&res, path, base, NULL,
3440
our, ".our", their, ".their", 0);
3541
if (merge_status < 0)

0 commit comments

Comments
 (0)