[PATCH v4 0/1] MR2771: winestreamer: Don't set twice the same format in stream description.
Prior to 98d209752cee9abd8dc31dfe1f28811066b0b83f, the base type format wasn't inserted twice in stream format description. An application (Idol Showdown) fails to play its intro video when the base type format is duplicated. Signed-off-by: Eric Pouech <epouech(a)codeweavers.com> -- v4: winegstreamer: Fix typo. https://gitlab.winehq.org/wine/wine/-/merge_requests/2771
From: Eric Pouech <epouech(a)codeweavers.com> Introduced in 98d209752cee9abd8dc31dfe1f28811066b0b83f. An application (Idol Showdown) fails to play its intro video without this change. Signed-off-by: Eric Pouech <epouech(a)codeweavers.com> --- dlls/winegstreamer/media_source.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/winegstreamer/media_source.c b/dlls/winegstreamer/media_source.c index 335b617becd..41271585454 100644 --- a/dlls/winegstreamer/media_source.c +++ b/dlls/winegstreamer/media_source.c @@ -947,7 +947,7 @@ static HRESULT media_stream_init_desc(struct media_stream *stream) if (FAILED(hr = MFCreateMediaType(&iyuv_type))) goto done; - if (FAILED(hr = IMFMediaType_CopyAllItems(iyuv_type, (IMFAttributes *)iyuv_type))) + if (FAILED(hr = IMFMediaType_CopyAllItems(new_type, (IMFAttributes *)iyuv_type))) goto done; if (FAILED(hr = IMFMediaType_SetGUID(iyuv_type, &MF_MT_SUBTYPE, &MFVideoFormat_IYUV))) goto done; -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/2771
On Wed May 10 14:58:42 2023 +0000, Zebediah Figura wrote:
V3 pushed to correct path title. Amusingly, that's actually not even the one I was thinking of ("winestreamer" instead of "winegstreamer"). ok, fixed also (V4)
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/2771#note_32442
participants (2)
-
Eric Pouech -
eric pouech (@epo)