Re: [PATCH v3 0/6] MR7715: Implement dynamic formats in directdraw media stream.

[Anton Baskanov (＠baskanov)]

Anton Baskanov (@baskanov) commented about dlls/amstream/ddrawstream.c:
> LeaveCriticalSection(&stream->cs);
>              return hr;
>          }
> -        old_peer = stream->peer;
> -        IPin_AddRef(old_peer);
>
> -        IFilterGraph_Disconnect(stream->graph, stream->peer);
> -        IFilterGraph_Disconnect(stream->graph, &stream->IPin_iface);
> -        hr = IFilterGraph_ConnectDirect(stream->graph, old_peer, &stream->IPin_iface, NULL);
> -        if (FAILED(hr))
> +        set_mt_from_desc(&stream->mt, format);

I think it would be better to do this after `IPin_QueryAccept()` returns `S_OK` and call `CopyMediaType()`/`FreeMediaType()` only on the reconnection path.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7715#note_100228