Re: [PATCH 0/8] MR6568: d3dx9: Pixel format handling changes, part 2.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/surface.c:
> break;
>              }
>          }
> +        else if (dst_ctype == CTYPE_LUMA)
> +        {
> +            *dst_component = dst->value.x;

This only works if we guarantee that all luma formats have a "red" component. Which is certainly expected to be the case, but...

I guess I'd like to see that guarantee enforced e.g. with an `assert(format->bits[1]);` here.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6568#note_84093