[PATCH v4 3/3] winegstreamer: Implement IMFMediaStream::GetStreamDescriptor.

Derek Lesho dlesho at codeweavers.com
Thu Sep 24 18:48:27 CDT 2020


On 9/24/20 6:42 PM, Zebediah Figura wrote:

>
> On 9/24/20 3:56 PM, Derek Lesho wrote:
>> On 9/24/20 12:27 PM, Zebediah Figura wrote:
>>>>        enum
>>>>        {
>>>>            STREAM_STUB,
>>>> +        STREAM_INACTIVE,
>>> What's the difference between these two states? There doesn't seem to be
>>> a difference in this patch.
>> I think it's just a vestige of a prior version of the patchset, I'll
>> merge STREAM_STUB into STREAM_INACTIVE.
> Are there other members of this enumeration that will be introduced, or
> can it become a simple boolean flag?
There's one more state I have in future patches, STREAM_RUNNING. [1]  I 
presume STREAM_PAUSED will also be necessary once pausing support is 
added.  I am open to turning these into simple boolean flags if you 
think that's better.

1: 
https://github.com/Guy1524/wine/blob/mfplat_cleanup/dlls/winegstreamer/media_source.c#L60





More information about the wine-devel mailing list