[libvirt] [PATCH 09/13] virsh: Implement vshTable API to domiflist
Signed-off-by: Simon Kobyda <skobyda(a)redhat.com>
---
tools/virsh-domain-monitor.c | 25 ++++++++++++++++---------
1 file changed, 16 insertions(+), 9 deletions(-)
diff --git a/tools/virsh-domain-monitor.c b/tools/virsh-domain-monitor.c
index b9b6739287..d31f02e5e6 100644
--- a/tools/virsh-domain-monitor.c
+++ b/tools/virsh-domain-monitor.c
@@ -693,6 +693,7 @@ cmdDomiflist(vshControl *ctl, const vshCmd *cmd)
int ninterfaces;
xmlNodePtr *interfaces = NULL;
size_t i;
+ vshTablePtr table = NULL;
if (vshCommandOptBool(cmd, "inactive"))
flags |= VIR_DOMAIN_XML_INACTIVE;
@@ -704,9 +705,10 @@ cmdDomiflist(vshControl *ctl, const vshCmd *cmd)
if (ninterfaces < 0)
goto cleanup;
- vshPrintExtra(ctl, "%-10s %-10s %-10s %-11s %s\n",
_("Interface"),
- _("Type"), _("Source"), _("Model"),
_("MAC"));
- vshPrintExtra(ctl,
"-------------------------------------------------------\n");
+ table = vshTableNew("Interface", "Type",
+ "Source", "Model", "MAC",
NULL);
+ if (!table)
+ goto cleanup;
for (i = 0; i < ninterfaces; i++) {
char *type = NULL;
@@ -727,12 +729,14 @@ cmdDomiflist(vshControl *ctl, const vshCmd *cmd)
model = virXPathString("string(./model/@type)", ctxt);
mac = virXPathString("string(./mac/@address)", ctxt);
- vshPrint(ctl, "%-10s %-10s %-10s %-11s %-10s\n",
- target ? target : "-",
- type,
- source ? source : "-",
- model ? model : "-",
- mac ? mac : "-");
+ if (vshTableRowAppend(table,
+ target ? target : "-",
+ type,
+ source ? source : "-",
+ model ? model : "-",
+ mac ? mac : "-",
+ NULL) < 0)
+ goto cleanup;
VIR_FREE(type);
VIR_FREE(source);
@@ -741,9 +745,12 @@ cmdDomiflist(vshControl *ctl, const vshCmd *cmd)
VIR_FREE(mac);
}
+ vshTablePrintToStdout(table, ctl);
+
ret = true;
cleanup:
+ vshTableFree(table);
VIR_FREE(interfaces);
xmlFreeDoc(xmldoc);
xmlXPathFreeContext(ctxt);
--
2.17.1