Re: [PATCH v3 06/16] builtin/multi-pack-index.c: display usage on unrecognized command
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Mar 29, 2021, 20:41 UTC
- 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:
Show 12 quoted lines
> 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