This macro neither takes nor produces an empty string.
Remove it in favor of NULLSTR_MINUS.
Signed-off-by: Ján Tomko <jtomko(a)redhat.com>
---
src/internal.h | 5 -----
tools/virsh-network.c | 10 +++++-----
2 files changed, 5 insertions(+), 10 deletions(-)
diff --git a/src/internal.h b/src/internal.h
index 7c11f57e33..cf03a82105 100644
--- a/src/internal.h
+++ b/src/internal.h
@@ -240,11 +240,6 @@
*/
# define NULLSTR(s) ((s) ? (s) : "<null>")
-/*
- * Similar to NULLSTR, but print '-' to make it more user friendly.
- */
-# define EMPTYSTR(s) ((s) ? (s) : "-")
-
/*
* Turn a NULL string into an empty string
*/
diff --git a/tools/virsh-network.c b/tools/virsh-network.c
index 94adbabff7..9b86ef8071 100644
--- a/tools/virsh-network.c
+++ b/tools/virsh-network.c
@@ -1398,11 +1398,11 @@ cmdNetworkDHCPLeases(vshControl *ctl, const vshCmd *cmd)
if (vshTableRowAppend(table,
expirytime,
- EMPTYSTR(lease->mac),
- EMPTYSTR(typestr),
- EMPTYSTR(cidr_format),
- EMPTYSTR(lease->hostname),
- EMPTYSTR(lease->clientid),
+ NULLSTR_MINUS(lease->mac),
+ NULLSTR_MINUS(typestr),
+ NULLSTR_MINUS(cidr_format),
+ NULLSTR_MINUS(lease->hostname),
+ NULLSTR_MINUS(lease->clientid),
NULL) < 0)
goto cleanup;
}
--
2.20.1