2023-09-04 Bruno Haible <bruno@clisp.org>
+ argv-iter tests: Avoid gcc -Wanalyzer-out-of-bounds warning.
+ * tests/test-argv-iter.c (main): Simplify logic.
+
crypto/{gc-rijndael,rijndael} tests: Fix some gcc -Wshadow warnings.
* tests/test-gc-rijndael.c (main): Rename local variable 'i' to 'round'.
* tests/test-rijndael.c (main): Likewise.
ASSERT ((i == n_found) == (ai_err == AI_ERR_EOF));
ASSERT ((s == NULL) ^ (ai_err == AI_ERR_OK));
ASSERT (ai_err == AI_ERR_OK || ai_err == AI_ERR_EOF);
- if (ai_err == AI_ERR_OK)
- ++n_found;
if (ai_err == AI_ERR_EOF)
break;
/* In stream mode, the strings are equal, but
in argv mode the actual pointers are equal. */
ASSERT (use_stream
- ? STREQ (s, av[i][n_found - 1])
- : s == av[i][n_found - 1]);
+ ? STREQ (s, av[i][n_found])
+ : s == av[i][n_found]);
+ ++n_found;
}
ASSERT (argv_iter_n_args (ai) == i);
argv_iter_free (ai);