From 2d61934a2163bcb8f11ddab4aad7c3b7b9f3a98b Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A1n=20Tomko?= Date: Thu, 28 Apr 2016 15:35:41 +0200 Subject: [PATCH] Remove useless variable in qemuDomainAssignAddresses We do not need to propagate the exact return values and the only possible ones are 0 and -1 anyway. Remove the temporary variable and use the usual pattern: if (f() < 0) return -1; --- src/qemu/qemu_domain_address.c | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/src/qemu/qemu_domain_address.c b/src/qemu/qemu_domain_address.c index f149ff9c67..25aab39675 100644 --- a/src/qemu/qemu_domain_address.c +++ b/src/qemu/qemu_domain_address.c @@ -1644,23 +1644,21 @@ qemuDomainAssignAddresses(virDomainDefPtr def, virQEMUCapsPtr qemuCaps, virDomainObjPtr obj) { - int rc; - - rc = qemuDomainAssignVirtioSerialAddresses(def, obj); - if (rc) - return rc; + if (qemuDomainAssignVirtioSerialAddresses(def, obj) < 0) + return -1; - rc = qemuDomainAssignSpaprVIOAddresses(def, qemuCaps); - if (rc) - return rc; + if (qemuDomainAssignSpaprVIOAddresses(def, qemuCaps) < 0) + return -1; - rc = qemuDomainAssignS390Addresses(def, qemuCaps, obj); - if (rc) - return rc; + if (qemuDomainAssignS390Addresses(def, qemuCaps, obj) < 0) + return -1; qemuDomainAssignARMVirtioMMIOAddresses(def, qemuCaps); - return qemuDomainAssignPCIAddresses(def, qemuCaps, obj); + if (qemuDomainAssignPCIAddresses(def, qemuCaps, obj) < 0) + return -1; + + return 0; } -- 2.47.2