Re: [PATCH v2 0/5] MR9089: d3dx10: Add partial support for D3DX10_IMAGE_LOAD_INFO structure fields.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx10_43/tests/d3dx10.c:
>  BOOL line_match;
>  HRESULT hr;
>
> - readback = get_texture2d_readback(texture);
> - readback = get_texture2d_readback_iface(texture);

Instead of introducing another, separate texture readback and check mechanism, maybe it makes sense to get rid of the old functions and make `check_resource_data()` (or `check_texture2d_data()` + `check_texture3d_data()`) use the new `resource_readback` machinery. Probably as its own patch at the start of the MR, no new tests, just plugging in `resource_readback` in place of the existing (`_iface`) stuff.