d3dx9: Add ID3DXSprite tests (review please)

Tony Wasserka tony.wasserka at freenet.de
Sun Nov 16 05:47:38 CST 2008


This is my first test, so I think it's better to ask for feedback on 
wine-devel, first.

There are some points, where I wasn't sure what to do:
line 26: Is using inline okay here? IIRC that's C++ only, but other 
tests do it the same way...
line 82: Is it legal to demand A8R8G8B8 support here? Otherwise, how 
would I do this without making the code overcomplicated? For what other 
formats should I check then?
line 120-123: I hope it's okay to use hardcoded matrix values here, as 
the point of that test is only to make sure that the same matrix gets 
returned...
line 215-253:
    1) Is it okay to initialize D3D that way?
    2) Can I simplify the initialization even further?

Also, I'd like to know wether the code in general is clean enough. Any 
other feedback is welcome, too.
Best regards,
    Tony



(Note1: This test only finishes successfully if my recent ID3DXSprite 
patch series is applied, but I already had all of them pass/todo'ed.
 Note2: visual tests will follow later, this is just for the basic 
functionality)

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-d3dx9-Add-ID3DXSprite-tests.patch
Url: http://www.winehq.org/pipermail/wine-devel/attachments/20081116/724b4c43/attachment.txt 


More information about the wine-devel mailing list