[PATCH 1/5] d3d9/tests: Test GetData behavior with various size values. (try 2)

Henri Verbeet hverbeet at gmail.com
Mon May 5 06:49:24 CDT 2014


On 2 May 2014 16:30, Matteo Bruni <mbruni at codeweavers.com> wrote:
> +    memset(&data, 0xff, sizeof(data));
This is a bit of an odd way to write "data = ~0u;".

> +    ok(*(WORD *)&data == 1 || broken(*(WORD *)&data != 1),
> +            "Occlusion query returned an unexpected result (0x%.8x).\n", data);
> +    todo_wine ok(*((WORD *)&data + 1) == 0xffff,
> +            "data was modified outside of the expected size (0x%.8x).\n", data);
I think this is a bit awkward, perhaps it makes more sense the declare
"data" as something like "WORD data[4];". Also, can this really ever
return 0 for the query data? Would it help to draw a different
primitive type, instead of a tiny point? (Because, well, point
rasterization is complicated.)



More information about the wine-devel mailing list