diff options
author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2006-08-02 18:44:00 +0200 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2006-09-17 19:31:20 +0200 |
commit | 611aa19fd60fe57059d9972fa6ae29f7472a13cf (patch) | |
tree | a87f71ef393eec3e904291d1d5da1f2fdc5bb11b /drivers/ieee1394/dv1394.c | |
parent | 4b60912e52bc6ccdf587f2b92f3435ee2678d730 (diff) | |
download | op-kernel-dev-611aa19fd60fe57059d9972fa6ae29f7472a13cf.zip op-kernel-dev-611aa19fd60fe57059d9972fa6ae29f7472a13cf.tar.gz |
ieee1394: safer definition of empty macros
A deactivated macro, defined as "#define foo(bar)", will result in
silent corruption if somebody forgets a semicolon after a call to foo.
Replace it by "#define foo(bar) do {} while (0)" which will reveal any
respective syntax errors.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/ieee1394/dv1394.c')
-rw-r--r-- | drivers/ieee1394/dv1394.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/ieee1394/dv1394.c b/drivers/ieee1394/dv1394.c index 6e71d68..6c72f04 100644 --- a/drivers/ieee1394/dv1394.c +++ b/drivers/ieee1394/dv1394.c @@ -137,13 +137,13 @@ #if DV1394_DEBUG_LEVEL >= 2 #define irq_printk( args... ) printk( args ) #else -#define irq_printk( args... ) +#define irq_printk( args... ) do {} while (0) #endif #if DV1394_DEBUG_LEVEL >= 1 #define debug_printk( args... ) printk( args) #else -#define debug_printk( args... ) +#define debug_printk( args... ) do {} while (0) #endif /* issue a dummy PCI read to force the preceding write |