Re: [PATCH 0/1] MR8381: winegstreamer: Support the Indeo 5.0 format in DirectShow.

[Alfred Agrell (＠Alcaro)]

Alfred Agrell (@Alcaro) commented about dlls/winegstreamer/quartz_parser.c:
> - const VIDEOINFOHEADER *video_format = (const VIDEOINFOHEADER *)mt->pbFormat;
> -
> - if (!IsEqualGUID(&mt->formattype, &FORMAT_VideoInfo))
> - {
> -
>    FIXME("Unknown format type %s.\n", debugstr_guid(&mt->formattype));
> -
>    return false;
> - }
> - if (mt->cbFormat < sizeof(VIDEOINFOHEADER) || !mt->pbFormat)
> - {
> -
>    ERR("Unexpected format size %lu.\n", mt->cbFormat);
> -
>    return false;
> - }
> -
> - format->major_type = WG_MAJOR_TYPE_VIDEO_CINEPAK;
> - if (IsEqualGUID(&mt->subtype, &MEDIASUBTYPE_IV50))
> -
>    format->u.video.version = 5;

Looks a little odd how this one doesn't have an else clause with a fixme... but this entire function is only reached for IV50, so your choice.