On Fri, Oct 02, 2009 at 03:50:12AM -0400, Jeff King wrote:
> 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.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html