From af18078d8057203b1ed26ac5534d233aabb36886 Mon Sep 17 00:00:00 2001 From: Peter Maydell Date: Sat, 2 Feb 2013 17:17:54 +0000 Subject: [PATCH] disas/i386.c: Add explicit braces round empty for-loop body Add explicit braces round an empty for-loop body; this fits QEMU style and is easier to read than an inconspicuous semicolon at the end of the line. It also silences a clang warning: disas/i386.c:4723:49: warning: for loop has empty body [-Wempty-body] for (i = 0; tmp[i] == '0' && tmp[i + 1]; i++); ^ disas/i386.c:4723:49: note: put the semicolon on a separate line to silence this warning [-Wempty-body] Signed-off-by: Peter Maydell Signed-off-by: Blue Swirl --- disas/i386.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/disas/i386.c b/disas/i386.c index 3b006b13e..dbecf1ffb 100644 --- a/disas/i386.c +++ b/disas/i386.c @@ -4720,7 +4720,8 @@ print_operand_value (char *buf, size_t bufsize, int hex, bfd_vma disp) buf[0] = '0'; buf[1] = 'x'; snprintf_vma (tmp, sizeof(tmp), disp); - for (i = 0; tmp[i] == '0' && tmp[i + 1]; i++); + for (i = 0; tmp[i] == '0' && tmp[i + 1]; i++) { + } pstrcpy (buf + 2, bufsize - 2, tmp + i); } else -- 2.39.2