lib: Drop intermediary return variables

In a few places we declare a variable (which is optionally
followed by a code not touching it) then set the variable to a
value and return the variable immediately. It's obvious that the
variable is needless and the value can be returned directly
instead.

This patch was generated using this semantic patch:

  @@
  type T;
  identifier ret;
  expression E;
  @@
  - T ret;
  ... when != ret
      when strict
  - ret = E;
  - return ret;
  + return E;

After that I fixed couple of formatting issues because coccinelle
formatted some lines differently than our coding style.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/bhyve/bhyve_domain.c b/src/bhyve/bhyve_domain.c
index 33e74e2e25d933b9c07dbfce8c2d0a63271d3eb9..b526235a4ed46abddb94fd92e82b0c13bfe2798f 100644 (file)
--- a/src/bhyve/bhyve_domain.c
+++ b/src/bhyve/bhyve_domain.c
@@ -36,11 +36,7 @@ VIR_LOG_INIT("bhyve.bhyve_domain");
 static void *
 bhyveDomainObjPrivateAlloc(void *opaque G_GNUC_UNUSED)
 {
-    bhyveDomainObjPrivate *priv;
-
-    priv = g_new0(bhyveDomainObjPrivate, 1);
-
-    return priv;
+    return g_new0(bhyveDomainObjPrivate, 1);
 }
 
 static void

diff --git a/src/conf/domain_addr.c b/src/conf/domain_addr.c
index fe6520cf3a506547a1718b2102024c835723bf33..a06721c35d09e2839f3a517b27c71723f3c5f2f8 100644 (file)
--- a/src/conf/domain_addr.c
+++ b/src/conf/domain_addr.c
@@ -1862,11 +1862,7 @@ virDomainUSBAddressPortFormat(unsigned int *port)
 virDomainUSBAddressSet *
 virDomainUSBAddressSetCreate(void)
 {
-    virDomainUSBAddressSet *addrs;
-
-    addrs = g_new0(virDomainUSBAddressSet, 1);
-
-    return addrs;
+    return g_new0(virDomainUSBAddressSet, 1);
 }
 
 

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 15228d1e38b8312d334c5498c2e7d621601f2c83..1b366997612d48090a3223c21b940e49606b635b 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -28879,10 +28879,7 @@ char
 *virDomainConfigFile(const char *dir,
                      const char *name)
 {
-    char *ret;
-
-    ret = g_strdup_printf("%s/%s.xml", dir, name);
-    return ret;
+    return g_strdup_printf("%s/%s.xml", dir, name);
 }
 
 /* Translates a device name of the form (regex) "[fhv]d[a-z]+" into

diff --git a/src/conf/virnetworkobj.c b/src/conf/virnetworkobj.c
index aa629af6f64af089ffc8e0d4d4214164d7491b7e..f1baffc516481419b6cc907b67fe5e279102b4cc 100644 (file)
--- a/src/conf/virnetworkobj.c
+++ b/src/conf/virnetworkobj.c
@@ -1601,9 +1601,7 @@ char *
 virNetworkObjGetPortStatusDir(virNetworkObj *net,
                               const char *stateDir)
 {
-    char *ret;
-    ret = g_strdup_printf("%s/%s/ports", stateDir, net->def->name);
-    return ret;
+    return g_strdup_printf("%s/%s/ports", stateDir, net->def->name);
 }
 
 int

diff --git a/src/conf/virnwfilterbindingobj.c b/src/conf/virnwfilterbindingobj.c
index 29fbb63b5f977b245ea6f4f43dd47414a1677658..acea240b5d128437b37b33e2bcdb831d681c8120 100644 (file)
--- a/src/conf/virnwfilterbindingobj.c
+++ b/src/conf/virnwfilterbindingobj.c
@@ -138,10 +138,7 @@ char *
 virNWFilterBindingObjConfigFile(const char *dir,
                                 const char *name)
 {
-    char *ret;
-
-    ret = g_strdup_printf("%s/%s.xml", dir, name);
-    return ret;
+    return g_strdup_printf("%s/%s.xml", dir, name);
 }
 
 

diff --git a/src/conf/virnwfilterobj.c b/src/conf/virnwfilterobj.c
index c3b2eb048cb4c1506f8c3050fd315815a5e0e03d..6bbdf6e6fa5f6e4914d7ccb35e7c8f795f56070c 100644 (file)
--- a/src/conf/virnwfilterobj.c
+++ b/src/conf/virnwfilterobj.c
@@ -117,10 +117,7 @@ virNWFilterObjListFree(virNWFilterObjList *nwfilters)
 virNWFilterObjList *
 virNWFilterObjListNew(void)
 {
-    virNWFilterObjList *nwfilters;
-
-    nwfilters = g_new0(virNWFilterObjList, 1);
-    return nwfilters;
+    return g_new0(virNWFilterObjList, 1);
 }
 
 

diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c
index 5d9687733f7064a32e39bb8907a056889e8e6bf1..ee752662ec47762434c0687a731feafdf6e73a7c 100644 (file)
--- a/src/esx/esx_driver.c
+++ b/src/esx/esx_driver.c
@@ -1800,10 +1800,7 @@ esxDomainDestroy(virDomainPtr dom)
 static char *
 esxDomainGetOSType(virDomainPtr domain G_GNUC_UNUSED)
 {
-    char *osType;
-
-    osType = g_strdup("hvm");
-    return osType;
+    return g_strdup("hvm");
 }
 
 

diff --git a/src/esx/esx_storage_backend_iscsi.c b/src/esx/esx_storage_backend_iscsi.c
index d89b5a4ba899d1ea95878a843b56b690a87e3a74..043de1690d0b7cd0ebe8a91b2391fa9118bc8802 100644 (file)
--- a/src/esx/esx_storage_backend_iscsi.c
+++ b/src/esx/esx_storage_backend_iscsi.c
@@ -766,10 +766,7 @@ esxStorageVolWipe(virStorageVolPtr volume G_GNUC_UNUSED,
 static char *
 esxStorageVolGetPath(virStorageVolPtr volume)
 {
-    char *path;
-
-    path = g_strdup(volume->name);
-    return path;
+    return g_strdup(volume->name);
 }
 
 

diff --git a/src/esx/esx_storage_backend_vmfs.c b/src/esx/esx_storage_backend_vmfs.c
index cb2be59a33f5954214f2cacdd57cae68c36e6dc4..5dba813111234f89d297c3a4c79849d3c482872e 100644 (file)
--- a/src/esx/esx_storage_backend_vmfs.c
+++ b/src/esx/esx_storage_backend_vmfs.c
@@ -1392,10 +1392,7 @@ esxStorageVolGetXMLDesc(virStorageVolPtr volume,
 static char *
 esxStorageVolGetPath(virStorageVolPtr volume)
 {
-    char *path;
-
-    path = g_strdup_printf("[%s] %s", volume->pool, volume->name);
-    return path;
+    return g_strdup_printf("[%s] %s", volume->pool, volume->name);
 }
 
 

diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c
index a672901a81eb7660a94ef826c77d0b3633d44208..3e6360f2a78c3bd7c47d9082ae352fadf1c157c4 100644 (file)
--- a/src/hyperv/hyperv_driver.c
+++ b/src/hyperv/hyperv_driver.c
@@ -2234,10 +2234,7 @@ hypervDomainDestroy(virDomainPtr domain)
 static char *
 hypervDomainGetOSType(virDomainPtr domain G_GNUC_UNUSED)
 {
-    char *osType;
-
-    osType = g_strdup("hvm");
-    return osType;
+    return g_strdup("hvm");
 }
 
 

diff --git a/src/locking/lock_daemon.c b/src/locking/lock_daemon.c
index fa08acbc7683aa2a914519ccbf368876375b0ac0..faca4a2485694045087e834172c92d77bc297758 100644 (file)
--- a/src/locking/lock_daemon.c
+++ b/src/locking/lock_daemon.c
@@ -616,10 +616,7 @@ virLockDaemonExecRestartStatePath(bool privileged,
 static char *
 virLockDaemonGetExecRestartMagic(void)
 {
-    char *ret;
-
-    ret = g_strdup_printf("%lld", (long long int)getpid());
-    return ret;
+    return g_strdup_printf("%lld", (long long int)getpid());
 }
 
 

diff --git a/src/locking/lock_driver_sanlock.c b/src/locking/lock_driver_sanlock.c
index 46e2cbc4f0683a2f6f77cf08c781d0d2c5a21a45..2dceb1eefcd4c140857e2af0e91512063f241f6d 100644 (file)
--- a/src/locking/lock_driver_sanlock.c
+++ b/src/locking/lock_driver_sanlock.c
@@ -174,13 +174,11 @@ static int
 virLockManagerSanlockInitLockspace(virLockManagerSanlockDriver *driver,
                                    struct sanlk_lockspace *ls)
 {
-    int ret;
-
     const int max_hosts = 0; /* defaults used in sanlock_init() implementation */
     const unsigned int lockspaceFlags = 0;
 
-    ret = sanlock_write_lockspace(ls, max_hosts, lockspaceFlags, driver->io_timeout);
-    return ret;
+    return sanlock_write_lockspace(ls, max_hosts, lockspaceFlags,
+                                   driver->io_timeout);
 }
 
 /* How many times try adding a lockspace? */

diff --git a/src/logging/log_daemon.c b/src/logging/log_daemon.c
index 1b04fbdae839a76a6f1a55bbc0095a42a406bc3f..cc7889399bf04f03c54f00cbd196aa0498709ed6 100644 (file)
--- a/src/logging/log_daemon.c
+++ b/src/logging/log_daemon.c
@@ -423,10 +423,7 @@ virLogDaemonExecRestartStatePath(bool privileged,
 static char *
 virLogDaemonGetExecRestartMagic(void)
 {
-    char *ret;
-
-    ret = g_strdup_printf("%lld", (long long int)getpid());
-    return ret;
+    return g_strdup_printf("%lld", (long long int)getpid());
 }
 
 

diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index 7027dc636c64fb05bd77d6322c7054c77b7b5ca7..498c45d0a7fc827764071da6bcad0c91b26c3da9 100644 (file)
--- a/src/network/bridge_driver.c
+++ b/src/network/bridge_driver.c
@@ -365,10 +365,7 @@ static char *
 networkDnsmasqLeaseFileNameDefault(virNetworkDriverState *driver,
                                    const char *netname)
 {
-    char *leasefile;
-
-    leasefile = g_strdup_printf("%s/%s.leases", driver->dnsmasqStateDir, netname);
-    return leasefile;
+    return g_strdup_printf("%s/%s.leases", driver->dnsmasqStateDir, netname);
 }
 
 
@@ -376,10 +373,7 @@ static char *
 networkDnsmasqLeaseFileNameCustom(virNetworkDriverState *driver,
                                   const char *bridge)
 {
-    char *leasefile;
-
-    leasefile = g_strdup_printf("%s/%s.status", driver->dnsmasqStateDir, bridge);
-    return leasefile;
+    return g_strdup_printf("%s/%s.status", driver->dnsmasqStateDir, bridge);
 }
 
 
@@ -387,21 +381,13 @@ static char *
 networkDnsmasqConfigFileName(virNetworkDriverState *driver,
                              const char *netname)
 {
-    char *conffile;
-
-    conffile = g_strdup_printf("%s/%s.conf", driver->dnsmasqStateDir, netname);
-    return conffile;
+    return g_strdup_printf("%s/%s.conf", driver->dnsmasqStateDir, netname);
 }
 
 
 static char *
 networkRadvdPidfileBasename(const char *netname)
-{
-    /* this is simple but we want to be sure it's consistently done */
-    char *pidfilebase;
-
-    pidfilebase = g_strdup_printf("%s-radvd", netname);
-    return pidfilebase;
+{return g_strdup_printf("%s-radvd", netname);
 }
 
 
@@ -409,10 +395,7 @@ static char *
 networkRadvdConfigFileName(virNetworkDriverState *driver,
                            const char *netname)
 {
-    char *configfile;
-
-    configfile = g_strdup_printf("%s/%s-radvd.conf", driver->radvdStateDir, netname);
-    return configfile;
+    return g_strdup_printf("%s/%s-radvd.conf", driver->radvdStateDir, netname);
 }
 
 

diff --git a/src/qemu/qemu_alias.c b/src/qemu/qemu_alias.c
index 276a03cb569622e871aff0d7c6c13f27a22a171e..a36f34659213ad4a5aa088d60846da205a05e574 100644 (file)
--- a/src/qemu/qemu_alias.c
+++ b/src/qemu/qemu_alias.c
@@ -804,11 +804,7 @@ qemuAliasFromHostdev(const virDomainHostdevDef *hostdev)
 char *
 qemuDomainGetMasterKeyAlias(void)
 {
-    char *alias;
-
-    alias = g_strdup("masterKey0");
-
-    return alias;
+    return g_strdup("masterKey0");
 }
 
 
@@ -837,11 +833,7 @@ qemuAliasForSecret(const char *parentalias,
 char *
 qemuAliasTLSObjFromSrcAlias(const char *srcAlias)
 {
-    char *ret;
-
-    ret = g_strdup_printf("obj%s_tls0", srcAlias);
-
-    return ret;
+    return g_strdup_printf("obj%s_tls0", srcAlias);
 }
 
 
@@ -853,11 +845,7 @@ qemuAliasTLSObjFromSrcAlias(const char *srcAlias)
 char *
 qemuAliasChardevFromDevAlias(const char *devAlias)
 {
-    char *ret;
-
-    ret = g_strdup_printf("char%s", devAlias);
-
-    return ret;
+    return g_strdup_printf("char%s", devAlias);
 }
 
 
@@ -871,11 +859,7 @@ qemuDomainGetManagedPRAlias(void)
 char *
 qemuDomainGetUnmanagedPRAlias(const char *parentalias)
 {
-    char *ret;
-
-    ret = g_strdup_printf("pr-helper-%s", parentalias);
-
-    return ret;
+    return g_strdup_printf("pr-helper-%s", parentalias);
 }
 
 

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index d6df50ec73cd6efb659d53991fb7e34274bbd7e1..335bad5616d4dfd1e7b15ec41f24622bce1f9f3e 100644 (file)
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -3811,18 +3811,15 @@ qemuBuildMemoryDeviceProps(const virDomainDef *def,
 static char *
 qemuBuildLegacyNicStr(virDomainNetDef *net)
 {
-    char *str;
     char macaddr[VIR_MAC_STRING_BUFLEN];
     const char *netmodel = virDomainNetGetModelString(net);
 
-    str = g_strdup_printf("nic,macaddr=%s,netdev=host%s%s%s%s%s",
-                          virMacAddrFormat(&net->mac, macaddr),
-                          net->info.alias,
-                          netmodel ? ",model=" : "",
-                          NULLSTR_EMPTY(netmodel),
-                          (net->info.alias ? ",id=" : ""),
-                          NULLSTR_EMPTY(net->info.alias));
-    return str;
+    return g_strdup_printf("nic,macaddr=%s,netdev=host%s%s%s%s%s",
+                           virMacAddrFormat(&net->mac, macaddr),
+                           net->info.alias, netmodel ? ",model=" : "",
+                           NULLSTR_EMPTY(netmodel),
+                           (net->info.alias ? ",id=" : ""),
+                           NULLSTR_EMPTY(net->info.alias));
 }
 
 

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index e719c7b8c7705589c2f0a091517da3acf4ab75ae..d0cb2eebedd866b16476e6bbaadce5ae9a3d5183 100644 (file)
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -5944,13 +5944,9 @@ qemuDomainDefFromXML(virQEMUDriver *driver,
                      virQEMUCaps *qemuCaps,
                      const char *xml)
 {
-    virDomainDef *def;
-
-    def = virDomainDefParseString(xml, driver->xmlopt, qemuCaps,
-                                  VIR_DOMAIN_DEF_PARSE_INACTIVE |
-                                  VIR_DOMAIN_DEF_PARSE_SKIP_VALIDATE);
-
-    return def;
+    return virDomainDefParseString(xml, driver->xmlopt, qemuCaps,
+                                   VIR_DOMAIN_DEF_PARSE_INACTIVE |
+                                   VIR_DOMAIN_DEF_PARSE_SKIP_VALIDATE);
 }
 
 

diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
index 6b6d6cd613dcd55a1854de737cc529d5a4cfb51b..81d90878393077c495426598b78ef5d50a9fc086 100644 (file)
--- a/src/qemu/qemu_monitor.c
+++ b/src/qemu/qemu_monitor.c
@@ -920,10 +920,7 @@ qemuMonitorClose(qemuMonitor *mon)
 char *
 qemuMonitorNextCommandID(qemuMonitor *mon)
 {
-    char *id;
-
-    id = g_strdup_printf("libvirt-%d", ++mon->nextSerial);
-    return id;
+    return g_strdup_printf("libvirt-%d", ++mon->nextSerial);
 }
 
 

diff --git a/src/security/security_apparmor.c b/src/security/security_apparmor.c
index 2cc15ba6af541a25331a7cd2e9c89cba25b6010a..dadcfd67db33caae1cdc5412531604ac6cf1f9f9 100644 (file)
--- a/src/security/security_apparmor.c
+++ b/src/security/security_apparmor.c
@@ -1169,10 +1169,7 @@ static char *
 AppArmorGetMountOptions(virSecurityManager *mgr G_GNUC_UNUSED,
                         virDomainDef *vm G_GNUC_UNUSED)
 {
-    char *opts;
-
-    opts = g_strdup("");
-    return opts;
+    return g_strdup("");
 }
 
 static const char *

diff --git a/src/security/security_nop.c b/src/security/security_nop.c
index a634e7f3fe667979cae72e55f0ac3e5bda92d318..0b3d37142a562901a1c4cfe1e547948e05d6c546 100644 (file)
--- a/src/security/security_nop.c
+++ b/src/security/security_nop.c
@@ -178,10 +178,7 @@ static char *
 virSecurityDomainGetMountOptionsNop(virSecurityManager *mgr G_GNUC_UNUSED,
                                     virDomainDef *vm G_GNUC_UNUSED)
 {
-    char *opts;
-
-    opts = g_strdup("");
-    return opts;
+    return g_strdup("");
 }
 
 static const char *

diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index b5413cc03d047425d432298270c476970d2738a7..fd7486b8ca575acee9637ab9236470aedd057c70 100644 (file)
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -1666,7 +1666,6 @@ static char *
 testConnectGetSysinfo(virConnectPtr conn G_GNUC_UNUSED,
                       unsigned int flags)
 {
-    char *ret;
     const char *sysinfo = "<sysinfo type='smbios'>\n"
            "  <bios>\n"
            "    <entry name='vendor'>LENOVO</entry>\n"
@@ -1678,8 +1677,7 @@ testConnectGetSysinfo(virConnectPtr conn G_GNUC_UNUSED,
 
     virCheckFlags(0, NULL);
 
-    ret = g_strdup(sysinfo);
-    return ret;
+    return g_strdup(sysinfo);
 }
 
 static const char *
@@ -2706,10 +2704,7 @@ testDomainCoreDump(virDomainPtr domain,
 static char *
 testDomainGetOSType(virDomainPtr dom G_GNUC_UNUSED)
 {
-    char *ret;
-
-    ret = g_strdup("linux");
-    return ret;
+    return g_strdup("linux");
 }
 
 

diff --git a/src/util/viraudit.c b/src/util/viraudit.c
index bbd2462897a74bfd8707c675feb86d7f52ec5dc9..76ad3a73ed61dd3237202c225b1f8b11573f4c27 100644 (file)
--- a/src/util/viraudit.c
+++ b/src/util/viraudit.c
@@ -140,8 +140,6 @@ char *virAuditEncode(const char *key, const char *value)
 #if WITH_AUDIT
     return audit_encode_nv_string(key, value, 0);
 #else
-    char *str;
-    str = g_strdup_printf("%s=%s", key, value);
-    return str;
+    return g_strdup_printf("%s=%s", key, value);
 #endif
 }

diff --git a/src/util/virfirewall.c b/src/util/virfirewall.c
index fb0299291236b68aa93971d8ceb10356b1ab9115..1a546335f6f573e82b844b3328a70e8fd196bca0 100644 (file)
--- a/src/util/virfirewall.c
+++ b/src/util/virfirewall.c
@@ -157,11 +157,7 @@ virFirewallSetBackend(virFirewallBackend backend)
 static virFirewallGroup *
 virFirewallGroupNew(void)
 {
-    virFirewallGroup *group;
-
-    group = g_new0(virFirewallGroup, 1);
-
-    return group;
+    return g_new0(virFirewallGroup, 1);
 }
 
 

diff --git a/src/util/virmacmap.c b/src/util/virmacmap.c
index 297d57a931591780edbc4a6dae3552d54e974c84..fb16062cd18a685ab844ef8e6c14eab642237bdb 100644 (file)
--- a/src/util/virmacmap.c
+++ b/src/util/virmacmap.c
@@ -267,11 +267,7 @@ char *
 virMacMapFileName(const char *dnsmasqStateDir,
                   const char *bridge)
 {
-    char *filename;
-
-    filename = g_strdup_printf("%s/%s.macs", dnsmasqStateDir, bridge);
-
-    return filename;
+    return g_strdup_printf("%s/%s.macs", dnsmasqStateDir, bridge);
 }
 
 

diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c
index 5b4c585716298e6da44611a3e1db49a7035b20e4..58f7360a0f895a6564d2b58a72dbdb722919e2ae 100644 (file)
--- a/src/util/virnetdev.c
+++ b/src/util/virnetdev.c
@@ -2803,10 +2803,7 @@ static int virNetDevGetMulticastTable(const char *ifname,
 virNetDevRxFilter *
 virNetDevRxFilterNew(void)
 {
-    virNetDevRxFilter *filter;
-
-    filter = g_new0(virNetDevRxFilter, 1);
-    return filter;
+    return g_new0(virNetDevRxFilter, 1);
 }
 
 

diff --git a/src/util/virpci.c b/src/util/virpci.c
index e746a2b25be5ed3a537c3ae370d40dfe06fcd719..2d12e280042cd754fbc3ccb9584ae43467e1301e 100644 (file)
--- a/src/util/virpci.c
+++ b/src/util/virpci.c
@@ -1438,14 +1438,8 @@ void virPCIDeviceAddressCopy(virPCIDeviceAddress *dst,
 char *
 virPCIDeviceAddressAsString(const virPCIDeviceAddress *addr)
 {
-    char *str;
-
-    str = g_strdup_printf(VIR_PCI_DEVICE_ADDRESS_FMT,
-                          addr->domain,
-                          addr->bus,
-                          addr->slot,
-                          addr->function);
-    return str;
+    return g_strdup_printf(VIR_PCI_DEVICE_ADDRESS_FMT, addr->domain,
+                           addr->bus, addr->slot, addr->function);
 }
 
 bool

diff --git a/src/vbox/vbox_common.c b/src/vbox/vbox_common.c
index 7334254a3600f62e4487b6d51de91904c0b0b293..45e7225ae14412bfb48f5ecaf64a069bbd10f12b 100644 (file)
--- a/src/vbox/vbox_common.c
+++ b/src/vbox/vbox_common.c
@@ -2699,16 +2699,15 @@ static int vboxDomainDestroy(virDomainPtr dom)
     return vboxDomainDestroyFlags(dom, 0);
 }
 
-static char *vboxDomainGetOSType(virDomainPtr dom G_GNUC_UNUSED) {
+static char *vboxDomainGetOSType(virDomainPtr dom G_GNUC_UNUSED)
+{
     /* Returning "hvm" always as suggested on list, cause
      * this functions seems to be badly named and it
      * is supposed to pass the ABI name and not the domain
      * operating system driver as I had imagined ;)
      */
-    char *osType;
 
-    osType = g_strdup("hvm");
-    return osType;
+    return g_strdup("hvm");
 }
 
 static int vboxDomainSetMemory(virDomainPtr dom, unsigned long memory)

diff --git a/tests/bhyvexml2argvmock.c b/tests/bhyvexml2argvmock.c
index f3820ef0325f860ff4cfe74b4ff35be84c8bb4a8..9b77f97e5fefdd974d0c478b31278cbf158f2e4a 100644 (file)
--- a/tests/bhyvexml2argvmock.c
+++ b/tests/bhyvexml2argvmock.c
@@ -74,10 +74,7 @@ int virNetDevTapCreateInBridgePort(const char *brname G_GNUC_UNUSED,
 
 char *virNetDevTapGetRealDeviceName(char *name G_GNUC_UNUSED)
 {
-    char *fakename;
-
-    fakename = g_strdup("faketapdev");
-    return fakename;
+    return g_strdup("faketapdev");
 }
 
 int virNetDevSetOnline(const char *ifname G_GNUC_UNUSED,

diff --git a/tests/qemumonitortestutils.c b/tests/qemumonitortestutils.c
index 35a1bce07142c321d7d3d6b682dac0e122c7e121..75a6a76b92d0c6a109805363e6dc80bd3fb35a68 100644 (file)
--- a/tests/qemumonitortestutils.c
+++ b/tests/qemumonitortestutils.c
@@ -1282,13 +1282,10 @@ qemuMonitorTestFullAddItem(qemuMonitorTest *test,
                            size_t line)
 {
     g_autofree char *cmderr = NULL;
-    int ret;
 
     cmderr = g_strdup_printf("wrong expected command in %s:%zu: ", filename, line);
 
-    ret = qemuMonitorTestAddItemVerbatim(test, command, cmderr, response);
-
-    return ret;
+    return qemuMonitorTestAddItemVerbatim(test, command, cmderr, response);
 }
 
 

diff --git a/tests/qemusecuritymock.c b/tests/qemusecuritymock.c
index 87aadf564eda4acb39d7bcaa6088e3bb4fe0708e..f092383b6252c55c74bc0efee66efe60fbd984f1 100644 (file)
--- a/tests/qemusecuritymock.c
+++ b/tests/qemusecuritymock.c
@@ -128,11 +128,7 @@ static char *
 get_key(const char *path,
         const char *name)
 {
-    char *ret;
-
-    ret = g_strdup_printf("%s:%s", path, name);
-
-    return ret;
+    return g_strdup_printf("%s:%s", path, name);
 }
 
 

diff --git a/tests/qemuxml2argvmock.c b/tests/qemuxml2argvmock.c
index 2265492f1e99df3879803fc368be798683c51c02..f4e2f52680a0b7a2d4ba18f3d33bbbba5f27a8f6 100644 (file)
--- a/tests/qemuxml2argvmock.c
+++ b/tests/qemuxml2argvmock.c
@@ -107,12 +107,9 @@ virNumaNodesetIsAvailable(virBitmap *nodeset)
 char *
 virTPMCreateCancelPath(const char *devpath)
 {
-    char *path;
     (void)devpath;
 
-    path = g_strdup("/sys/class/misc/tpm0/device/cancel");
-
-    return path;
+    return g_strdup("/sys/class/misc/tpm0/device/cancel");
 }
 
 /**

diff --git a/tests/virnetserverclienttest.c b/tests/virnetserverclienttest.c
index 942901bf6010a055cace084a5aecffcd6f9f6e4e..d3c817f70d64192b78f53e8df8398c05acff42ce 100644 (file)
--- a/tests/virnetserverclienttest.c
+++ b/tests/virnetserverclienttest.c
@@ -30,11 +30,7 @@ static void *
 testClientNew(virNetServerClient *client G_GNUC_UNUSED,
               void *opaque G_GNUC_UNUSED)
 {
-    char *dummy;
-
-    dummy = g_new0(char, 1);
-
-    return dummy;
+    return g_new0(char, 1);
 }
 
 

diff --git a/tests/virpcimock.c b/tests/virpcimock.c
index 0f8d5ad54f266ecc40557855feea4c6c26ed3846..f65ae7c0c51e48a9c6b39e953f97ed7a489c95b9 100644 (file)
--- a/tests/virpcimock.c
+++ b/tests/virpcimock.c
@@ -351,12 +351,8 @@ remove_fd(int fd)
 static char *
 pci_address_format(struct pciDeviceAddress const *addr)
 {
-    char *ret;
-
-    ret = g_strdup_printf(ADDR_STR_FMT,
-                          addr->domain, addr->bus,
-                          addr->device, addr->function);
-    return ret;
+    return g_strdup_printf(ADDR_STR_FMT, addr->domain, addr->bus,
+                           addr->device, addr->function);
 }
 
 static int

diff --git a/tests/virusbmock.c b/tests/virusbmock.c
index a1f539b8b16149cd5c83d36455ba4de557b3d9b3..5b09895197e2dcc3d1aff9f8fdc197274d2f5438 100644 (file)
--- a/tests/virusbmock.c
+++ b/tests/virusbmock.c
@@ -64,14 +64,12 @@ static char *get_fake_path(const char *real_path)
 DIR *opendir(const char *name)
 {
     g_autofree char *path = NULL;
-    DIR* ret;
 
     init_syms();
 
     path = get_fake_path(name);
 
-    ret = realopendir(path);
-    return ret;
+    return realopendir(path);
 }
 
 int open(const char *pathname, int flags, ...)