[PATCH] xaudio2: Added missing break in a switch (Coverity)

[Andrew Eikum]

On Thu, Jan 07, 2016 at 05:13:45PM +0300, Nikolay Sivov wrote:
> On 07.01.2016 0:06, Andrew Eikum wrote:
> > On Wed, Jan 06, 2016 at 05:56:05PM +0300, Nikolay Sivov wrote:
> >> Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
> >> ---
> >>  dlls/xaudio2_7/xaudio_dll.c | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/dlls/xaudio2_7/xaudio_dll.c b/dlls/xaudio2_7/xaudio_dll.c
> >> index e7176b2..1f5a91f 100644
> >> --- a/dlls/xaudio2_7/xaudio_dll.c
> >> +++ b/dlls/xaudio2_7/xaudio_dll.c
> >> @@ -449,6 +449,7 @@ static ALenum get_al_format(const WAVEFORMATEX *fmt)
> >>              case 8:
> >>                  return AL_FORMAT_71CHN8;
> >>              }
> >> +            break;
> >>          case 16:
> > 
> > Could you also add a break to the 16 case?
> 
> I can, but it ends on case 16, so there's nothing to fall through to. Or
> you mean purely for consistency?
> 

Yeah, just for consistency, and to prevent the same error in the
future.  But the patch already got in, so no big deal if you don't
bother.

Andrew