diff options
author | Amit Shah <amit.shah@redhat.com> | 2011-07-20 13:14:12 +0530 |
---|---|---|
committer | Amit Shah <amit.shah@redhat.com> | 2011-07-26 11:21:13 +0530 |
commit | f1ee0a0ebda763f39f8b8aa289fd771133c9971b (patch) | |
tree | 1eeae82b19bd7b10883244240fb2110ea4c2f4c6 | |
parent | balloon: Add braces around if statements (diff) | |
download | qemu-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.c | 16 |
1 files changed, 7 insertions, 9 deletions
@@ -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) |