Re: [libvirt] [PATCH 05/11] Switch the virsh XML generation to use virBuffer instead of virAsprintf

The code generating XML for storage pool source discovery is hardcoded to only allow a hostname and optional port number. Refactor this code to make it easier to add support for extra parameters. * tools/virsh.c: Refactor XML generator --- tools/virsh.c | 26 ++++++++++---------------- 1 files changed, 10 insertions(+), 16 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index d15a8df..6fc1b47 100644 --- a/tools/virsh.c

+ virBuffer buf = VIR_BUFFER_INITIALIZER; + virBufferAddLit(&buf, "<source>\n"); + virBufferVSprintf(&buf, " <host name='%.*s'",(int)hostlen, host); + if (port) + virBufferVSprintf(&buf, " port='%s'", port); + virBufferAddLit(&buf, "/>\n"); + virBufferAddLit(&buf, "</source>\n");