Re: [PATCH 2/2] allow mangling short options which take integer arguments
- From
Clemens Buchacher <drizzd@aon.at>
- Date
- Oct 3, 2009, 09:23 UTC
- Message-ID
- <20091003092332.GA10570@localhost>
- In-Reply-To
- <20091002075012.GB27664@coredump.intra.peff.net>
On Fri, Oct 02, 2009 at 03:50:12AM -0400, Jeff King wrote:
Show 27 quoted lines
> On Fri, Oct 02, 2009 at 09:43:17AM +0200, Clemens Buchacher wrote:
>
> > On Thu, Oct 01, 2009 at 11:55:03PM +0200, Johannes Schindelin wrote:
> >
> > > And this patch looks even more straight-forward than 1/2, _but_... what
> > > about cases where there are short options that are digits?
> >
> > Could you point me to one of those? I did not find any during my
> > non-exhaustive search. We should be able to handle them easily by adding
> > PARSE_OPT_MANY.
>
> The one that comes readily to mind is "git log -1", but that is actually
> parsed by the revision options parser, which doesn't use parseopt. But
> there are a few done by parseopt:
>
> $ git grep "OPT_.*'[0-9]'"
> archive.c: OPT__COMPR('1', &compression_level, "compress faster", 1),
> archive.c: OPT__COMPR_HIDDEN('2', &compression_level, 2),
> archive.c: OPT__COMPR_HIDDEN('3', &compression_level, 3),
> archive.c: OPT__COMPR_HIDDEN('4', &compression_level, 4),
> archive.c: OPT__COMPR_HIDDEN('5', &compression_level, 5),
> archive.c: OPT__COMPR_HIDDEN('6', &compression_level, 6),
> archive.c: OPT__COMPR_HIDDEN('7', &compression_level, 7),
> archive.c: OPT__COMPR_HIDDEN('8', &compression_level, 8),
> archive.c: OPT__COMPR('9', &compression_level, "compress better", 9),
> builtin-checkout.c: OPT_SET_INT('2', "ours", &opts.writeout_stage, "stage",
> builtin-checkout.c: OPT_SET_INT('3', "theirs", &opts.writeout_stage, "stage",Those are not affected by this patch series. They can be cuddled with other short options just like before, since they don't take arguments.