diff options
author | Junio C Hamano <gitster@pobox.com> | 2023-02-14 14:15:50 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2023-02-14 14:15:50 -0800 |
commit | 7cbfd0e572560040df3381efb91e3b52b524e128 (patch) | |
tree | 07de4477baafc17bb06513d85a4bab78f38336f3 /builtin | |
parent | 259988af42caa4e17d410790d75dd939144bad24 (diff) | |
parent | 6d5e9e53aa46b97227a25c85400fd00b8237411a (diff) | |
download | git-7cbfd0e572560040df3381efb91e3b52b524e128.tar.gz |
Merge branch 'ab/bundle-wo-args' into maint-2.39
Fix to a small regression in 2.38 days.
* ab/bundle-wo-args:
bundle <cmd>: have usage_msg_opt() note the missing "<file>"
builtin/bundle.c: remove superfluous "newargc" variable
bundle: don't segfault on "git bundle <subcmd>"
Diffstat (limited to 'builtin')
-rw-r--r-- | builtin/bundle.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/builtin/bundle.c b/builtin/bundle.c index c12c09f854..acceef6200 100644 --- a/builtin/bundle.c +++ b/builtin/bundle.c @@ -55,13 +55,12 @@ static int parse_options_cmd_bundle(int argc, const char * const usagestr[], const struct option options[], char **bundle_file) { - int newargc; - newargc = parse_options(argc, argv, NULL, options, usagestr, + argc = parse_options(argc, argv, NULL, options, usagestr, PARSE_OPT_STOP_AT_NON_OPTION); - if (argc < 1) - usage_with_options(usagestr, options); + if (!argc) + usage_msg_opt(_("need a <file> argument"), usagestr, options); *bundle_file = prefix_filename(prefix, argv[0]); - return newargc; + return argc; } static int cmd_bundle_create(int argc, const char **argv, const char *prefix) { |