[2/2] winepulse: Add missing return after default case in a switch (Coverity)

Andrew Eikum aeikum at codeweavers.com
Wed Nov 18 09:36:50 CST 2015


Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>

On Mon, Nov 16, 2015 at 10:36:12PM +0100, André Hentschel wrote:
> Signed-off-by: André Hentschel <nerv at dawncrow.de>
> ---
> CID 1339385
> 
>  dlls/winepulse.drv/mmdevdrv.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/dlls/winepulse.drv/mmdevdrv.c b/dlls/winepulse.drv/mmdevdrv.c
> index 7369802..215a53e 100644
> --- a/dlls/winepulse.drv/mmdevdrv.c
> +++ b/dlls/winepulse.drv/mmdevdrv.c
> @@ -285,6 +285,8 @@ static void pulse_contextcallback(pa_context *c, void *userdata)
>      switch (pa_context_get_state(c)) {
>          default:
>              FIXME("Unhandled state: %i\n", pa_context_get_state(c));
> +            return;
> +
>          case PA_CONTEXT_CONNECTING:
>          case PA_CONTEXT_UNCONNECTED:
>          case PA_CONTEXT_AUTHORIZING:
> -- 
> 1.9.1
> 
> 
> 
> 



More information about the wine-patches mailing list