tree-walk.c: remove the_repo from fill_tree_descriptor()
While at there, clean up the_repo usage in builtin/merge-tree.c a tiny bit. Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
d3b4705ab8
commit
5e57580733
@ -205,6 +205,7 @@ static void resolve(const struct traverse_info *info, struct name_entry *ours, s
|
||||
static void unresolved_directory(const struct traverse_info *info,
|
||||
struct name_entry n[3])
|
||||
{
|
||||
struct repository *r = the_repository;
|
||||
char *newbase;
|
||||
struct name_entry *p;
|
||||
struct tree_desc t[3];
|
||||
@ -220,9 +221,9 @@ static void unresolved_directory(const struct traverse_info *info,
|
||||
newbase = traverse_path(info, p);
|
||||
|
||||
#define ENTRY_OID(e) (((e)->mode && S_ISDIR((e)->mode)) ? &(e)->oid : NULL)
|
||||
buf0 = fill_tree_descriptor(t + 0, ENTRY_OID(n + 0));
|
||||
buf1 = fill_tree_descriptor(t + 1, ENTRY_OID(n + 1));
|
||||
buf2 = fill_tree_descriptor(t + 2, ENTRY_OID(n + 2));
|
||||
buf0 = fill_tree_descriptor(r, t + 0, ENTRY_OID(n + 0));
|
||||
buf1 = fill_tree_descriptor(r, t + 1, ENTRY_OID(n + 1));
|
||||
buf2 = fill_tree_descriptor(r, t + 2, ENTRY_OID(n + 2));
|
||||
#undef ENTRY_OID
|
||||
|
||||
merge_trees(t, newbase);
|
||||
@ -351,14 +352,16 @@ static void merge_trees(struct tree_desc t[3], const char *base)
|
||||
traverse_trees(&the_index, 3, t, &info);
|
||||
}
|
||||
|
||||
static void *get_tree_descriptor(struct tree_desc *desc, const char *rev)
|
||||
static void *get_tree_descriptor(struct repository *r,
|
||||
struct tree_desc *desc,
|
||||
const char *rev)
|
||||
{
|
||||
struct object_id oid;
|
||||
void *buf;
|
||||
|
||||
if (get_oid(rev, &oid))
|
||||
if (repo_get_oid(r, rev, &oid))
|
||||
die("unknown rev %s", rev);
|
||||
buf = fill_tree_descriptor(desc, &oid);
|
||||
buf = fill_tree_descriptor(r, desc, &oid);
|
||||
if (!buf)
|
||||
die("%s is not a tree", rev);
|
||||
return buf;
|
||||
@ -366,15 +369,16 @@ static void *get_tree_descriptor(struct tree_desc *desc, const char *rev)
|
||||
|
||||
int cmd_merge_tree(int argc, const char **argv, const char *prefix)
|
||||
{
|
||||
struct repository *r = the_repository;
|
||||
struct tree_desc t[3];
|
||||
void *buf1, *buf2, *buf3;
|
||||
|
||||
if (argc != 4)
|
||||
usage(merge_tree_usage);
|
||||
|
||||
buf1 = get_tree_descriptor(t+0, argv[1]);
|
||||
buf2 = get_tree_descriptor(t+1, argv[2]);
|
||||
buf3 = get_tree_descriptor(t+2, argv[3]);
|
||||
buf1 = get_tree_descriptor(r, t+0, argv[1]);
|
||||
buf2 = get_tree_descriptor(r, t+1, argv[2]);
|
||||
buf3 = get_tree_descriptor(r, t+2, argv[3]);
|
||||
merge_trees(t, "");
|
||||
free(buf1);
|
||||
free(buf2);
|
||||
|
Reference in New Issue
Block a user