diff options
| author | Sam James <sam@gentoo.org> | 2022-10-20 02:18:35 +0300 |
|---|---|---|
| committer | Richard Levitte <levitte@openssl.org> | 2022-10-27 12:39:17 +0300 |
| commit | 1cfc95b25c930df4de19c428a74d35f7a7411ab5 (patch) | |
| tree | 0a53f9edf646ee5de2c914853c1e04b9ce900950 | |
| parent | 1b85fc806a7eeef14ee269761bf051c3abdca073 (diff) | |
test: driver: fix -Wunused-but-set-variable
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>
(cherry picked from commit 6a94c5849ea7d1f08d4fcaa9a6fc0a947e19da66)
Signed-off-by: Sam James <sam@gentoo.org>
(Merged from https://github.com/openssl/openssl/pull/19500)
| -rw-r--r-- | test/testutil/driver.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/test/testutil/driver.c b/test/testutil/driver.c index 8568a51fd7..efb5785d7d 100644 --- a/test/testutil/driver.c +++ b/test/testutil/driver.c @@ -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; |