summaryrefslogtreecommitdiffstats
path: root/tests/acceptance
diff options
context:
space:
mode:
authorPeter Maydell2020-03-19 20:33:20 +0100
committerPeter Maydell2020-04-14 10:44:31 +0200
commit2e7980244585a5be6f2c1b4e14c19d7932ee2fcb (patch)
tree9b0fb86ba711d8c9b510ed631fcb310ab742223b /tests/acceptance
parentthread.h: Fix Coverity version of qemu_cond_timedwait() (diff)
downloadqemu-2e7980244585a5be6f2c1b4e14c19d7932ee2fcb.tar.gz
qemu-2e7980244585a5be6f2c1b4e14c19d7932ee2fcb.tar.xz
qemu-2e7980244585a5be6f2c1b4e14c19d7932ee2fcb.zip
thread.h: Remove trailing semicolons from Coverity qemu_mutex_lock() etc
All the Coverity-specific definitions of qemu_mutex_lock() and friends have a trailing semicolon. This works fine almost everywhere because of QEMU's mandatory-braces coding style and because most callsites are simple, but target/s390x/sigp.c has a use of qemu_mutex_trylock() as an if() statement, which makes the ';' a syntax error: "../target/s390x/sigp.c", line 461: warning #18: expected a ")" if (qemu_mutex_trylock(&qemu_sigp_mutex)) { ^ Remove the bogus semicolons from the macro definitions. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Message-id: 20200319193323.2038-4-peter.maydell@linaro.org
Diffstat (limited to 'tests/acceptance')
0 files changed, 0 insertions, 0 deletions