[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-trivial] [PATCH] monitor: Fix warning from clang
From: |
Stefan Weil |
Subject: |
[Qemu-trivial] [PATCH] monitor: Fix warning from clang |
Date: |
Fri, 17 Aug 2012 15:34:04 +0200 |
ccc-analyzer reports these warnings:
monitor.c:3532:21: warning: Division by zero
val %= val2;
^
monitor.c:3530:21: warning: Division by zero
val /= val2;
^
Rewriting the code fixes this (and also a style issue).
Signed-off-by: Stefan Weil <address@hidden>
---
monitor.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/monitor.c b/monitor.c
index 0c34934..0ea2c14 100644
--- a/monitor.c
+++ b/monitor.c
@@ -3524,12 +3524,13 @@ static int64_t expr_prod(Monitor *mon)
break;
case '/':
case '%':
- if (val2 == 0)
+ if (val2 == 0) {
expr_error(mon, "division by zero");
- if (op == '/')
+ } else if (op == '/') {
val /= val2;
- else
+ } else {
val %= val2;
+ }
break;
}
}
--
1.7.10
- [Qemu-trivial] [PATCH] monitor: Fix warning from clang,
Stefan Weil <=
- Re: [Qemu-trivial] [PATCH] monitor: Fix warning from clang, Luiz Capitulino, 2012/08/17
- Re: [Qemu-trivial] [Qemu-devel] [PATCH] monitor: Fix warning from clang, Markus Armbruster, 2012/08/17
- Re: [Qemu-trivial] [Qemu-devel] [PATCH] monitor: Fix warning from clang, Luiz Capitulino, 2012/08/17
- Re: [Qemu-trivial] [Qemu-devel] [PATCH] monitor: Fix warning from clang, Markus Armbruster, 2012/08/17
- Re: [Qemu-trivial] [Qemu-devel] [PATCH] monitor: Fix warning from clang, Luiz Capitulino, 2012/08/17
- Re: [Qemu-trivial] [Qemu-devel] [PATCH] monitor: Fix warning from clang, Stefan Weil, 2012/08/18
- Re: [Qemu-trivial] [Qemu-devel] [PATCH] monitor: Fix warning from clang, Markus Armbruster, 2012/08/20
- Re: [Qemu-trivial] [Qemu-devel] [PATCH] monitor: Fix warning from clang, Stefan Hajnoczi, 2012/08/24