[PATCH v2 1/4] winegstreamer: Introduce audio conversion transform.

[Derek Lesho]

On 12/1/20 3:00 PM, Zebediah Figura (she/her) wrote:
>> +static HRESULT WINAPI audio_converter_ProcessMessage(IMFTransform *iface, MFT_MESSAGE_TYPE message, ULONG_PTR param)
>> +{
>> +    FIXME("%p, %u.\n", iface, message);
>> +
>> +    return S_OK;
>> +}
> Why S_OK?
Because the media session sends some messages to the transform such as 
MFT_MESSAGE_NOTIFY_BEGIN_STREAMING and fails to play if S_OK isn't 
returned.  I take it you'd like me to actually implement this method 
instead.
> Why not CLSID_CResamplerMediaObject? In fact, we already have one
> application that needs it (bug 47781).
The only reason I didn't expose it as CLSID_CResamplerMediaObject is 
that I didn't want to imply that I was basing the interface and types 
supported off of that specific object.  For instance, that object, from 
what I can see on the MSDN, doesn't support PCM<->Float conversions, and 
vice versa.  Is this not a big enough deal to keep it separate?