Re: [PATCH v5 0/10] MR2908: wine{alsa,coreaudio,oss,pulse}: Move AudioClient's GetService into mmdevapi.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/winepulse.drv/mmdevdrv.c:
>      void **ppv)
>
> {
>      ACImpl *This = impl_from_IAudioClient3(iface);
> - BOOLEAN add_ref = TRUE;

I'm afraid I'm not a fan of this add_ref boolean.  I'd prefer something like the current alsa driver's code with perhaps the grouping of the three session interfaces like it's currently done in the pulse driver and the moving of `sessions_unlock()` so it's called unconditionally near the end.