Re: [PATCH v5 0/7] MR6360: d3dx9: Beginning of pixel format handling changes.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/surface.c:
> -
>    if (SUCCEEDED(hr))
> -
>    {
> -
>        const uint32_t src_pitch = get_bpp_for_d3dformat(tests[i].src_format) * tests[i].src_rect.right;
> -
>        const uint32_t dst_pitch = get_bpp_for_d3dformat(tests[i].dst_format) * tests[i].src_rect.right;
> -
>        IDirect3DSurface9 *surf;
> -
> -
>        hr = IDirect3DTexture9_GetSurfaceLevel(tex, 0, &surf);
> -
>        ok(hr == D3D_OK, "Failed to get the surface, hr %#lx.\n", hr);
> -
> -
>        hr = D3DXLoadSurfaceFromMemory(surf, NULL, NULL, tests[i].src_data, tests[i].src_format,
> -
>                src_pitch, tests[i].src_palette, &tests[i].src_rect, D3DX_FILTER_NONE, 0);
> -
>        todo_wine_if(tests[i].todo) ok(hr == D3D_OK, "Unexpected hr %#lx.\n", hr);
> -
>        if (SUCCEEDED(hr))
> -
>        {
> -
>            const uint32_t dst_fmt_bpp = get_bpp_for_d3dformat(tests[i].dst_format);
> -
>            uint32_t match_count, mismatch_count;

Similarly for `match_count`, is there any use for it?