Commit 82029cba authored by Matteo Bruni's avatar Matteo Bruni Committed by Alexandre Julliard

d3dx9/tests: Add a comment about the negation in boolean constant checks.

parent 26175489
...@@ -4350,6 +4350,8 @@ static void test_effect_preshader_compare_pbool_consts_(unsigned int line, IDire ...@@ -4350,6 +4350,8 @@ static void test_effect_preshader_compare_pbool_consts_(unsigned int line, IDire
{ {
for (i = 0; i < ARRAY_SIZE(test_effect_preshader_bconsts); ++i) for (i = 0; i < ARRAY_SIZE(test_effect_preshader_bconsts); ++i)
{ {
/* The negation on both sides is actually needed, sometimes you
* get 0xffffffff instead of 1 on native. */
ok_(__FILE__, line)(!bdata[i] == !test_effect_preshader_bconsts[i], ok_(__FILE__, line)(!bdata[i] == !test_effect_preshader_bconsts[i],
"Pixel shader boolean constants do not match, expected %#x, got %#x, i %u.\n", "Pixel shader boolean constants do not match, expected %#x, got %#x, i %u.\n",
test_effect_preshader_bconsts[i], bdata[i], i); test_effect_preshader_bconsts[i], bdata[i], i);
...@@ -4362,6 +4364,8 @@ static void test_effect_preshader_compare_pbool_consts_(unsigned int line, IDire ...@@ -4362,6 +4364,8 @@ static void test_effect_preshader_compare_pbool_consts_(unsigned int line, IDire
if (const_updated_mask[i / TEST_EFFECT_BITMASK_BLOCK_SIZE] if (const_updated_mask[i / TEST_EFFECT_BITMASK_BLOCK_SIZE]
& (1u << (i % TEST_EFFECT_BITMASK_BLOCK_SIZE))) & (1u << (i % TEST_EFFECT_BITMASK_BLOCK_SIZE)))
{ {
/* The negation on both sides is actually needed, sometimes
* you get 0xffffffff instead of 1 on native. */
ok_(__FILE__, line)(!bdata[i] == !test_effect_preshader_bconsts[i], ok_(__FILE__, line)(!bdata[i] == !test_effect_preshader_bconsts[i],
"Pixel shader boolean constants do not match, expected %#x, got %#x, i %u, parameter %s.\n", "Pixel shader boolean constants do not match, expected %#x, got %#x, i %u, parameter %s.\n",
test_effect_preshader_bconsts[i], bdata[i], i, updated_param); test_effect_preshader_bconsts[i], bdata[i], i, updated_param);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment