[PATCH 2/3] parse-options: add PARSE_OPT_CUSTOM_ARGH for complicated argh's
From: Stephen Boyd <hidden>
Date: 2016-06-15 22:46:47
Subsystem:
the rest · Maintainer:
Linus Torvalds
If argh is complicated, i.e. the option takes more than one argument,
don't add the brackets around argh in the usage message.
Signed-off-by: Stephen Boyd <redacted>
---
This is used in the next patch to for --reflog[=<n>[,<base>]]
Maybe someone can think of a better name?
parse-options.c | 26 +++++++++++++++++---------
parse-options.h | 4 ++++
2 files changed, 21 insertions(+), 9 deletions(-)
diff --git a/parse-options.c b/parse-options.c
index cf71bcf..cd9ab4f 100644
--- a/parse-options.c
+++ b/parse-options.c@@ -361,6 +361,20 @@ int parse_options(int argc, const char **argv, const struct option *options, return parse_options_end(&ctx); } +static int usage_argh(const struct option *opts, int pos) +{ + const char *s; + int custom = opts->flags & PARSE_OPT_CUSTOM_ARGH; + if (opts->flags & PARSE_OPT_OPTARG) + if (opts->long_name) + s = custom ? "[=%s]" : "[=<%s>]"; + else + s = custom ? "[%s]" : "[<%s>]"; + else + s = custom ? " %s" : " <%s>"; + return pos + fprintf(stderr, s, opts->argh); +} + #define USAGE_OPTS_WIDTH 24 #define USAGE_GAP 2
@@ -421,15 +435,9 @@ int usage_with_options_internal(const char * const *usagestr, break; /* FALLTHROUGH */ case OPTION_STRING: - if (opts->argh) { - if (opts->flags & PARSE_OPT_OPTARG) - if (opts->long_name) - pos += fprintf(stderr, "[=<%s>]", opts->argh); - else - pos += fprintf(stderr, "[<%s>]", opts->argh); - else - pos += fprintf(stderr, " <%s>", opts->argh); - } else { + if (opts->argh) + pos += usage_argh(opts, pos); + else { if (opts->flags & PARSE_OPT_OPTARG) if (opts->long_name) pos += fprintf(stderr, "[=...]");
diff --git a/parse-options.h b/parse-options.h
index b54eec1..671a635 100644
--- a/parse-options.h
+++ b/parse-options.h@@ -31,6 +31,7 @@ enum parse_opt_option_flags { PARSE_OPT_NONEG = 4, PARSE_OPT_HIDDEN = 8, PARSE_OPT_LASTARG_DEFAULT = 16, + PARSE_OPT_CUSTOM_ARGH = 64, }; struct option;
@@ -66,6 +67,9 @@ typedef int parse_opt_cb(const struct option *, const char *arg, int unset); * PARSE_OPT_NONEG: says that this option cannot be negated * PARSE_OPT_HIDDEN this option is skipped in the default usage, showed in * the long one. + * PARSE_OPT_CUSTOM_ARGH: says that argh shouldn't be enclosed in brackets + * (i.e. '<argh>') in the help message. + * Useful for options with multiple parameters. * * `callback`:: * pointer to the callback to use for OPTION_CALLBACK.
--
1.6.3.1.30.g55524