From nobody Sun May 19 02:27:00 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1661519464; cv=none; d=zohomail.com; s=zohoarc; b=JFXWcTdULBagdhNwlYuHgL3Vd1N87FeS5obUAfxY9zeM2ICUCMC8naEr0LIuNI1/1tnrffTFWAif0gcfDWj/2VH/NC36gzTdPubmU59TgFy6tMIb72ZGyDgq+PXl3uaaAzILq4thmloBs5FJzyZSsw4giH9rFhbU4EN8geylcJM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1661519464; h=Content-Type:Content-Transfer-Encoding:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=jfenYtxdh/QSaJMfhVNe9GUTkl+woIUx6ztisgz1Ens=; b=mU3K1z/g7EkPrnu4c/al6UhguLiyK6a3IQKEForepwH2BBrkOQjon+l4RKY3BNs4pdVmITZGuCNT8qm+Cccq9rN8Q1WOEQoGkhRa89p5dvV4DswgFarrMBxJbG3oLLV50nlVG8/6B7V3UaXrW42UkB8NvNeloyU+rFa0ojyZsoc= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1661519464368565.4572040548215; Fri, 26 Aug 2022 06:11:04 -0700 (PDT) Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-45-wPUwCXo8NIKtWSzRUWGy4g-1; Fri, 26 Aug 2022 09:10:34 -0400 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 842DD29AB417; Fri, 26 Aug 2022 13:10:32 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id 29EEB945DD; Fri, 26 Aug 2022 13:10:32 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id F15EB1946A5B; Fri, 26 Aug 2022 13:10:31 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 7C9C41946A43 for ; Fri, 26 Aug 2022 13:10:30 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 6AE6F1121315; Fri, 26 Aug 2022 13:10:30 +0000 (UTC) Received: from maggie.redhat.com (unknown [10.43.2.39]) by smtp.corp.redhat.com (Postfix) with ESMTP id D26081121314 for ; Fri, 26 Aug 2022 13:10:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1661519463; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=jfenYtxdh/QSaJMfhVNe9GUTkl+woIUx6ztisgz1Ens=; b=X8PWMMjMR3b7er1TMiW62NtaDr0nZeWqolVC8rnyAeqSMd2ure2BHzifRw1q0btC1XTmXa ED+Ie1sJkwK2sxRWeh6eRf2m8bnmD6QjHv/MOaIaxMUx1kNMCxVcFTLMwWWFBOMqTdMDE/ x9LZ4MNA9MsThuyPtUVbtogKRddM2zE= X-MC-Unique: wPUwCXo8NIKtWSzRUWGy4g-1 X-Original-To: libvir-list@listman.corp.redhat.com From: Michal Privoznik To: libvir-list@redhat.com Subject: [PATCH] lib: Drop needless ret variables Date: Fri, 26 Aug 2022 15:10:28 +0200 Message-Id: <4f1e4e0d63189cf574c31071d1d18f685c5fa6e6.1661519428.git.mprivozn@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.78 on 10.11.54.3 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 2.79 on 10.11.54.5 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1661519465263100001 Content-Type: text/plain; charset="utf-8"; x-default="true" This patch was generated using the following spatch: @@ type T; constant C; expression f; identifier ret; @@ ( -T ret =3D C; | -T ret; ) ... when !=3D ret when strict ( -ret =3D f; -return ret; +return f; | -return ret; +return C; ) Basically, if there's a variable declared at the beginning of a function and the variable is not changed until the very last moment, just before return, such variable is useless. Signed-off-by: Michal Privoznik --- src/conf/checkpoint_conf.c | 4 +--- src/conf/domain_addr.c | 6 +----- src/conf/domain_conf.c | 8 ++------ src/esx/esx_driver.c | 5 +---- src/libxl/libxl_domain.c | 4 +--- src/libxl/xen_xl.c | 5 +---- src/openvz/openvz_conf.c | 3 +-- src/qemu/qemu_driver.c | 6 ++---- src/qemu/qemu_process.c | 5 +---- src/test/test_driver.c | 18 +++++++----------- src/util/virbitmap.c | 4 +--- src/util/virerror.c | 3 +-- src/util/virxml.c | 5 +---- tests/genericxml2xmltest.c | 10 ++++------ tests/interfacexml2xmltest.c | 5 +---- tests/libxlxml2domconfigtest.c | 5 +---- tests/lxcconf2xmltest.c | 10 ++-------- tests/lxcxml2xmltest.c | 11 ++++------- tests/networkxml2conftest.c | 5 +---- tests/networkxml2firewalltest.c | 5 +---- tests/networkxml2xmltest.c | 7 ++----- tests/networkxml2xmlupdatetest.c | 9 +++------ tests/nodedevxml2xmltest.c | 5 +---- tests/nwfilterxml2firewalltest.c | 5 +---- tests/nwfilterxml2xmltest.c | 5 +---- tests/secretxml2xmltest.c | 5 +---- tests/vboxsnapshotxmltest.c | 5 +---- tests/virnwfilterbindingxml2xmltest.c | 5 +---- tests/vmx2xmltest.c | 5 +---- tests/xml2vmxtest.c | 5 +---- 30 files changed, 48 insertions(+), 135 deletions(-) diff --git a/src/conf/checkpoint_conf.c b/src/conf/checkpoint_conf.c index 338cf10d12..17aa98dc2a 100644 --- a/src/conf/checkpoint_conf.c +++ b/src/conf/checkpoint_conf.c @@ -119,7 +119,6 @@ virDomainCheckpointDefParse(xmlXPathContextPtr ctxt, void *parseOpaque, unsigned int flags) { - virDomainCheckpointDef *ret =3D NULL; size_t i; int n; g_autofree xmlNodePtr *nodes =3D NULL; @@ -177,8 +176,7 @@ virDomainCheckpointDefParse(xmlXPathContextPtr ctxt, return NULL; } =20 - ret =3D g_steal_pointer(&def); - return ret; + return g_steal_pointer(&def); } =20 static virDomainCheckpointDef * diff --git a/src/conf/domain_addr.c b/src/conf/domain_addr.c index 76f9c12ca6..f7b2acdcc9 100644 --- a/src/conf/domain_addr.c +++ b/src/conf/domain_addr.c @@ -1429,11 +1429,7 @@ virDomainCCWAddressSetCreateFromDomain(virDomainDef = *def) static virDomainVirtioSerialAddrSet * virDomainVirtioSerialAddrSetCreate(void) { - virDomainVirtioSerialAddrSet *ret =3D NULL; - - ret =3D g_new0(virDomainVirtioSerialAddrSet, 1); - - return ret; + return g_new0(virDomainVirtioSerialAddrSet, 1); } =20 static void diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 6950f7ec1d..bc74965d91 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -28728,7 +28728,6 @@ virSecurityLabelDef * virDomainDefGetSecurityLabelDef(const virDomainDef *def, const char *model) { size_t i; - virSecurityLabelDef *seclabel =3D NULL; =20 if (def =3D=3D NULL || model =3D=3D NULL) return NULL; @@ -28740,7 +28739,7 @@ virDomainDefGetSecurityLabelDef(const virDomainDef = *def, const char *model) return def->seclabels[i]; } =20 - return seclabel; + return NULL; } =20 =20 @@ -30592,7 +30591,6 @@ virDomainObjGetMessages(virDomainObj *vm, size_t i =3D 0; size_t n =3D 0; int nmsgs =3D 0; - int rv =3D -1; =20 *msgs =3D NULL; =20 @@ -30623,9 +30621,7 @@ virDomainObjGetMessages(virDomainObj *vm, if (*msgs) (*msgs)[nmsgs] =3D NULL; =20 - rv =3D nmsgs; - - return rv; + return nmsgs; } =20 bool diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c index 9dc5489411..a0540f585d 100644 --- a/src/esx/esx_driver.c +++ b/src/esx/esx_driver.c @@ -2671,7 +2671,6 @@ esxConnectDomainXMLToNative(virConnectPtr conn, const= char *nativeFormat, virVMXContext ctx; esxVMX_Data data; g_autoptr(virDomainDef) def =3D NULL; - char *vmx =3D NULL; =20 virCheckFlags(0, NULL); =20 @@ -2705,9 +2704,7 @@ esxConnectDomainXMLToNative(virConnectPtr conn, const= char *nativeFormat, ctx.datacenterPath =3D NULL; ctx.moref =3D NULL; =20 - vmx =3D virVMXFormatConfig(&ctx, priv->xmlopt, def, virtualHW_version); - - return vmx; + return virVMXFormatConfig(&ctx, priv->xmlopt, def, virtualHW_version); } =20 =20 diff --git a/src/libxl/libxl_domain.c b/src/libxl/libxl_domain.c index 52e0aa1e60..1cbb007f13 100644 --- a/src/libxl/libxl_domain.c +++ b/src/libxl/libxl_domain.c @@ -712,11 +712,9 @@ libxlDomainEventHandler(void *data, libxl_event *event) char * libxlDomainManagedSavePath(libxlDriverPrivate *driver, virDomainObj *vm) { - char *ret; g_autoptr(libxlDriverConfig) cfg =3D libxlDriverConfigGet(driver); =20 - ret =3D g_strdup_printf("%s/%s.save", cfg->saveDir, vm->def->name); - return ret; + return g_strdup_printf("%s/%s.save", cfg->saveDir, vm->def->name); } =20 /* diff --git a/src/libxl/xen_xl.c b/src/libxl/xen_xl.c index 6919325623..188cecb98e 100644 --- a/src/libxl/xen_xl.c +++ b/src/libxl/xen_xl.c @@ -1318,7 +1318,6 @@ xenFormatXLVnuma(virConfValue *list, size_t node, size_t nr_nodes) { - int ret =3D -1; size_t i; g_auto(virBuffer) buf =3D VIR_BUFFER_INITIALIZER; virConfValue *numaVnode; @@ -1365,9 +1364,7 @@ xenFormatXLVnuma(virConfValue *list, tmp->next =3D numaVnode; else list->list =3D numaVnode; - ret =3D 0; - - return ret; + return 0; } =20 static int diff --git a/src/openvz/openvz_conf.c b/src/openvz/openvz_conf.c index c94f9b8577..368e747a34 100644 --- a/src/openvz/openvz_conf.c +++ b/src/openvz/openvz_conf.c @@ -772,7 +772,6 @@ static int openvzLocateConfFileDefault(int vpsid, char **conffile, const char *ext) { char *confdir; - int ret =3D 0; =20 confdir =3D openvzLocateConfDir(); if (confdir =3D=3D NULL) @@ -781,7 +780,7 @@ openvzLocateConfFileDefault(int vpsid, char **conffile,= const char *ext) *conffile =3D g_strdup_printf("%s/%d.%s", confdir, vpsid, ext ? ext : = "conf"); =20 VIR_FREE(confdir); - return ret; + return 0; } =20 static char * diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 707f4cc1bb..3f9638bc8e 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -6029,7 +6029,6 @@ qemuDomainRestoreParams(virConnectPtr conn, { const char *path =3D NULL; const char *dxml =3D NULL; - int ret =3D -1; =20 if (virTypedParamsValidate(params, nparams, VIR_DOMAIN_SAVE_PARAM_FILE, VIR_TYPED_PARAM= _STRING, @@ -6050,9 +6049,8 @@ qemuDomainRestoreParams(virConnectPtr conn, return -1; } =20 - ret =3D qemuDomainRestoreInternal(conn, path, dxml, flags, - virDomainRestoreParamsEnsureACL); - return ret; + return qemuDomainRestoreInternal(conn, path, dxml, flags, + virDomainRestoreParamsEnsureACL); } =20 static char * diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index 32f03ff79a..f7cf56904b 100644 --- a/src/qemu/qemu_process.c +++ b/src/qemu/qemu_process.c @@ -2974,7 +2974,6 @@ qemuProcessUpdateVideoRamSize(virQEMUDriver *driver, virDomainObj *vm, int asyncJob) { - int ret =3D -1; ssize_t i; qemuDomainObjPrivate *priv =3D vm->privateData; virDomainVideoDef *video =3D NULL; @@ -3035,9 +3034,7 @@ qemuProcessUpdateVideoRamSize(virQEMUDriver *driver, qemuDomainObjExitMonitor(vm); =20 cfg =3D virQEMUDriverGetConfig(driver); - ret =3D virDomainObjSave(vm, driver->xmlopt, cfg->stateDir); - - return ret; + return virDomainObjSave(vm, driver->xmlopt, cfg->stateDir); =20 error: qemuDomainObjExitMonitor(vm); diff --git a/src/test/test_driver.c b/src/test/test_driver.c index 641a141b6a..15fe9404e4 100644 --- a/src/test/test_driver.c +++ b/src/test/test_driver.c @@ -7751,7 +7751,6 @@ testNodeDeviceCreateXML(virConnectPtr conn, static int testNodeDeviceDestroy(virNodeDevicePtr dev) { - int ret =3D 0; testDriver *driver =3D dev->conn->privateData; virNodeDeviceObj *obj =3D NULL; virNodeDeviceObj *parentobj =3D NULL; @@ -7794,7 +7793,7 @@ testNodeDeviceDestroy(virNodeDevicePtr dev) cleanup: virNodeDeviceObjEndAPI(&obj); virObjectEventStateQueue(driver->eventState, event); - return ret; + return 0; } =20 =20 @@ -9547,25 +9546,24 @@ testDomainChgIOThread(virDomainObj *vm, unsigned int flags) { virDomainDef *def; - int ret =3D -1; =20 if (!(def =3D virDomainObjGetOneDef(vm, flags))) - return ret; + return -1; =20 if (def) { switch (action) { case VIR_DOMAIN_IOTHREAD_ACTION_ADD: if (virDomainDriverAddIOThreadCheck(def, iothread_id) < 0) - return ret; + return -1; =20 if (!virDomainIOThreadIDAdd(def, iothread_id)) - return ret; + return -1; =20 break; =20 case VIR_DOMAIN_IOTHREAD_ACTION_DEL: if (virDomainDriverDelIOThreadCheck(def, iothread_id) < 0) - return ret; + return -1; =20 virDomainIOThreadIDDel(def, iothread_id); =20 @@ -9576,16 +9574,14 @@ testDomainChgIOThread(virDomainObj *vm, virReportError(VIR_ERR_INVALID_ARG, _("cannot find IOThread '%u' in iothreadids= "), iothread_id); - return ret; + return -1; } =20 break; } } =20 - ret =3D 0; - - return ret; + return 0; } =20 static int diff --git a/src/util/virbitmap.c b/src/util/virbitmap.c index 5b9204cbd7..786b9adb06 100644 --- a/src/util/virbitmap.c +++ b/src/util/virbitmap.c @@ -330,9 +330,7 @@ virBitmapFormat(virBitmap *bitmap) int start, cur, prev; =20 if (!bitmap || (cur =3D virBitmapNextSetBit(bitmap, -1)) < 0) { - char *ret; - ret =3D g_strdup(""); - return ret; + return g_strdup(""); } =20 start =3D prev =3D cur; diff --git a/src/util/virerror.c b/src/util/virerror.c index d114c0a346..e41544cdb8 100644 --- a/src/util/virerror.c +++ b/src/util/virerror.c @@ -199,7 +199,6 @@ static int virCopyError(virErrorPtr from, virErrorPtr to) { - int ret =3D 0; if (!to) return 0; virResetError(to); @@ -217,7 +216,7 @@ virCopyError(virErrorPtr from, /* * Deliberately not setting 'conn', 'dom', 'net' references */ - return ret; + return 0; } =20 =20 diff --git a/src/util/virxml.c b/src/util/virxml.c index d6e2e5dd91..1f64cf151b 100644 --- a/src/util/virxml.c +++ b/src/util/virxml.c @@ -1591,13 +1591,10 @@ virXMLValidateAgainstSchema(const char *schemafile, int virXMLValidateNodeAgainstSchema(const char *schemafile, xmlNodePtr node) { - int ret; g_autoptr(xmlDoc) copy =3D xmlNewDoc(NULL); =20 xmlDocSetRootElement(copy, xmlCopyNode(node, true)); - ret =3D virXMLValidateAgainstSchema(schemafile, copy); - - return ret; + return virXMLValidateAgainstSchema(schemafile, copy); } =20 =20 diff --git a/tests/genericxml2xmltest.c b/tests/genericxml2xmltest.c index 3501eadf55..70b69101d8 100644 --- a/tests/genericxml2xmltest.c +++ b/tests/genericxml2xmltest.c @@ -27,18 +27,16 @@ testCompareXMLToXMLHelper(const void *data) const struct testInfo *info =3D data; g_autofree char *xml_in =3D NULL; g_autofree char *xml_out =3D NULL; - int ret =3D -1; =20 xml_in =3D g_strdup_printf("%s/genericxml2xmlindata/%s.xml", abs_srcdir, info->name); xml_out =3D g_strdup_printf("%s/genericxml2xmloutdata/%s.xml", abs_srcdir, info->name); =20 - ret =3D testCompareDomXML2XMLFiles(caps, xmlopt, xml_in, - info->different ? xml_out : xml_in, - info->active_only, 0, - info->expectResult); - return ret; + return testCompareDomXML2XMLFiles(caps, xmlopt, xml_in, + info->different ? xml_out : xml_in, + info->active_only, 0, + info->expectResult); } =20 =20 diff --git a/tests/interfacexml2xmltest.c b/tests/interfacexml2xmltest.c index b23e0f589b..95fee4d744 100644 --- a/tests/interfacexml2xmltest.c +++ b/tests/interfacexml2xmltest.c @@ -38,15 +38,12 @@ testCompareXMLToXMLFiles(const char *xml) static int testCompareXMLToXMLHelper(const void *data) { - int result =3D -1; g_autofree char *xml =3D NULL; =20 xml =3D g_strdup_printf("%s/interfaceschemadata/%s.xml", abs_srcdir, (const char *)data); =20 - result =3D testCompareXMLToXMLFiles(xml); - - return result; + return testCompareXMLToXMLFiles(xml); } =20 =20 diff --git a/tests/libxlxml2domconfigtest.c b/tests/libxlxml2domconfigtest.c index a7080e0fa2..7d59bfe886 100644 --- a/tests/libxlxml2domconfigtest.c +++ b/tests/libxlxml2domconfigtest.c @@ -139,7 +139,6 @@ struct testInfo { static int testCompareXMLToDomConfigHelper(const void *data) { - int ret =3D -1; const struct testInfo *info =3D data; g_autofree char *xmlfile =3D NULL; g_autofree char *jsonfile =3D NULL; @@ -147,9 +146,7 @@ testCompareXMLToDomConfigHelper(const void *data) xmlfile =3D g_strdup_printf("%s/libxlxml2domconfigdata/%s.xml", abs_sr= cdir, info->name); jsonfile =3D g_strdup_printf("%s/libxlxml2domconfigdata/%s.json", abs_= srcdir, info->name); =20 - ret =3D testCompareXMLToDomConfig(xmlfile, jsonfile); - - return ret; + return testCompareXMLToDomConfig(xmlfile, jsonfile); } =20 =20 diff --git a/tests/lxcconf2xmltest.c b/tests/lxcconf2xmltest.c index 09f6465567..f384264e59 100644 --- a/tests/lxcconf2xmltest.c +++ b/tests/lxcconf2xmltest.c @@ -58,7 +58,6 @@ struct testInfo { static int testCompareXMLToConfigHelperLegacy(const void *data) { - int result =3D -1; const struct testInfo *info =3D data; g_autofree char *xml =3D NULL; g_autofree char *config =3D NULL; @@ -68,15 +67,12 @@ testCompareXMLToConfigHelperLegacy(const void *data) config =3D g_strdup_printf("%s/lxcconf2xmldata/lxcconf2xml-%s.config", abs_srcdir, info->name); =20 - result =3D testCompareXMLToConfigFiles(xml, config, info->expectError); - - return result; + return testCompareXMLToConfigFiles(xml, config, info->expectError); } =20 static int testCompareXMLToConfigHelperV3(const void *data) { - int result =3D -1; const struct testInfo *info =3D data; g_autofree char *xml =3D NULL; g_autofree char *config =3D NULL; @@ -86,9 +82,7 @@ testCompareXMLToConfigHelperV3(const void *data) config =3D g_strdup_printf("%s/lxcconf2xmldata/lxcconf2xml-%s-v3.confi= g", abs_srcdir, info->name); =20 - result =3D testCompareXMLToConfigFiles(xml, config, info->expectError); - - return result; + return testCompareXMLToConfigFiles(xml, config, info->expectError); } =20 =20 diff --git a/tests/lxcxml2xmltest.c b/tests/lxcxml2xmltest.c index 2c5d294f18..abb48ff993 100644 --- a/tests/lxcxml2xmltest.c +++ b/tests/lxcxml2xmltest.c @@ -30,19 +30,16 @@ testCompareXMLToXMLHelper(const void *data) const struct testInfo *info =3D data; g_autofree char *xml_in =3D NULL; g_autofree char *xml_out =3D NULL; - int ret =3D -1; =20 xml_in =3D g_strdup_printf("%s/lxcxml2xmldata/lxc-%s.xml", abs_srcdir, info->name); xml_out =3D g_strdup_printf("%s/lxcxml2xmloutdata/lxc-%s.xml", abs_srcdir, info->name); =20 - ret =3D testCompareDomXML2XMLFiles(driver->caps, driver->xmlopt, xml_i= n, - info->different ? xml_out : xml_in, - info->active_only, - info->parse_flags, - TEST_COMPARE_DOM_XML2XML_RESULT_SUCCE= SS); - return ret; + return testCompareDomXML2XMLFiles(driver->caps, driver->xmlopt, xml_in, + info->different ? xml_out : xml_in, + info->active_only, info->parse_flags, + TEST_COMPARE_DOM_XML2XML_RESULT_SUCC= ESS); } =20 =20 diff --git a/tests/networkxml2conftest.c b/tests/networkxml2conftest.c index bbeb849a39..81f2b4b9ae 100644 --- a/tests/networkxml2conftest.c +++ b/tests/networkxml2conftest.c @@ -93,7 +93,6 @@ typedef struct { static int testCompareXMLToConfHelper(const void *data) { - int result =3D -1; const testInfo *info =3D data; g_autofree char *inxml =3D NULL; g_autofree char *outconf =3D NULL; @@ -103,9 +102,7 @@ testCompareXMLToConfHelper(const void *data) outconf =3D g_strdup_printf("%s/networkxml2confdata/%s.conf", abs_srcd= ir, info->name); outhostsfile =3D g_strdup_printf("%s/networkxml2confdata/%s.hostsfile"= , abs_srcdir, info->name); =20 - result =3D testCompareXMLToConfFiles(inxml, outconf, outhostsfile, inf= o->caps); - - return result; + return testCompareXMLToConfFiles(inxml, outconf, outhostsfile, info->c= aps); } =20 static void diff --git a/tests/networkxml2firewalltest.c b/tests/networkxml2firewalltes= t.c index ca793fd4ea..9cf00d9369 100644 --- a/tests/networkxml2firewalltest.c +++ b/tests/networkxml2firewalltest.c @@ -125,7 +125,6 @@ struct testInfo { static int testCompareXMLToIPTablesHelper(const void *data) { - int result =3D -1; const struct testInfo *info =3D data; g_autofree char *xml =3D NULL; g_autofree char *args =3D NULL; @@ -135,9 +134,7 @@ testCompareXMLToIPTablesHelper(const void *data) args =3D g_strdup_printf("%s/networkxml2firewalldata/%s-%s.args", abs_srcdir, info->name, RULESTYPE); =20 - result =3D testCompareXMLToArgvFiles(xml, args, info->baseargs); - - return result; + return testCompareXMLToArgvFiles(xml, args, info->baseargs); } =20 =20 diff --git a/tests/networkxml2xmltest.c b/tests/networkxml2xmltest.c index 521f058acc..86a9cdd137 100644 --- a/tests/networkxml2xmltest.c +++ b/tests/networkxml2xmltest.c @@ -81,17 +81,14 @@ static int testCompareXMLToXMLHelper(const void *data) { const struct testInfo *info =3D data; - int result =3D -1; g_autofree char *inxml =3D NULL; g_autofree char *outxml =3D NULL; =20 inxml =3D g_strdup_printf("%s/networkxml2xmlin/%s.xml", abs_srcdir, in= fo->name); outxml =3D g_strdup_printf("%s/networkxml2xmlout/%s.xml", abs_srcdir, = info->name); =20 - result =3D testCompareXMLToXMLFiles(inxml, outxml, info->flags, - info->expectResult); - - return result; + return testCompareXMLToXMLFiles(inxml, outxml, info->flags, + info->expectResult); } =20 static int diff --git a/tests/networkxml2xmlupdatetest.c b/tests/networkxml2xmlupdatet= est.c index 58b6f70c96..392507cc5c 100644 --- a/tests/networkxml2xmlupdatetest.c +++ b/tests/networkxml2xmlupdatetest.c @@ -71,7 +71,6 @@ static int testCompareXMLToXMLHelper(const void *data) { const struct testInfo *info =3D data; - int result =3D -1; g_autofree char *netxml =3D NULL; g_autofree char *updatexml =3D NULL; g_autofree char *outxml =3D NULL; @@ -83,11 +82,9 @@ testCompareXMLToXMLHelper(const void *data) outxml =3D g_strdup_printf("%s/networkxml2xmlupdateout/%s.xml", abs_srcdir, info->outxml); =20 - result =3D testCompareXMLToXMLFiles(netxml, updatexml, outxml, info->f= lags, - info->command, info->section, - info->parentIndex, info->expectFailu= re); - - return result; + return testCompareXMLToXMLFiles(netxml, updatexml, outxml, info->flags, + info->command, info->section, + info->parentIndex, info->expectFailure= ); } =20 static int diff --git a/tests/nodedevxml2xmltest.c b/tests/nodedevxml2xmltest.c index f82027fb87..6f67ffd73f 100644 --- a/tests/nodedevxml2xmltest.c +++ b/tests/nodedevxml2xmltest.c @@ -64,7 +64,6 @@ testCompareXMLToXMLFiles(const char *xml, const char *out= file) static int testCompareXMLToXMLHelper(const void *data) { - int result =3D -1; g_autofree char *xml =3D NULL; g_autofree char *outfile =3D NULL; =20 @@ -74,9 +73,7 @@ testCompareXMLToXMLHelper(const void *data) outfile =3D g_strdup_printf("%s/nodedevxml2xmlout/%s.xml", abs_srcdir, (const char *)data); =20 - result =3D testCompareXMLToXMLFiles(xml, outfile); - - return result; + return testCompareXMLToXMLFiles(xml, outfile); } =20 =20 diff --git a/tests/nwfilterxml2firewalltest.c b/tests/nwfilterxml2firewallt= est.c index 1514002b8f..6c540f90af 100644 --- a/tests/nwfilterxml2firewalltest.c +++ b/tests/nwfilterxml2firewalltest.c @@ -389,7 +389,6 @@ struct testInfo { static int testCompareXMLToIPTablesHelper(const void *data) { - int result =3D -1; const struct testInfo *info =3D data; g_autofree char *xml =3D NULL; g_autofree char *args =3D NULL; @@ -399,9 +398,7 @@ testCompareXMLToIPTablesHelper(const void *data) args =3D g_strdup_printf("%s/nwfilterxml2firewalldata/%s-%s.args", abs_srcdir, info->name, RULESTYPE); =20 - result =3D testCompareXMLToArgvFiles(xml, args); - - return result; + return testCompareXMLToArgvFiles(xml, args); } =20 =20 diff --git a/tests/nwfilterxml2xmltest.c b/tests/nwfilterxml2xmltest.c index c2b6dc575b..1924cfdbde 100644 --- a/tests/nwfilterxml2xmltest.c +++ b/tests/nwfilterxml2xmltest.c @@ -51,7 +51,6 @@ typedef struct test_parms { static int testCompareXMLToXMLHelper(const void *data) { - int result =3D -1; const test_parms *tp =3D data; g_autofree char *inxml =3D NULL; g_autofree char *outxml =3D NULL; @@ -59,9 +58,7 @@ testCompareXMLToXMLHelper(const void *data) inxml =3D g_strdup_printf("%s/nwfilterxml2xmlin/%s.xml", abs_srcdir, t= p->name); outxml =3D g_strdup_printf("%s/nwfilterxml2xmlout/%s.xml", abs_srcdir,= tp->name); =20 - result =3D testCompareXMLToXMLFiles(inxml, outxml, tp->expect_warning); - - return result; + return testCompareXMLToXMLFiles(inxml, outxml, tp->expect_warning); } =20 static int diff --git a/tests/secretxml2xmltest.c b/tests/secretxml2xmltest.c index 828e44a32b..4aa7463e83 100644 --- a/tests/secretxml2xmltest.c +++ b/tests/secretxml2xmltest.c @@ -33,7 +33,6 @@ struct testInfo { static int testCompareXMLToXMLHelper(const void *data) { - int result =3D -1; g_autofree char *inxml =3D NULL; g_autofree char *outxml =3D NULL; const struct testInfo *info =3D data; @@ -44,9 +43,7 @@ testCompareXMLToXMLHelper(const void *data) info->different ? "out" : "in", info->name); =20 - result =3D testCompareXMLToXMLFiles(inxml, outxml); - - return result; + return testCompareXMLToXMLFiles(inxml, outxml); } =20 static int diff --git a/tests/vboxsnapshotxmltest.c b/tests/vboxsnapshotxmltest.c index 3ad8298895..ed9bb92018 100644 --- a/tests/vboxsnapshotxmltest.c +++ b/tests/vboxsnapshotxmltest.c @@ -87,15 +87,12 @@ testCompareXMLtoXMLFiles(const char *xml) static int testCompareXMLToXMLHelper(const void *data) { - int result =3D -1; g_autofree char *xml =3D NULL; =20 xml =3D g_strdup_printf("%s/vboxsnapshotxmldata/%s.vbox", abs_srcdir, (const char *)data); =20 - result =3D testCompareXMLtoXMLFiles(xml); - - return result; + return testCompareXMLtoXMLFiles(xml); } =20 static int diff --git a/tests/virnwfilterbindingxml2xmltest.c b/tests/virnwfilterbindi= ngxml2xmltest.c index 0c93b58a0c..afde2e3a4d 100644 --- a/tests/virnwfilterbindingxml2xmltest.c +++ b/tests/virnwfilterbindingxml2xmltest.c @@ -64,16 +64,13 @@ typedef struct test_parms { static int testCompareXMLToXMLHelper(const void *data) { - int result =3D -1; const test_parms *tp =3D data; g_autofree char *xml =3D NULL; =20 xml =3D g_strdup_printf("%s/virnwfilterbindingxml2xmldata/%s.xml", abs= _srcdir, tp->name); =20 - result =3D testCompareXMLToXMLFiles(xml); - - return result; + return testCompareXMLToXMLFiles(xml); } =20 static int diff --git a/tests/vmx2xmltest.c b/tests/vmx2xmltest.c index a43cab3108..d864843c19 100644 --- a/tests/vmx2xmltest.c +++ b/tests/vmx2xmltest.c @@ -89,7 +89,6 @@ struct testInfo { static int testCompareHelper(const void *data) { - int ret =3D -1; const struct testInfo *info =3D data; g_autofree char *vmx =3D NULL; g_autofree char *xml =3D NULL; @@ -99,9 +98,7 @@ testCompareHelper(const void *data) xml =3D g_strdup_printf("%s/vmx2xmldata/%s.xml", abs_srcdir, info->file); =20 - ret =3D testCompareFiles(vmx, xml, info->should_fail); - - return ret; + return testCompareFiles(vmx, xml, info->should_fail); } =20 static int diff --git a/tests/xml2vmxtest.c b/tests/xml2vmxtest.c index 1cadb9500a..be9d5f3e6b 100644 --- a/tests/xml2vmxtest.c +++ b/tests/xml2vmxtest.c @@ -83,7 +83,6 @@ struct testInfo { static int testCompareHelper(const void *data) { - int result =3D -1; const struct testInfo *info =3D data; g_autofree char *xml =3D NULL; g_autofree char *vmx =3D NULL; @@ -93,9 +92,7 @@ testCompareHelper(const void *data) vmx =3D g_strdup_printf("%s/xml2vmxdata/xml2vmx-%s.vmx", abs_srcdir, info->output); =20 - result =3D testCompareFiles(xml, vmx, info->virtualHW_version); - - return result; + return testCompareFiles(xml, vmx, info->virtualHW_version); } =20 static int --=20 2.35.1