ddraw: Use SetRectEmpty() instead of open coding it.

Michael Stefaniuc mstefani at redhat.de
Thu Mar 3 02:26:06 CST 2016


Signed-off-by: Michael Stefaniuc <mstefani at redhat.de>
---
Inspired by one of Henri's patch reviews on wine-devel.


 dlls/ddraw/surface.c   | 2 +-
 dlls/ddraw/tests/d3d.c | 5 +----
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c
index e9c9c65..d133e7e 100644
--- a/dlls/ddraw/surface.c
+++ b/dlls/ddraw/surface.c
@@ -1436,7 +1436,7 @@ static HRESULT ddraw_surface_blt_clipped(struct ddraw_surface *dst_surface, cons
     }
     else
     {
-        SetRect(&src_rect, 0, 0, 0, 0);
+        SetRectEmpty(&src_rect);
         wined3d_src_texture = NULL;
         src_sub_resource_idx = 0;
     }
diff --git a/dlls/ddraw/tests/d3d.c b/dlls/ddraw/tests/d3d.c
index 06f16f3..d877020 100644
--- a/dlls/ddraw/tests/d3d.c
+++ b/dlls/ddraw/tests/d3d.c
@@ -1832,10 +1832,7 @@ static void DeviceLoadTest(void)
 
     /* First test some broken coordinates. */
     loadpoint.x = loadpoint.y = 0;
-    loadrect.left = 0;
-    loadrect.top = 0;
-    loadrect.right = 0;
-    loadrect.bottom = 0;
+    SetRectEmpty(&loadrect);
     hr = IDirect3DDevice7_Load(lpD3DDevice, texture_levels[1][0], &loadpoint, texture_levels[0][0], &loadrect, 0);
     ok(hr==DDERR_INVALIDPARAMS, "IDirect3DDevice7_Load returned: %x\n",hr);
 
-- 
2.5.0



More information about the wine-patches mailing list