diff options
| author | Stefan Weil <sw@weilnetz.de> | 2013-05-24 12:19:25 +0200 |
|---|---|---|
| committer | Michael Tokarev <mjt@tls.msk.ru> | 2013-06-01 14:25:38 +0400 |
| commit | 6681fca3fc5f44eff68a0c0b2e62694f8579ce28 (patch) | |
| tree | 889f7dedd18f92ac932f85fbff205691e7c8c7a7 /target-s390x/mem_helper.c | |
| parent | a3605bf696a30e793d375b6c4ff1fd88ec546010 (diff) | |
| download | qemu-6681fca3fc5f44eff68a0c0b2e62694f8579ce28.zip qemu-6681fca3fc5f44eff68a0c0b2e62694f8579ce28.tar.gz qemu-6681fca3fc5f44eff68a0c0b2e62694f8579ce28.tar.bz2 | |
Remove unnecessary break statements
Fix these warnings from cppcheck:
hw/display/cirrus_vga.c:2603:
hw/sd/sd.c:348:
hw/timer/exynos4210_mct.c:1033:
target-arm/translate.c:9886:
target-s390x/mem_helper.c:518:
target-unicore32/translate.c:1936:
style: Consecutive return, break, continue, goto or throw statements are unnecessary.
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Andreas Färber <afaerber@suse.de>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Diffstat (limited to 'target-s390x/mem_helper.c')
| -rw-r--r-- | target-s390x/mem_helper.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/target-s390x/mem_helper.c b/target-s390x/mem_helper.c index 02bc432..1422ae9 100644 --- a/target-s390x/mem_helper.c +++ b/target-s390x/mem_helper.c @@ -515,7 +515,6 @@ uint32_t HELPER(ex)(CPUS390XState *env, uint32_t cc, uint64_t v1, break; default: goto abort; - break; } } else if ((insn & 0xff00) == 0x0a00) { /* supervisor call */ |