Re: [PATCH 0/1] MR5157: evr/dshow: List NV12 as supported, improve warnings

[Loïc Rebmeister]

Loïc Rebmeister (@Fox2Code) commented about dlls/evr/evr.c:
> {
>          width = (3 * width + 3) & ~3;
>      }
> -    else
> +    else if (IsEqualGUID(subtype, &MFVideoFormat_ARGB32)
> +                || IsEqualGUID(subtype, &MFVideoFormat_RGB32))
>      {
>          width *= 4;
>      }
> +    else
> +    {
> +        FIXME("unsupported video format %s\n", debugstr_guid(subtype));

This change behavior and might introduce regression due to `width *= 4;` not being called.

Adding `width *= 4;` as a default assumption like before is probably better.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5157#note_62815