Huw Davies (@huw) commented about dlls/mmdevapi/client.c:
WINE_UNIX_CALL(is_format_supported, ¶ms);
- if (params.result == S_FALSE)
- if (params.result == S_FALSE && out)
I believe this is unnecessary as the driver should only return `S_FALSE` in the `AUDCLNT_SHAREMODE_SHARED` case where `out` will be non-NULL. However, it would be a good idea to move the parameter checks that are currently done in the first few lines of each driver into `mmdevapi` - @davidebeatrici something to add to the list (not really a priority though).