January 2009 - Wine-commits

Juan Lang : crypt32: Only open the read-only root store for the local machine location.
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 56634dce7b0dddde8b4a3681c1bf7db3d3f123c6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=56634dce7b0dddde8b4a3681c… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Jan 16 07:40:55 2009 -0800 crypt32: Only open the read-only root store for the local machine location. --- dlls/crypt32/store.c | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/dlls/crypt32/store.c b/dlls/crypt32/store.c index 3b42893..602bd40 100644 --- a/dlls/crypt32/store.c +++ b/dlls/crypt32/store.c @@ -348,7 +348,14 @@ static PWINECRYPT_CERTSTORE CRYPT_SysRegOpenStoreW(HCRYPTPROV hCryptProv, SetLastError(E_INVALIDARG); return NULL; } - if (!lstrcmpiW(storeName, rootW)) + /* FIXME: In Windows, the root store (even the current user location) is + * protected: adding to it or removing from it present a user interface, + * and the keys are owned by the system process, not the current user. + * Wine's registry doesn't implement access controls, so a similar + * mechanism isn't possible yet. + */ + if ((dwFlags & CERT_SYSTEM_STORE_LOCATION_MASK) == + CERT_SYSTEM_STORE_LOCAL_MACHINE && !lstrcmpiW(storeName, rootW)) return CRYPT_RootOpenStore(hCryptProv, dwFlags); switch (dwFlags & CERT_SYSTEM_STORE_LOCATION_MASK)

1 0

Juan Lang : crypt32: Remove location checks when opening root store.
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 69e43ef10835a7a526fe5ff4db55dfa080f1575c URL: http://source.winehq.org/git/wine.git/?a=commit;h=69e43ef10835a7a526fe5ff4d… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Jan 16 07:49:08 2009 -0800 crypt32: Remove location checks when opening root store. --- dlls/crypt32/rootstore.c | 11 ----------- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/dlls/crypt32/rootstore.c b/dlls/crypt32/rootstore.c index 116c934..a2ed6e5 100644 --- a/dlls/crypt32/rootstore.c +++ b/dlls/crypt32/rootstore.c @@ -744,17 +744,6 @@ PWINECRYPT_CERTSTORE CRYPT_RootOpenStore(HCRYPTPROV hCryptProv, DWORD dwFlags) SetLastError(ERROR_ACCESS_DENIED); return NULL; } - switch (dwFlags & CERT_SYSTEM_STORE_LOCATION_MASK) - { - case CERT_SYSTEM_STORE_LOCAL_MACHINE: - case CERT_SYSTEM_STORE_CURRENT_USER: - break; - default: - TRACE("location %08x unsupported\n", - dwFlags & CERT_SYSTEM_STORE_LOCATION_MASK); - SetLastError(E_INVALIDARG); - return NULL; - } if (!CRYPT_rootStore) { HCERTSTORE root = create_root_store();

1 0

Andrew Nguyen : kernel32/tests: Simplify a test function's parameters.
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 1571c05ad59041819de4cda4b186a527a48139f2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1571c05ad59041819de4cda4b… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Sun Jan 18 23:51:14 2009 -0600 kernel32/tests: Simplify a test function's parameters. --- dlls/kernel32/tests/console.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/tests/console.c b/dlls/kernel32/tests/console.c index 6eed56b..a90fe70 100644 --- a/dlls/kernel32/tests/console.c +++ b/dlls/kernel32/tests/console.c @@ -173,7 +173,7 @@ static void testCursorInfo(HANDLE hCon) ERROR_INVALID_ACCESS, GetLastError()); } -static void testWriteSimple(HANDLE hCon, COORD sbSize) +static void testWriteSimple(HANDLE hCon) { COORD c; DWORD len; @@ -407,7 +407,7 @@ static void testWrite(HANDLE hCon, COORD sbSize) /* FIXME: should in fact insure that the sb is at least 10 character wide */ ok(SetConsoleTextAttribute(hCon, TEST_ATTRIB), "Setting default text color\n"); resetContent(hCon, sbSize, FALSE); - testWriteSimple(hCon, sbSize); + testWriteSimple(hCon); resetContent(hCon, sbSize, FALSE); testWriteNotWrappedNotProcessed(hCon, sbSize); resetContent(hCon, sbSize, FALSE);

1 0

Andrew Nguyen : kernel32: Ensure that WriteConsoleW succeeds with an empty input buffer.
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 976d5cae8d58b29564cd802b4330ad211557f518 URL: http://source.winehq.org/git/wine.git/?a=commit;h=976d5cae8d58b29564cd802b4… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Sun Jan 18 23:52:11 2009 -0600 kernel32: Ensure that WriteConsoleW succeeds with an empty input buffer. --- dlls/kernel32/console.c | 2 ++ dlls/kernel32/tests/console.c | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 43 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/console.c b/dlls/kernel32/console.c index a872b1f..d247e1b 100644 --- a/dlls/kernel32/console.c +++ b/dlls/kernel32/console.c @@ -2057,6 +2057,8 @@ BOOL WINAPI WriteConsoleW(HANDLE hConsoleOutput, LPCVOID lpBuffer, DWORD nNumber !GetConsoleScreenBufferInfo(hConsoleOutput, &csbi)) return FALSE; + if (!nNumberOfCharsToWrite) return TRUE; + if (mode & ENABLE_PROCESSED_OUTPUT) { unsigned int i; diff --git a/dlls/kernel32/tests/console.c b/dlls/kernel32/tests/console.c index a90fe70..eb51f7a 100644 --- a/dlls/kernel32/tests/console.c +++ b/dlls/kernel32/tests/console.c @@ -173,6 +173,45 @@ static void testCursorInfo(HANDLE hCon) ERROR_INVALID_ACCESS, GetLastError()); } +static void testEmptyWrite(HANDLE hCon) +{ + COORD c; + DWORD len; + const char* mytest = ""; + + c.X = c.Y = 0; + ok(SetConsoleCursorPosition(hCon, c) != 0, "Cursor in upper-left\n"); + + len = -1; + ok(WriteConsole(hCon, NULL, 0, &len, NULL) != 0 && len == 0, "WriteConsole\n"); + okCURSOR(hCon, c); + + /* Passing a NULL lpBuffer with sufficiently large non-zero length succeeds + * on native Windows and result in memory-like contents being written to + * the console. Calling WriteConsoleW like this will crash on Wine. */ + if (0) + { + len = -1; + ok(!WriteConsole(hCon, NULL, 16, &len, NULL) && len == -1, "WriteConsole\n"); + okCURSOR(hCon, c); + + /* Cursor advances for this call. */ + len = -1; + ok(WriteConsole(hCon, NULL, 128, &len, NULL) != 0 && len == 128, "WriteConsole\n"); + } + + len = -1; + ok(WriteConsole(hCon, mytest, 0, &len, NULL) != 0 && len == 0, "WriteConsole\n"); + okCURSOR(hCon, c); + + /* WriteConsole does not halt on a null terminator and is happy to write + * memory contents beyond the actual size of the buffer. */ + len = -1; + ok(WriteConsole(hCon, mytest, 16, &len, NULL) != 0 && len == 16, "WriteConsole\n"); + c.X += 16; + okCURSOR(hCon, c); +} + static void testWriteSimple(HANDLE hCon) { COORD c; @@ -407,6 +446,8 @@ static void testWrite(HANDLE hCon, COORD sbSize) /* FIXME: should in fact insure that the sb is at least 10 character wide */ ok(SetConsoleTextAttribute(hCon, TEST_ATTRIB), "Setting default text color\n"); resetContent(hCon, sbSize, FALSE); + testEmptyWrite(hCon); + resetContent(hCon, sbSize, FALSE); testWriteSimple(hCon); resetContent(hCon, sbSize, FALSE); testWriteNotWrappedNotProcessed(hCon, sbSize);

1 0

Henri Verbeet : dxgi: Implement IDXGIDevice::CreateSurface().
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: c9f116a56f519fea0c171362d2e00dddfb908ce9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c9f116a56f519fea0c171362d… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 19 10:39:06 2009 +0100 dxgi: Implement IDXGIDevice::CreateSurface(). --- dlls/d3d10core/d3d10core_private.h | 1 + dlls/d3d10core/device.c | 11 ++++ dlls/d3d10core/tests/device.c | 2 +- dlls/d3d10core/texture2d.c | 9 ++++ dlls/dxgi/device.c | 89 ++++++++++++++++++++++++++++++------ dlls/dxgi/tests/device.c | 2 +- include/wine/winedxgi.idl | 7 +++ 7 files changed, 105 insertions(+), 16 deletions(-) diff --git a/dlls/d3d10core/d3d10core_private.h b/dlls/d3d10core/d3d10core_private.h index 1b28d5e..5453600 100644 --- a/dlls/d3d10core/d3d10core_private.h +++ b/dlls/d3d10core/d3d10core_private.h @@ -58,6 +58,7 @@ struct d3d10_texture2d const struct ID3D10Texture2DVtbl *vtbl; LONG refcount; + IUnknown *dxgi_surface; IWineD3DSurface *wined3d_surface; }; diff --git a/dlls/d3d10core/device.c b/dlls/d3d10core/device.c index 034c9b9..2e4c5c1 100644 --- a/dlls/d3d10core/device.c +++ b/dlls/d3d10core/device.c @@ -602,6 +602,16 @@ static HRESULT STDMETHODCALLTYPE d3d10_device_CreateTexture2D(ID3D10Device *ifac return E_FAIL; } + hr = IWineDXGIDevice_create_surface(wine_device, NULL, 0, NULL, + (IUnknown *)object, (void **)&object->dxgi_surface); + if (FAILED(hr)) + { + ERR("Failed to create DXGI surface, returning %#x\n", hr); + HeapFree(GetProcessHeap(), 0, object); + IWineDXGIDevice_Release(wine_device); + return hr; + } + wined3d_device = IWineDXGIDevice_get_wined3d_device(wine_device); IWineDXGIDevice_Release(wine_device); @@ -614,6 +624,7 @@ static HRESULT STDMETHODCALLTYPE d3d10_device_CreateTexture2D(ID3D10Device *ifac if (FAILED(hr)) { ERR("CreateSurface failed, returning %#x\n", hr); + IDXGISurface_Release(object->dxgi_surface); HeapFree(GetProcessHeap(), 0, object); return hr; } diff --git a/dlls/d3d10core/tests/device.c b/dlls/d3d10core/tests/device.c index 5493d23..1a3f331 100644 --- a/dlls/d3d10core/tests/device.c +++ b/dlls/d3d10core/tests/device.c @@ -116,7 +116,7 @@ static void test_create_texture(ID3D10Device *device) ok(SUCCEEDED(hr), "Failed to create a 2d texture, hr %#x\n", hr); hr = ID3D10Texture2D_QueryInterface(texture, &IID_IDXGISurface, (void **)&surface); - todo_wine ok(SUCCEEDED(hr), "Texture should implement IDXGISurface\n"); + ok(SUCCEEDED(hr), "Texture should implement IDXGISurface\n"); if (SUCCEEDED(hr)) IDXGISurface_Release(surface); ID3D10Texture2D_Release(texture); diff --git a/dlls/d3d10core/texture2d.c b/dlls/d3d10core/texture2d.c index d0fb0f5..afb6167 100644 --- a/dlls/d3d10core/texture2d.c +++ b/dlls/d3d10core/texture2d.c @@ -28,6 +28,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d10core); static HRESULT STDMETHODCALLTYPE d3d10_texture2d_QueryInterface(ID3D10Texture2D *iface, REFIID riid, void **object) { + struct d3d10_texture2d *This = (struct d3d10_texture2d *)iface; + TRACE("iface %p, riid %s, object %p\n", iface, debugstr_guid(riid), object); if (IsEqualGUID(riid, &IID_ID3D10Texture2D) @@ -40,6 +42,12 @@ static HRESULT STDMETHODCALLTYPE d3d10_texture2d_QueryInterface(ID3D10Texture2D return S_OK; } + if (This->dxgi_surface) + { + TRACE("Forwarding to dxgi surface\n"); + return IDXGISurface_QueryInterface(This->dxgi_surface, riid, object); + } + WARN("%s not implemented, returning E_NOINTERFACE\n", debugstr_guid(riid)); *object = NULL; @@ -65,6 +73,7 @@ static ULONG STDMETHODCALLTYPE d3d10_texture2d_Release(ID3D10Texture2D *iface) if (!refcount) { + if (This->dxgi_surface) IDXGISurface_Release(This->dxgi_surface); if (This->wined3d_surface) IWineD3DSurface_Release(This->wined3d_surface); HeapFree(GetProcessHeap(), 0, This); } diff --git a/dlls/dxgi/device.c b/dlls/dxgi/device.c index 235e1f7..3c10f99 100644 --- a/dlls/dxgi/device.c +++ b/dlls/dxgi/device.c @@ -145,40 +145,65 @@ static HRESULT STDMETHODCALLTYPE dxgi_device_CreateSurface(IWineDXGIDevice *ifac const DXGI_SURFACE_DESC *desc, UINT surface_count, DXGI_USAGE usage, const DXGI_SHARED_RESOURCE *shared_resource, IDXGISurface **surface) { - struct dxgi_surface *object; + IWineD3DDeviceParent *device_parent; HRESULT hr; UINT i; + UINT j; - FIXME("iface %p, desc %p, surface_count %u, usage %#x, shared_resource %p, surface %p partial stub!\n", + TRACE("iface %p, desc %p, surface_count %u, usage %#x, shared_resource %p, surface %p\n", iface, desc, surface_count, usage, shared_resource, surface); + hr = IWineDXGIDevice_QueryInterface(iface, &IID_IWineD3DDeviceParent, (void **)&device_parent); + if (FAILED(hr)) + { + ERR("Device should implement IWineD3DDeviceParent\n"); + return E_FAIL; + } + + FIXME("Implement DXGI<->wined3d format and usage conversion\n"); + memset(surface, 0, surface_count * sizeof(*surface)); for (i = 0; i < surface_count; ++i) { - object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)); - if (!object) + IWineD3DSurface *wined3d_surface; + IUnknown *parent; + + hr = IWineD3DDeviceParent_CreateSurface(device_parent, NULL, desc->Width, desc->Height, desc->Format, + usage, WINED3DPOOL_DEFAULT, 0, WINED3DCUBEMAP_FACE_POSITIVE_X, &wined3d_surface); + if (FAILED(hr)) { - ERR("Failed to allocate DXGI surface object memory\n"); - hr = E_OUTOFMEMORY; + ERR("CreateSurface failed, returning %#x\n", hr); goto fail; } - object->vtbl = &dxgi_surface_vtbl; - object->inner_unknown_vtbl = &dxgi_surface_inner_unknown_vtbl; - object->refcount = 1; - object->outer_unknown = (IUnknown *)&object->inner_unknown_vtbl; - surface[i] = (IDXGISurface *)object; + hr = IWineD3DSurface_GetParent(wined3d_surface, &parent); + IWineD3DSurface_Release(wined3d_surface); + if (FAILED(hr)) + { + ERR("GetParent failed, returning %#x\n", hr); + goto fail; + } + + hr = IUnknown_QueryInterface(parent, &IID_IDXGISurface, (void **)&surface[i]); + IUnknown_Release(parent); + if (FAILED(hr)) + { + ERR("Surface should implement IDXGISurface\n"); + goto fail; + } - TRACE("Created IDXGISurface %p (%u/%u)\n", object, i + 1, surface_count); + TRACE("Created IDXGISurface %p (%u/%u)\n", surface[i], i + 1, surface_count); } + IWineD3DDeviceParent_Release(device_parent); return S_OK; fail: - for (i = 0; i < surface_count; ++i) + for (j = 0; j < i; ++j) { - HeapFree(GetProcessHeap(), 0, surface[i]); + IDXGISurface_Release(surface[i]); } + IWineD3DDeviceParent_Release(device_parent); return hr; } @@ -219,6 +244,41 @@ static IWineD3DDevice * STDMETHODCALLTYPE dxgi_device_get_wined3d_device(IWineDX return This->wined3d_device; } +static HRESULT STDMETHODCALLTYPE dxgi_device_create_surface(IWineDXGIDevice *iface, const DXGI_SURFACE_DESC *desc, + DXGI_USAGE usage, const DXGI_SHARED_RESOURCE *shared_resource, IUnknown *outer, void **surface) +{ + struct dxgi_surface *object; + + FIXME("iface %p, desc %p, usage %#x, shared_resource %p, outer %p, surface %p partial stub!\n", + iface, desc, usage, shared_resource, outer, surface); + + object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)); + if (!object) + { + ERR("Failed to allocate DXGI surface object memory\n"); + return E_OUTOFMEMORY; + } + + object->vtbl = &dxgi_surface_vtbl; + object->inner_unknown_vtbl = &dxgi_surface_inner_unknown_vtbl; + object->refcount = 1; + + if (outer) + { + object->outer_unknown = outer; + *surface = &object->inner_unknown_vtbl; + } + else + { + object->outer_unknown = (IUnknown *)&object->inner_unknown_vtbl; + *surface = object; + } + + TRACE("Created IDXGISurface %p\n", object); + + return S_OK; +} + const struct IWineDXGIDeviceVtbl dxgi_device_vtbl = { /* IUnknown methods */ @@ -238,4 +298,5 @@ const struct IWineDXGIDeviceVtbl dxgi_device_vtbl = dxgi_device_GetGPUThreadPriority, /* IWineDXGIAdapter methods */ dxgi_device_get_wined3d_device, + dxgi_device_create_surface, }; diff --git a/dlls/dxgi/tests/device.c b/dlls/dxgi/tests/device.c index 39d780a..12893b1 100644 --- a/dlls/dxgi/tests/device.c +++ b/dlls/dxgi/tests/device.c @@ -115,7 +115,7 @@ static void test_create_surface(IDXGIDevice *device) ok(SUCCEEDED(hr), "Failed to create a dxgi surface, hr %#x\n", hr); hr = IDXGISurface_QueryInterface(surface, &IID_ID3D10Texture2D, (void **)&texture); - todo_wine ok(SUCCEEDED(hr), "Surface should implement ID3D10Texture2D\n"); + ok(SUCCEEDED(hr), "Surface should implement ID3D10Texture2D\n"); if (SUCCEEDED(hr)) ID3D10Texture2D_Release(texture); IDXGISurface_Release(surface); diff --git a/include/wine/winedxgi.idl b/include/wine/winedxgi.idl index 860b695..3fc7883 100644 --- a/include/wine/winedxgi.idl +++ b/include/wine/winedxgi.idl @@ -46,4 +46,11 @@ interface IWineDXGIAdapter : IDXGIAdapter interface IWineDXGIDevice : IDXGIDevice { struct IWineD3DDevice *get_wined3d_device(); + HRESULT create_surface( + [in] const DXGI_SURFACE_DESC *desc, + [in] DXGI_USAGE usage, + [in] const DXGI_SHARED_RESOURCE *shared_resource, + [in] IUnknown *outer, + [out] void **surface + ); }

1 0

Henri Verbeet : d3d10core: Implement device_parent_CreateSurface().
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 43f469ec6a0518028c12c757895e2c483eafd4ec URL: http://source.winehq.org/git/wine.git/?a=commit;h=43f469ec6a0518028c12c7578… Author: Henri Verbeet <hverbeet(a)gmail.com> Date: Mon Jan 19 10:39:05 2009 +0100 d3d10core: Implement device_parent_CreateSurface(). --- .gitignore | 2 +- dlls/d3d10core/d3d10core_private.h | 3 + dlls/d3d10core/device.c | 64 +++++++++++++++++++- dlls/d3d10core/texture2d.c | 6 ++ dlls/dxgi/Makefile.in | 2 - dlls/dxgi/device.c | 48 ++++++++++---- dlls/dxgi/dxgi_private.h | 6 +- include/Makefile.in | 3 +- .../wine/winedxgi.idl | 12 ++++- tools/make_makefiles | 1 + 10 files changed, 123 insertions(+), 24 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=43f469ec6a0518028c12c…

1 0

Henri Verbeet : d3d10core: Implement device_parent_CreateRenderTarget().
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 57242b848f926fddb4323959b45be74f01ae39d0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=57242b848f926fddb4323959b… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 19 10:39:06 2009 +0100 d3d10core: Implement device_parent_CreateRenderTarget(). --- dlls/d3d10core/device.c | 30 +++++++++++++++++++++++++++++- 1 files changed, 29 insertions(+), 1 deletions(-) diff --git a/dlls/d3d10core/device.c b/dlls/d3d10core/device.c index 965bd49..3c19c39 100644 --- a/dlls/d3d10core/device.c +++ b/dlls/d3d10core/device.c @@ -1008,11 +1008,39 @@ static HRESULT STDMETHODCALLTYPE device_parent_CreateRenderTarget(IWineD3DDevice IUnknown *superior, UINT width, UINT height, WINED3DFORMAT format, WINED3DMULTISAMPLE_TYPE multisample_type, DWORD multisample_quality, BOOL lockable, IWineD3DSurface **surface) { + struct d3d10_device *This = device_from_device_parent(iface); + struct d3d10_texture2d *texture; + D3D10_TEXTURE2D_DESC desc; + HRESULT hr; + FIXME("iface %p, superior %p, width %u, height %u, format %#x, multisample_type %#x,\n" "\tmultisample_quality %u, lockable %u, surface %p stub!\n", iface, superior, width, height, format, multisample_type, multisample_quality, lockable, surface); - return E_NOTIMPL; + FIXME("Implement DXGI<->wined3d format and usage conversion\n"); + + desc.Width = width; + desc.Height = height; + desc.MipLevels = 1; + desc.ArraySize = 1; + desc.Format = format; + desc.SampleDesc.Count = multisample_type; + desc.SampleDesc.Quality = multisample_quality; + desc.Usage = D3D10_USAGE_DEFAULT; + desc.BindFlags = D3D10_BIND_RENDER_TARGET; + desc.CPUAccessFlags = 0; + desc.MiscFlags = 0; + + hr = d3d10_device_CreateTexture2D((ID3D10Device *)This, &desc, NULL, (ID3D10Texture2D **)&texture); + if (FAILED(hr)) + { + ERR("CreateTexture2D failed, returning %#x\n", hr); + return hr; + } + + *surface = texture->wined3d_surface; + + return S_OK; } static HRESULT STDMETHODCALLTYPE device_parent_CreateDepthStencilSurface(IWineD3DDeviceParent *iface,

1 0

Henri Verbeet : d3d10core: Implement device_parent_CreateDepthStencilSurface().
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: bb0e940e16c59e563957e75d8a45a7ab49f8d4a6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bb0e940e16c59e563957e75d8… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 19 10:39:06 2009 +0100 d3d10core: Implement device_parent_CreateDepthStencilSurface(). --- dlls/d3d10core/device.c | 30 +++++++++++++++++++++++++++++- 1 files changed, 29 insertions(+), 1 deletions(-) diff --git a/dlls/d3d10core/device.c b/dlls/d3d10core/device.c index 3c19c39..034c9b9 100644 --- a/dlls/d3d10core/device.c +++ b/dlls/d3d10core/device.c @@ -1047,11 +1047,39 @@ static HRESULT STDMETHODCALLTYPE device_parent_CreateDepthStencilSurface(IWineD3 IUnknown *superior, UINT width, UINT height, WINED3DFORMAT format, WINED3DMULTISAMPLE_TYPE multisample_type, DWORD multisample_quality, BOOL discard, IWineD3DSurface **surface) { + struct d3d10_device *This = device_from_device_parent(iface); + struct d3d10_texture2d *texture; + D3D10_TEXTURE2D_DESC desc; + HRESULT hr; + FIXME("iface %p, superior %p, width %u, height %u, format %#x, multisample_type %#x,\n" "\tmultisample_quality %u, discard %u, surface %p stub!\n", iface, superior, width, height, format, multisample_type, multisample_quality, discard, surface); - return E_NOTIMPL; + FIXME("Implement DXGI<->wined3d format and usage conversion\n"); + + desc.Width = width; + desc.Height = height; + desc.MipLevels = 1; + desc.ArraySize = 1; + desc.Format = format; + desc.SampleDesc.Count = multisample_type; + desc.SampleDesc.Quality = multisample_quality; + desc.Usage = D3D10_USAGE_DEFAULT; + desc.BindFlags = D3D10_BIND_RENDER_TARGET; + desc.CPUAccessFlags = 0; + desc.MiscFlags = 0; + + hr = d3d10_device_CreateTexture2D((ID3D10Device *)This, &desc, NULL, (ID3D10Texture2D **)&texture); + if (FAILED(hr)) + { + ERR("CreateTexture2D failed, returning %#x\n", hr); + return hr; + } + + *surface = texture->wined3d_surface; + + return S_OK; } static HRESULT STDMETHODCALLTYPE device_parent_CreateVolume(IWineD3DDeviceParent *iface,

1 0

Henri Verbeet : dxgi: Allow dxgi_surface to be aggregated.
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 23094bfad83ac6109b681d7a9faad387d4a9e7c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=23094bfad83ac6109b681d7a9… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 19 10:39:06 2009 +0100 dxgi: Allow dxgi_surface to be aggregated. --- dlls/dxgi/device.c | 2 + dlls/dxgi/dxgi_private.h | 3 ++ dlls/dxgi/surface.c | 54 +++++++++++++++++++++++++++++++++++++++------ 3 files changed, 51 insertions(+), 8 deletions(-) diff --git a/dlls/dxgi/device.c b/dlls/dxgi/device.c index 28bc9be..235e1f7 100644 --- a/dlls/dxgi/device.c +++ b/dlls/dxgi/device.c @@ -164,7 +164,9 @@ static HRESULT STDMETHODCALLTYPE dxgi_device_CreateSurface(IWineDXGIDevice *ifac } object->vtbl = &dxgi_surface_vtbl; + object->inner_unknown_vtbl = &dxgi_surface_inner_unknown_vtbl; object->refcount = 1; + object->outer_unknown = (IUnknown *)&object->inner_unknown_vtbl; surface[i] = (IDXGISurface *)object; TRACE("Created IDXGISurface %p (%u/%u)\n", object, i + 1, surface_count); diff --git a/dlls/dxgi/dxgi_private.h b/dlls/dxgi/dxgi_private.h index d19b9b1..4e6901c 100644 --- a/dlls/dxgi/dxgi_private.h +++ b/dlls/dxgi/dxgi_private.h @@ -81,9 +81,12 @@ struct dxgi_swapchain /* IDXGISurface */ extern const struct IDXGISurfaceVtbl dxgi_surface_vtbl; +extern const struct IUnknownVtbl dxgi_surface_inner_unknown_vtbl; struct dxgi_surface { const struct IDXGISurfaceVtbl *vtbl; + const struct IUnknownVtbl *inner_unknown_vtbl; + IUnknown *outer_unknown; LONG refcount; }; diff --git a/dlls/dxgi/surface.c b/dlls/dxgi/surface.c index 82db08b..bdbeaf3 100644 --- a/dlls/dxgi/surface.c +++ b/dlls/dxgi/surface.c @@ -24,10 +24,17 @@ WINE_DEFAULT_DEBUG_CHANNEL(dxgi); -/* IUnknown methods */ +/* Inner IUnknown methods */ -static HRESULT STDMETHODCALLTYPE dxgi_surface_QueryInterface(IDXGISurface *iface, REFIID riid, void **object) +static inline struct dxgi_surface *dxgi_surface_from_inner_unknown(IUnknown *iface) +{ + return (struct dxgi_surface *)((char*)iface - FIELD_OFFSET(struct dxgi_surface, inner_unknown_vtbl)); +} + +static HRESULT STDMETHODCALLTYPE dxgi_surface_inner_QueryInterface(IUnknown *iface, REFIID riid, void **object) { + struct dxgi_surface *This = dxgi_surface_from_inner_unknown(iface); + TRACE("iface %p, riid %s, object %p\n", iface, debugstr_guid(riid), object); if (IsEqualGUID(riid, &IID_IDXGISurface) @@ -35,8 +42,8 @@ static HRESULT STDMETHODCALLTYPE dxgi_surface_QueryInterface(IDXGISurface *iface || IsEqualGUID(riid, &IID_IDXGIObject) || IsEqualGUID(riid, &IID_IUnknown)) { - IUnknown_AddRef(iface); - *object = iface; + IUnknown_AddRef((IUnknown *)This); + *object = This; return S_OK; } @@ -46,9 +53,9 @@ static HRESULT STDMETHODCALLTYPE dxgi_surface_QueryInterface(IDXGISurface *iface return E_NOINTERFACE; } -static ULONG STDMETHODCALLTYPE dxgi_surface_AddRef(IDXGISurface *iface) +static ULONG STDMETHODCALLTYPE dxgi_surface_inner_AddRef(IUnknown *iface) { - struct dxgi_surface *This = (struct dxgi_surface *)iface; + struct dxgi_surface *This = dxgi_surface_from_inner_unknown(iface); ULONG refcount = InterlockedIncrement(&This->refcount); TRACE("%p increasing refcount to %u\n", This, refcount); @@ -56,9 +63,9 @@ static ULONG STDMETHODCALLTYPE dxgi_surface_AddRef(IDXGISurface *iface) return refcount; } -static ULONG STDMETHODCALLTYPE dxgi_surface_Release(IDXGISurface *iface) +static ULONG STDMETHODCALLTYPE dxgi_surface_inner_Release(IUnknown *iface) { - struct dxgi_surface *This = (struct dxgi_surface *)iface; + struct dxgi_surface *This = dxgi_surface_from_inner_unknown(iface); ULONG refcount = InterlockedDecrement(&This->refcount); TRACE("%p decreasing refcount to %u\n", This, refcount); @@ -71,6 +78,29 @@ static ULONG STDMETHODCALLTYPE dxgi_surface_Release(IDXGISurface *iface) return refcount; } +/* IUnknown methods */ + +static HRESULT STDMETHODCALLTYPE dxgi_surface_QueryInterface(IDXGISurface *iface, REFIID riid, void **object) +{ + struct dxgi_surface *This = (struct dxgi_surface *)iface; + TRACE("Forwarding to outer IUnknown\n"); + return IUnknown_QueryInterface(This->outer_unknown, riid, object); +} + +static ULONG STDMETHODCALLTYPE dxgi_surface_AddRef(IDXGISurface *iface) +{ + struct dxgi_surface *This = (struct dxgi_surface *)iface; + TRACE("Forwarding to outer IUnknown\n"); + return IUnknown_AddRef(This->outer_unknown); +} + +static ULONG STDMETHODCALLTYPE dxgi_surface_Release(IDXGISurface *iface) +{ + struct dxgi_surface *This = (struct dxgi_surface *)iface; + TRACE("Forwarding to outer IUnknown\n"); + return IUnknown_Release(This->outer_unknown); +} + /* IDXGIObject methods */ static HRESULT STDMETHODCALLTYPE dxgi_surface_SetPrivateData(IDXGISurface *iface, @@ -153,3 +183,11 @@ const struct IDXGISurfaceVtbl dxgi_surface_vtbl = dxgi_surface_Map, dxgi_surface_Unmap, }; + +const struct IUnknownVtbl dxgi_surface_inner_unknown_vtbl = +{ + /* IUnknown methods */ + dxgi_surface_inner_QueryInterface, + dxgi_surface_inner_AddRef, + dxgi_surface_inner_Release, +};

1 0

Michael Stefaniuc : dbghelp: Remove superfluous pointer casts.
by Alexandre Julliard 19 Jan '09

19 Jan '09

Module: wine Branch: master Commit: 3c3a9b50812a4e2ca967e1158f286a4e4bab059b URL: http://source.winehq.org/git/wine.git/?a=commit;h=3c3a9b50812a4e2ca967e1158… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jan 19 10:24:06 2009 +0100 dbghelp: Remove superfluous pointer casts. --- dlls/dbghelp/dwarf.c | 2 +- dlls/dbghelp/minidump.c | 8 ++++---- dlls/dbghelp/msc.c | 6 +++--- dlls/dbghelp/path.c | 10 +++++----- dlls/dbghelp/pe_module.c | 2 +- dlls/dbghelp/symbol.c | 8 ++++---- 6 files changed, 18 insertions(+), 18 deletions(-) diff --git a/dlls/dbghelp/dwarf.c b/dlls/dbghelp/dwarf.c index 3667c67..44e2cad 100644 --- a/dlls/dbghelp/dwarf.c +++ b/dlls/dbghelp/dwarf.c @@ -72,7 +72,7 @@ static void dump(const void* ptr, unsigned len) int i, j; BYTE msg[128]; static const char hexof[] = "0123456789abcdef"; - const BYTE* x = (const BYTE*)ptr; + const BYTE* x = ptr; for (i = 0; i < len; i += 16) { diff --git a/dlls/dbghelp/minidump.c b/dlls/dbghelp/minidump.c index 0419bc8..da0632a 100644 --- a/dlls/dbghelp/minidump.c +++ b/dlls/dbghelp/minidump.c @@ -282,13 +282,13 @@ static BOOL add_module(struct dump_context* dc, const WCHAR* name, static BOOL WINAPI fetch_pe_module_info_cb(PCWSTR name, DWORD64 base, ULONG size, PVOID user) { - struct dump_context* dc = (struct dump_context*)user; + struct dump_context* dc = user; IMAGE_NT_HEADERS nth; if (!validate_addr64(base)) return FALSE; if (pe_load_nt_header(dc->hProcess, base, &nth)) - add_module((struct dump_context*)user, name, base, size, + add_module(user, name, base, size, nth.FileHeader.TimeDateStamp, nth.OptionalHeader.CheckSum, FALSE); return TRUE; @@ -302,7 +302,7 @@ static BOOL WINAPI fetch_pe_module_info_cb(PCWSTR name, DWORD64 base, ULONG size static BOOL fetch_elf_module_info_cb(const WCHAR* name, unsigned long base, void* user) { - struct dump_context* dc = (struct dump_context*)user; + struct dump_context* dc = user; DWORD rbase, size, checksum; /* FIXME: there's no relevant timestamp on ELF modules */ @@ -971,7 +971,7 @@ BOOL WINAPI MiniDumpReadDumpStream(PVOID base, ULONG str_idx, PMINIDUMP_DIRECTORY* pdir, PVOID* stream, ULONG* size) { - MINIDUMP_HEADER* mdHead = (MINIDUMP_HEADER*)base; + MINIDUMP_HEADER* mdHead = base; if (mdHead->Signature == MINIDUMP_SIGNATURE) { diff --git a/dlls/dbghelp/msc.c b/dlls/dbghelp/msc.c index 564e745..f894afb 100644 --- a/dlls/dbghelp/msc.c +++ b/dlls/dbghelp/msc.c @@ -71,7 +71,7 @@ static void dump(const void* ptr, unsigned len) unsigned int i, j; char msg[128]; const char* hexof = "0123456789abcdef"; - const BYTE* x = (const BYTE*)ptr; + const BYTE* x = ptr; for (i = 0; i < len; i += 16) { @@ -2150,7 +2150,7 @@ static void pdb_convert_symbol_file(const PDB_SYMBOLS* symbols, { if (symbols->version < 19970000) { - const PDB_SYMBOL_FILE *sym_file = (const PDB_SYMBOL_FILE*)image; + const PDB_SYMBOL_FILE *sym_file = image; memset(sfile, 0, sizeof(*sfile)); sfile->file = sym_file->file; sfile->range.index = sym_file->range.index; @@ -2385,7 +2385,7 @@ static void pdb_process_symbol_imports(const struct process* pcs, imp = (const PDB_SYMBOL_IMPORT*)((const char*)symbols_image + sizeof(PDB_SYMBOLS) + symbols->module_size + symbols->offset_size + symbols->hash_size + symbols->srcmodule_size); - first = (const char*)imp; + first = imp; last = (const char*)imp + symbols->pdbimport_size; while (imp < (const PDB_SYMBOL_IMPORT*)last) { diff --git a/dlls/dbghelp/path.c b/dlls/dbghelp/path.c index e0cfc97..6af51d0 100644 --- a/dlls/dbghelp/path.c +++ b/dlls/dbghelp/path.c @@ -234,7 +234,7 @@ static BOOL do_searchW(PCWSTR file, PWSTR buffer, BOOL recurse, strcpyW(buffer + pos, fd.cFileName); if (recurse && (fd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)) found = do_searchW(file, buffer, TRUE, cb, user); - else if (SymMatchFileNameW(buffer, (WCHAR*)file, NULL, NULL)) + else if (SymMatchFileNameW(buffer, file, NULL, NULL)) { if (!cb || cb(buffer, user)) found = TRUE; } @@ -340,13 +340,13 @@ struct sffip */ static BOOL CALLBACK sffip_cb(PCWSTR buffer, PVOID user) { - struct sffip* s = (struct sffip*)user; + struct sffip* s = user; if (!s->cb) return TRUE; /* yes, EnumDirTree/do_search and SymFindFileInPath callbacks use the opposite * convention to stop/continue enumeration. sigh. */ - return !(s->cb)((WCHAR*)buffer, s->user); + return !(s->cb)(buffer, s->user); } /****************************************************************** @@ -461,7 +461,7 @@ struct module_find */ static BOOL CALLBACK module_find_cb(PCWSTR buffer, PVOID user) { - struct module_find* mf = (struct module_find*)user; + struct module_find* mf = user; DWORD size, checksum, timestamp; unsigned matched = 0; @@ -580,7 +580,7 @@ static BOOL CALLBACK module_find_cb(PCWSTR buffer, PVOID user) if ((mapping = MapViewOfFile(hMap, FILE_MAP_READ, 0, 0, 0)) != NULL) { const IMAGE_SEPARATE_DEBUG_HEADER* hdr; - hdr = (const IMAGE_SEPARATE_DEBUG_HEADER*)mapping; + hdr = mapping; if (hdr->Signature == IMAGE_SEPARATE_DEBUG_SIGNATURE) { diff --git a/dlls/dbghelp/pe_module.c b/dlls/dbghelp/pe_module.c index a79396f..4e77790 100644 --- a/dlls/dbghelp/pe_module.c +++ b/dlls/dbghelp/pe_module.c @@ -435,7 +435,7 @@ PVOID WINAPI ImageDirectoryEntryToDataEx( PVOID base, BOOLEAN image, USHORT dir, *size = nt->OptionalHeader.DataDirectory[dir].Size; if (image || addr < nt->OptionalHeader.SizeOfHeaders) return (char *)base + addr; - return RtlImageRvaToVa( nt, (HMODULE)base, addr, section ); + return RtlImageRvaToVa( nt, base, addr, section ); } /*********************************************************************** diff --git a/dlls/dbghelp/symbol.c b/dlls/dbghelp/symbol.c index 280666d..4df69d1 100644 --- a/dlls/dbghelp/symbol.c +++ b/dlls/dbghelp/symbol.c @@ -1046,7 +1046,7 @@ struct sym_enumerate static BOOL CALLBACK sym_enumerate_cb(PSYMBOL_INFO syminfo, ULONG size, void* ctx) { - struct sym_enumerate* se = (struct sym_enumerate*)ctx; + struct sym_enumerate* se = ctx; return (se->cb)(syminfo->Name, syminfo->Address, syminfo->Size, se->ctx); } @@ -1073,7 +1073,7 @@ struct sym_enumerate64 static BOOL CALLBACK sym_enumerate_cb64(PSYMBOL_INFO syminfo, ULONG size, void* ctx) { - struct sym_enumerate64* se = (struct sym_enumerate64*)ctx; + struct sym_enumerate64* se = ctx; return (se->cb)(syminfo->Name, syminfo->Address, syminfo->Size, se->ctx); } @@ -1473,7 +1473,7 @@ BOOL WINAPI SymGetLinePrev(HANDLE hProcess, PIMAGEHLP_LINE Line) if (!module_get_debug(&pair)) return FALSE; if (Line->Key == 0) return FALSE; - li = (struct line_info*)Line->Key; + li = Line->Key; /* things are a bit complicated because when we encounter a DLIT_SOURCEFILE * element we have to go back until we find the prev one to get the real * source file name for the DLIT_OFFSET element just before @@ -1523,7 +1523,7 @@ BOOL symt_get_func_line_next(const struct module* module, PIMAGEHLP_LINE line) struct line_info* li; if (line->Key == 0) return FALSE; - li = (struct line_info*)line->Key; + li = line->Key; while (!li->is_last) { li++;

1 0