[PATCH 2/5] ddraw/tests: Check some more return values (LLVM/Clang).

Henri Verbeet hverbeet at codeweavers.com
Fri Oct 16 03:38:40 CDT 2009


---
 dlls/ddraw/tests/d3d.c      |    5 +++++
 dlls/ddraw/tests/dsurface.c |    1 +
 dlls/ddraw/tests/visual.c   |    2 ++
 3 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/dlls/ddraw/tests/d3d.c b/dlls/ddraw/tests/d3d.c
index d9b1c72..9104147 100644
--- a/dlls/ddraw/tests/d3d.c
+++ b/dlls/ddraw/tests/d3d.c
@@ -961,6 +961,7 @@ static BOOL D3D1_createObjects(void)
     ddsd.dwWidth = 256;
     ddsd.dwHeight = 256;
     hr = IDirectDraw_CreateSurface(DirectDraw1, &ddsd, &Surface1, NULL);
+    ok(SUCCEEDED(hr), "IDirectDraw_CreateSurface returned %#x.\n", hr);
     if (!Surface1) {
         skip("DDSCAPS_3DDEVICE surface not available\n");
         return FALSE;
@@ -1388,6 +1389,7 @@ static void Direct3D1Test(void)
     vp_data.dvScaleX = 1;
     vp_data.dvScaleY = 1;
     hr = IDirect3DViewport_SetViewport(Viewport, &vp_data);
+    ok(SUCCEEDED(hr), "IDirect3DViewport_SetViewport returned %#x.\n", hr);
     i = 12345;
     hr = IDirect3DViewport_TransformVertices(Viewport, sizeof(offscreentest) / sizeof(offscreentest[0]),
                                              &transformdata, D3DTRANSFORM_CLIPPED,
@@ -1397,6 +1399,7 @@ static void Direct3D1Test(void)
     vp_data.dwWidth = 256;
     vp_data.dwHeight = 256;
     hr = IDirect3DViewport_SetViewport(Viewport, &vp_data);
+    ok(SUCCEEDED(hr), "IDirect3DViewport_SetViewport returned %#x.\n", hr);
     i = 12345;
     hr = IDirect3DViewport_TransformVertices(Viewport, sizeof(offscreentest) / sizeof(offscreentest[0]),
                                              &transformdata, D3DTRANSFORM_CLIPPED,
@@ -2598,10 +2601,12 @@ static void DeviceLoadTest(void)
             memset(&ddsd, 0, sizeof(DDSURFACEDESC2));
             ddsd.dwSize = sizeof(ddsd);
             hr = IDirectDrawSurface7_GetSurfaceDesc(texture_levels[0][i1], &ddsd);
+            ok(SUCCEEDED(hr), "IDirectDrawSurface7_GetSurfaceDesc returned %#x.\n", hr);
 
             memset(&ddsd2, 0, sizeof(DDSURFACEDESC2));
             ddsd2.dwSize = sizeof(ddsd2);
             hr = IDirectDrawSurface7_GetSurfaceDesc(texture_levels[1][i], &ddsd2);
+            ok(SUCCEEDED(hr), "IDirectDrawSurface7_GetSurfaceDesc returned %#x.\n", hr);
 
             if (ddsd.dwWidth == ddsd2.dwWidth && ddsd.dwHeight == ddsd2.dwHeight)
             {
diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c
index 1feab9b..44be603 100644
--- a/dlls/ddraw/tests/dsurface.c
+++ b/dlls/ddraw/tests/dsurface.c
@@ -2433,6 +2433,7 @@ static void PrivateDataTest(void)
     ref2 = getref((IUnknown *) lpDD);
     ok(ref2 == ref + 1, "Object reference is %d, expected %d\n", ref2, ref + 1);
     hr = IDirectDrawSurface7_FreePrivateData(surface7, &IID_IDirectDrawSurface7);
+    ok(SUCCEEDED(hr), "IDirectDrawSurface7_FreePrivateData returned %#x.\n", hr);
     ref2 = getref((IUnknown *) lpDD);
     ok(ref2 == ref, "Object reference is %d, expected %d\n", ref2, ref);
 
diff --git a/dlls/ddraw/tests/visual.c b/dlls/ddraw/tests/visual.c
index 450d231..ac70b31 100644
--- a/dlls/ddraw/tests/visual.c
+++ b/dlls/ddraw/tests/visual.c
@@ -615,10 +615,12 @@ static void offscreen_test(IDirect3DDevice7 *device)
 
 out:
     hr = IDirect3DDevice7_SetTexture(device, 0, NULL);
+    ok(SUCCEEDED(hr), "IDirect3DDevice7_SetTexture returned %#x.\n", hr);
 
     /* restore things */
     if(backbuffer) {
         hr = IDirect3DDevice7_SetRenderTarget(device, backbuffer, 0);
+        ok(SUCCEEDED(hr), "IDirect3DDevice7_SetRenderTarget returned %#x.\n", hr);
         IDirectDrawSurface7_Release(backbuffer);
     }
     if(offscreen) {
-- 
1.6.4.4




More information about the wine-patches mailing list