[libvirt] [PATCH 2/9] vsh-table: allow empty columns
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(a)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