Surprise surprise. This is just rename and introduce resolve_ref_recursively() to the public. Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> --- refs.c | 9 +++++---- refs.h | 6 ++++++ refs/files-backend.c | 2 +- refs/refs-internal.h | 5 ----- 4 files changed, 12 insertions(+), 10 deletions(-) diff --git a/refs.c b/refs.c index ba4d9420c..e0e191107 100644 --- a/refs.c +++ b/refs.c @@ -1226,7 +1226,7 @@ int for_each_rawref(each_ref_fn fn, void *cb_data) } /* This function needs to return a meaningful errno on failure */ -const char *resolve_ref_recursively(struct ref_store *refs, +const char *refs_resolve_ref_unsafe(struct ref_store *refs, const char *refname, int resolve_flags, unsigned char *sha1, int *flags) @@ -1313,8 +1313,9 @@ int refs_init_db(struct strbuf *err) const char *resolve_ref_unsafe(const char *refname, int resolve_flags, unsigned char *sha1, int *flags) { - return resolve_ref_recursively(get_main_ref_store(), refname, - resolve_flags, sha1, flags); + return refs_resolve_ref_unsafe(get_main_ref_store(), + refname, resolve_flags, + sha1, flags); } int resolve_gitlink_ref(const char *submodule, const char *refname, @@ -1343,7 +1344,7 @@ int resolve_gitlink_ref(const char *submodule, const char *refname, if (!refs) return -1; - if (!resolve_ref_recursively(refs, refname, 0, sha1, &flags) || + if (!refs_resolve_ref_unsafe(refs, refname, 0, sha1, &flags) || is_null_sha1(sha1)) return -1; return 0; diff --git a/refs.h b/refs.h index 464cc384a..10c2cfc00 100644 --- a/refs.h +++ b/refs.h @@ -568,4 +568,10 @@ struct ref_store *get_submodule_ref_store(const char *submodule); struct worktree; struct ref_store *get_worktree_ref_store(const struct worktree *wt); +const char *refs_resolve_ref_unsafe(struct ref_store *refs, + const char *refname, + int resolve_flags, + unsigned char *sha1, + int *flags); + #endif /* REFS_H */ diff --git a/refs/files-backend.c b/refs/files-backend.c index 685ea5c14..f3be620ab 100644 --- a/refs/files-backend.c +++ b/refs/files-backend.c @@ -1276,7 +1276,7 @@ static void read_loose_refs(const char *dirname, struct ref_dir *dir) create_dir_entry(refs, refname.buf, refname.len, 1)); } else { - if (!resolve_ref_recursively(&refs->base, + if (!refs_resolve_ref_unsafe(&refs->base, refname.buf, RESOLVE_REF_READING, sha1, &flag)) { diff --git a/refs/refs-internal.h b/refs/refs-internal.h index cb6882779..6b29dc3b1 100644 --- a/refs/refs-internal.h +++ b/refs/refs-internal.h @@ -634,11 +634,6 @@ struct ref_store { void base_ref_store_init(struct ref_store *refs, const struct ref_storage_be *be); -const char *resolve_ref_recursively(struct ref_store *refs, - const char *refname, - int resolve_flags, - unsigned char *sha1, int *flags); - static inline int is_per_worktree_ref(const char *refname) { return !starts_with(refname, "refs/") || -- 2.11.0.157.gd943d85