[PATCH 13/17] refs: replace `refs_for_each_rawref_in()`
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Feb 20, 2026, 08:24 UTC
- Message-ID
- <20260220-pks-refs-for-each-unification-v1-13-17170bd99de1@pks.im>
- In-Reply-To
- <20260220-pks-refs-for-each-unification-v1-0-17170bd99de1@pks.im>
Replace calls to `refs_for_each_rawref_in()` with the newly introduced `refs_for_each_ref_ext()` function.
Signed-off-by: Patrick Steinhardt <ps@pks.im> --- builtin/remote.c | 8 ++++++-- refs.c | 10 ---------- refs.h | 4 ---- 3 files changed, 6 insertions(+), 16 deletions(-)
diff --git a/builtin/remote.c b/builtin/remote.c index ace390c671..0fddaa1773 100644 --- a/builtin/remote.c +++ b/builtin/remote.c @@ -912,6 +912,9 @@ static int mv(int argc, const char **argv, const char *prefix, old_remote_context.buf); if (refspecs_need_update) { + struct refs_for_each_ref_options opts = { + .flags = REFS_FOR_EACH_INCLUDE_BROKEN, + }; rename.transaction = ref_store_transaction_begin(get_main_ref_store(the_repository), 0, &err); if (!rename.transaction) @@ -923,9 +926,10 @@ static int mv(int argc, const char **argv, const char *prefix, strbuf_reset(&buf); strbuf_addf(&buf, "refs/remotes/%s/", rename.old_name); + opts.prefix = buf.buf; - result = refs_for_each_rawref_in(get_main_ref_store(the_repository), buf.buf, - rename_one_ref, &rename); + result = refs_for_each_ref_ext(get_main_ref_store(the_repository), + rename_one_ref, &rename, &opts); if (result < 0) die(_("queueing remote ref renames failed: %s"), rename.err->buf); diff --git a/refs.c b/refs.c index 5739804870..454c9dd017 100644 --- a/refs.c +++ b/refs.c @@ -1972,16 +1972,6 @@ int refs_for_each_namespaced_ref(struct ref_store *refs, return refs_for_each_ref_ext(refs, cb, cb_data, &opts); } -int refs_for_each_rawref_in(struct ref_store *refs, const char *prefix, - refs_for_each_cb cb, void *cb_data) -{ - struct refs_for_each_ref_options opts = { - .prefix = prefix, - .flags = REFS_FOR_EACH_INCLUDE_BROKEN, - }; - return refs_for_each_ref_ext(refs, cb, cb_data, &opts); -} - static int qsort_strcmp(const void *va, const void *vb) { const char *a = *(const char **)va; diff --git a/refs.h b/refs.h index 7a0c9c3693..2915ff081a 100644 --- a/refs.h +++ b/refs.h @@ -541,10 +541,6 @@ int refs_for_each_namespaced_ref(struct ref_store *refs, const char **exclude_patterns, refs_for_each_cb fn, void *cb_data); -/* can be used to learn about broken ref and symref */ -int refs_for_each_rawref_in(struct ref_store *refs, const char *prefix, - refs_for_each_cb fn, void *cb_data); - /* * Normalizes partial refs to their fully qualified form. * Will prepend <prefix> to the <pattern> if it doesn't start with 'refs/'.
-- 2.53.0.414.gf7e9f6c205.dirty