diff options
| author | René Scharfe <l.s.r@web.de> | 2018-08-02 19:18:14 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-08-03 15:36:20 (GMT) |
| commit | 5f0df44cd79a953c287ccb598896ea7aaa2cc9e3 (patch) | |
| tree | d438018d4b5dec908b60bd46913e36a8e6304ade /builtin/show-branch.c | |
| parent | b8ade4c5766774207e05c8f92e262f8f6835bd3d (diff) | |
| download | git-5f0df44cd79a953c287ccb598896ea7aaa2cc9e3.zip git-5f0df44cd79a953c287ccb598896ea7aaa2cc9e3.tar.gz git-5f0df44cd79a953c287ccb598896ea7aaa2cc9e3.tar.bz2 | |
parse-options: automatically infer PARSE_OPT_LITERAL_ARGHELP
Parseopt wraps argument help strings in a pair of angular brackets by
default, to tell users that they need to replace it with an actual
value. This is useful in most cases, because most option arguments
are indeed single values of a certain type. The option
PARSE_OPT_LITERAL_ARGHELP needs to be used in option definitions with
arguments that have multiple parts or are literal strings.
Stop adding these angular brackets if special characters are present,
as they indicate that we don't deal with a simple placeholder. This
simplifies the code a bit and makes defining special options slightly
easier.
Remove the flag PARSE_OPT_LITERAL_ARGHELP in the cases where the new
and more cautious handling suffices.
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/show-branch.c')
| -rw-r--r-- | builtin/show-branch.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/show-branch.c b/builtin/show-branch.c index e8a4aa4..371b8c8 100644 --- a/builtin/show-branch.c +++ b/builtin/show-branch.c @@ -655,7 +655,7 @@ int cmd_show_branch(int ac, const char **av, const char *prefix) { OPTION_CALLBACK, 'g', "reflog", &reflog_base, N_("<n>[,<base>]"), N_("show <n> most recent ref-log entries starting at " "base"), - PARSE_OPT_OPTARG | PARSE_OPT_LITERAL_ARGHELP, + PARSE_OPT_OPTARG, parse_reflog_param }, OPT_END() }; |