d3d8/tests: make sure to use return values (LLVM/Clang)

[Stefan Dösinger]

On Tuesday 26 July 2011 17:33:54 Jacek Caban wrote:
> On 07/26/11 17:22, Henri Verbeet wrote:
> > No, I disagree with the basic idea that stricter tests are necessarily
> > better. The advantage of SUCCEEDED is that it's *less* strict.
> 
> We won't agree on that.
I prefer comparison against S_OK(or D3D_OK, which is just S_OK) for the ok() 
lines and SUCCEEDED/FAILED for figuring out control flow inside the tests. At 
least until a Windows version or driver returns a success value other than 
D3D_OK.

But either way that's not a strong opinion. I am fine with either way, and I 
think I've used SUCCEEDED/FAILED myself for ok() statements mostly because 
Henri has used it and to keep the code consistent.
-------------- 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/20110726/a736e53b/attachment.pgp>