From: Taylor Blau <me@ttaylorr.com>
Date: Mon, 29 Mar 2021 20:41:19 GMT
Subject: Re: [PATCH v3 06/16] builtin/multi-pack-index.c: display usage on unrecognized command
Message-ID: <YGI7b1xTYOQeyEy3@nand.local>
In-Reply-To: <YGG9GogdjlijGxSF@coredump.intra.peff.net>

On Mon, Mar 29, 2021 at 07:42:18AM -0400, Jeff King wrote:
> I.e., I'd suggest changing this:
>
> >  	else
> > -		die(_("unrecognized subcommand: %s"), argv[0]);
> > +usage:
> > +		usage_with_options(builtin_multi_pack_index_usage,
> > +				   builtin_multi_pack_index_options);
>
> to:
>
>   error(_("unrecognized subcommand: %s"), argv[0]);
>   usage_with_options(...);

Thanks, that's a helpful suggestion (and pretty easy to change, too).

Thanks,
Taylor