]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
test: driver: fix -Wunused-but-set-variable
authorSam James <sam@gentoo.org>
Wed, 19 Oct 2022 23:18:35 +0000 (00:18 +0100)
committerTomas Mraz <tomas@openssl.org>
Fri, 21 Oct 2022 13:56:32 +0000 (15:56 +0200)
The value of 'num_failed_inner' isn't ever used.

Fixes this error with Clang 15:
```
test/testutil/driver.c:341:17: error: variable 'num_failed_inner' set but not used [-Werror,-Wunused-but-set-variable]
            int num_failed_inner = 0;
                ^
1 error generated.
```

Signed-off-by: Sam James <sam@gentoo.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19450)

test/testutil/driver.c

index 8568a51fd7075dded3bf1acf1e31eda3f405ffc9..efb5785d7d2a1621fdcada886136cbbb30317833 100644 (file)
@@ -338,8 +338,6 @@ int run_tests(const char *test_prog_name)
                 num_failed++;
             test_case_count++;
         } else {
-            int num_failed_inner = 0;
-
             verdict = TEST_SKIP_CODE;
             set_test_title(all_tests[i].test_case_name);
             if (all_tests[i].subtest) {
@@ -370,7 +368,6 @@ int run_tests(const char *test_prog_name)
                 v = all_tests[i].param_test_fn(j);
 
                 if (v == 0) {
-                    ++num_failed_inner;
                     verdict = 0;
                 } else if (v != TEST_SKIP_CODE && verdict != 0) {
                     verdict = 1;