1/8 has a whitespace error.
So it does. Thank you for pointing that out. That's the one formatting error I normally do catch.
3/8: Why do you need both of those macros?
I think my intention here was to highlight the different usage. I'm not convinced I need it - it's gone.
3/8: I suppose the other drivers do this too, but I'm not convinced it's useful.
I only included it because a precedent has been set. I'm happy to remove it if you think that is the right way to go?
7/8: That should depend on the sign of the height.
YUV is always top-down and Windows actually rejects a negative sign for the output (see the test [here](https://gitlab.winehq.org/wine/wine/-/blob/master/dlls/iyuv_32/tests/iyuv_32...)). So the output stride is always negative (as it's always a bottom-up RGB image). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10549#note_135163