Re: [PATCH 0/5] MR6776: d3dx9: D3DXSaveSurfaceToFileInMemory() refactoring, part 1.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/surface.c:
> -
>    uint32_t height;
> -
>    const PALETTEENTRY *palette;
> -
> -
>    HRESULT expected_hr;
> -
>    struct dds_pixel_format expected_pixel_format;
> -
>    uint32_t expected_flags;
> -
>    uint32_t expected_width;
> -
>    uint32_t expected_height;
> -
>    uint32_t expected_pitch;
> -
>    uint32_t expected_depth;
> -
>    uint32_t expected_mip_levels;
> -
>    uint32_t expected_caps;
> -
>    uint32_t expected_caps2;
> -
>    uint32_t expected_buffer_size;
> -
>    BOOL todo_hr;
> -
>    BOOL todo_expected;

An option would be to collect all of that into an `expected` structure, which would be easier to handle (e.g. to pass the data to `check_dds_header()`).