Re: [PATCH 0/8] MR8705: d3dx10: Exclusively use shared code to implement get_image_info().

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx10_43/tests/d3dx10.c:
> +        DXGI_FORMAT format;
> +        D3D10_RESOURCE_DIMENSION resource_dimension;
> +    };
> +    static const struct
> +    {
> +        uint32_t flags;
> +        uint32_t width;
> +        uint32_t height;
> +        uint32_t depth;
> +        uint32_t row_pitch;
> +        uint32_t mip_levels;
> +        uint32_t caps;
> +        uint32_t caps2;
> +        struct expected expected;
> +        uint32_t pixel_data_size;
> +        BOOL todo_hr;
> Kind of a nitpick and I could see arguments against it, but I'd put pixel_data_size (an input to `D3DX10GetImageInfoFromMemory()`) before the expected outputs.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8705#note_112442