[PATCH] winepulse: Use FAILED() instead of non-zero result check.

Andrew Eikum aeikum at codeweavers.com
Tue Feb 15 08:39:44 CST 2022


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

On Mon, Feb 14, 2022 at 02:50:47PM +0300, Nikolay Sivov wrote:
> Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
> ---
>  dlls/winepulse.drv/mmdevdrv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/dlls/winepulse.drv/mmdevdrv.c b/dlls/winepulse.drv/mmdevdrv.c
> index 35a66e18d8a..e3d5ea91cf5 100644
> --- a/dlls/winepulse.drv/mmdevdrv.c
> +++ b/dlls/winepulse.drv/mmdevdrv.c
> @@ -344,7 +344,7 @@ HRESULT WINAPI AUDDRV_GetAudioEndpoint(GUID *guid, IMMDevice *dev, IAudioClient
>      This->parent = dev;
>  
>      hr = CoCreateFreeThreadedMarshaler((IUnknown*)&This->IAudioClient3_iface, &This->marshal);
> -    if (hr) {
> +    if (FAILED(hr)) {
>          HeapFree(GetProcessHeap(), 0, This);
>          return hr;
>      }
> -- 
> 2.34.1
> 
> 



More information about the wine-devel mailing list