Re: [PATCH 0/6] MR5999: d3dx9: Miscellaneous fixes.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/surface.c:
> -
>    uint32_t row_pitch;
> -
>    uint32_t mip_levels;
> -
>    uint32_t caps;
> -
>    uint32_t caps2;
> -
>    struct
> -
>    {
> -
>        HRESULT hr;
> -
>        uint32_t width;
> -
>        uint32_t height;
> -
>        uint32_t depth;
> -
>        uint32_t mip_levels;
> -
>        D3DRESOURCETYPE resource_type;
> -
>    }
> -
>    expected;
> -
>    BOOL todo;
> -
>    uint32_t pixel_data_size;

I'd swap these two, so that you can leave out initializing the `todo` field instead of explicitly setting it to `false`.