[PATCH 05/12] qemu: Use macro for setting string migration parameters

[Jiri Denemark <jdenemar@xxxxxxxxxx>]

Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx>
---
 src/qemu/qemu_monitor_json.c | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 218bbd8bd..826133543 100644
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -2749,24 +2749,21 @@ qemuMonitorJSONSetMigrationParams(qemuMonitorPtr mon,
     APPEND(params->VAR ## _set,                                             \
            virJSONValueObjectAppendNumberInt, VAR, FIELD)
 
+#define APPEND_STR(VAR, FIELD)                                              \
+    APPEND(params->VAR,                                                     \
+           virJSONValueObjectAppendString, VAR, FIELD)
+
     APPEND_INT(compressLevel, "compress-level");
     APPEND_INT(compressThreads, "compress-threads");
     APPEND_INT(decompressThreads, "decompress-threads");
     APPEND_INT(cpuThrottleInitial, "cpu-throttle-initial");
     APPEND_INT(cpuThrottleIncrement, "cpu-throttle-increment");
+    APPEND_STR(migrateTLSAlias, "tls-creds");
+    APPEND_STR(migrateTLSHostname, "tls-hostname");
 
 #undef APPEND
 #undef APPEND_INT
-
-    if (params->migrateTLSAlias &&
-        virJSONValueObjectAppendString(args, "tls-creds",
-                                       params->migrateTLSAlias) < 0)
-        goto cleanup;
-
-    if (params->migrateTLSHostname &&
-        virJSONValueObjectAppendString(args, "tls-hostname",
-                                       params->migrateTLSHostname) < 0)
-        goto cleanup;
+#undef APPEND_STR
 
     if (virJSONValueObjectAppend(cmd, "arguments", args) < 0)
         goto cleanup;
-- 
2.14.3

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