From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:34145) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S1Hxn-000625-JQ for qemu-devel@nongnu.org; Sat, 25 Feb 2012 08:47:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S1Hxi-00069s-7I for qemu-devel@nongnu.org; Sat, 25 Feb 2012 08:47:03 -0500 Received: from v220110690675601.yourvserver.net ([78.47.199.172]:56151) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S1Hxi-00069i-1F for qemu-devel@nongnu.org; Sat, 25 Feb 2012 08:46:58 -0500 From: Stefan Weil Date: Sat, 25 Feb 2012 14:46:55 +0100 Message-Id: <1330177615-25375-1-git-send-email-sw@weilnetz.de> Subject: [Qemu-devel] [PATCH] audio: Add some fall through comments List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: malc Cc: Stefan Weil , qemu-devel@nongnu.org Static code analysers expect these comments for case statements without a break statement. Signed-off-by: Stefan Weil --- audio/audio.c | 3 +++ audio/esdaudio.c | 2 +- 2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/audio/audio.c b/audio/audio.c index 5fff6de..398763f 100644 --- a/audio/audio.c +++ b/audio/audio.c @@ -585,17 +585,20 @@ static int audio_pcm_info_eq (struct audio_pcm_info *info, struct audsettings *a switch (as->fmt) { case AUD_FMT_S8: sign = 1; + /* fall through */ case AUD_FMT_U8: break; case AUD_FMT_S16: sign = 1; + /* fall through */ case AUD_FMT_U16: bits = 16; break; case AUD_FMT_S32: sign = 1; + /* fall through */ case AUD_FMT_U32: bits = 32; break; diff --git a/audio/esdaudio.c b/audio/esdaudio.c index bd6e1cc..eea9cce 100644 --- a/audio/esdaudio.c +++ b/audio/esdaudio.c @@ -201,7 +201,7 @@ static int qesd_init_out (HWVoiceOut *hw, struct audsettings *as) case AUD_FMT_S32: case AUD_FMT_U32: dolog ("Will use 16 instead of 32 bit samples\n"); - + /* fall through */ case AUD_FMT_S16: case AUD_FMT_U16: deffmt: -- 1.7.9