Re: [PATCH v13 0/7] MR3938: winegstreamer: Implement CLSID_CMpegVideoCodec

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/wg_format.c:
> return NULL;
>
> gst_video_info_set_format(&info, video_format, format->u.video.width, abs(format->u.video.height));
> +    info.fps_n = format->u.video.fps_n;
> +    info.fps_d = format->u.video.fps_d;
> +    if (!format->u.video.fps_d && !format->u.video.fps_n)
> +        info.fps_d = 1;

This may be correct (although I'm curious if it actually matters here?) but should be a separate patch.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3938#note_48901