Revert "pcm_native.c: remove unused label"
authorLinus Torvalds <torvalds@linux-foundation.org>
Thu, 7 Aug 2008 02:24:47 +0000 (19:24 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 7 Aug 2008 02:26:20 +0000 (19:26 -0700)
This reverts commit 680db0136e0778a0d7e025af7572c6a8d82279e2.  The label
is actually used, but hidden behind CONFIG_SND_DEBUG and the horrible
snd_assert() macro.

That macro could probably be improved to be along the lines of

#define snd_assert(expr, args...) do { if ((void)(expr),0) { args; } } while (0)

or similar to make sure that we always both evaluate 'expr' and parse
'args', but while gcc should optimize it all away, I'm too lazy to
really verify that.  So I'll just admit defeat and will continue to live
with the annoying warning.

Noted-by: Robert P. J. Day <rpjday@crashcourse.ca>
Signed-off-by: Linus "Grr.." Torvalds
sound/core/pcm_native.c

index 333cff6..c49b9d9 100644 (file)
@@ -3259,6 +3259,7 @@ static int snd_pcm_fasync(int fd, struct file * file, int on)
        runtime = substream->runtime;
 
        err = fasync_helper(fd, file, on, &runtime->fasync);
+out:
        unlock_kernel();
        if (err < 0)
                return err;