Re: [PATCH 0/6] MR7711: d3dx9: Rework color key handling.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/surface.c:
> -
>    {
> -
>        const uint8_t *dst_expected_row = ((uint8_t *)tests[i].expected_dst_data) + (16 * y);
> -
>        const uint8_t *dst_row = ((uint8_t *)lock_rect.pBits) + (lock_rect.Pitch * y);
> -
> -
>        for (x = 0; x < 4; ++x)
> -
>        {
> -
>            const uint8_t *dst_expected_pixel = dst_expected_row + (4 * x);
> -
>            const uint8_t *dst_pixel = dst_row + (4 * x);
> -
> -
>            if (memcmp(dst_pixel, dst_expected_pixel, 4))
> -
>                mismatch_count++;
> -
>        }
> -
>    }
> -
>    IDirect3DSurface9_UnlockRect(surf);
> -
> -
>    todo_wine_if(tests[i].todo) ok(!mismatch_count, "Unexpected number of mismatched pixels %u.\n", mismatch_count);

Technically you're counting bytes on a 4 byte-per-pixel format, so the message isn't entirely correct :nerd: