]> Kevux Git Server - rit/commitdiff
refs: remove `dwim_log()`
authorPatrick Steinhardt <ps@pks.im>
Fri, 17 May 2024 08:19:14 +0000 (10:19 +0200)
committerJunio C Hamano <gitster@pobox.com>
Fri, 17 May 2024 17:33:39 +0000 (10:33 -0700)
Remove `dwim_log()` in favor of `repo_dwim_log()` so that we can get rid
of one more dependency on `the_repository`.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/reflog.c
reflog-walk.c
reflog.c
refs.c
refs.h

index b4650ce..0d2ff95 100644 (file)
@@ -378,7 +378,7 @@ static int cmd_reflog_expire(int argc, const char **argv, const char *prefix)
                char *ref;
                struct expire_reflog_policy_cb cb = { .cmd = cmd };
 
-               if (!dwim_log(argv[i], strlen(argv[i]), NULL, &ref)) {
+               if (!repo_dwim_log(the_repository, argv[i], strlen(argv[i]), NULL, &ref)) {
                        status |= error(_("%s points nowhere!"), argv[i]);
                        continue;
                }
index f11b97e..5f09552 100644 (file)
@@ -191,8 +191,8 @@ int add_reflog_for_walk(struct reflog_walk_info *info,
                reflogs = read_complete_reflog(branch);
                if (!reflogs || reflogs->nr == 0) {
                        char *b;
-                       int ret = dwim_log(branch, strlen(branch),
-                                          NULL, &b);
+                       int ret = repo_dwim_log(the_repository, branch, strlen(branch),
+                                               NULL, &b);
                        if (ret > 1)
                                free(b);
                        else if (ret == 1) {
index 8861c2d..3c80950 100644 (file)
--- a/reflog.c
+++ b/reflog.c
@@ -409,7 +409,7 @@ int reflog_delete(const char *rev, enum expire_reflog_flags flags, int verbose)
        if (!spec)
                return error(_("not a reflog: %s"), rev);
 
-       if (!dwim_log(rev, spec - rev, NULL, &ref)) {
+       if (!repo_dwim_log(the_repository, rev, spec - rev, NULL, &ref)) {
                status |= error(_("no reflog for '%s'"), rev);
                goto cleanup;
        }
diff --git a/refs.c b/refs.c
index 83cd965..43c5fef 100644 (file)
--- a/refs.c
+++ b/refs.c
@@ -775,11 +775,6 @@ int repo_dwim_log(struct repository *r, const char *str, int len,
        return logs_found;
 }
 
-int dwim_log(const char *str, int len, struct object_id *oid, char **log)
-{
-       return repo_dwim_log(the_repository, str, len, oid, log);
-}
-
 int is_per_worktree_ref(const char *refname)
 {
        return starts_with(refname, "refs/worktree/") ||
diff --git a/refs.h b/refs.h
index e906dbb..82022d3 100644 (file)
--- a/refs.h
+++ b/refs.h
@@ -164,7 +164,6 @@ int expand_ref(struct repository *r, const char *str, int len, struct object_id
 int repo_dwim_ref(struct repository *r, const char *str, int len,
                  struct object_id *oid, char **ref, int nonfatal_dangling_mark);
 int repo_dwim_log(struct repository *r, const char *str, int len, struct object_id *oid, char **ref);
-int dwim_log(const char *str, int len, struct object_id *oid, char **ref);
 
 /*
  * Retrieves the default branch name for newly-initialized repositories.