[libvirt] [PATCH 3/6] Remove EMPTYSTR macro

Ján Tomko jtomko at redhat.com
Tue Feb 12 16:40:37 UTC 2019


This macro neither takes nor produces an empty string.
Remove it in favor of NULLSTR_MINUS.

Signed-off-by: Ján Tomko <jtomko at 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




More information about the libvir-list mailing list