[PATCH 04/24] domaincapstest: Restructure code to avoid disabling of warnings

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

 



Construct the capsName/emulator strings as initialization of variable
definition and move definition of 'struct testData' above the code.

This means that 'name' field will be initialized later.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 tests/domaincapstest.c | 32 +++++++++++++-------------------
 1 file changed, 13 insertions(+), 19 deletions(-)

diff --git a/tests/domaincapstest.c b/tests/domaincapstest.c
index 2120e7efd5..90693aac68 100644
--- a/tests/domaincapstest.c
+++ b/tests/domaincapstest.c
@@ -250,11 +250,20 @@ doTestQemuInternal(const char *version,
                    void *opaque)
 {
     g_autofree char *name = NULL;
-    g_autofree char *capsName = NULL;
-    g_autofree char *emulator = NULL;
+    g_autofree char *capsName = g_strdup_printf("caps_%s", version);
+    g_autofree char *emulator = g_strdup_printf("/usr/bin/qemu-system-%s", arch);
     const char *typestr = NULL;
     g_autofree char *mach = NULL;
     int rc;
+    struct testData data = {
+        .emulator = emulator,
+        .machine = machine,
+        .arch = arch,
+        .type = type,
+        .capsType = CAPS_QEMU,
+        .capsName = capsName,
+        .capsOpaque = opaque,
+    };

     switch ((unsigned int) type) {
     case VIR_DOMAIN_VIRT_QEMU:
@@ -275,23 +284,8 @@ doTestQemuInternal(const char *version,
     else
         mach = g_strdup("");

-    name = g_strdup_printf("qemu_%s%s%s.%s",
-                           version, typestr, mach, arch);
-    capsName = g_strdup_printf("caps_%s", version);
-    emulator = g_strdup_printf("/usr/bin/qemu-system-%s", arch);
-
-    VIR_WARNINGS_NO_DECLARATION_AFTER_STATEMENT
-    struct testData data = {
-        .name = name,
-        .emulator = emulator,
-        .machine = machine,
-        .arch = arch,
-        .type = type,
-        .capsType = CAPS_QEMU,
-        .capsName = capsName,
-        .capsOpaque = opaque,
-    };
-    VIR_WARNINGS_RESET
+    data.name = name = g_strdup_printf("qemu_%s%s%s.%s",
+                                       version, typestr, mach, arch);

     if (STRPREFIX(version, "3.") ||
         STRPREFIX(version, "4.") ||
-- 
2.39.2




[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