From bf5addedfed50216774c4227da108baf148fd581 Mon Sep 17 00:00:00 2001 From: dmitrykos Date: Sun, 14 Mar 2010 17:25:48 +0000 Subject: [PATCH] wasapi: removed excessive argument checks in stream management functions --- src/hostapi/wasapi/pa_win_wasapi.c | 39 ++++-------------------------- 1 file changed, 5 insertions(+), 34 deletions(-) diff --git a/src/hostapi/wasapi/pa_win_wasapi.c b/src/hostapi/wasapi/pa_win_wasapi.c index 31e1675..7154aa3 100644 --- a/src/hostapi/wasapi/pa_win_wasapi.c +++ b/src/hostapi/wasapi/pa_win_wasapi.c @@ -2263,10 +2263,6 @@ static PaError CloseStream( PaStream* s ) PaError result = paNoError; PaWasapiStream *stream = (PaWasapiStream*)s; - // validate - if (s == NULL) - return paBadStreamPtr; - // abort active stream if (IsStreamActive(s)) { @@ -2300,10 +2296,6 @@ static PaError StartStream( PaStream *s ) HRESULT hr; PaWasapiStream *stream = (PaWasapiStream*)s; - // validate - if (s == NULL) - return paBadStreamPtr; - // check if stream is active already if (IsStreamActive(s)) return paStreamIsNotStopped; @@ -2418,45 +2410,29 @@ static void _FinishStream(PaWasapiStream *stream) // ------------------------------------------------------------------------------------------ static PaError StopStream( PaStream *s ) { - PaError result = paNoError; - - // validate - if (s == NULL) - return paBadStreamPtr; - // Finish stream _FinishStream((PaWasapiStream *)s); - - return result; + return paNoError; } // ------------------------------------------------------------------------------------------ static PaError AbortStream( PaStream *s ) { - PaError result = paNoError; - - // validate - if (s == NULL) - return paBadStreamPtr; - // Finish stream _FinishStream((PaWasapiStream *)s); - - return result; + return paNoError; } // ------------------------------------------------------------------------------------------ static PaError IsStreamStopped( PaStream *s ) { - PaWasapiStream *stream = (PaWasapiStream*)s; - return (stream ? !stream->running : paBadStreamPtr); + return !((PaWasapiStream *)s)->running; } // ------------------------------------------------------------------------------------------ static PaError IsStreamActive( PaStream *s ) { - PaWasapiStream *stream = (PaWasapiStream*)s; - return (stream ? stream->running : paBadStreamPtr); + return ((PaWasapiStream *)s)->running; } // ------------------------------------------------------------------------------------------ @@ -2464,10 +2440,6 @@ static PaTime GetStreamTime( PaStream *s ) { PaWasapiStream *stream = (PaWasapiStream*)s; - // validate - if (s == NULL) - return paBadStreamPtr; - /* suppress unused variable warnings */ (void) stream; @@ -2482,8 +2454,7 @@ static PaTime GetStreamTime( PaStream *s ) // ------------------------------------------------------------------------------------------ static double GetStreamCpuLoad( PaStream* s ) { - PaWasapiStream *stream = (PaWasapiStream *)s; - return (stream ? PaUtil_GetCpuLoad( &stream->cpuLoadMeasurer ) : 0.0); + return PaUtil_GetCpuLoad(&((PaWasapiStream *)s)->cpuLoadMeasurer); } // ------------------------------------------------------------------------------------------ -- 2.43.0