[libvirt PATCH 09/16] domain_conf: graphics: extract EGL-Headless formatting to separate function

Pavel Hrdina posted 16 patches 6 days ago
[libvirt PATCH 09/16] domain_conf: graphics: extract EGL-Headless formatting to separate function
Posted by Pavel Hrdina 6 days ago
virDomainGraphicsDefFormat function was way too long so split it into
separate functions for each graphics type.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
---
 src/conf/domain_conf.c | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 636d4a5099..83dccd87f1 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -26565,6 +26565,14 @@ virDomainGraphicsDefFormatSpice(virBuffer *attrBuf,
     return 0;
 }
 
+static void
+virDomainGraphicsDefFormatEGLHeadless(virBuffer *childBuf,
+                                      virDomainGraphicsDef *def)
+{
+    virDomainGraphicsDefFormatGL(childBuf, VIR_TRISTATE_BOOL_ABSENT,
+                                 def->data.egl_headless.rendernode);
+}
+
 static int
 virDomainGraphicsDefFormat(virBuffer *buf,
                            virDomainGraphicsDef *def,
@@ -26607,9 +26615,9 @@ virDomainGraphicsDefFormat(virBuffer *buf,
         break;
 
     case VIR_DOMAIN_GRAPHICS_TYPE_EGL_HEADLESS:
-        virDomainGraphicsDefFormatGL(&childBuf, VIR_TRISTATE_BOOL_ABSENT,
-                                     def->data.egl_headless.rendernode);
+        virDomainGraphicsDefFormatEGLHeadless(&childBuf, def);
         break;
+
     case VIR_DOMAIN_GRAPHICS_TYPE_DBUS:
         if (def->data.dbus.p2p)
             virBufferAddLit(&attrBuf, " p2p='yes'");
-- 
2.48.1