From: Jonathan Tan <jonathantanmy@google.com>
Date: Thu, 04 Mar 2021 01:54:44 GMT
Subject: Re: [PATCH v2 04/15] builtin/multi-pack-index.c: split sub-commands
Message-ID: <20210304015444.1801419-1-jonathantanmy@google.com>
In-Reply-To: <YD6Ly6EIcz4PT6uB@nand.local>

> On Mon, Mar 01, 2021 at 08:06:25PM -0800, Jonathan Tan wrote:
> > > +static char const * const builtin_multi_pack_index_write_usage[] = {
> > >  #define BUILTIN_MIDX_WRITE_USAGE \
> > >  	N_("git multi-pack-index [<options>] write")
> > > +	BUILTIN_MIDX_WRITE_USAGE,
> > > +	NULL
> > > +};
> >
> > I think this way of writing is vulnerable to confusing errors if a
> > missing or extra backslash happens, so I would prefer the #define to be
> > outside the variable declaration.
> 
> Yeah, I can't say that I disagree with you. Of course, having the
> #define's outside of the declaration makes the whole thing a little more
> verbose, which isn't a huge deal.

I think it's the same verbosity - you just need to move the lines?

> But I was mirroring what Ævar was doing in the sub-thread he started at:
> 
>     https://public-inbox.org/git/20210215184118.11306-1-avarab@gmail.com/
> 
> Unless you feel strongly, I think that what we have isn't so bad here.

Yeah I don't feel that strongly about it.