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

[Zebediah Figura]

On 9/25/20 9:49 AM, Derek Lesho wrote:
> @@ -775,6 +812,19 @@ static HRESULT media_source_constructor(IMFByteStream *bytestream, struct media_
>      }
>  
>      WaitForSingleObject(object->no_more_pads_event, INFINITE);
> +    for (i = 0; i < object->stream_count; i++)
> +    {
> +        GstSample *preroll;
> +        g_signal_emit_by_name(object->streams[i]->appsink, "pull-preroll", &preroll);
> +        hr = E_FAIL;

This statement is superfluous now.

> +        if (FAILED(hr = media_stream_init_desc(object->streams[i])))
> +        {
> +            ERR("Failed to finish initialization of media stream %p, hr %x.\n", object->streams[i], hr);
> +            IMFMediaStream_Release(&object->streams[i]->IMFMediaStream_iface);
> +            goto fail;
> +        }
> +        gst_sample_unref(preroll);
> +    }
>  
>      object->state = SOURCE_STOPPED;
>  

With that addressed, I think I have no further concerns regarding this
patch series; I'll leave it to Nikolay to undo all of my review ;-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20200928/d32c7a3e/attachment.sig>