Re: [Qemu-devel] [PATCH] Remove unnecessary break statements

2013-05-24 Thread Peter Maydell
On 24 May 2013 11:19, Stefan Weil s...@weilnetz.de wrote: 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

[Qemu-devel] [PATCH] Remove unnecessary break statements

2013-05-24 Thread Stefan Weil
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.

Re: [Qemu-devel] [PATCH] Remove unnecessary break statements

2013-05-24 Thread Andreas Färber
Am 24.05.2013 12:19, schrieb Stefan Weil: 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,

Re: [Qemu-devel] [PATCH] Remove unnecessary break statements

2013-05-24 Thread Richard W.M. Jones
On Fri, May 24, 2013 at 12:19:25PM +0200, Stefan Weil wrote: 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

Re: [Qemu-devel] [PATCH] Remove unnecessary break statements

2013-05-24 Thread Stefan Weil
Am 24.05.2013 14:33, schrieb Richard W.M. Jones: On Fri, May 24, 2013 at 12:19:25PM +0200, Stefan Weil wrote: 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: