[v2,04/16] conf: domcaps: Add single line formatting macro

Message ID a4ceeb8884704c7996e7b0e99a0e552e68a9c0b5.1551914794.git.crobinso@redhat.com
State Accepted
Commit bf68454c46ba5b7d56990bbcc857e646f16aab94
Headers show
Series
  • domcaps: use virTristateBool
Related show

Commit Message

Cole Robinson March 6, 2019, 11:36 p.m.
Similar to the macros we have for formatting enums, add a macro to
simplify formatting the pattern:

  <FOO supported='yes|no'/>

Signed-off-by: Cole Robinson <crobinso@redhat.com>

---
 src/conf/domain_capabilities.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Patch

diff --git a/src/conf/domain_capabilities.c b/src/conf/domain_capabilities.c
index a6104920ab..5a26329176 100644
--- a/src/conf/domain_capabilities.c
+++ b/src/conf/domain_capabilities.c
@@ -384,6 +384,12 @@  virDomainCapsStringValuesFormat(virBufferPtr buf,
         virBufferAddLit(buf, "</" #item ">\n"); \
     } while (0)
 
+#define FORMAT_SINGLE(name, supported) \
+    do { \
+        virBufferAsprintf(&buf, "<%s supported='%s'/>\n", name, \
+                          supported ? "yes" : "no"); \
+    } while (0)
+
 #define ENUM_PROCESS(master, capsEnum, valToStr) \
     do { \
         virDomainCapsEnumFormat(buf, &master->capsEnum, \
@@ -594,8 +600,7 @@  virDomainCapsFormat(virDomainCapsPtr const caps)
     if (caps->maxvcpus)
         virBufferAsprintf(&buf, "<vcpu max='%d'/>\n", caps->maxvcpus);
 
-    virBufferAsprintf(&buf, "<iothreads supported='%s'/>\n",
-                      caps->iothreads ? "yes" : "no");
+    FORMAT_SINGLE("iothreads", caps->iothreads);
 
     virDomainCapsOSFormat(&buf, &caps->os);
     virDomainCapsCPUFormat(&buf, &caps->cpu);
@@ -615,11 +620,8 @@  virDomainCapsFormat(virDomainCapsPtr const caps)
     virBufferAdjustIndent(&buf, 2);
 
     virDomainCapsFeatureGICFormat(&buf, &caps->gic);
-    virBufferAsprintf(&buf, "<vmcoreinfo supported='%s'/>\n",
-                      caps->vmcoreinfo ? "yes" : "no");
-
-    virBufferAsprintf(&buf, "<genid supported='%s'/>\n",
-                      caps->genid ? "yes" : "no");
+    FORMAT_SINGLE("vmcoreinfo", caps->vmcoreinfo);
+    FORMAT_SINGLE("genid", caps->genid);
     virDomainCapsFeatureSEVFormat(&buf, caps->sev);
 
     virBufferAdjustIndent(&buf, -2);