From ec03c757070adf4b460eaa56da284f2370c69cd4 Mon Sep 17 00:00:00 2001 From: Patrick Steinhardt Date: Fri, 9 Jan 2026 13:39:31 +0100 Subject: [PATCH] refs/files: move fsck functions into global scope When performing consistency checks we pass the functions that perform the verification down the calling stack. This is somewhat unnecessary though, as the set of functions doesn't ever change. Simplify the code by moving the array into global scope and remove the parameter. Signed-off-by: Patrick Steinhardt Signed-off-by: Junio C Hamano --- refs/files-backend.c | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/refs/files-backend.c b/refs/files-backend.c index 297739f203..feba3ee58b 100644 --- a/refs/files-backend.c +++ b/refs/files-backend.c @@ -3890,11 +3890,16 @@ cleanup: return ret; } +static const files_fsck_refs_fn fsck_refs_fn[]= { + files_fsck_refs_name, + files_fsck_refs_content, + NULL, +}; + static int files_fsck_refs_dir(struct ref_store *ref_store, struct fsck_options *o, const char *refs_check_dir, - struct worktree *wt, - files_fsck_refs_fn *fsck_refs_fn) + struct worktree *wt) { struct strbuf refname = STRBUF_INIT; struct strbuf sb = STRBUF_INIT; @@ -3955,13 +3960,7 @@ static int files_fsck_refs(struct ref_store *ref_store, struct fsck_options *o, struct worktree *wt) { - files_fsck_refs_fn fsck_refs_fn[]= { - files_fsck_refs_name, - files_fsck_refs_content, - NULL, - }; - - return files_fsck_refs_dir(ref_store, o, "refs", wt, fsck_refs_fn); + return files_fsck_refs_dir(ref_store, o, "refs", wt); } static int files_fsck(struct ref_store *ref_store,