Hi Stefan,
On Thursday 22 July 2010 22:05:21 Stefan Dösinger wrote:
Am 22.07.2010 um 21:13 schrieb Oldřich Jedlička:
- hr = IDirectDraw_QueryInterface(lpDD, &IID_IDirectDraw, (void **)
&dd); + ok(SUCCEEDED(hr), "IDirectDraw_QueryInterface failed: 0x%08x\n", hr);
This is redundant, lpDD is already an IDirectDraw interface.
You are right, I thought it is some default (=latest, V7) interface. I will change it.
- hr = IDirectDraw_CreateSurface(dd, &ddsd, &surf, NULL);
- ok(SUCCEEDED(hr), "IDirectDraw_CreateSurface failed: 0x%08x\n", hr);
This test will fail before your 3/4 patch is in, so you'll have to use todo_wine for now, and remove in the patch that implements the correct behavior. You'll have to handle the NULL pointer that is returned in case of a failure.
The same is for my 2/4. I will add the todo_wine in 1/4 and 2/4 and remove them in 3/4 and 4/4.
Which NULL pointer do you mean? Does the todo_wine mean that the test reports problem and just continues (so that surf is NULL)?
Thanks, Oldrich.