This function always returns zero, so it might as well be void.
Signed-off-by: Matt Coleman <matt(a)datto.com>
---
src/conf/domain_conf.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index d716db596a..edfaf7c8af 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -29372,7 +29372,7 @@ virDomainDefIothreadShouldFormat(virDomainDefPtr def)
}
-static int
+static void
virDomainIOMMUDefFormat(virBufferPtr buf,
const virDomainIOMMUDef *iommu)
{
@@ -29407,8 +29407,6 @@ virDomainIOMMUDefFormat(virBufferPtr buf,
virDomainIOMMUModelTypeToString(iommu->model));
virXMLFormatElement(buf, "iommu", &attrBuf, &childBuf);
-
- return 0;
}
@@ -30384,9 +30382,8 @@ virDomainDefFormatInternalSetRootName(virDomainDefPtr def,
return -1;
}
- if (def->iommu &&
- virDomainIOMMUDefFormat(buf, def->iommu) < 0)
- return -1;
+ if (def->iommu)
+ virDomainIOMMUDefFormat(buf, def->iommu);
if (def->vsock &&
virDomainVsockDefFormat(buf, def->vsock) < 0)
--
2.27.0