On Thu, Sep 09, 2021 at 12:38:52PM -0700, Junio C Hamano wrote:
> Taylor Blau <me@xxxxxxxxxxxx> writes:
> > diff --git a/builtin/repack.c b/builtin/repack.c
> > index 5f9bc74adc..c3e4771609 100644
> > --- a/builtin/repack.c
> > +++ b/builtin/repack.c
> > @@ -208,10 +208,10 @@ static struct {
> > unsigned optional:1;
> > } exts[] = {
> > {".pack"},
> > - {".idx"},
> > {".rev", 1},
> > {".bitmap", 1},
> > {".promisor", 1},
> > + {".idx"},
> > };
>
> And the .idx entry MUST stay the last. I wonder if dropping the
> trailing comma and replace it with a "/* must be at the end */"
> comment would be an effective way to ensure that.
Heh, Ævar even wrote that up, and I responded that I did not think it
helped much. Again, I'm happy to add it back if anybody feels strongly.
Thanks,
Taylor
