diff options
| author | Patrick Steinhardt <ps@pks.im> | 2026-01-12 10:02:51 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2026-01-12 06:55:40 -0800 |
| commit | e615643d2ecf2d4e44b0ca29cf949a5212618245 (patch) | |
| tree | e9d13d2a13cddebbee3b6c54f8f75e9785f86099 | |
| parent | df971a7c42fa5506cb7e845ac175104093c35e8d (diff) | |
| download | git-e615643d2ecf2d4e44b0ca29cf949a5212618245.tar.gz git-e615643d2ecf2d4e44b0ca29cf949a5212618245.zip | |
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 <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | refs/files-backend.c | 17 |
1 files 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, |