From e778ecbcee3a4a14e46c06d66328fdfc0af8780d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=86var=20Arnfj=C3=B6r=C3=B0=20Bjarmason?= Date: Tue, 27 Dec 2022 19:39:09 +0100 Subject: builtin/bundle.c: remove superfluous "newargc" variable MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit As noted in 891cb09db6c (bundle: don't segfault on "git bundle ", 2022-12-20) the "newargc" in this function is redundant to using our own "argc". Let's refactor the function to avoid needlessly introducing another variable. Signed-off-by: Ævar Arnfjörð Bjarmason Signed-off-by: Junio C Hamano --- builtin/bundle.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'builtin') diff --git a/builtin/bundle.c b/builtin/bundle.c index 13309f2ea9..34b4aa611b 100644 --- a/builtin/bundle.c +++ b/builtin/bundle.c @@ -45,13 +45,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 (!newargc) + if (!argc) usage_with_options(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) { -- cgit v1.2.3