[PATCH v2 01/10] tests: glib-ify vircaps2xmltest

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

 



Ideally, turning pointers into g_auto* would be done in one step
and dropping cleanup label and unused @ret variable in second
step, but since this is a test we don't care that much, do we?

Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
Reviewed-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
---
 tests/vircaps2xmltest.c | 31 +++++++++++--------------------
 1 file changed, 11 insertions(+), 20 deletions(-)

diff --git a/tests/vircaps2xmltest.c b/tests/vircaps2xmltest.c
index 5b1b60124a..1e6c113c77 100644
--- a/tests/vircaps2xmltest.c
+++ b/tests/vircaps2xmltest.c
@@ -38,12 +38,11 @@ test_virCapabilities(const void *opaque)
 {
     struct virCapabilitiesData *data = (struct virCapabilitiesData *) opaque;
     const char *archStr = virArchToString(data->arch);
-    virCaps *caps = NULL;
-    char *capsXML = NULL;
-    char *path = NULL;
-    char *system = NULL;
-    char *resctrl = NULL;
-    int ret = -1;
+    g_autoptr(virCaps) caps = NULL;
+    g_autofree char *capsXML = NULL;
+    g_autofree char *path = NULL;
+    g_autofree char *system = NULL;
+    g_autofree char *resctrl = NULL;
 
     system = g_strdup_printf("%s/vircaps2xmldata/linux-%s/system", abs_srcdir,
                              data->filename);
@@ -56,34 +55,26 @@ test_virCapabilities(const void *opaque)
     caps = virCapabilitiesNew(data->arch, data->offlineMigrate, data->liveMigrate);
 
     if (!caps)
-        goto cleanup;
+        return -1;
 
     if (!(caps->host.numa = virCapabilitiesHostNUMANewHost()))
-        goto cleanup;
+        return -1;
 
     if (virCapabilitiesInitCaches(caps) < 0)
-        goto cleanup;
+        return -1;
 
     virFileWrapperClearPrefixes();
 
     if (!(capsXML = virCapabilitiesFormatXML(caps)))
-        goto cleanup;
+        return -1;
 
     path = g_strdup_printf("%s/vircaps2xmldata/vircaps-%s-%s.xml", abs_srcdir,
                            archStr, data->filename);
 
     if (virTestCompareToFile(capsXML, path) < 0)
-        goto cleanup;
+        return -1;
 
-    ret = 0;
-
- cleanup:
-    VIR_FREE(system);
-    VIR_FREE(resctrl);
-    VIR_FREE(path);
-    VIR_FREE(capsXML);
-    virObjectUnref(caps);
-    return ret;
+    return 0;
 }
 
 static int
-- 
2.31.1




[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