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

[Rémi Bernon (＠rbernon)]

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?