[libvirt] [PATCH 2/9] vsh-table: allow empty columns
Ján Tomko
jtomko at redhat.com
Tue Feb 12 09:21:20 UTC 2019
Trivially implement this by deleting the bogus check in
vshTableSafeEncode.
Now it returns an empty string for an empty string instead
of returning NULL without setting an error.
Signed-off-by: Ján Tomko <jtomko at redhat.com>
---
tests/vshtabletest.c | 4 +++-
tools/vsh-table.c | 3 ---
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/tests/vshtabletest.c b/tests/vshtabletest.c
index 79285b14a9..b07db3cf23 100644
--- a/tests/vshtabletest.c
+++ b/tests/vshtabletest.c
@@ -291,7 +291,8 @@ testNTables(const void *opaque ATTRIBUTE_UNUSED)
" Id Name Status\n"
"--------------------------\n"
" 1 fedora28 running\n"
- " 2 rhel7.5 running\n";
+ " 2 rhel7.5 running\n"
+ " 3 gazpacho \n";
const char *exp2 =
" Id Name Status\n"
"---------------------\n";
@@ -311,6 +312,7 @@ testNTables(const void *opaque ATTRIBUTE_UNUSED)
goto cleanup;
vshTableRowAppend(table1, "1", "fedora28", "running", NULL);
vshTableRowAppend(table1, "2", "rhel7.5", "running", NULL);
+ vshTableRowAppend(table1, "3", "gazpacho", "", NULL);
act1 = vshTablePrintToString(table1, true);
table2 = vshTableNew("Id", "Name", "Status", NULL);
diff --git a/tools/vsh-table.c b/tools/vsh-table.c
index fda8f15879..8bd6d99778 100644
--- a/tools/vsh-table.c
+++ b/tools/vsh-table.c
@@ -219,9 +219,6 @@ vshTableSafeEncode(const char *s, size_t *width)
memset(&st, 0, sizeof(st));
- if (!sz)
- return NULL;
-
if (VIR_ALLOC_N(buf, (sz * HEX_ENCODE_LENGTH) + 1) < 0)
return NULL;
--
2.19.2
More information about the libvir-list
mailing list