> > +`PARSE_OPT_KEEP_DASHDASH`::
> > + Usually long options (`\--long-opt`) are boiled down to their
> > + short option equivalent, if available.
> > + Using this flag, long options are kept.
>
> No: PARSE_OPT_KEEP_DASHDASH keeps the "--" that usually separates options
> from files, as in
Eh, right, of course! What I wrote doesn't make any sense. ;-)
> > +`OPT__ABBREV(&int_var)`::
> > + Add `\--abbrev [<n>]`.
>
> better:
> + Add `\--abbrev[=<n>]`.
>
> since the <n> is optional.
That's right and I also expected that print_usage_with_options() would
do so, since the other variant is ambiguous, BUT:
$ ./test-parse-options -h 2>&1 | grep abbrev
--abbrev [<n>] use <n> digits to display SHA-1s
And I didn't want to change parse-options.c ;-)
Any other comments?
Regards,
Stephan
--
Stephan Beyer <s-beyer@xxxxxxx>, PGP 0x6EDDD207FCC5040F
--
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
