Re: [PATCH v4 6/7] reftable: add code to facilitate consistency checks
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Oct 2, 2025, 11:44 UTC
- Message-ID
- <aN5lngvE39JGwGSx@pks.im>
- In-Reply-To
- <20250926-228-reftable-introduce-consistency-checks-v4-6-c96fd8551c0d@gmail.com>
On Fri, Sep 26, 2025 at 09:25:49AM +0200, Karthik Nayak wrote:
Show 6 quoted lines
> diff --git a/reftable/fsck.c b/reftable/fsck.c > new file mode 100644 > index 0000000000..26b9115b14 > --- /dev/null > +++ b/reftable/fsck.c > @@ -0,0 +1,100 @@
[snip]
Show 16 quoted lines
> +static int table_checks(struct reftable_table *table,
> + reftable_fsck_report_fn report_fn,
> + reftable_fsck_verbose_fn verbose_fn UNUSED,
> + void *cb_data)
> +{
> + table_check_fn table_check_fns[] = {
> + table_check_name,
> + NULL,
> + };
> + int err = 0;
> +
> + for (size_t i = 0; table_check_fns[i]; i++)
> + err |= table_check_fns[i](table, report_fn, cb_data);
> +
> + return err;
> +}Okay, good. We now only verify individual table names part of the stack, and don't scan the directory anymore. Furthermore, it is easy to add more tests by adding to the function array.
Patrick