Re: [PATCH v11 0/4] MR2669: wine{alsa,coreaudio,oss,pulse}: Move AudioSessionControl into mmdevapi.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/mmdevapi/audiosession.c:
> -
>
> +#define NULL_PTR_ERR MAKE_HRESULT(SEVERITY_ERROR, FACILITY_WIN32, RPC_X_NULL_REF_POINTER)
> -
>
> +WINE_DEFAULT_DEBUG_CHANNEL(mmdevapi);
> -
>
> +extern void sessions_lock(void) DECLSPEC_HIDDEN;
> +extern void sessions_unlock(void) DECLSPEC_HIDDEN;
> -
>
> +extern const IAudioClient3Vtbl AudioClient3_Vtbl;
> -
>
> +static inline struct audio_session_wrapper *impl_from_IAudioSessionControl2(IAudioSessionControl2 *iface)
> +{
> - return CONTAINING_RECORD(iface, struct audio_session_wrapper, IAudioSessionControl2_iface);
>
> +}
> -
>
> +static HRESULT WINAPI ASC_QueryInterface(IAudioSessionControl2 *iface, REFIID riid, void **ppv)

Actually while I'm nitpicking, how about:
```suggestion:-0+0
static HRESULT WINAPI control_QueryInterface(IAudioSessionControl2 *iface, REFIID riid, void **ppv)
```