[PATCH] d3dx10: Parially implement async data loaders interfaces

Nikolay Sivov nsivov at codeweavers.com
Sun Nov 6 12:45:02 CST 2016


Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
 dlls/d3dx10_43/async.c        | 296 ++++++++++++++++++++++++++++++++++++++++--
 dlls/d3dx10_43/tests/d3dx10.c |  36 ++---
 2 files changed, 306 insertions(+), 26 deletions(-)

diff --git a/dlls/d3dx10_43/async.c b/dlls/d3dx10_43/async.c
index f49eb92..a9cbde5 100644
--- a/dlls/d3dx10_43/async.c
+++ b/dlls/d3dx10_43/async.c
@@ -27,6 +27,184 @@
 
 WINE_DEFAULT_DEBUG_CHANNEL(d3dx);
 
+struct asyncdataloader
+{
+    ID3DX10DataLoader ID3DX10DataLoader_iface;
+
+    union
+    {
+        HANDLE file;
+        struct
+        {
+            HMODULE module;
+            HRSRC rsrc;
+        } resource;
+    } u;
+    void *data;
+    SIZE_T size;
+};
+
+static inline struct asyncdataloader *impl_from_ID3DX10DataLoader(ID3DX10DataLoader *iface)
+{
+    return CONTAINING_RECORD(iface, struct asyncdataloader, ID3DX10DataLoader_iface);
+}
+
+/* Memory data loader. */
+static HRESULT WINAPI memorydataloader_Load(ID3DX10DataLoader *loader)
+{
+    TRACE("loader %p\n", loader);
+    return S_OK;
+}
+
+static HRESULT WINAPI memorydataloader_Decompress(ID3DX10DataLoader *iface, void **data, SIZE_T *size)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+
+    FIXME("loader %p, data %p, size %p semi-stub\n", loader, data, size);
+
+    *data = loader->data;
+    *size = loader->size;
+
+    return S_OK;
+}
+
+static HRESULT WINAPI memorydataloader_Destroy(ID3DX10DataLoader *iface)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+
+    TRACE("loader %p\n", loader);
+
+    HeapFree(GetProcessHeap(), 0, loader);
+    return S_OK;
+}
+
+static const ID3DX10DataLoaderVtbl memorydataloadervtbl =
+{
+    memorydataloader_Load,
+    memorydataloader_Decompress,
+    memorydataloader_Destroy
+};
+
+/* File data loader. */
+static HRESULT WINAPI filedataloader_Load(ID3DX10DataLoader *iface)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+    HANDLE mapping;
+
+    TRACE("loader %p\n", loader);
+
+    if (loader->u.file == INVALID_HANDLE_VALUE)
+        return D3D10_ERROR_FILE_NOT_FOUND;
+
+    if (loader->data)
+        return S_OK;
+
+    GetFileSize(loader->u.file, (DWORD*)&loader->size);
+    mapping = CreateFileMappingW(loader->u.file, NULL, PAGE_READONLY, 0, 0, NULL);
+    if (!mapping)
+        return E_FAIL;
+
+    loader->data = MapViewOfFile(mapping, FILE_MAP_READ, 0, 0, 0);
+    CloseHandle(mapping);
+    if (!loader->data)
+        return E_FAIL;
+
+    return S_OK;
+}
+
+static HRESULT WINAPI filedataloader_Decompress(ID3DX10DataLoader *iface, void **data, SIZE_T *size)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+
+    FIXME("loader %p, data %p, size %p semi-stub\n", loader, data, size);
+
+    if (!loader->data)
+        return E_FAIL;
+
+    *data = loader->data;
+    *size = loader->size;
+
+    return S_OK;
+}
+
+static HRESULT WINAPI filedataloader_Destroy(ID3DX10DataLoader *iface)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+
+    TRACE("loader %p\n", loader);
+
+    UnmapViewOfFile(loader->data);
+    CloseHandle(loader->u.file);
+
+    HeapFree(GetProcessHeap(), 0, loader);
+
+    return S_OK;
+}
+
+static const ID3DX10DataLoaderVtbl filedataloadervtbl =
+{
+    filedataloader_Load,
+    filedataloader_Decompress,
+    filedataloader_Destroy
+};
+
+/* Resource data loader. */
+static HRESULT WINAPI resourcedataloader_Load(ID3DX10DataLoader *iface)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+    HGLOBAL hglobal;
+
+    TRACE("loader %p\n", loader);
+
+    if (loader->data)
+        return S_OK;
+
+    hglobal = LoadResource(loader->u.resource.module, loader->u.resource.rsrc);
+    if (!hglobal)
+    {
+        ERR("Failed to load resource.\n");
+        return E_FAIL;
+    }
+
+    loader->data = LockResource(hglobal);
+    loader->size = SizeofResource(loader->u.resource.module, loader->u.resource.rsrc);
+
+    return S_OK;
+}
+
+static HRESULT WINAPI resourcedataloader_Decompress(ID3DX10DataLoader *iface, void **data, SIZE_T *size)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+
+    FIXME("loader %p, data %p, size %p semi-stub\n", loader, data, size);
+
+    if (!loader->data)
+        return E_FAIL;
+
+    *data = loader->data;
+    *size = loader->size;
+
+    return S_OK;
+}
+
+static HRESULT WINAPI resourcedataloader_Destroy(ID3DX10DataLoader *iface)
+{
+    struct asyncdataloader *loader = impl_from_ID3DX10DataLoader(iface);
+
+    TRACE("loader %p\n", loader);
+
+    HeapFree(GetProcessHeap(), 0, loader);
+
+    return S_OK;
+}
+
+static const ID3DX10DataLoaderVtbl resourcedataloadervtbl =
+{
+    resourcedataloader_Load,
+    resourcedataloader_Decompress,
+    resourcedataloader_Destroy
+};
+
 HRESULT WINAPI D3DX10CompileFromMemory(const char *data, SIZE_T data_size, const char *filename,
         const D3D10_SHADER_MACRO *defines, ID3D10Include *include, const char *entry_point,
         const char *target, UINT sflags, UINT eflags, ID3DX10ThreadPump *pump, ID3D10Blob **shader,
@@ -70,35 +248,133 @@ HRESULT WINAPI D3DX10CreateEffectPoolFromFileW(const WCHAR *filename, const D3D1
 
 HRESULT WINAPI D3DX10CreateAsyncMemoryLoader(const void *data, SIZE_T data_size, ID3DX10DataLoader **loader)
 {
-    FIXME("data %p, data_size %lu, loader %p stub!\n", data, data_size, loader);
+    struct asyncdataloader *object;
 
-    return E_NOTIMPL;
+    TRACE("data %p, data_size %lu, loader %p\n", data, data_size, loader);
+
+    if (!data || !loader)
+        return E_FAIL;
+
+    object = HeapAlloc(GetProcessHeap(), 0, sizeof(*object));
+    if (!object)
+        return E_OUTOFMEMORY;
+
+    object->ID3DX10DataLoader_iface.lpVtbl = &memorydataloadervtbl;
+    object->u.file = NULL;
+    object->data = (void*)data;
+    object->size = data_size;
+
+    *loader = &object->ID3DX10DataLoader_iface;
+
+    return S_OK;
 }
 
 HRESULT WINAPI D3DX10CreateAsyncFileLoaderA(const char *filename, ID3DX10DataLoader **loader)
 {
-    FIXME("filename %s, loader %p stub!\n", debugstr_a(filename), loader);
+    WCHAR *filenameW;
+    HRESULT hr;
+    int len;
 
-    return E_NOTIMPL;
+    TRACE("filename %s, loader %p\n", debugstr_a(filename), loader);
+
+    if (!filename || !loader)
+        return E_FAIL;
+
+    len = MultiByteToWideChar(CP_ACP, 0, filename, -1, NULL, 0);
+    filenameW = HeapAlloc(GetProcessHeap(), 0, len * sizeof(*filenameW));
+    MultiByteToWideChar(CP_ACP, 0, filename, -1, filenameW, len);
+
+    hr = D3DX10CreateAsyncFileLoaderW(filenameW, loader);
+
+    HeapFree(GetProcessHeap(), 0, filenameW);
+
+    return hr;
 }
 
 HRESULT WINAPI D3DX10CreateAsyncFileLoaderW(const WCHAR *filename, ID3DX10DataLoader **loader)
 {
-    FIXME("filename %s, loader %p stub!\n", debugstr_w(filename), loader);
+    struct asyncdataloader *object;
 
-    return E_NOTIMPL;
+    TRACE("filename %s, loader %p\n", debugstr_w(filename), loader);
+
+    if (!filename || !loader)
+        return E_FAIL;
+
+    object = HeapAlloc(GetProcessHeap(), 0, sizeof(*object));
+    if (!object)
+        return E_OUTOFMEMORY;
+
+    object->ID3DX10DataLoader_iface.lpVtbl = &filedataloadervtbl;
+    object->u.file = CreateFileW(filename, GENERIC_READ, FILE_SHARE_READ|FILE_SHARE_WRITE,
+            NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);
+    object->data = NULL;
+    object->size = 0;
+
+    *loader = &object->ID3DX10DataLoader_iface;
+
+    return S_OK;
 }
 
 HRESULT WINAPI D3DX10CreateAsyncResourceLoaderA(HMODULE module, const char *resource, ID3DX10DataLoader **loader)
 {
-    FIXME("module %p, resource %s, loader %p stub!\n", module, debugstr_a(resource), loader);
+    struct asyncdataloader *object;
+    HRSRC rsrc;
 
-    return E_NOTIMPL;
+    TRACE("module %p, resource %s, loader %p.\n", module, debugstr_a(resource), loader);
+
+    if (!loader)
+        return E_FAIL;
+
+    object = HeapAlloc(GetProcessHeap(), 0, sizeof(*object));
+    if (!object)
+        return E_OUTOFMEMORY;
+
+    if (!(rsrc = FindResourceA(module, resource, (LPSTR)RT_RCDATA)))
+    {
+        ERR("Failed to find resource\n");
+        HeapFree(GetProcessHeap(), 0, object);
+        return D3DX10_ERR_INVALID_DATA;
+    }
+
+    object->ID3DX10DataLoader_iface.lpVtbl = &resourcedataloadervtbl;
+    object->u.resource.module = module;
+    object->u.resource.rsrc = rsrc;
+    object->data = NULL;
+    object->size = 0;
+
+    *loader = &object->ID3DX10DataLoader_iface;
+
+    return S_OK;
 }
 
 HRESULT WINAPI D3DX10CreateAsyncResourceLoaderW(HMODULE module, const WCHAR *resource, ID3DX10DataLoader **loader)
 {
-    FIXME("module %p, resource %s, loader %p stub!\n", module, debugstr_w(resource), loader);
+    struct asyncdataloader *object;
+    HRSRC rsrc;
 
-    return E_NOTIMPL;
+    TRACE("module %p, resource %s, loader %p\n", module, debugstr_w(resource), loader);
+
+    if (!loader)
+        return E_FAIL;
+
+    object = HeapAlloc(GetProcessHeap(), 0, sizeof(*object));
+    if (!object)
+        return E_OUTOFMEMORY;
+
+    if (!(rsrc = FindResourceW(module, resource, (LPWSTR)RT_RCDATA)))
+    {
+        ERR("Failed to find resource\n");
+        HeapFree(GetProcessHeap(), 0, object);
+        return D3DX10_ERR_INVALID_DATA;
+    }
+
+    object->ID3DX10DataLoader_iface.lpVtbl = &resourcedataloadervtbl;
+    object->u.resource.module = module;
+    object->u.resource.rsrc = rsrc;
+    object->data = NULL;
+    object->size = 0;
+
+    *loader = &object->ID3DX10DataLoader_iface;
+
+    return S_OK;
 }
diff --git a/dlls/d3dx10_43/tests/d3dx10.c b/dlls/d3dx10_43/tests/d3dx10.c
index 2eccb69..4dc9423 100644
--- a/dlls/d3dx10_43/tests/d3dx10.c
+++ b/dlls/d3dx10_43/tests/d3dx10.c
@@ -614,16 +614,13 @@ static void test_D3DX10CreateAsyncMemoryLoader(void)
     void *ptr;
 
     hr = D3DX10CreateAsyncMemoryLoader(NULL, 0, NULL);
-    todo_wine ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
+    ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
 
     hr = D3DX10CreateAsyncMemoryLoader(NULL, 0, &loader);
-    todo_wine ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
+    ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
 
     hr = D3DX10CreateAsyncMemoryLoader(&data, 0, &loader);
-    todo_wine ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
-
-    if (FAILED(hr))
-        return;
+    ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
 
     size = 100;
     hr = ID3DX10DataLoader_Decompress(loader, &ptr, &size);
@@ -663,16 +660,13 @@ static void test_D3DX10CreateAsyncFileLoader(void)
     void *ptr;
 
     hr = D3DX10CreateAsyncFileLoaderA(NULL, NULL);
-    todo_wine ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
+    ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
 
     hr = D3DX10CreateAsyncFileLoaderA(NULL, &loader);
-    todo_wine ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
+    ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
 
     hr = D3DX10CreateAsyncFileLoaderA("nonexistentfilename", &loader);
-    todo_wine ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
-
-    if (FAILED(hr))
-        return;
+    ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
 
     hr = ID3DX10DataLoader_Decompress(loader, &ptr, &size);
     ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
@@ -689,17 +683,27 @@ static void test_D3DX10CreateAsyncFileLoader(void)
 
 static void test_D3DX10CreateAsyncResourceLoader(void)
 {
+    static const WCHAR resnameW[] = {'n','o','n','a','m','e',0};
     ID3DX10DataLoader *loader;
     HRESULT hr;
 
     hr = D3DX10CreateAsyncResourceLoaderA(NULL, NULL, NULL);
-    todo_wine ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
+    ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
 
     hr = D3DX10CreateAsyncResourceLoaderA(NULL, NULL, &loader);
-    todo_wine ok(hr == D3DX10_ERR_INVALID_DATA, "Got unexpected hr %#x.\n", hr);
+    ok(hr == D3DX10_ERR_INVALID_DATA, "Got unexpected hr %#x.\n", hr);
+
+    hr = D3DX10CreateAsyncResourceLoaderA(NULL, "noname", &loader);
+    ok(hr == D3DX10_ERR_INVALID_DATA, "Got unexpected hr %#x.\n", hr);
+
+    hr = D3DX10CreateAsyncResourceLoaderW(NULL, NULL, NULL);
+    ok(hr == E_FAIL, "Got unexpected hr %#x.\n", hr);
+
+    hr = D3DX10CreateAsyncResourceLoaderW(NULL, NULL, &loader);
+    ok(hr == D3DX10_ERR_INVALID_DATA, "Got unexpected hr %#x.\n", hr);
 
-    hr = D3DX10CreateAsyncResourceLoaderA(NULL, "nonexistentresourcename", &loader);
-    todo_wine ok(hr == D3DX10_ERR_INVALID_DATA, "Got unexpected hr %#x.\n", hr);
+    hr = D3DX10CreateAsyncResourceLoaderW(NULL, resnameW, &loader);
+    ok(hr == D3DX10_ERR_INVALID_DATA, "Got unexpected hr %#x.\n", hr);
 }
 
 START_TEST(d3dx10)
-- 
2.10.2




More information about the wine-patches mailing list