On 01/01/2008, Stefan Dösinger stefan@codeweavers.com wrote:
It is likely that Alexandre was waiting for an OK from me or Henri. He is the only committer, but when someone he doesn't know sends a patch he usually waits for the OK from someone who is maintaining the part of Wine. I think I forgot to send an OK mail to the test when you submitted it.
I didn't see anything obviously wrong with the test, except perhaps checking hr against D3D_OK, but ddraw is more your area.
- You don't have to set the render states unless you need a different setting
than the previous tests leave behind
I don't think it's a bad thing to do though... it's pretty useful the be able to comment out the other tests without breaking the current test, especially when writing new tests. In general, we might want to consider using a new device for each test to prevent states from one test interfering with those from another test.