diff options
| author | Paul-Sebastian Ungureanu <ungureanupaulsebastian@gmail.com> | 2018-03-22 20:43:51 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-03-22 12:10:08 -0700 |
| commit | 3bb0923f06c55ea44569f547cefa9e1a59069ff2 (patch) | |
| tree | 9667154be2d13f516298897c4b132191db38b6f4 /builtin/shortlog.c | |
| parent | 38e79b1fdab9244e1727d0698afcf3bb8956c0a4 (diff) | |
| download | git-3bb0923f06c55ea44569f547cefa9e1a59069ff2.tar.gz | |
parse-options: do not show usage upon invalid option valueps/contains-id-error-message
Usually, the usage should be shown only if the user does not know what
options are available. If the user specifies an invalid value, the user
is already aware of the available options. In this case, there is no
point in displaying the usage anymore.
This patch applies to "git tag --contains", "git branch --contains",
"git branch --points-at", "git for-each-ref --contains" and many more.
Signed-off-by: Paul-Sebastian Ungureanu <ungureanupaulsebastian@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/shortlog.c')
| -rw-r--r-- | builtin/shortlog.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/builtin/shortlog.c b/builtin/shortlog.c index e29875b843..be4df6a037 100644 --- a/builtin/shortlog.c +++ b/builtin/shortlog.c @@ -283,6 +283,7 @@ int cmd_shortlog(int argc, const char **argv, const char *prefix) for (;;) { switch (parse_options_step(&ctx, options, shortlog_usage)) { case PARSE_OPT_HELP: + case PARSE_OPT_ERROR: exit(129); case PARSE_OPT_DONE: goto parse_done; |