diff options
| author | Michael Brown | 2017-03-22 07:19:33 +0100 |
|---|---|---|
| committer | Michael Brown | 2017-03-22 07:25:04 +0100 |
| commit | e88e2a29657824963c4fdb948e5ac99dfe6472ee (patch) | |
| tree | 042cfb77175c7b073881b63168b9234505dfa9f8 /src/include/compiler.h | |
| parent | [xhci] Avoid accessing beyond end of endpoint context array (diff) | |
| download | ipxe-e88e2a29657824963c4fdb948e5ac99dfe6472ee.tar.gz ipxe-e88e2a29657824963c4fdb948e5ac99dfe6472ee.tar.xz ipxe-e88e2a29657824963c4fdb948e5ac99dfe6472ee.zip | |
[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>
Diffstat (limited to 'src/include/compiler.h')
| -rw-r--r-- | src/include/compiler.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/include/compiler.h b/src/include/compiler.h index 56a5f63d..a936425d 100644 --- 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(); \ } \ |