diff options
| author | Takashi Iwai <tiwai@suse.de> | 2008-11-07 09:08:15 +0100 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2008-11-07 09:08:15 +0100 |
| commit | 945eed02cd619f525e097319cd3d18c58d01da87 (patch) | |
| tree | b6cf5b5b17fea72d4c2140f0307f1790c173e595 /include/sound | |
| parent | 63cf123bf5d538a066e4a899390c9ce56ab6df9f (diff) | |
| download | lwn-945eed02cd619f525e097319cd3d18c58d01da87.tar.gz lwn-945eed02cd619f525e097319cd3d18c58d01da87.zip | |
ALSA: Evaluate condition in snd_BUG_ON() in non-debugging case
Change snd_BUG_ON() to evaluate the given condition, at least, in syntax
for avoiding compile warnings such as unused variables. The compiler
should optimize out the condition evaluation in the real code, though.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'include/sound')
| -rw-r--r-- | include/sound/core.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/sound/core.h b/include/sound/core.h index 7e5589472681..6fa4c7b1970a 100644 --- a/include/sound/core.h +++ b/include/sound/core.h @@ -390,11 +390,11 @@ void snd_verbose_printd(const char *file, int line, const char *format, ...) #define snd_printd(fmt, args...) do { } while (0) #define snd_BUG() do { } while (0) -static inline int __snd_bug_on(void) +static inline int __snd_bug_on(int cond) { return 0; } -#define snd_BUG_ON(cond) __snd_bug_on() /* always false */ +#define snd_BUG_ON(cond) __snd_bug_on(0 && (cond)) /* always false */ #endif /* CONFIG_SND_DEBUG */ |