path: root/builtin/merge-index.c
diff options
authorJunio C Hamano <>2012-12-06 23:08:01 (GMT)
committerJunio C Hamano <>2012-12-10 07:05:27 (GMT)
commitfa2364ec3457cc107e47d617779f1ffa23647ca4 (patch)
tree0c40c2a03a8f65d5abf5b14c0b8a167ef7288b5b /builtin/merge-index.c
parentfb4c62235fee8008d99ef55c4adcb1f7ea9508a3 (diff)
Which merge_file() function do you mean?
There are two different static functions and one global function, all of them called "merge_file()", with different signatures and purposes. Rename them all to reduce confusion in "git grep" output: * Rename the static one in merge-index to "merge_one_path(const char *path)" as that function is about asking an external command to resolve conflicts in one path. * Rename the global one in merge-file.c that is only used by merge-tree to "merge_blobs()", as the function takes three blobs and returns the merged result only in-core, without doing anything to the filesystem. * Rename the one in merge-recursive to "merge_one_file()", just to be fair. Also rename merge-file.[ch] to merge-blobs.[ch]. Signed-off-by: Junio C Hamano <>
Diffstat (limited to 'builtin/merge-index.c')
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/merge-index.c b/builtin/merge-index.c
index 2338832..be5e514 100644
--- a/builtin/merge-index.c
+++ b/builtin/merge-index.c
@@ -42,7 +42,7 @@ static int merge_entry(int pos, const char *path)
return found;
-static void merge_file(const char *path)
+static void merge_one_path(const char *path)
int pos = cache_name_pos(path, strlen(path));
@@ -102,7 +102,7 @@ int cmd_merge_index(int argc, const char **argv, const char *prefix)
die("git merge-index: unknown option %s", arg);
- merge_file(arg);
+ merge_one_path(arg);
if (err && !quiet)
die("merge program failed");