Add "do { } while (0)" to macros used to remove logging calls, to
ensure there's no code structure change when enabling/disabling
logging.

Signed-off-by: Andre Muezerie <andre...@linux.microsoft.com>
---
 lib/port/rte_swx_port_ethdev.c      | 2 +-
 lib/port/rte_swx_port_fd.c          | 2 +-
 lib/port/rte_swx_port_ring.c        | 2 +-
 lib/port/rte_swx_port_source_sink.c | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/lib/port/rte_swx_port_ethdev.c b/lib/port/rte_swx_port_ethdev.c
index 7cb3c4dfb1..71dabae826 100644
--- a/lib/port/rte_swx_port_ethdev.c
+++ b/lib/port/rte_swx_port_ethdev.c
@@ -25,7 +25,7 @@ do {                                                          
                 \
 #if TRACE_LEVEL
 #define TRACE(...) printf(__VA_ARGS__)
 #else
-#define TRACE(...)
+#define TRACE(...) do { } while (0)
 #endif
 
 /*
diff --git a/lib/port/rte_swx_port_fd.c b/lib/port/rte_swx_port_fd.c
index 4bbad2eaa7..f315db285f 100644
--- a/lib/port/rte_swx_port_fd.c
+++ b/lib/port/rte_swx_port_fd.c
@@ -18,7 +18,7 @@
 #if TRACE_LEVEL
 #define TRACE(...) printf(__VA_ARGS__)
 #else
-#define TRACE(...)
+#define TRACE(...) do { } while (0)
 #endif
 
 /*
diff --git a/lib/port/rte_swx_port_ring.c b/lib/port/rte_swx_port_ring.c
index af541a2fa2..1f7abc0584 100644
--- a/lib/port/rte_swx_port_ring.c
+++ b/lib/port/rte_swx_port_ring.c
@@ -18,7 +18,7 @@
 #if TRACE_LEVEL
 #define TRACE(...) printf(__VA_ARGS__)
 #else
-#define TRACE(...)
+#define TRACE(...) do { } while (0)
 #endif
 
 /*
diff --git a/lib/port/rte_swx_port_source_sink.c 
b/lib/port/rte_swx_port_source_sink.c
index 757a2c4a2e..9f080f2e93 100644
--- a/lib/port/rte_swx_port_source_sink.c
+++ b/lib/port/rte_swx_port_source_sink.c
@@ -28,7 +28,7 @@ do {                                                          
                 \
 #if TRACE_LEVEL
 #define TRACE(...) printf(__VA_ARGS__)
 #else
-#define TRACE(...)
+#define TRACE(...) do { } while (0)
 #endif
 
 /*
-- 
2.47.0.vfs.0.3

Reply via email to