[PATCH 0/1] MR2570: winegstreamer: Fix check for non-zero padding in h264 transform.
IsRectEmpty() doesn't do the right thing as padding is not a rectangle. E. g., for rect (0, 0, 0, 8) IsRectEmpty() will return TRUE while the padding is apparently not empty. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2570
From: Paul Gofman <pgofman(a)codeweavers.com> --- dlls/winegstreamer/h264_decoder.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/winegstreamer/h264_decoder.c b/dlls/winegstreamer/h264_decoder.c index 439701098ad..7d9c38837d3 100644 --- a/dlls/winegstreamer/h264_decoder.c +++ b/dlls/winegstreamer/h264_decoder.c @@ -183,7 +183,8 @@ static HRESULT fill_output_media_type(struct h264_decoder *decoder, IMFMediaType } if (FAILED(hr = IMFMediaType_GetItem(media_type, &MF_MT_MINIMUM_DISPLAY_APERTURE, NULL)) - && !IsRectEmpty(&wg_format->u.video.padding)) + && (wg_format->u.video.padding.left || wg_format->u.video.padding.right || wg_format->u.video.padding.top + || wg_format->u.video.padding.bottom)) { MFVideoArea aperture = { -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/2570
This merge request was approved by Rémi Bernon. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2570
participants (3)
-
Paul Gofman -
Paul Gofman (@gofman) -
Rémi Bernon