ddraw: Assign to structs instead of using memcpy

[Andrew Talbot]

Here, I am assuming that the "dwSize" elements in all these cases should be set
to the size of the struct each is in, respectively. Please advise if this
assumption is wrong.

Thanks,

-- Andy.
---
Changelog:
    ddraw: Assign to structs instead of using memcpy.

diff --git a/dlls/ddraw/device.c b/dlls/ddraw/device.c
index b60b6f3..03ccea8 100644
--- a/dlls/ddraw/device.c
+++ b/dlls/ddraw/device.c
@@ -704,7 +704,7 @@ IDirect3DDeviceImpl_1_CreateExecuteBuffer(IDirect3DDevice *iface,
     object->d3ddev = This;
 
     /* Initializes memory */
-    memcpy(&object->desc, Desc, Desc->dwSize);
+    object->desc = *Desc;
 
     /* No buffer given */
     if ((object->desc.dwFlags & D3DDEB_LPDATA) == 0)
diff --git a/dlls/ddraw/material.c b/dlls/ddraw/material.c
index d6b9a7c..8e17174 100644
--- a/dlls/ddraw/material.c
+++ b/dlls/ddraw/material.c
@@ -250,7 +250,7 @@ IDirect3DMaterialImpl_SetMaterial(IDirect3DMaterial3 *iface,
     /* Stores the material */
     EnterCriticalSection(&ddraw_cs);
     memset(&This->mat, 0, sizeof(This->mat));
-    memcpy(&This->mat, lpMat, lpMat->dwSize);
+    This->mat = *lpMat;
     LeaveCriticalSection(&ddraw_cs);
 
     return DD_OK;
@@ -285,7 +285,7 @@ IDirect3DMaterialImpl_GetMaterial(IDirect3DMaterial3 *iface,
     EnterCriticalSection(&ddraw_cs);
     dwSize = lpMat->dwSize;
     memset(lpMat, 0, dwSize);
-    memcpy(lpMat, &This->mat, dwSize);
+    *lpMat = This->mat;
     LeaveCriticalSection(&ddraw_cs);
 
     return DD_OK;
diff --git a/dlls/ddraw/viewport.c b/dlls/ddraw/viewport.c
index deb4146..8dfc015 100644
--- a/dlls/ddraw/viewport.c
+++ b/dlls/ddraw/viewport.c
@@ -261,7 +261,7 @@ IDirect3DViewportImpl_GetViewport(IDirect3DViewport3 *iface,
     }
     dwSize = lpData->dwSize;
     memset(lpData, 0, dwSize);
-    memcpy(lpData, &(This->viewports.vp1), dwSize);
+    *lpData = This->viewports.vp1;
 
     if (TRACE_ON(d3d7)) {
         TRACE("  returning D3DVIEWPORT :\n");
@@ -301,7 +301,7 @@ IDirect3DViewportImpl_SetViewport(IDirect3DViewport3 *iface,
     EnterCriticalSection(&ddraw_cs);
     This->use_vp2 = 0;
     memset(&(This->viewports.vp1), 0, sizeof(This->viewports.vp1));
-    memcpy(&(This->viewports.vp1), lpData, lpData->dwSize);
+    This->viewports.vp1 = *lpData;
 
     /* Tests on two games show that these values are never used properly so override
        them with proper ones :-)
@@ -856,7 +856,7 @@ IDirect3DViewportImpl_GetViewport2(IDirect3DViewport3 *iface,
     }
     dwSize = lpData->dwSize;
     memset(lpData, 0, dwSize);
-    memcpy(lpData, &(This->viewports.vp2), dwSize);
+    *lpData = This->viewports.vp2;
 
     if (TRACE_ON(d3d7)) {
         TRACE("  returning D3DVIEWPORT2 :\n");
@@ -895,7 +895,7 @@ IDirect3DViewportImpl_SetViewport2(IDirect3DViewport3 *iface,
     EnterCriticalSection(&ddraw_cs);
     This->use_vp2 = 1;
     memset(&(This->viewports.vp2), 0, sizeof(This->viewports.vp2));
-    memcpy(&(This->viewports.vp2), lpData, lpData->dwSize);
+    This->viewports.vp2 = *lpData;
 
     if (This->active_device) {
       IDirect3DDevice3_GetCurrentViewport(ICOM_INTERFACE(This->active_device, IDirect3DDevice3), &current_viewport);