Added unbound sampler test for pixel shaders.

[Stefan Dösinger]

On Tuesday 31 May 2011 17:48:14 Sean Sube wrote:
> +    static const DWORD ps_code[] = 
> +    {
> +        0xFFFF0200, 0x0200001F, 0x90000000, 0xA00F0800, 
> +        0x0200001F, 0x80000000, 0xB00F0000, 0x03000042, 
> +        0x800F0000, 0xB0E40000, 0xA0E40000, 0x02000001,
> +        0x800F0800, 0x80E40000, 0x0000FFFF,
> +    };
Please write the assembler code into comments, this makes the test easier to 
read.

> +    hr = IDirect3DDevice9_GetPixelShader(device, &old_ps);
> +    ok(SUCCEEDED(hr), "IDirect3DDevice9_GetPixelShader failed, hr %#x.\n", 
hr);
The individual tests should unset the pixel shader, if old_ps is not NULL here 
that's a bug.

> +        color = getPixelColorFromSurface(rt, 32, 32);
> +        ok(color == 0xFF000000, "Color is %#x.\n", color);
My understanding is that the test currently fails, so you should mark it as 
todo:

todo_wine ok(...);

> +    hr = IDirect3DDevice9_CreatePixelShader(device, ps_code, &ps);
> +    ok(SUCCEEDED(hr), "IDirect3DDevice9_CreatePixelShader failed, hr %#x.
\n", hr);
Your test must check is PS 2.0 shaders are supported. We already have a pixel 
shader >= 2.0 supported block in the START_TEST(visual) function, please call 
your test function inside it. (e.g. after dp2add_ps_test)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110531/30b4d8d2/attachment.pgp>