<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jan 11, 2016 at 10:46 AM, Alistair Leslie-Hughes <span dir="ltr"><<a href="mailto:leslie_alistair@hotmail.com" target="_blank">leslie_alistair@hotmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Signed-off-by: Alistair Leslie-Hughes <<a href="mailto:leslie_alistair@hotmail.com" target="_blank">leslie_alistair@hotmail.com</a>><br>
---<br>
 dlls/d3dx9_36/tests/surface.c | 19 ++++++++++++-------<br>
 1 file changed, 12 insertions(+), 7 deletions(-)<br>
<br>
diff --git a/dlls/d3dx9_36/tests/surface.c b/dlls/d3dx9_36/tests/surface.c<br>
index 183086f..5a20658 100644<br>
--- a/dlls/d3dx9_36/tests/surface.c<br>
+++ b/dlls/d3dx9_36/tests/surface.c<br>
@@ -200,7 +200,6 @@ struct dds_pixel_format<br>
<br>
 struct dds_header<br>
 {<br>
-    DWORD magic;<br>
     DWORD size;<br>
     DWORD flags;<br>
     DWORD height;<br>
@@ -222,7 +221,6 @@ static void fill_dds_header(struct dds_header *header)<br>
 {<br>
     memset(header, 0, sizeof(*header));<br>
<br>
-    header->magic = MAKEFOURCC('D','D','S',' ');<br>
     header->size = sizeof(*header);<br>
     header->flags = DDS_CAPS | DDS_WIDTH | DDS_HEIGHT | DDS_PIXELFORMAT;<br>
     header->height = 4;<br>
@@ -250,10 +248,12 @@ static void check_dds_pixel_format_(unsigned int line,<br>
     D3DXIMAGE_INFO info;<br>
     struct<br>
     {<br>
+        DWORD magic;<br>
         struct dds_header header;<br>
         BYTE data[256];<br>
     } dds;<br>
<br>
+    dds.magic = MAKEFOURCC('D','D','S',' ');<br>
     fill_dds_header(&dds.header);<br>
     dds.header.pixel_format.flags = flags;<br>
     dds.header.pixel_format.fourcc = fourcc;<br>
@@ -281,6 +281,7 @@ static void test_dds_header_handling(void)<br>
     D3DXIMAGE_INFO info;<br>
     struct<br>
     {<br>
+        DWORD magic;<br>
         struct dds_header header;<br>
         BYTE data[4096 * 1024];<br>
     } *dds;<br>
@@ -399,9 +400,10 @@ static void test_dds_header_handling(void)<br>
<br>
     for (i = 0; i < sizeof(tests) / sizeof(tests[0]); i++)<br>
     {<br>
-        DWORD file_size = sizeof(dds->header) + tests[i].pixel_data_size;<br>
+        DWORD file_size = sizeof(dds->magic) + sizeof(dds->header) + tests[i].pixel_data_size;<br>
         assert(file_size <= sizeof(*dds));<br>
<br>
+        dds->magic = MAKEFOURCC('D','D','S',' ');<br>
         fill_dds_header(&dds->header);<br>
         dds->header.flags |= tests[i].flags;<br>
         dds->header.width = tests[i].width;<br>
@@ -1235,6 +1237,7 @@ static void test_D3DXSaveSurfaceToFileInMemory(IDirect3DDevice9 *device)<br>
     ID3DXBuffer *buffer;<br>
     IDirect3DSurface9 *surface;<br>
     struct dds_header *header;<br>
+    BYTE *data;<br>
<br>
     hr = IDirect3DDevice9_CreateOffscreenPlainSurface(device, 4, 4, D3DFMT_A8R8G8B8, D3DPOOL_SCRATCH, &surface, NULL);<br>
     if (FAILED(hr)) {<br>
@@ -1257,9 +1260,10 @@ static void test_D3DXSaveSurfaceToFileInMemory(IDirect3DDevice9 *device)<br>
     todo_wine ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr);<br>
     if (SUCCEEDED(hr))<br>
     {<br>
-        header = ID3DXBuffer_GetBufferPointer(buffer);<br>
+        data = ID3DXBuffer_GetBufferPointer(buffer);<br>
+        header = (struct dds_header *)(data + sizeof(DWORD));<br>
<br>
-        ok(header->magic == MAKEFOURCC('D','D','S',' '), "Invalid DDS signature.\n");<br>
+        ok( *(DWORD*)data == MAKEFOURCC('D','D','S',' '), "Invalid DDS signature.\n");<br>
         ok(header->size == 124, "Invalid DDS size %u.\n", header->size);<br>
         ok(!header->height, "Got unexpected height %u.\n", header->height);<br>
         ok(!header->width, "Got unexpected width %u.\n", header->width);<br>
@@ -1274,9 +1278,10 @@ static void test_D3DXSaveSurfaceToFileInMemory(IDirect3DDevice9 *device)<br>
<br>
     hr = D3DXSaveSurfaceToFileInMemory(&buffer, D3DXIFF_DDS, surface, NULL, NULL);<br>
     ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr);<br>
-    header = ID3DXBuffer_GetBufferPointer(buffer);<br>
<br>
-    ok(header->magic == MAKEFOURCC('D','D','S',' '), "Invalid DDS signature.\n");<br>
+    data = ID3DXBuffer_GetBufferPointer(buffer);<br>
+    header = (struct dds_header *)(data + sizeof(DWORD));<br>
+    ok( *(DWORD*)data == MAKEFOURCC('D','D','S',' '), "Invalid DDS signature.\n");<br>
     ok(header->size == 124, "Invalid DDS size %u.\n", header->size);<br>
     ok(header->height == 4, "Got unexpected height %u.\n", header->height);<br>
     ok(header->width == 4, "Got unexpected width %u.\n", header->width);<br>
<span><font color="#888888">--<br>
2.6.4<br>
<br>
<br>
<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">What's the point of this change? It doesn't fix anything and doesn't make code more readable. If the file signature isn't included in the DDS header size it doesn't necessarily mean we have to move it out of struct dds_header.<br></div></div>