On Wed, Jul 19, 2017 at 5:21 PM, Jonathan Tan <jonathantanmy@xxxxxxxxxx> wrote: > The "used" field in struct object is only used by builtin/fsck. Remove > that field and modify builtin/fsck to use a flag instead. The patch looks good to me (I would even claim this could go in as an independent cleanup, not tied to the RFCish nature of the later patches), though I have a question: How did you select 0x0008 for USED, i.e. does it collide with other flags (theoretically?), and if so how do we make sure to avoid the collusion in the future?
