Re: [PATCH 0/5] MR3938: winegstreamer: Implement CLSID_CMpegVideoCodec

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/wg_transform.c:
> switch (input_format.major_type)
>      {
> +        case WG_MAJOR_TYPE_VIDEO_MPEG1:
> +            if (!(element = create_element("mpegvideoparse", "bad"))
> +                    || !append_element(transform->container, element, &first, &last))
> +            {
> +                gst_caps_unref(raw_caps);
> +                goto out;
> +            }

We used to have a "h264parse" hardcoded too, but it ended up being apparently unnecessary. Maybe there's a better way here, like for instance use find_element to optionally add a parser element?

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