[build] Avoid confusing sparse in single-argument DBG() macros

For visual consistency with surrounding lines, the definitions of
DBG_MORE(), DBG_PAUSE(), etc include an unnecessary ##__VA_ARGS__
argument which is always elided.  This confuses sparse, which
complains about DBG_MORE_IF() being called with more than one
argument.

Work around this problem by adding an unused variable argument list to
the single-argument macros DBG_MORE_IF() and DBG_PAUSE_IF().

Signed-off-by: Michael Brown <mcb30@ipxe.org>

diff --git a/src/include/compiler.h b/src/include/compiler.h
index 56a5f63d51901246c88c6a573e4c877f3ae4c1c9..a936425de322461e09e7a1cbb7e64c4fffa76481 100644 (file)
--- a/src/include/compiler.h
+++ b/src/include/compiler.h
@@ -403,7 +403,7 @@ char __debug_disable(OBJECT) = ( DBGLVL_MAX & ~DBGLVL_DFLT );
  *
  * @v level            Debug level
  */
-#define DBG_PAUSE_IF( level ) do {                             \
+#define DBG_PAUSE_IF( level, ... ) do {                                \
                if ( DBG_ ## level ) {                          \
                        dbg_pause();                            \
                }                                               \
@@ -414,7 +414,7 @@ char __debug_disable(OBJECT) = ( DBGLVL_MAX & ~DBGLVL_DFLT );
  *
  * @v level            Debug level
  */
-#define DBG_MORE_IF( level ) do {                              \
+#define DBG_MORE_IF( level, ... ) do {                         \
                if ( DBG_ ## level ) {                          \
                        dbg_more();                             \
                }                                               \