Re: [PATCH 0/2] MR3457: wine{alsa,pulse}: Move GetPropValue into mmdevapi.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/mmdevapi/devenum.c:
>  PROPVARIANT pv;
> - char *dev_name;
> - unsigned int size = 0;
> -
> - TRACE("%s, (%s,%lu)\n", wine_dbgstr_guid(id), wine_dbgstr_guid(&prop->fmtid), prop->pid);
> - if (!drvs.pGetPropValue)
> -
>    return E_NOTIMPL;
> - if (!drvs.pget_device_name_from_guid(id, &dev_name, &params.flow))
> -
>    return E_FAIL;
>
> - hr = drvs.pGetPropValue(id, prop, &pv);
> - params.device      = dev_name;
> - params.guid        = id;
> - params.prop        = prop;
> - params.result      = E_NOTIMPL;

It would be cleaner to add stub `get_prop_value()` implementations to the drivers that don't currently have it.