aboutsummaryrefslogtreecommitdiff
path: root/tools/lib
diff options
context:
space:
mode:
authorGravatar Arnaldo Carvalho de Melo <acme@redhat.com> 2023-04-14 11:26:42 -0300
committerGravatar Arnaldo Carvalho de Melo <acme@redhat.com> 2023-04-14 21:43:39 -0300
commit17354d1528f902b3a4f66023cd24012e7999c4dc (patch)
tree0d0fcc29cb2b2f4ca3ae18cba58123fa7653a772 /tools/lib
parentperf map: Delete two variable initialisations before null pointer checks in s... (diff)
downloadlinux-17354d1528f902b3a4f66023cd24012e7999c4dc.tar.gz
linux-17354d1528f902b3a4f66023cd24012e7999c4dc.tar.bz2
linux-17354d1528f902b3a4f66023cd24012e7999c4dc.zip
perf test: Simplify for_each_test() to avoid tripping on -Werror=array-bounds
When cross building on debian to the mips 32-bit arch we get these warnings: In function '__cmd_test', inlined from 'cmd_test' at tests/builtin-test.c:561:9: tests/builtin-test.c:260:66: error: array subscript 1 is outside array bounds of 'struct test_suite *[1]' [-Werror=array-bounds] 260 | for (k = 0, t = tests[j][k]; tests[j][k]; k++, t = tests[j][k]) | ^ tests/builtin-test.c:369:9: note: in expansion of macro 'for_each_test' 369 | for_each_test(j, k, t) { | ^~~~~~~~~~~~~ tests/builtin-test.c: In function 'cmd_test': tests/builtin-test.c:36:27: note: at offset 4 into object 'arch_tests' of size 4 36 | struct test_suite *__weak arch_tests[] = { | ^~~~~~~~~~ cc1: all warnings being treated as errors Switch to using a while(!sentinel) for the second level of the 'tests' array to avoid that compiler complaint. Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/lib')
0 files changed, 0 insertions, 0 deletions