diff options
| author | Paul-Sebastian Ungureanu <ungureanupaulsebastian@gmail.com> | 2018-03-22 18:43:51 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-03-22 19:10:08 (GMT) |
| commit | 3bb0923f06c55ea44569f547cefa9e1a59069ff2 (patch) | |
| tree | 9667154be2d13f516298897c4b132191db38b6f4 /builtin/shortlog.c | |
| parent | 38e79b1fdab9244e1727d0698afcf3bb8956c0a4 (diff) | |
| download | git-3bb0923f06c55ea44569f547cefa9e1a59069ff2.zip git-3bb0923f06c55ea44569f547cefa9e1a59069ff2.tar.gz git-3bb0923f06c55ea44569f547cefa9e1a59069ff2.tar.bz2 | |
parse-options: do not show usage upon invalid option value
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 e29875b..be4df6a 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; |