Re: [PATCH 1/4] ddraw/tests: New CreateSurface for DDSCAPS_BACKBUFFER tests.

[Stefan Dösinger]

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.
> - 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.