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.