Re: [PATCH 5/9] builtin/repack.c: move `.idx` files into place last

From: Taylor Blau <me@ttaylorr.com>
Date: Sep 9, 2021, 21:08 UTC
Message-ID: <YTp365kOEpRenuml@nand.local>
In-Reply-To: <xmqqa6klzghf.fsf@gitster.g>

On Thu, Sep 09, 2021 at 12:38:52PM -0700, Junio C Hamano wrote:

Show 19 quoted lines

> Taylor Blau <me@ttaylorr.com> 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