Ziqing Hui : windowscodecs: Implement DdsDecoder_Initialize().

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 8467b61a3e55eff0bbad08898fd723ead805b895
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=8467b61a3e55eff0bbad08898fd723ead805b895

Author: Ziqing Hui <zhui at codeweavers.com>
Date:   Thu Apr 16 17:43:18 2020 +0800

windowscodecs: Implement DdsDecoder_Initialize().

Signed-off-by: Ziqing Hui <zhui at codeweavers.com>
Signed-off-by: Vincent Povirk <vincent at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/windowscodecs/ddsformat.c       | 81 +++++++++++++++++++++++++++++++++++-
 dlls/windowscodecs/tests/ddsformat.c |  4 +-
 2 files changed, 81 insertions(+), 4 deletions(-)

diff --git a/dlls/windowscodecs/ddsformat.c b/dlls/windowscodecs/ddsformat.c
index 67680e9a0a..ac832e2116 100644
--- a/dlls/windowscodecs/ddsformat.c
+++ b/dlls/windowscodecs/ddsformat.c
@@ -33,12 +33,43 @@
 
 WINE_DEFAULT_DEBUG_CHANNEL(wincodecs);
 
+#define DDS_MAGIC 0x20534444
+
+typedef struct {
+    DWORD size;
+    DWORD flags;
+    DWORD fourCC;
+    DWORD rgbBitCount;
+    DWORD rBitMask;
+    DWORD gBitMask;
+    DWORD bBitMask;
+    DWORD aBitMask;
+} DDS_PIXELFORMAT;
+
+typedef struct {
+    DWORD size;
+    DWORD flags;
+    DWORD height;
+    DWORD width;
+    DWORD pitchOrLinearSize;
+    DWORD depth;
+    DWORD mipMapCount;
+    DWORD reserved1[11];
+    DDS_PIXELFORMAT ddspf;
+    DWORD caps;
+    DWORD caps2;
+    DWORD caps3;
+    DWORD caps4;
+    DWORD reserved2;
+} DDS_HEADER;
+
 typedef struct DdsDecoder {
     IWICBitmapDecoder IWICBitmapDecoder_iface;
     LONG ref;
     BOOL initialized;
     IStream *stream;
     CRITICAL_SECTION lock;
+    DDS_HEADER header;
 } DdsDecoder;
 
 static inline DdsDecoder *impl_from_IWICBitmapDecoder(IWICBitmapDecoder *iface)
@@ -108,9 +139,55 @@ static HRESULT WINAPI DdsDecoder_QueryCapability(IWICBitmapDecoder *iface, IStre
 static HRESULT WINAPI DdsDecoder_Initialize(IWICBitmapDecoder *iface, IStream *pIStream,
                                             WICDecodeOptions cacheOptions)
 {
-    FIXME("(%p,%p,%x): stub.\n", iface, pIStream, cacheOptions);
+    DdsDecoder *This = impl_from_IWICBitmapDecoder(iface);
+    HRESULT hr;
+    LARGE_INTEGER seek;
+    DWORD magic;
+    ULONG bytesread;
 
-    return E_NOTIMPL;
+    TRACE("(%p,%p,%x)\n", iface, pIStream, cacheOptions);
+
+    EnterCriticalSection(&This->lock);
+
+    if (This->initialized) {
+        hr = WINCODEC_ERR_WRONGSTATE;
+        goto end;
+    }
+
+    seek.QuadPart = 0;
+    hr = IStream_Seek(pIStream, seek, SEEK_SET, NULL);
+    if (FAILED(hr)) goto end;
+
+    hr = IStream_Read(pIStream, &magic, sizeof(magic), &bytesread);
+    if (FAILED(hr)) goto end;
+    if (bytesread != sizeof(magic)) {
+        hr = WINCODEC_ERR_STREAMREAD;
+        goto end;
+    }
+    if (magic != DDS_MAGIC) {
+        hr = WINCODEC_ERR_UNKNOWNIMAGEFORMAT;
+        goto end;
+    }
+
+    hr = IStream_Read(pIStream, &This->header, sizeof(This->header), &bytesread);
+    if (FAILED(hr)) goto end;
+    if (bytesread != sizeof(This->header)) {
+        hr = WINCODEC_ERR_STREAMREAD;
+        goto end;
+    }
+    if (This->header.size != sizeof(This->header)) {
+        hr = WINCODEC_ERR_BADHEADER;
+        goto end;
+    }
+
+    This->initialized = TRUE;
+    This->stream = pIStream;
+    IStream_AddRef(pIStream);
+
+end:
+    LeaveCriticalSection(&This->lock);
+
+    return hr;
 }
 
 static HRESULT WINAPI DdsDecoder_GetContainerFormat(IWICBitmapDecoder *iface,
diff --git a/dlls/windowscodecs/tests/ddsformat.c b/dlls/windowscodecs/tests/ddsformat.c
index 6abee4dea5..8569d0e291 100644
--- a/dlls/windowscodecs/tests/ddsformat.c
+++ b/dlls/windowscodecs/tests/ddsformat.c
@@ -86,9 +86,9 @@ static HRESULT init_decoder(IWICBitmapDecoder *decoder, IWICStream *stream, HRES
 
     hr = IWICBitmapDecoder_Initialize(decoder, (IStream*)stream, WICDecodeMetadataCacheOnDemand);
     if (index == -1) {
-        todo_wine ok(SUCCEEDED(hr), "Decoder Initialize failed, hr=%x\n", hr);
+        ok(SUCCEEDED(hr), "Decoder Initialize failed, hr=%x\n", hr);
     } else {
-        todo_wine ok(hr == expected, "%d: Expected hr=%x, got %x\n", index, expected, hr);
+        ok(hr == expected, "%d: Expected hr=%x, got %x\n", index, expected, hr);
     }
     return hr;
 }