Re: [PATCH v2 5/7] winepulse: Simplify IAudioStreamVolume::GetChannelVolume implementation.

[Andrew Eikum]

Signed-off-by: Andrew Eikum

On Mon, May 24, 2021 at 06:22:10PM +0200, Jacek Caban wrote:
> Signed-off-by: Jacek Caban
>
> dlls/winepulse.drv/mmdevdrv.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> diff --git a/dlls/winepulse.drv/mmdevdrv.c b/dlls/winepulse.drv/mmdevdrv.c
> index 84ab5b6adec..df38bb594c7 100644
> --- a/dlls/winepulse.drv/mmdevdrv.c
> +++ b/dlls/winepulse.drv/mmdevdrv.c
> @@ -1494,21 +1494,21 @@ static HRESULT WINAPI AudioStreamVolume_GetChannelVolume(
>          IAudioStreamVolume *iface, UINT32 index, float *level)
>  {
>      ACImpl *This = impl_from_IAudioStreamVolume(iface);
> -
>
> float volumes[PA_CHANNELS_MAX];
> -
>
> HRESULT hr;
>
> TRACE("(%p)->(%d, %p)\n", This, index, level);
>
> if (!level)
>     return E_POINTER;
> - if (!This->pulse_stream)
> -
>    return AUDCLNT_E_NOT_INITIALIZED;
> if (index >= This->channel_count)
>        return E_INVALIDARG;
> - hr = AudioStreamVolume_GetAllVolumes(iface, This->channel_count, volumes);
> - if (SUCCEEDED(hr))
> -
>    *level = volumes[index];
> - return hr;
> - pulse->lock();
> - *level = This->vol[index];
> - pulse->unlock();
> - return S_OK;
>
> }
>
> static const IAudioStreamVolumeVtbl AudioStreamVolume_Vtbl =