[PATCH 9/9] qemu: Unify style for qemuDomain*()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



These functions do mostly the same things, and it would be
preferrable if they did them in mostly the same ways. This
also fixes a few violations to our code style guidelines.

Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
---
 src/qemu/qemu_domain.c | 89 +++++++++++++++++++++++++-----------------
 1 file changed, 54 insertions(+), 35 deletions(-)

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index f222f78654..eb4de1e6bd 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -9877,8 +9877,12 @@ qemuDomainMachineIsQ35(const char *machine,
     if (!ARCH_IS_X86(arch))
         return false;
 
-    return (STRPREFIX(machine, "pc-q35-") ||
-            STREQ(machine, "q35"));
+    if (STREQ(machine, "q35") ||
+        STRPREFIX(machine, "pc-q35-")) {
+        return true;
+    }
+
+    return false;
 }
 
 
@@ -9889,11 +9893,15 @@ qemuDomainMachineIsI440FX(const char *machine,
     if (!ARCH_IS_X86(arch))
         return false;
 
-    return (STREQ(machine, "pc") ||
-            STRPREFIX(machine, "pc-0.") ||
-            STRPREFIX(machine, "pc-1.") ||
-            STRPREFIX(machine, "pc-i440fx-") ||
-            STRPREFIX(machine, "rhel"));
+    if (STREQ(machine, "pc") ||
+        STRPREFIX(machine, "pc-0.") ||
+        STRPREFIX(machine, "pc-1.") ||
+        STRPREFIX(machine, "pc-i440fx-") ||
+        STRPREFIX(machine, "rhel")) {
+        return true;
+    }
+
+    return false;
 }
 
 
@@ -9904,7 +9912,10 @@ qemuDomainMachineIsS390CCW(const char *machine,
     if (!ARCH_IS_S390(arch))
         return false;
 
-    return STRPREFIX(machine, "s390-ccw");
+    if (STRPREFIX(machine, "s390-ccw"))
+        return true;
+
+    return false;
 }
 
 
@@ -9916,14 +9927,16 @@ qemuDomainMachineIsARMVirt(const char *machine,
 {
     if (arch != VIR_ARCH_ARMV6L &&
         arch != VIR_ARCH_ARMV7L &&
-        arch != VIR_ARCH_AARCH64)
+        arch != VIR_ARCH_AARCH64) {
         return false;
+    }
 
-    if (STRNEQ(machine, "virt") &&
-        !STRPREFIX(machine, "virt-"))
-        return false;
+    if (STREQ(machine, "virt") ||
+        STRPREFIX(machine, "virt-")) {
+        return true;
+    }
 
-    return true;
+    return false;
 }
 
 
@@ -9934,11 +9947,12 @@ qemuDomainMachineIsRISCVVirt(const char *machine,
     if (!ARCH_IS_RISCV(arch))
         return false;
 
-    if (STRNEQ(machine, "virt") &&
-        !STRPREFIX(machine, "virt-"))
-        return false;
+    if (STREQ(machine, "virt") ||
+        STRPREFIX(machine, "virt-")) {
+        return true;
+    }
 
-    return true;
+    return false;
 }
 
 
@@ -9951,11 +9965,12 @@ qemuDomainMachineIsPSeries(const char *machine,
     if (!ARCH_IS_PPC64(arch))
         return false;
 
-    if (STRNEQ(machine, "pseries") &&
-        !STRPREFIX(machine, "pseries-"))
-        return false;
+    if (STREQ(machine, "pseries") ||
+        STRPREFIX(machine, "pseries-")) {
+        return true;
+    }
 
-    return true;
+    return false;
 }
 
 
@@ -9981,16 +9996,18 @@ qemuDomainMachineNeedsFDC(const char *machine,
     if (!ARCH_IS_X86(arch))
         return false;
 
-    if (p) {
-        if (STRPREFIX(p, "1.") ||
-            STREQ(p, "2.0") ||
-            STREQ(p, "2.1") ||
-            STREQ(p, "2.2") ||
-            STREQ(p, "2.3"))
-            return false;
-        return true;
+    if (!p)
+        return false;
+
+    if (STRPREFIX(p, "1.") ||
+        STREQ(p, "2.0") ||
+        STREQ(p, "2.1") ||
+        STREQ(p, "2.2") ||
+        STREQ(p, "2.3")) {
+        return false;
     }
-    return false;
+
+    return true;
 }
 
 
@@ -10084,19 +10101,21 @@ bool
 qemuDomainSupportsPCI(virDomainDefPtr def,
                       virQEMUCapsPtr qemuCaps)
 {
-    if ((def->os.arch != VIR_ARCH_ARMV6L) &&
-        (def->os.arch != VIR_ARCH_ARMV7L) &&
-        (def->os.arch != VIR_ARCH_AARCH64) &&
-        !ARCH_IS_RISCV(def->os.arch))
+    if (def->os.arch != VIR_ARCH_ARMV6L &&
+        def->os.arch != VIR_ARCH_ARMV7L &&
+        def->os.arch != VIR_ARCH_AARCH64 &&
+        !ARCH_IS_RISCV(def->os.arch)) {
         return true;
+    }
 
     if (STREQ(def->os.machine, "versatilepb"))
         return true;
 
     if ((qemuDomainIsARMVirt(def) ||
          qemuDomainIsRISCVVirt(def)) &&
-        virQEMUCapsGet(qemuCaps, QEMU_CAPS_OBJECT_GPEX))
+        virQEMUCapsGet(qemuCaps, QEMU_CAPS_OBJECT_GPEX)) {
         return true;
+    }
 
     return false;
 }
-- 
2.20.1

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux