On 24/3/25 10:09, Daniel P. Berrangé wrote:
On Mon, Mar 24, 2025 at 12:00:06AM +0100, Philippe Mathieu-Daudé wrote:
The break in the QEMU_OPTION_machine case is mis-placed.

I think that's largely a bikeshed colouring question. If you
look at other places in the outer switch using a block in
the case, eg

    case FOO:
       {
          .....
       }

or

    case FOO: {
          .....
    }

they'll also have 'break' inside the '{}', so either this patch
should change all, or change none.

OK, none then! Thanks :)



Not a big deal, since producing the same outcome, but
suspicious, so put it in the correct place.

Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
---
  system/vl.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)



Reply via email to