Re: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-25 Thread Dmitry Timoshkov
"Maarten Lankhorst" <[EMAIL PROTECTED]> wrote: >> Perhaps there is something less hacky way like GdiFlush() which is more >> appropriate for this. > I created a patch that does GdiFlush and waits up to .5 seconds after > a resolution change, are you ok with that? Perhaps you could investigate f

Re: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-25 Thread Maarten Lankhorst
2008/2/13, Dmitry Timoshkov <[EMAIL PROTECTED]>: > "Maarten Lankhorst" <[EMAIL PROTECTED]> wrote: > > > This makes the tests pass on my pc. > > Perhaps there is something less hacky way like GdiFlush() which is more > appropriate for this. I created a patch that does GdiFlush and waits up to .5

Re: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-25 Thread Maarten Lankhorst
Hi Dmitry, 2008/2/13, Dmitry Timoshkov <[EMAIL PROTECTED]>: > "Maarten Lankhorst" <[EMAIL PROTECTED]> wrote: > > > This makes the tests pass on my pc. > > Perhaps there is something less hacky way like GdiFlush() which is more > appropriate for this. I tested GdiFlush to do an XSync and XFlush,

Re: d3d: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-13 Thread Peter Dons Tychsen
OK. I still think he should put a FIXME remark in the test-case, and document why Sleep() is needed. Someone might fix the driver someday. The only reason i yelled "alert!", is that i have seen so much bad code which has come to a state where the are Sleep(143) here and Sleep(1332) there, where n

Re: d3d: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-13 Thread Stefan Dösinger
Am Mittwoch, 13. Februar 2008 03:11:24 schrieb Peter Dons Tychsen: > Eh, this does not seem like a proper way to get a unit test to pass. > > Sleep()... that leads to the dark side! :-) I talked to Maarten on IRC about this. The problem seems to be that his monitor takes ages to mode switch, but C

Re: d3d: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-13 Thread Kai Blin
On Wednesday 13 February 2008 03:11:24 Peter Dons Tychsen wrote: > The string "This makes the tests pass on my pc" isn't very good either. > What system? Given that the default mode for the tests in the last years has been "Needs to pass on Alexandre's system", I don't think it's that bad. ;) C

Re: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-13 Thread Dmitry Timoshkov
"Maarten Lankhorst" <[EMAIL PROTECTED]> wrote: > This makes the tests pass on my pc. Perhaps there is something less hacky way like GdiFlush() which is more appropriate for this. -- Dmitry.

Re: d3d: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-12 Thread Peter Dons Tychsen
Eh, this does not seem like a proper way to get a unit test to pass. Sleep()... that leads to the dark side! :-) Why does it need a Sleep()? If its failing on Wine, then shouldn't it be fixed in the libs? And if its on orig-winxp-vista then there must be a better way to wait? Or no? The string "

Re: d3d: Add a slight delay before testing the ddraw and d3d9 visual tests

2008-02-12 Thread Stefan Dösinger
Am Mittwoch, 13. Februar 2008 02:25:30 schrieb Maarten Lankhorst: > This makes the tests pass on my pc. d3d8 might need it as well