February 2017 - Wine-Commits

Nikolay Sivov : uxtheme: Partial implementation of BeginBufferedPaint().
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 35ae7c8f5939c0ba77b4023a14596706852f5acd URL: http://source.winehq.org/git/wine.git/?a=commit;h=35ae7c8f5939c0ba77b4023a1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Feb 9 01:42:34 2017 +0300 uxtheme: Partial implementation of BeginBufferedPaint(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/uxtheme/buffer.c | 92 +++++++++++++++++++++++++++++++++++++-------- dlls/uxtheme/tests/system.c | 40 ++++++++++++++++---- 2 files changed, 109 insertions(+), 23 deletions(-) diff --git a/dlls/uxtheme/buffer.c b/dlls/uxtheme/buffer.c index 8583f1a..972df68 100644 --- a/dlls/uxtheme/buffer.c +++ b/dlls/uxtheme/buffer.c @@ -2,6 +2,7 @@ * uxtheme Double-buffered Drawing API * * Copyright (C) 2008 Reece H. Dunn + * Copyright 2017 Nikolay Sivov for CodeWeavers * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -34,6 +35,20 @@ WINE_DEFAULT_DEBUG_CHANNEL(uxtheme); +struct paintbuffer +{ + HDC targetdc; + HDC memorydc; + RECT rect; + void *bits; +}; + +static void free_paintbuffer(struct paintbuffer *buffer) +{ + DeleteDC(buffer->memorydc); + HeapFree(GetProcessHeap(), 0, buffer); +} + /*********************************************************************** * BufferedPaintInit (UXTHEME.@) */ @@ -55,24 +70,71 @@ HRESULT WINAPI BufferedPaintUnInit(VOID) /*********************************************************************** * BeginBufferedPaint (UXTHEME.@) */ -HPAINTBUFFER WINAPI BeginBufferedPaint(HDC hdcTarget, - const RECT * prcTarget, - BP_BUFFERFORMAT dwFormat, - BP_PAINTPARAMS *pPaintParams, - HDC *phdc) +HPAINTBUFFER WINAPI BeginBufferedPaint(HDC targetdc, const RECT *rect, + BP_BUFFERFORMAT format, BP_PAINTPARAMS *params, HDC *retdc) { - static int i; - - TRACE("Stub (%p %p %d %p %p)\n", hdcTarget, prcTarget, dwFormat, - pPaintParams, phdc); - - if (!i++) - FIXME("Stub (%p %p %d %p %p)\n", hdcTarget, prcTarget, dwFormat, - pPaintParams, phdc); - return NULL; + char bmibuf[FIELD_OFFSET(BITMAPINFO, bmiColors[256])]; + BITMAPINFO *bmi = (BITMAPINFO *)bmibuf; + struct paintbuffer *buffer; + HBITMAP hbm; + + TRACE("(%p %s %d %p %p)\n", targetdc, wine_dbgstr_rect(rect), format, + params, retdc); + + if (retdc) + *retdc = NULL; + + if (!targetdc || IsRectEmpty(rect)) + return NULL; + + if (params) + FIXME("painting parameters are ignored\n"); + + buffer = HeapAlloc(GetProcessHeap(), 0, sizeof(*buffer)); + buffer->targetdc = targetdc; + buffer->rect = *rect; + buffer->memorydc = CreateCompatibleDC(targetdc); + + switch (format) + { + case BPBF_COMPATIBLEBITMAP: + hbm = CreateCompatibleBitmap(buffer->memorydc, rect->right - rect->left, rect->bottom - rect->top); + buffer->bits = NULL; + break; + case BPBF_DIB: + case BPBF_TOPDOWNDIB: + case BPBF_TOPDOWNMONODIB: + /* create DIB section */ + memset(bmi, 0, sizeof(bmibuf)); + bmi->bmiHeader.biSize = sizeof(bmi->bmiHeader); + bmi->bmiHeader.biHeight = format == BPBF_DIB ? rect->bottom - rect->top : + -(rect->bottom - rect->top); + bmi->bmiHeader.biWidth = rect->right - rect->left; + bmi->bmiHeader.biBitCount = format == BPBF_TOPDOWNMONODIB ? 1 : 32; + bmi->bmiHeader.biPlanes = 1; + bmi->bmiHeader.biCompression = BI_RGB; + hbm = CreateDIBSection(buffer->memorydc, bmi, DIB_RGB_COLORS, &buffer->bits, NULL, 0); + break; + default: + WARN("Unknown buffer format %d\n", format); + free_paintbuffer(buffer); + return NULL; + } + + if (!hbm) + { + WARN("Failed to create buffer bitmap\n"); + free_paintbuffer(buffer); + return NULL; + } + + DeleteObject(SelectObject(buffer->memorydc, hbm)); + + *retdc = buffer->memorydc; + + return (HPAINTBUFFER)buffer; } - /*********************************************************************** * EndBufferedPaint (UXTHEME.@) */ diff --git a/dlls/uxtheme/tests/system.c b/dlls/uxtheme/tests/system.c index d7ba43c..66f696b 100644 --- a/dlls/uxtheme/tests/system.c +++ b/dlls/uxtheme/tests/system.c @@ -552,31 +552,55 @@ static void test_buffered_paint(void) &params, NULL); ok(buffer == NULL, "Unexpected buffer %p\n", buffer); + src = (void *)0xdeadbeef; buffer = pBeginBufferedPaint(target, NULL, BPBF_COMPATIBLEBITMAP, &params, &src); ok(buffer == NULL, "Unexpected buffer %p\n", buffer); + ok(src == NULL, "Unexpected buffered dc %p\n", src); /* target rect is mandatory */ - rect.left = rect.top = 0; - rect.right = rect.bottom = 0; + SetRectEmpty(&rect); + src = (void *)0xdeadbeef; buffer = pBeginBufferedPaint(target, &rect, BPBF_COMPATIBLEBITMAP, &params, &src); ok(buffer == NULL, "Unexpected buffer %p\n", buffer); + ok(src == NULL, "Unexpected buffered dc %p\n", src); - rect.left = rect.top = 0; - rect.right = rect.bottom = 5; + /* inverted rectangle */ + SetRect(&rect, 10, 0, 5, 5); + src = (void *)0xdeadbeef; + buffer = pBeginBufferedPaint(target, &rect, BPBF_COMPATIBLEBITMAP, + &params, &src); + ok(buffer == NULL, "Unexpected buffer %p\n", buffer); + ok(src == NULL, "Unexpected buffered dc %p\n", src); + + SetRect(&rect, 0, 10, 5, 0); + src = (void *)0xdeadbeef; + buffer = pBeginBufferedPaint(target, &rect, BPBF_COMPATIBLEBITMAP, + &params, &src); + ok(buffer == NULL, "Unexpected buffer %p\n", buffer); + ok(src == NULL, "Unexpected buffered dc %p\n", src); + + /* valid rectangle, no target dc */ + SetRect(&rect, 0, 0, 5, 5); + src = (void *)0xdeadbeef; + buffer = pBeginBufferedPaint(NULL, &rect, BPBF_COMPATIBLEBITMAP, + &params, &src); + ok(buffer == NULL, "Unexpected buffer %p\n", buffer); + ok(src == NULL, "Unexpected buffered dc %p\n", src); + + SetRect(&rect, 0, 0, 5, 5); + src = NULL; buffer = pBeginBufferedPaint(target, &rect, BPBF_COMPATIBLEBITMAP, &params, &src); -todo_wine ok(buffer != NULL, "Unexpected buffer %p\n", buffer); + ok(src != NULL, "Expected buffered dc\n"); hr = pEndBufferedPaint(buffer, FALSE); ok(hr == S_OK, "Unexpected return code %#x\n", hr); - rect.left = rect.top = 0; - rect.right = rect.bottom = 5; + SetRect(&rect, 0, 0, 5, 5); buffer = pBeginBufferedPaint(target, &rect, BPBF_COMPATIBLEBITMAP, &params, &src); -todo_wine ok(buffer != NULL, "Unexpected buffer %p\n", buffer); /* clearing */

1 0

Francois Gouget : schedsvc: Simplify and standardize the heap_xxx() declarations.
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 223ea1312be779843dcca7b07cc954720240a219 URL: http://source.winehq.org/git/wine.git/?a=commit;h=223ea1312be779843dcca7b07… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Feb 8 17:56:06 2017 +0100 schedsvc: Simplify and standardize the heap_xxx() declarations. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/schedsvc/schedsvc_private.h | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/dlls/schedsvc/schedsvc_private.h b/dlls/schedsvc/schedsvc_private.h index 4138aa0..3abce07 100644 --- a/dlls/schedsvc/schedsvc_private.h +++ b/dlls/schedsvc/schedsvc_private.h @@ -23,22 +23,19 @@ void schedsvc_auto_start(void) DECLSPEC_HIDDEN; -static void *heap_alloc_zero(SIZE_T size) __WINE_ALLOC_SIZE(1); -static inline void *heap_alloc_zero(SIZE_T size) +static inline void* __WINE_ALLOC_SIZE(1) heap_alloc(SIZE_T size) { - void *ptr = MIDL_user_allocate(size); - if (ptr) memset(ptr, 0, size); - return ptr; + return MIDL_user_allocate(size); } -static void *heap_alloc(SIZE_T size) __WINE_ALLOC_SIZE(1); -static inline void *heap_alloc(SIZE_T size) +static inline void* __WINE_ALLOC_SIZE(1) heap_alloc_zero(SIZE_T size) { - return MIDL_user_allocate(size); + void *ptr = MIDL_user_allocate(size); + if (ptr) memset(ptr, 0, size); + return ptr; } -static void *heap_realloc(void *ptr, SIZE_T size) __WINE_ALLOC_SIZE(2); -static inline void *heap_realloc(void *ptr, SIZE_T size) +static inline void* __WINE_ALLOC_SIZE(2) heap_realloc(void *ptr, SIZE_T size) { return HeapReAlloc(GetProcessHeap(), 0, ptr, size); }

1 0

Francois Gouget : ole32: Add a __WINE_ALLOC_SIZE attribute to heap_alloc() .
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 93a8ede0e854cb954d81c43c5220481f917189d4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=93a8ede0e854cb954d81c43c5… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Feb 8 17:53:00 2017 +0100 ole32: Add a __WINE_ALLOC_SIZE attribute to heap_alloc(). Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/compobj_private.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ole32/compobj_private.h b/dlls/ole32/compobj_private.h index 7ceee2c..b9d1e85 100644 --- a/dlls/ole32/compobj_private.h +++ b/dlls/ole32/compobj_private.h @@ -320,7 +320,7 @@ extern BOOL actctx_get_miscstatus(const CLSID*, DWORD, DWORD*) DECLSPEC_HIDDEN; extern const char *debugstr_formatetc(const FORMATETC *formatetc) DECLSPEC_HIDDEN; -static inline void *heap_alloc(size_t len) +static inline void* __WINE_ALLOC_SIZE(1) heap_alloc(size_t len) { return HeapAlloc(GetProcessHeap(), 0, len); }

1 0

Francois Gouget : appwiz.cpl: Add __WINE_ALLOC_SIZE attributes to heap_xxx () functions.
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 71aa8ee1a2b854405851fc92054d330698b81653 URL: http://source.winehq.org/git/wine.git/?a=commit;h=71aa8ee1a2b854405851fc920… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Feb 8 17:52:14 2017 +0100 appwiz.cpl: Add __WINE_ALLOC_SIZE attributes to heap_xxx() functions. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/appwiz.cpl/appwiz.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/appwiz.cpl/appwiz.h b/dlls/appwiz.cpl/appwiz.h index 77636ae..1e60daf 100644 --- a/dlls/appwiz.cpl/appwiz.h +++ b/dlls/appwiz.cpl/appwiz.h @@ -27,12 +27,12 @@ BOOL install_addon(addon_t) DECLSPEC_HIDDEN; extern HINSTANCE hInst DECLSPEC_HIDDEN; -static inline void *heap_alloc(size_t len) +static inline void* __WINE_ALLOC_SIZE(1) heap_alloc(size_t len) { return HeapAlloc(GetProcessHeap(), 0, len); } -static inline void *heap_realloc(void *mem, size_t len) +static inline void* __WINE_ALLOC_SIZE(2) heap_realloc(void *mem, size_t len) { return HeapReAlloc(GetProcessHeap(), 0, mem, len); }

1 0

Henri Verbeet : d3d8: Do not modify the "texture" pointer for D3DFMT_UNKNOWN in d3d8_device_CreateCubeTexture().
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 9e16f478c1c1d18e35d0964ca93378626fede569 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e16f478c1c1d18e35d0964ca… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Feb 9 12:52:39 2017 +0100 d3d8: Do not modify the "texture" pointer for D3DFMT_UNKNOWN in d3d8_device_CreateCubeTexture(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d8/device.c | 3 +++ dlls/d3d8/tests/device.c | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/dlls/d3d8/device.c b/dlls/d3d8/device.c index 989c1b0..ad5880c 100644 --- a/dlls/d3d8/device.c +++ b/dlls/d3d8/device.c @@ -853,6 +853,9 @@ static HRESULT WINAPI d3d8_device_CreateCubeTexture(IDirect3DDevice8 *iface, UIN TRACE("iface %p, edge_length %u, levels %u, usage %#x, format %#x, pool %#x, texture %p.\n", iface, edge_length, levels, usage, format, pool, texture); + if (!format) + return D3DERR_INVALIDCALL; + *texture = NULL; object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)); if (!object) diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c index c05c9f2..ff3c9b4 100644 --- a/dlls/d3d8/tests/device.c +++ b/dlls/d3d8/tests/device.c @@ -8236,7 +8236,7 @@ static void test_format_unknown(void) hr = IDirect3DDevice8_CreateCubeTexture(device, 64, 1, 0, D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DCubeTexture8 **)&iface); ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); - todo_wine ok(iface == (void *)0xdeadbeef, "Got unexpected iface %p.\n", iface); + ok(iface == (void *)0xdeadbeef, "Got unexpected iface %p.\n", iface); iface = (void *)0xdeadbeef; hr = IDirect3DDevice8_CreateVolumeTexture(device, 64, 64, 1, 1, 0,

1 0

Henri Verbeet : d3d9/tests: Add a test for resource creation with D3DFMT_UNKNOWN.
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: b28631f62f2a20c61a656d0f810710407424b899 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b28631f62f2a20c61a656d0f8… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Feb 9 12:52:36 2017 +0100 d3d9/tests: Add a test for resource creation with D3DFMT_UNKNOWN. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d9/tests/d3d9ex.c | 76 ++++++++++++++++++++++++++++++++++++++++++++++++ dlls/d3d9/tests/device.c | 64 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 140 insertions(+) diff --git a/dlls/d3d9/tests/d3d9ex.c b/dlls/d3d9/tests/d3d9ex.c index f949f5c..6348ee7 100644 --- a/dlls/d3d9/tests/d3d9ex.c +++ b/dlls/d3d9/tests/d3d9ex.c @@ -3701,6 +3701,81 @@ static void test_backbuffer_resize(void) DestroyWindow(window); } +static void test_format_unknown(void) +{ + IDirect3DDevice9Ex *device; + UINT refcount; + HWND window; + void *iface; + HRESULT hr; + + window = create_window(); + if (!(device = create_device(window, NULL))) + { + skip("Failed to create a D3D device.\n"); + DestroyWindow(window); + return; + } + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateRenderTarget(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, 0, FALSE, (IDirect3DSurface9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateRenderTargetEx(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, 0, FALSE, (IDirect3DSurface9 **)&iface, NULL, 0); + todo_wine ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateDepthStencilSurface(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, 0, TRUE, (IDirect3DSurface9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateDepthStencilSurfaceEx(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, 0, TRUE, (IDirect3DSurface9 **)&iface, NULL, 0); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + todo_wine ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateOffscreenPlainSurface(device, 64, 64, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DSurface9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateOffscreenPlainSurfaceEx(device, 64, 64, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DSurface9 **)&iface, NULL, 9); + todo_wine ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + todo_wine ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateTexture(device, 64, 64, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DTexture9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateCubeTexture(device, 64, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DCubeTexture9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9Ex_CreateVolumeTexture(device, 64, 64, 1, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DVolumeTexture9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + refcount = IDirect3DDevice9Ex_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); + DestroyWindow(window); +} + START_TEST(d3d9ex) { DEVMODEW current_mode; @@ -3749,4 +3824,5 @@ START_TEST(d3d9ex) test_window_style(); test_swapchain_parameters(); test_backbuffer_resize(); + test_format_unknown(); } diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c index e776515..feb1efc 100644 --- a/dlls/d3d9/tests/device.c +++ b/dlls/d3d9/tests/device.c @@ -11443,6 +11443,69 @@ static void test_render_target_device_mismatch(void) DestroyWindow(window); } +static void test_format_unknown(void) +{ + IDirect3DDevice9 *device; + IDirect3D9 *d3d; + UINT refcount; + HWND window; + void *iface; + HRESULT hr; + + window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, NULL, NULL, NULL, NULL); + d3d = Direct3DCreate9(D3D_SDK_VERSION); + ok(!!d3d, "Failed to create a D3D object.\n"); + if (!(device = create_device(d3d, window, NULL))) + { + skip("Failed to create a D3D device.\n"); + IDirect3D9_Release(d3d); + DestroyWindow(window); + return; + } + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9_CreateRenderTarget(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, 0, FALSE, (IDirect3DSurface9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9_CreateDepthStencilSurface(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, 0, TRUE, (IDirect3DSurface9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9_CreateOffscreenPlainSurface(device, 64, 64, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DSurface9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9_CreateTexture(device, 64, 64, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DTexture9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9_CreateCubeTexture(device, 64, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DCubeTexture9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice9_CreateVolumeTexture(device, 64, 64, 1, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DVolumeTexture9 **)&iface, NULL); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + refcount = IDirect3DDevice9_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); + IDirect3D9_Release(d3d); + DestroyWindow(window); +} + START_TEST(device) { WNDCLASSA wc = {0}; @@ -11560,6 +11623,7 @@ START_TEST(device) test_miptree_layout(); test_get_render_target_data(); test_render_target_device_mismatch(); + test_format_unknown(); UnregisterClassA("d3d9_test_wc", GetModuleHandleA(NULL)); }

1 0

Henri Verbeet : d3d8/tests: Add a test for resource creation with D3DFMT_UNKNOWN.
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 1b007ad38d2d01f7c8830f9f33d3cec69dc5cc4b URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b007ad38d2d01f7c8830f9f3… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Feb 9 12:52:35 2017 +0100 d3d8/tests: Add a test for resource creation with D3DFMT_UNKNOWN. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d8/tests/device.c | 102 +++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 102 insertions(+) diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c index 87d2433..da67fb3 100644 --- a/dlls/d3d8/tests/device.c +++ b/dlls/d3d8/tests/device.c @@ -8149,6 +8149,107 @@ static void test_render_target_device_mismatch(void) DestroyWindow(window); } +static void test_format_unknown(void) +{ + IDirect3DDevice8 *device; + IDirect3D8 *d3d; + UINT refcount; + HWND window; + void *iface; + HRESULT hr; + + window = CreateWindowA("static", "d3d8_test", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, NULL, NULL, NULL, NULL); + d3d = Direct3DCreate8(D3D_SDK_VERSION); + ok(!!d3d, "Failed to create a D3D object.\n"); + if (!(device = create_device(d3d, window, NULL))) + { + skip("Failed to create a D3D device.\n"); + IDirect3D8_Release(d3d); + DestroyWindow(window); + return; + } + + if (SUCCEEDED(IDirect3D8_CheckDeviceFormat(d3d, D3DADAPTER_DEFAULT, D3DDEVTYPE_HAL, + D3DFMT_X8R8G8B8, 0, D3DRTYPE_TEXTURE, D3DFMT_P8))) + { + skip("P8 textures are supported, skipping some tests.\n"); + } + else + { + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateRenderTarget(device, 64, 64, + D3DFMT_P8, D3DMULTISAMPLE_NONE, FALSE, (IDirect3DSurface8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateDepthStencilSurface(device, 64, 64, + D3DFMT_P8, D3DMULTISAMPLE_NONE, (IDirect3DSurface8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateTexture(device, 64, 64, 1, 0, + D3DFMT_P8, D3DPOOL_DEFAULT, (IDirect3DTexture8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateCubeTexture(device, 64, 1, 0, + D3DFMT_P8, D3DPOOL_DEFAULT, (IDirect3DCubeTexture8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateVolumeTexture(device, 64, 64, 1, 1, 0, + D3DFMT_P8, D3DPOOL_DEFAULT, (IDirect3DVolumeTexture8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + } + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateRenderTarget(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, FALSE, (IDirect3DSurface8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + todo_wine ok(iface == (void *)0xdeadbeef, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateDepthStencilSurface(device, 64, 64, + D3DFMT_UNKNOWN, D3DMULTISAMPLE_NONE, (IDirect3DSurface8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + todo_wine ok(iface == (void *)0xdeadbeef, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateImageSurface(device, 64, 64, + D3DFMT_UNKNOWN, (IDirect3DSurface8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + ok(!iface, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateTexture(device, 64, 64, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DTexture8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + todo_wine ok(iface == (void *)0xdeadbeef, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateCubeTexture(device, 64, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DCubeTexture8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + todo_wine ok(iface == (void *)0xdeadbeef, "Got unexpected iface %p.\n", iface); + + iface = (void *)0xdeadbeef; + hr = IDirect3DDevice8_CreateVolumeTexture(device, 64, 64, 1, 1, 0, + D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, (IDirect3DVolumeTexture8 **)&iface); + ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); + todo_wine ok(iface == (void *)0xdeadbeef, "Got unexpected iface %p.\n", iface); + + refcount = IDirect3DDevice8_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); + IDirect3D8_Release(d3d); + DestroyWindow(window); +} + START_TEST(device) { HMODULE d3d8_handle = LoadLibraryA( "d3d8.dll" ); @@ -8254,6 +8355,7 @@ START_TEST(device) test_check_device_format(); test_miptree_layout(); test_render_target_device_mismatch(); + test_format_unknown(); UnregisterClassA("d3d8_test_wc", GetModuleHandleA(NULL)); }

1 0

Nikolay Sivov : uxtheme: Implement GetBufferedPaintTargetRect().
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 89fe0a3f26d6e2ce6eb3d3d42f560c6c6c9b3e23 URL: http://source.winehq.org/git/wine.git/?a=commit;h=89fe0a3f26d6e2ce6eb3d3d42… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Feb 9 01:42:38 2017 +0300 uxtheme: Implement GetBufferedPaintTargetRect(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/uxtheme/buffer.c | 16 +++++++++++++--- dlls/uxtheme/tests/system.c | 10 +++++++--- 2 files changed, 20 insertions(+), 6 deletions(-) diff --git a/dlls/uxtheme/buffer.c b/dlls/uxtheme/buffer.c index e2c5756..853fe82 100644 --- a/dlls/uxtheme/buffer.c +++ b/dlls/uxtheme/buffer.c @@ -223,10 +223,20 @@ HDC WINAPI GetBufferedPaintTargetDC(HPAINTBUFFER bufferhandle) /*********************************************************************** * GetBufferedPaintTargetRect (UXTHEME.@) */ -HRESULT WINAPI GetBufferedPaintTargetRect(HPAINTBUFFER hBufferedPaint, RECT *prc) +HRESULT WINAPI GetBufferedPaintTargetRect(HPAINTBUFFER bufferhandle, RECT *rect) { - FIXME("Stub (%p %p)\n", hBufferedPaint, prc); - return E_NOTIMPL; + struct paintbuffer *buffer = get_buffer_obj(bufferhandle); + + TRACE("(%p %p)\n", buffer, rect); + + if (!rect) + return E_POINTER; + + if (!buffer) + return E_FAIL; + + *rect = buffer->rect; + return S_OK; } /*********************************************************************** diff --git a/dlls/uxtheme/tests/system.c b/dlls/uxtheme/tests/system.c index 44aebb8..6c08635 100644 --- a/dlls/uxtheme/tests/system.c +++ b/dlls/uxtheme/tests/system.c @@ -619,16 +619,20 @@ todo_wine hdc = pGetBufferedPaintTargetDC(buffer); ok(hdc == target, "Unexpected target hdc %p, original %p\n", hdc, target); + hr = pGetBufferedPaintTargetRect(NULL, NULL); + ok(hr == E_POINTER, "Unexpected return code %#x\n", hr); + hr = pGetBufferedPaintTargetRect(buffer, NULL); -todo_wine ok(hr == E_POINTER, "Unexpected return code %#x\n", hr); + hr = pGetBufferedPaintTargetRect(NULL, &rect2); + ok(hr == E_FAIL, "Unexpected return code %#x\n", hr); + SetRectEmpty(&rect2); hr = pGetBufferedPaintTargetRect(buffer, &rect2); -todo_wine { ok(hr == S_OK, "Unexpected return code %#x\n", hr); ok(EqualRect(&rect, &rect2), "Wrong target rect\n"); -} + hr = pEndBufferedPaint(buffer, FALSE); ok(hr == S_OK, "Unexpected return code %#x\n", hr);

1 0

Nikolay Sivov : uxtheme: Implement GetBufferedPaintTargetDC().
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 9f9842ba192b2fd28d2773d3e3ac490f301be173 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9f9842ba192b2fd28d2773d3e… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Feb 9 01:42:37 2017 +0300 uxtheme: Implement GetBufferedPaintTargetDC(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/uxtheme/buffer.c | 9 ++++++--- dlls/uxtheme/tests/system.c | 1 - 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/dlls/uxtheme/buffer.c b/dlls/uxtheme/buffer.c index f85761f..e2c5756 100644 --- a/dlls/uxtheme/buffer.c +++ b/dlls/uxtheme/buffer.c @@ -211,10 +211,13 @@ HDC WINAPI GetBufferedPaintDC(HPAINTBUFFER bufferhandle) /*********************************************************************** * GetBufferedPaintTargetDC (UXTHEME.@) */ -HDC WINAPI GetBufferedPaintTargetDC(HPAINTBUFFER hBufferedPaint) +HDC WINAPI GetBufferedPaintTargetDC(HPAINTBUFFER bufferhandle) { - FIXME("Stub (%p)\n", hBufferedPaint); - return NULL; + struct paintbuffer *buffer = get_buffer_obj(bufferhandle); + + TRACE("(%p)\n", buffer); + + return buffer ? buffer->targetdc : NULL; } /*********************************************************************** diff --git a/dlls/uxtheme/tests/system.c b/dlls/uxtheme/tests/system.c index 83e8cde..44aebb8 100644 --- a/dlls/uxtheme/tests/system.c +++ b/dlls/uxtheme/tests/system.c @@ -617,7 +617,6 @@ todo_wine ok(hdc == src, "Unexpected hdc, %p, buffered dc %p\n", hdc, src); hdc = pGetBufferedPaintTargetDC(buffer); -todo_wine ok(hdc == target, "Unexpected target hdc %p, original %p\n", hdc, target); hr = pGetBufferedPaintTargetRect(buffer, NULL);

1 0

Francois Gouget : wmp: Add __WINE_ALLOC_SIZE attributes to heap_xxx() functions.
by Alexandre Julliard Feb. 9, 2017

Feb. 9, 2017

Module: wine Branch: master Commit: 97fa31eef22507057ce74abc904d6fb0105c44c4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=97fa31eef22507057ce74abc9… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Feb 8 17:53:56 2017 +0100 wmp: Add __WINE_ALLOC_SIZE attributes to heap_xxx() functions. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wmp/wmp_private.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wmp/wmp_private.h b/dlls/wmp/wmp_private.h index 91e8465..a44aa35 100644 --- a/dlls/wmp/wmp_private.h +++ b/dlls/wmp/wmp_private.h @@ -46,12 +46,12 @@ void unregister_wmp_class(void) DECLSPEC_HIDDEN; extern HINSTANCE wmp_instance DECLSPEC_HIDDEN; -static inline void *heap_alloc(size_t len) +static inline void* __WINE_ALLOC_SIZE(1) heap_alloc(size_t len) { return HeapAlloc(GetProcessHeap(), 0, len); } -static inline void *heap_alloc_zero(size_t len) +static inline void* __WINE_ALLOC_SIZE(1) heap_alloc_zero(size_t len) { return HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, len); }

1 0