On a Wednesday in 2020, Daniel Henrique Barboza wrote:
qemuExtDevicesInitPaths() does not need 'ret'.
Tested-by: Satheesh Rajendran <sathnaga(a)linux.vnet.ibm.com>
Reviewed-by: Stefan Berger <stefanb(a)linux.ibm.com>
Signed-off-by: Daniel Henrique Barboza <danielhb413(a)gmail.com>
---
src/qemu/qemu_extdevice.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/qemu/qemu_extdevice.c b/src/qemu/qemu_extdevice.c
index dc188e6042..8f1bd9311e 100644
--- a/src/qemu/qemu_extdevice.c
+++ b/src/qemu/qemu_extdevice.c
@@ -73,12 +73,10 @@ static int
qemuExtDevicesInitPaths(virQEMUDriverPtr driver,
virDomainDefPtr def)
{
- int ret = 0;
-
if (def->tpm)
- ret = qemuExtTPMInitPaths(driver, def);
+ return qemuExtTPMInitPaths(driver, def);
Neither the old nor the new pattern allows for easily extending
this function.
if (tpm) {
if (InitPaths() < 0)
return -1;
}
But we probably aren't going to need it.
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
Jano