Re: [PATCH 08/10] builtin/fsck: move bitmap verification into the packed source
- From
Karthik Nayak <karthik.188@gmail.com>
- Date
- Aug 31, 2026, 09:26 UTC
- Message-ID
- <CAOLa=ZSVe2okfJZL-xt1PkADF67z8JZrtcFce+mecsoMaseKuA@mail.gmail.com>
- In-Reply-To
- <apUYiv36xvWe-oj7@pks.im>
Patrick Steinhardt <ps@pks.im> writes:
Show 24 quoted lines
> On Thu, Aug 27, 2026 at 06:54:51AM -0400, Karthik Nayak wrote: >> Patrick Steinhardt <ps@pks.im> writes: >> >> > The checks for bitmaps live in `verify_bitmap_files()`, which is called >> > by "builtin/fsck.c". These checks are obviously specific to the "packed" >> > backend. >> > >> > Move the logic into `odb_source_packed_fsck()`. As in preceding commits, >> > this means that we now properly honor both "--connectivity-only" and >> > "--no-full". Furthermore, we drop the dedicated `ERROR_BITMAP` bit and >> > instead use the generic `ERROR_OBJECT` bit. >> > >> > Note that this change also adapts `verify_bitmap_files()` to be >> > focussed on a single "packed" source instead of verifying bitmaps from >> >> nit: s/focussed/focused > > You can actually use both spellings [1], where "focussed" is more > commonly used in the UK. Anyway, I'll change this to help our American > friends out there. > > Patrick > > [1]: https://en.wiktionary.org/wiki/focussed
I usually follow the UK spellings, I didn't know the focussed <> focused variability.