summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAmit Shah <amit.shah@redhat.com>2011-07-20 13:14:12 +0530
committerAmit Shah <amit.shah@redhat.com>2011-07-26 11:21:13 +0530
commitf1ee0a0ebda763f39f8b8aa289fd771133c9971b (patch)
tree1eeae82b19bd7b10883244240fb2110ea4c2f4c6
parentballoon: Add braces around if statements (diff)
downloadqemu-kvm-f1ee0a0ebda763f39f8b8aa289fd771133c9971b.tar.gz
qemu-kvm-f1ee0a0ebda763f39f8b8aa289fd771133c9971b.tar.bz2
qemu-kvm-f1ee0a0ebda763f39f8b8aa289fd771133c9971b.zip
balloon: Simplify code flow
Replace: if (foo) { ... } else { return 0; } by if (!foo) { return 0; } ... Signed-off-by: Amit Shah <amit.shah@redhat.com> Reviewed-by: Markus Armbruster <armbru@redhat.com>
-rw-r--r--balloon.c16
1 files changed, 7 insertions, 9 deletions
diff --git a/balloon.c b/balloon.c
index 86f629e77..d40be39b3 100644
--- a/balloon.c
+++ b/balloon.c
@@ -42,23 +42,21 @@ void qemu_add_balloon_handler(QEMUBalloonEvent *func, void *opaque)
static int qemu_balloon(ram_addr_t target, MonitorCompletion cb, void *opaque)
{
- if (balloon_event_fn) {
- trace_balloon_event(balloon_opaque, target);
- balloon_event_fn(balloon_opaque, target, cb, opaque);
- return 1;
- } else {
+ if (!balloon_event_fn) {
return 0;
}
+ trace_balloon_event(balloon_opaque, target);
+ balloon_event_fn(balloon_opaque, target, cb, opaque);
+ return 1;
}
static int qemu_balloon_status(MonitorCompletion cb, void *opaque)
{
- if (balloon_event_fn) {
- balloon_event_fn(balloon_opaque, 0, cb, opaque);
- return 1;
- } else {
+ if (!balloon_event_fn) {
return 0;
}
+ balloon_event_fn(balloon_opaque, 0, cb, opaque);
+ return 1;
}
static void print_balloon_stat(const char *key, QObject *obj, void *opaque)