Re: [PATCH 3/3] d3dcompiler/tests: Use winetest_push_context().

[Matteo Bruni]

On Sun, Apr 25, 2021 at 10:10 PM Zebediah Figura  wrote:
> Signed-off-by: Zebediah Figura
>
> This is an example where the nested context support is particularly helpful.
>
> dlls/d3dcompiler_43/tests/hlsl_d3d11.c | 160 ++++++++++++++-----------
>  1 file changed, 91 insertions(+), 69 deletions(-)
>
> diff --git a/dlls/d3dcompiler_43/tests/hlsl_d3d11.c b/dlls/d3dcompiler_43/tests/hlsl_d3d11.c
> index 53dbd3e2df7..36716ed753d 100644
> --- a/dlls/d3dcompiler_43/tests/hlsl_d3d11.c
> +++ b/dlls/d3dcompiler_43/tests/hlsl_d3d11.c
> @@ -621,6 +621,8 @@ static void test_sampling(void)
>  for (i = 0; i < ARRAY_SIZE(tests); ++i)
>  {
> -
>    winetest_push_context("Test %u", i);
> -
>    ID3D11DeviceContext_ClearRenderTargetView(test_context.immediate_context, test_context.rtv, red);
>    todo_wine ps_code = compile_shader_flags(tests[i], "ps_4_0", D3DCOMPILE_ENABLE_BACKWARDS_COMPATIBILITY);
>    if (ps_code)

Not a proper review yet, especially of patches 1 and 2, but I
certainly like this patch series in principle.