Only report the first failed row for each bias clamp test.
Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
---
It was established that ok() does not print anything when successful so
this patch actually works (which I can confirm from my tests). So I'm
resubmitting.
dlls/d3d10core/tests/d3d10core.c | 12 ++++++++----
dlls/d3d11/tests/d3d11.c | 12 ++++++++----
2 files changed, 16 insertions(+), 8 deletions(-)
diff --git a/dlls/d3d10core/tests/d3d10core.c b/dlls/d3d10core/tests/d3d10core.c
index f799922ea28..b0ff09d3697 100644
--- a/dlls/d3d10core/tests/d3d10core.c
+++ b/dlls/d3d10core/tests/d3d10core.c
@@ -15765,6 +15765,7 @@ static void test_depth_bias(void)
for (k = 0; k < ARRAY_SIZE(bias_clamp_tests); ++k)
{
+ BOOL all_match = TRUE;
rasterizer_desc.DepthBiasClamp = bias_clamp_tests[k];
ID3D10Device_CreateRasterizerState(device, &rasterizer_desc, &rs);
ok(SUCCEEDED(hr), "Failed to create rasterizer state, hr %#x.\n", hr);
@@ -15775,21 +15776,23 @@ static void test_depth_bias(void)
m = quad_slopes[i] / texture_desc.Height;
bias = clamp_depth_bias(rasterizer_desc.SlopeScaledDepthBias * m, rasterizer_desc.DepthBiasClamp);
get_texture_readback(texture, 0, &rb);
- for (y = 0; y < texture_desc.Height; ++y)
+ for (y = 0; y < texture_desc.Height && all_match; ++y)
{
depth = min(max(0.0f, depth_values[y] + bias), 1.0f);
switch (format)
{
case DXGI_FORMAT_D32_FLOAT:
data = get_readback_float(&rb, 0, y);
- ok(compare_float(data, depth, 64),
+ all_match = compare_float(data, depth, 64);
+ ok(all_match,
"Got depth %.8e, expected %.8e.\n", data, depth);
break;
case DXGI_FORMAT_D24_UNORM_S8_UINT:
u32 = get_readback_data(&rb, 0, y, sizeof(*u32));
u32_value = *u32 >> shift;
expected_value = depth * 16777215.0f + 0.5f;
- ok(abs(u32_value - expected_value) <= 3,
+ all_match = abs(u32_value - expected_value) <= 3;
+ ok(all_match,
"Got value %#x (%.8e), expected %#x (%.8e).\n",
u32_value, u32_value / 16777215.0f,
expected_value, expected_value / 16777215.0f);
@@ -15797,7 +15800,8 @@ static void test_depth_bias(void)
case DXGI_FORMAT_D16_UNORM:
u16 = get_readback_data(&rb, 0, y, sizeof(*u16));
expected_value = depth * 65535.0f + 0.5f;
- ok(abs(*u16 - expected_value) <= 1,
+ all_match = abs(*u16 - expected_value) <= 1;
+ ok(all_match,
"Got value %#x (%.8e), expected %#x (%.8e).\n",
*u16, *u16 / 65535.0f, expected_value, expected_value / 65535.0f);
break;
diff --git a/dlls/d3d11/tests/d3d11.c b/dlls/d3d11/tests/d3d11.c
index 3d6cbf57e20..2850d59757e 100644
--- a/dlls/d3d11/tests/d3d11.c
+++ b/dlls/d3d11/tests/d3d11.c
@@ -25681,6 +25681,7 @@ static void test_depth_bias(void)
for (k = 0; k < ARRAY_SIZE(bias_clamp_tests); ++k)
{
+ BOOL all_match = TRUE;
rasterizer_desc.DepthBiasClamp = bias_clamp_tests[k];
ID3D11Device_CreateRasterizerState(device, &rasterizer_desc, &rs);
ok(SUCCEEDED(hr), "Failed to create rasterizer state, hr %#x.\n", hr);
@@ -25691,21 +25692,23 @@ static void test_depth_bias(void)
m = quad_slopes[i] / texture_desc.Height;
bias = clamp_depth_bias(rasterizer_desc.SlopeScaledDepthBias * m, rasterizer_desc.DepthBiasClamp);
get_texture_readback(texture, 0, &rb);
- for (y = 0; y < texture_desc.Height; ++y)
+ for (y = 0; y < texture_desc.Height && all_match; ++y)
{
depth = min(max(0.0f, depth_values[y] + bias), 1.0f);
switch (format)
{
case DXGI_FORMAT_D32_FLOAT:
data = get_readback_float(&rb, 0, y);
- ok(compare_float(data, depth, 64),
+ all_match = compare_float(data, depth, 64);
+ ok(all_match,
"Got depth %.8e, expected %.8e.\n", data, depth);
break;
case DXGI_FORMAT_D24_UNORM_S8_UINT:
u32 = get_readback_data(&rb, 0, y, 0, sizeof(*u32));
u32_value = *u32 >> shift;
expected_value = depth * 16777215.0f + 0.5f;
- ok(abs(u32_value - expected_value) <= 3,
+ all_match = abs(u32_value - expected_value) <= 3;
+ ok(all_match,
"Got value %#x (%.8e), expected %#x (%.8e).\n",
u32_value, u32_value / 16777215.0f,
expected_value, expected_value / 16777215.0f);
@@ -25713,7 +25716,8 @@ static void test_depth_bias(void)
case DXGI_FORMAT_D16_UNORM:
u16 = get_readback_data(&rb, 0, y, 0, sizeof(*u16));
expected_value = depth * 65535.0f + 0.5f;
- ok(abs(*u16 - expected_value) <= 1,
+ all_match = abs(*u16 - expected_value) <= 1;
+ ok(all_match,
"Got value %#x (%.8e), expected %#x (%.8e).\n",
*u16, *u16 / 65535.0f, expected_value, expected_value / 65535.0f);
break;
--
2.20.1