vl: don't use 'Yoda conditions'
authorGonglei <arei.gonglei@huawei.com>
Mon, 11 Aug 2014 13:00:57 +0000 (21:00 +0800)
committerMichael Tokarev <mjt@tls.msk.ru>
Fri, 15 Aug 2014 14:54:07 +0000 (18:54 +0400)
imitate nearby code about using '!value' or 'value == NULL'

Signed-off-by: Gonglei <arei.gonglei@huawei.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
vl.c

diff --git a/vl.c b/vl.c
index a8029d59d7d135c5fd1d0186eb7dab5d36d73576..97831337d63bdf64433eecd1f53d8e07cde36ffd 100644 (file)
--- a/vl.c
+++ b/vl.c
@@ -1154,7 +1154,7 @@ static int drive_init_func(QemuOpts *opts, void *opaque)
 
 static int drive_enable_snapshot(QemuOpts *opts, void *opaque)
 {
-    if (NULL == qemu_opt_get(opts, "snapshot")) {
+    if (qemu_opt_get(opts, "snapshot") == NULL) {
         qemu_opt_set(opts, "snapshot", "on");
     }
     return 0;
@@ -2506,8 +2506,9 @@ static int foreach_device_config(int type, int (*func)(const char *cmdline))
         loc_push_restore(&conf->loc);
         rc = func(conf->cmdline);
         loc_pop(&conf->loc);
-        if (0 != rc)
+        if (rc) {
             return rc;
+        }
     }
     return 0;
 }