February 2014 - Wine-commits

Alexandre Julliard : makefiles: Generate the testlist.c files at make depend time.
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: 3bae3c6514475b17ac884508c26363c6bfd28d0d URL: http://source.winehq.org/git/wine.git/?a=commit;h=3bae3c6514475b17ac884508c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 10 10:55:18 2014 +0100 makefiles: Generate the testlist.c files at make depend time. --- aclocal.m4 | 4 +- configure | 4 +- tools/Makefile.in | 5 -- tools/make_ctests.c | 160 --------------------------------------------------- tools/makedep.c | 108 ++++++++++++++++++++++++++++------ 5 files changed, 97 insertions(+), 184 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=3bae3c6514475b17ac884…

1 0

Alexandre Julliard : makefiles: Add makefile variables for the winegcc parameters.
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: 658057ae0b7ec0230c179616ac2d7228e5a7eeae URL: http://source.winehq.org/git/wine.git/?a=commit;h=658057ae0b7ec0230c179616a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 10 10:57:53 2014 +0100 makefiles: Add makefile variables for the winegcc parameters. --- Make.vars.in | 2 ++ tools/winegcc/Makefile.in | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/Make.vars.in b/Make.vars.in index 684d673..ee8ee39 100644 --- a/Make.vars.in +++ b/Make.vars.in @@ -18,6 +18,8 @@ SHELL = /bin/sh RM = rm -f MV = mv CC = @CC@ +CXX = @CXX@ +CPPBIN = @CPPBIN@ CROSSCC = @CROSSCC@ CFLAGS = @CFLAGS@ CPPFLAGS = @CPPFLAGS@ diff --git a/tools/winegcc/Makefile.in b/tools/winegcc/Makefile.in index bb7e4a2..3975ca5 100644 --- a/tools/winegcc/Makefile.in +++ b/tools/winegcc/Makefile.in @@ -14,8 +14,8 @@ winegcc_EXTRADEFS = \ -DDLLDIR="\"${dlldir}\"" \ -DLIBDIR="\"${libdir}\"" \ -DCC="\"$(CC)\"" \ - -DCPP="\"@CPPBIN@\"" \ - -DCXX="\"@CXX@\"" \ + -DCPP="\"$(CPPBIN)\"" \ + -DCXX="\"$(CXX)\"" \ -DPRELINK="\"$(PRELINK)\"" winegcc$(EXEEXT): winegcc.o utils.o

1 0

Henri Verbeet : dxgi/tests: Cleanup test_output().
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: 19f204437529d42f579581d427655655369c46f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=19f204437529d42f579581d42… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Feb 10 10:37:53 2014 +0100 dxgi/tests: Cleanup test_output(). In particular, actually test the return codes for most calls. --- dlls/dxgi/tests/device.c | 54 +++++++++++++++++++++++++--------------------- 1 file changed, 30 insertions(+), 24 deletions(-) diff --git a/dlls/dxgi/tests/device.c b/dlls/dxgi/tests/device.c index b8e7b33..ba1a29f 100644 --- a/dlls/dxgi/tests/device.c +++ b/dlls/dxgi/tests/device.c @@ -255,44 +255,47 @@ static void test_output(IDXGIDevice *device) hr = IDXGIAdapter_EnumOutputs(adapter, 0, &output); if (hr == DXGI_ERROR_NOT_FOUND) { - skip("Adapter has not outputs, skipping output tests.\n"); + skip("Adapter doesn't have any outputs, skipping tests.\n"); IDXGIAdapter_Release(adapter); return; } - ok(SUCCEEDED(hr), "EnumOutputs failed, hr %#x.\n", hr); - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, 0, NULL, NULL); - ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, 0, NULL, NULL); + todo_wine ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr); - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, 0, &mode_count, NULL); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, 0, &mode_count, NULL); ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); mode_count_comp = mode_count; - IDXGIOutput_GetDisplayModeList(output, 0, 0, &mode_count, NULL); + hr = IDXGIOutput_GetDisplayModeList(output, 0, 0, &mode_count, NULL); ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); - ok(!mode_count, "Expected 0 got %d\n", mode_count); + ok(!mode_count, "Got unexpected mode_count %u.\n", mode_count); - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, &mode_count, NULL); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, + DXGI_ENUM_MODES_SCALING, &mode_count, NULL); ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); - ok(mode_count >= mode_count_comp, "Flag implies trying to enumerate more modes\n"); + ok(mode_count >= mode_count_comp, "Got unexpected mode_count %u, expected >= %u.\n", mode_count, mode_count_comp); mode_count_comp = mode_count; - modes = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(DXGI_MODE_DESC) * mode_count+10); + modes = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*modes) * (mode_count + 10)); - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, NULL, modes); - ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); - ok(!modes[0].Height, "No output was expected\n"); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, + DXGI_ENUM_MODES_SCALING, NULL, modes); + todo_wine ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr); + ok(!modes[0].Height, "No output was expected.\n"); mode_count = 0; - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, &mode_count, modes); - ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); - ok(!modes[0].Height, "No output was expected\n"); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, + DXGI_ENUM_MODES_SCALING, &mode_count, modes); + todo_wine ok(hr == DXGI_ERROR_MORE_DATA, "Got unexpected hr %#x.\n", hr); + ok(!modes[0].Height, "No output was expected.\n"); mode_count = mode_count_comp; - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, &mode_count, modes); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, + DXGI_ENUM_MODES_SCALING, &mode_count, modes); ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); - ok(mode_count == mode_count_comp, "Expected %d, got %d\n", mode_count_comp, mode_count); + ok(mode_count == mode_count_comp, "Got unexpected mode_count %u, expected %u.\n", mode_count, mode_count_comp); for (i = 0; i < mode_count; i++) { @@ -300,20 +303,23 @@ static void test_output(IDXGIDevice *device) } mode_count += 5; - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, &mode_count, modes); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, + DXGI_ENUM_MODES_SCALING, &mode_count, modes); ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); - ok(mode_count == mode_count_comp, "Expected %d, got %d\n", mode_count_comp, mode_count); + ok(mode_count == mode_count_comp, "Got unexpected mode_count %u, expected %u.\n", mode_count, mode_count_comp); if (mode_count_comp) { mode_count = mode_count_comp - 1; - IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, &mode_count, modes); - ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); - ok(mode_count == mode_count_comp -1, "Expected %d, got %d\n", mode_count_comp, mode_count); + hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, + DXGI_ENUM_MODES_SCALING, &mode_count, modes); + todo_wine ok(hr == DXGI_ERROR_MORE_DATA, "Got unexpected hr %#x.\n", hr); + ok(mode_count == mode_count_comp - 1, "Got unexpected mode_count %u, expected %u.\n", + mode_count, mode_count_comp - 1); } else { - skip("Not enough modes for test, skipping\n"); + skip("Not enough modes for test, skipping.\n"); } HeapFree(GetProcessHeap(), 0, modes);

1 0

Henri Verbeet : dxgi/tests: Use D3D10CreateDevice() to create a device.
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: f97d599eb212c76b345a7d308e066287e19b1daa URL: http://source.winehq.org/git/wine.git/?a=commit;h=f97d599eb212c76b345a7d308… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Feb 10 10:37:54 2014 +0100 dxgi/tests: Use D3D10CreateDevice() to create a device. At some point DXGID3D10CreateDevice() started always returning E_NOTIMPL. My guess would be that that was with the introduction of d3d11. --- dlls/dxgi/tests/Makefile.in | 2 +- dlls/dxgi/tests/device.c | 82 +++++++++++++------------------------------ 2 files changed, 25 insertions(+), 59 deletions(-) diff --git a/dlls/dxgi/tests/Makefile.in b/dlls/dxgi/tests/Makefile.in index c624fec..d706f16 100644 --- a/dlls/dxgi/tests/Makefile.in +++ b/dlls/dxgi/tests/Makefile.in @@ -1,5 +1,5 @@ TESTDLL = dxgi.dll -IMPORTS = dxgi user32 +IMPORTS = d3d10 dxgi user32 C_SRCS = \ device.c diff --git a/dlls/dxgi/tests/device.c b/dlls/dxgi/tests/device.c index ba1a29f..e8d2827 100644 --- a/dlls/dxgi/tests/device.c +++ b/dlls/dxgi/tests/device.c @@ -21,58 +21,25 @@ #include "d3d10.h" #include "wine/test.h" -HRESULT WINAPI DXGID3D10CreateDevice(HMODULE d3d10core, IDXGIFactory *factory, - IDXGIAdapter *adapter, UINT flags, void *unknown0, void **device); - -static IDXGIDevice *create_device(HMODULE d3d10core) +static IDXGIDevice *create_device(void) { - IDXGIDevice *dxgi_device = NULL; - IDXGIFactory *factory = NULL; - IDXGIAdapter *adapter = NULL; - IUnknown *device = NULL; + IDXGIDevice *dxgi_device; + ID3D10Device *device; HRESULT hr; - hr = CreateDXGIFactory(&IID_IDXGIFactory, (void *)&factory); - if (FAILED(hr)) goto cleanup; + if (SUCCEEDED(D3D10CreateDevice(NULL, D3D10_DRIVER_TYPE_HARDWARE, NULL, 0, D3D10_SDK_VERSION, &device))) + goto success; + if (SUCCEEDED(D3D10CreateDevice(NULL, D3D10_DRIVER_TYPE_WARP, NULL, 0, D3D10_SDK_VERSION, &device))) + goto success; + if (SUCCEEDED(D3D10CreateDevice(NULL, D3D10_DRIVER_TYPE_REFERENCE, NULL, 0, D3D10_SDK_VERSION, &device))) + goto success; - hr = IDXGIFactory_EnumAdapters(factory, 0, &adapter); - if (SUCCEEDED(hr)) - { - hr = DXGID3D10CreateDevice(d3d10core, factory, adapter, 0, NULL, (void **)&device); - } + return NULL; - if (FAILED(hr)) - { - HMODULE d3d10ref; - - trace("Failed to create a HW device, trying REF\n"); - if (adapter) IDXGIAdapter_Release(adapter); - adapter = NULL; - - d3d10ref = LoadLibraryA("d3d10ref.dll"); - if (!d3d10ref) - { - trace("d3d10ref.dll not available, unable to create a REF device\n"); - goto cleanup; - } - - hr = IDXGIFactory_CreateSoftwareAdapter(factory, d3d10ref, &adapter); - FreeLibrary(d3d10ref); - ok(SUCCEEDED(hr), "CreateSoftwareAdapter failed, hr %#x\n", hr); - if (FAILED(hr)) goto cleanup; - - hr = DXGID3D10CreateDevice(d3d10core, factory, adapter, 0, NULL, (void **)&device); - ok(SUCCEEDED(hr), "Failed to create a REF device, hr %#x\n", hr); - if (FAILED(hr)) goto cleanup; - } - - hr = IUnknown_QueryInterface(device, &IID_IDXGIDevice, (void **)&dxgi_device); +success: + hr = ID3D10Device_QueryInterface(device, &IID_IDXGIDevice, (void **)&dxgi_device); ok(SUCCEEDED(hr), "Created device does not implement IDXGIDevice\n"); - IUnknown_Release(device); - -cleanup: - if (adapter) IDXGIAdapter_Release(adapter); - if (factory) IDXGIFactory_Release(factory); + ID3D10Device_Release(device); return dxgi_device; } @@ -265,7 +232,16 @@ static void test_output(IDXGIDevice *device) todo_wine ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr); hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, 0, &mode_count, NULL); - ok(SUCCEEDED(hr), "Failed to list modes, hr %#x.\n", hr); + ok(SUCCEEDED(hr) + || broken(hr == DXGI_ERROR_NOT_CURRENTLY_AVAILABLE), /* Remote Desktop Services / Win 7 testbot */ + "Failed to list modes, hr %#x.\n", hr); + if (hr == DXGI_ERROR_NOT_CURRENTLY_AVAILABLE) + { + skip("GetDisplayModeList() not supported, skipping tests.\n"); + IDXGIOutput_Release(output); + IDXGIAdapter_Release(adapter); + return; + } mode_count_comp = mode_count; hr = IDXGIOutput_GetDisplayModeList(output, 0, 0, &mode_count, NULL); @@ -463,21 +439,12 @@ static void test_createswapchain(IDXGIDevice *device) START_TEST(device) { - HMODULE d3d10core = LoadLibraryA("d3d10core.dll"); IDXGIDevice *device; ULONG refcount; - if (!d3d10core) - { - win_skip("d3d10core.dll not available, skipping tests\n"); - return; - } - - device = create_device(d3d10core); - if (!device) + if (!(device = create_device())) { skip("Failed to create device, skipping tests\n"); - FreeLibrary(d3d10core); return; } @@ -490,5 +457,4 @@ START_TEST(device) refcount = IDXGIDevice_Release(device); ok(!refcount, "Device has %u references left\n", refcount); - FreeLibrary(d3d10core); }

1 0

Henri Verbeet : dxgi/tests: Don't reuse the device.
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: 4d7bd4ce4a02d59bf79f3b9af3f37edecfb95bfc URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d7bd4ce4a02d59bf79f3b9af… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Feb 10 10:37:55 2014 +0100 dxgi/tests: Don't reuse the device. --- dlls/dxgi/tests/device.c | 98 ++++++++++++++++++++++++++++++++++------------ 1 file changed, 74 insertions(+), 24 deletions(-) diff --git a/dlls/dxgi/tests/device.c b/dlls/dxgi/tests/device.c index e8d2827..b907a23 100644 --- a/dlls/dxgi/tests/device.c +++ b/dlls/dxgi/tests/device.c @@ -44,11 +44,19 @@ success: return dxgi_device; } -static void test_device_interfaces(IDXGIDevice *device) +static void test_device_interfaces(void) { + IDXGIDevice *device; + ULONG refcount; IUnknown *obj; HRESULT hr; + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + if (SUCCEEDED(hr = IDXGIDevice_QueryInterface(device, &IID_IUnknown, (void **)&obj))) IUnknown_Release(obj); ok(SUCCEEDED(hr), "IDXGIDevice does not implement IUnknown\n"); @@ -64,16 +72,27 @@ static void test_device_interfaces(IDXGIDevice *device) if (SUCCEEDED(hr = IDXGIDevice_QueryInterface(device, &IID_ID3D10Device, (void **)&obj))) IUnknown_Release(obj); ok(SUCCEEDED(hr), "IDXGIDevice does not implement ID3D10Device\n"); + + refcount = IDXGIDevice_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); } -static void test_adapter_desc(IDXGIDevice *device) +static void test_adapter_desc(void) { DXGI_ADAPTER_DESC1 desc1; IDXGIAdapter1 *adapter1; DXGI_ADAPTER_DESC desc; IDXGIAdapter *adapter; + IDXGIDevice *device; + ULONG refcount; HRESULT hr; + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + hr = IDXGIDevice_GetAdapter(device, &adapter); ok(SUCCEEDED(hr), "GetAdapter failed, hr %#x.\n", hr); @@ -123,15 +142,25 @@ static void test_adapter_desc(IDXGIDevice *device) done: IDXGIAdapter_Release(adapter); + refcount = IDXGIDevice_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); } -static void test_create_surface(IDXGIDevice *device) +static void test_create_surface(void) { ID3D10Texture2D *texture; IDXGISurface *surface; DXGI_SURFACE_DESC desc; + IDXGIDevice *device; + ULONG refcount; HRESULT hr; + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + desc.Width = 512; desc.Height = 512; desc.Format = DXGI_FORMAT_R8G8B8A8_UNORM; @@ -146,18 +175,28 @@ static void test_create_surface(IDXGIDevice *device) if (SUCCEEDED(hr)) ID3D10Texture2D_Release(texture); IDXGISurface_Release(surface); + refcount = IDXGIDevice_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); } -static void test_parents(IDXGIDevice *device) +static void test_parents(void) { DXGI_SURFACE_DESC surface_desc; IDXGISurface *surface; IDXGIFactory *factory; IDXGIAdapter *adapter; + IDXGIDevice *device; IDXGIOutput *output; IUnknown *parent; + ULONG refcount; HRESULT hr; + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + surface_desc.Width = 512; surface_desc.Height = 512; surface_desc.Format = DXGI_FORMAT_R8G8B8A8_UNORM; @@ -206,16 +245,26 @@ static void test_parents(IDXGIDevice *device) IUnknown_Release(parent); IDXGIAdapter_Release(adapter); + refcount = IDXGIDevice_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); } -static void test_output(IDXGIDevice *device) +static void test_output(void) { IDXGIAdapter *adapter; + IDXGIDevice *device; HRESULT hr; IDXGIOutput *output; + ULONG refcount; UINT mode_count, mode_count_comp, i; DXGI_MODE_DESC *modes; + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + hr = IDXGIDevice_GetAdapter(device, &adapter); ok(SUCCEEDED(hr), "GetAdapter failed, hr %#x.\n", hr); @@ -224,6 +273,7 @@ static void test_output(IDXGIDevice *device) { skip("Adapter doesn't have any outputs, skipping tests.\n"); IDXGIAdapter_Release(adapter); + IDXGIDevice_Release(device); return; } ok(SUCCEEDED(hr), "EnumOutputs failed, hr %#x.\n", hr); @@ -240,6 +290,7 @@ static void test_output(IDXGIDevice *device) skip("GetDisplayModeList() not supported, skipping tests.\n"); IDXGIOutput_Release(output); IDXGIAdapter_Release(adapter); + IDXGIDevice_Release(device); return; } mode_count_comp = mode_count; @@ -301,6 +352,8 @@ static void test_output(IDXGIDevice *device) HeapFree(GetProcessHeap(), 0, modes); IDXGIOutput_Release(output); IDXGIAdapter_Release(adapter); + refcount = IDXGIDevice_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); } struct refresh_rates @@ -311,11 +364,13 @@ struct refresh_rates BOOL denominator_should_pass; }; -static void test_createswapchain(IDXGIDevice *device) +static void test_createswapchain(void) { IUnknown *obj; IDXGIAdapter *adapter; IDXGIFactory *factory; + IDXGIDevice *device; + ULONG refcount; IDXGISwapChain *swapchain; DXGI_SWAP_CHAIN_DESC creation_desc, result_desc; HRESULT hr; @@ -331,6 +386,11 @@ static void test_createswapchain(IDXGIDevice *device) { 0, 0, TRUE, FALSE}, }; + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } wc.lpfnWndProc = DefWindowProcA; wc.lpszClassName = "dxgi_test_wc"; @@ -435,26 +495,16 @@ static void test_createswapchain(IDXGIDevice *device) IDXGIFactory_Release(factory); IDXGIAdapter_Release(adapter); IUnknown_Release(obj); + refcount = IDXGIDevice_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); } START_TEST(device) { - IDXGIDevice *device; - ULONG refcount; - - if (!(device = create_device())) - { - skip("Failed to create device, skipping tests\n"); - return; - } - - test_adapter_desc(device); - test_device_interfaces(device); - test_create_surface(device); - test_parents(device); - test_output(device); - test_createswapchain(device); - - refcount = IDXGIDevice_Release(device); - ok(!refcount, "Device has %u references left\n", refcount); + test_adapter_desc(); + test_device_interfaces(); + test_create_surface(); + test_parents(); + test_output(); + test_createswapchain(); }

1 0

Henri Verbeet : dxgi: Return DXGI_ERROR_INVALID_CALL when given a NULL mode_count in dxgi_output_GetDisplayModeList ().
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: d9570c3935f66d91583b75fb5865707c066ef6fa URL: http://source.winehq.org/git/wine.git/?a=commit;h=d9570c3935f66d91583b75fb5… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Feb 10 10:37:56 2014 +0100 dxgi: Return DXGI_ERROR_INVALID_CALL when given a NULL mode_count in dxgi_output_GetDisplayModeList(). --- dlls/dxgi/output.c | 4 +--- dlls/dxgi/tests/device.c | 4 ++-- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/dlls/dxgi/output.c b/dlls/dxgi/output.c index 4cdd06b..ec9d01b 100644 --- a/dlls/dxgi/output.c +++ b/dlls/dxgi/output.c @@ -132,9 +132,7 @@ static HRESULT STDMETHODCALLTYPE dxgi_output_GetDisplayModeList(IDXGIOutput *ifa iface, debug_dxgi_format(format), flags, mode_count, desc); if (!mode_count) - { - return S_OK; - } + return DXGI_ERROR_INVALID_CALL; if (format == DXGI_FORMAT_UNKNOWN) { diff --git a/dlls/dxgi/tests/device.c b/dlls/dxgi/tests/device.c index b907a23..d648dc7 100644 --- a/dlls/dxgi/tests/device.c +++ b/dlls/dxgi/tests/device.c @@ -279,7 +279,7 @@ static void test_output(void) ok(SUCCEEDED(hr), "EnumOutputs failed, hr %#x.\n", hr); hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, 0, NULL, NULL); - todo_wine ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr); + ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr); hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, 0, &mode_count, NULL); ok(SUCCEEDED(hr) @@ -309,7 +309,7 @@ static void test_output(void) hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, NULL, modes); - todo_wine ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr); + ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr); ok(!modes[0].Height, "No output was expected.\n"); mode_count = 0;

1 0

Henri Verbeet : dxgi: Properly check if the output array is large enough in dxgi_output_GetDisplayModeList ().
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: 852c61c5959e4d92dc9a45166cb4b3a78df678d6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=852c61c5959e4d92dc9a45166… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Feb 10 10:37:57 2014 +0100 dxgi: Properly check if the output array is large enough in dxgi_output_GetDisplayModeList(). --- dlls/dxgi/output.c | 9 ++++++++- dlls/dxgi/tests/device.c | 4 ++-- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/dlls/dxgi/output.c b/dlls/dxgi/output.c index ec9d01b..580a8b4 100644 --- a/dlls/dxgi/output.c +++ b/dlls/dxgi/output.c @@ -155,7 +155,14 @@ static HRESULT STDMETHODCALLTYPE dxgi_output_GetDisplayModeList(IDXGIOutput *ifa return S_OK; } - *mode_count = min(*mode_count,max_count); + if (max_count > *mode_count) + { + wined3d_decref(wined3d); + LeaveCriticalSection(&dxgi_cs); + return DXGI_ERROR_MORE_DATA; + } + + *mode_count = max_count; for (i = 0; i < *mode_count; ++i) { diff --git a/dlls/dxgi/tests/device.c b/dlls/dxgi/tests/device.c index d648dc7..f44e831 100644 --- a/dlls/dxgi/tests/device.c +++ b/dlls/dxgi/tests/device.c @@ -315,7 +315,7 @@ static void test_output(void) mode_count = 0; hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, &mode_count, modes); - todo_wine ok(hr == DXGI_ERROR_MORE_DATA, "Got unexpected hr %#x.\n", hr); + ok(hr == DXGI_ERROR_MORE_DATA, "Got unexpected hr %#x.\n", hr); ok(!modes[0].Height, "No output was expected.\n"); mode_count = mode_count_comp; @@ -340,7 +340,7 @@ static void test_output(void) mode_count = mode_count_comp - 1; hr = IDXGIOutput_GetDisplayModeList(output, DXGI_FORMAT_R8G8B8A8_UNORM, DXGI_ENUM_MODES_SCALING, &mode_count, modes); - todo_wine ok(hr == DXGI_ERROR_MORE_DATA, "Got unexpected hr %#x.\n", hr); + ok(hr == DXGI_ERROR_MORE_DATA, "Got unexpected hr %#x.\n", hr); ok(mode_count == mode_count_comp - 1, "Got unexpected mode_count %u, expected %u.\n", mode_count, mode_count_comp - 1); }

1 0

Jacek Caban : wmp: Added SetClientSite implementation.
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: 54e3e0de4801639a7a39174b30147f18d6094cd4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=54e3e0de4801639a7a39174b3… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Feb 10 11:50:34 2014 +0100 wmp: Added SetClientSite implementation. --- dlls/wmp/oleobj.c | 45 +++++++++++++++++++++++++++++++++++++++++---- dlls/wmp/wmp_private.h | 5 +++++ 2 files changed, 46 insertions(+), 4 deletions(-) diff --git a/dlls/wmp/oleobj.c b/dlls/wmp/oleobj.c index 380ae1f..d5c2f85 100644 --- a/dlls/wmp/oleobj.c +++ b/dlls/wmp/oleobj.c @@ -28,8 +28,19 @@ struct WindowsMediaPlayer { IPersistStreamInit IPersistStreamInit_iface; LONG ref; + + IOleClientSite *client_site; }; +static void release_client_site(WindowsMediaPlayer *This) +{ + if(!This->client_site) + return; + + IOleClientSite_Release(This->client_site); + This->client_site = NULL; +} + static inline WindowsMediaPlayer *impl_from_IOleObject(IOleObject *iface) { return CONTAINING_RECORD(iface, WindowsMediaPlayer, IOleObject_iface); @@ -84,8 +95,10 @@ static ULONG WINAPI OleObject_Release(IOleObject *iface) TRACE("(%p) ref=%d\n", This, ref); - if(!ref) + if(!ref) { + release_client_site(This); heap_free(This); + } return ref; } @@ -93,8 +106,32 @@ static ULONG WINAPI OleObject_Release(IOleObject *iface) static HRESULT WINAPI OleObject_SetClientSite(IOleObject *iface, IOleClientSite *pClientSite) { WindowsMediaPlayer *This = impl_from_IOleObject(iface); - FIXME("(%p)->(%p)\n", This, pClientSite); - return E_NOTIMPL; + IOleControlSite *control_site; + HRESULT hres; + + TRACE("(%p)->(%p)\n", This, pClientSite); + + release_client_site(This); + if(!pClientSite) + return S_OK; + + IOleClientSite_AddRef(pClientSite); + This->client_site = pClientSite; + + hres = IOleClientSite_QueryInterface(pClientSite, &IID_IOleControlSite, (void**)&control_site); + if(SUCCEEDED(hres)) { + IDispatch *disp; + + hres = IOleControlSite_GetExtendedControl(control_site, &disp); + if(SUCCEEDED(hres) && disp) { + FIXME("Use extended control\n"); + IDispatch_Release(disp); + } + + IOleControlSite_Release(control_site); + } + + return S_OK; } static HRESULT WINAPI OleObject_GetClientSite(IOleObject *iface, IOleClientSite **ppClientSite) @@ -407,7 +444,7 @@ HRESULT WINAPI WMPFactory_CreateInstance(IClassFactory *iface, IUnknown *outer, TRACE("(%p %s %p)\n", outer, debugstr_guid(riid), ppv); - wmp = heap_alloc(sizeof(*wmp)); + wmp = heap_alloc_zero(sizeof(*wmp)); if(!wmp) return E_OUTOFMEMORY; diff --git a/dlls/wmp/wmp_private.h b/dlls/wmp/wmp_private.h index f294519..68c3dd2 100644 --- a/dlls/wmp/wmp_private.h +++ b/dlls/wmp/wmp_private.h @@ -29,6 +29,11 @@ static inline void *heap_alloc(size_t len) return HeapAlloc(GetProcessHeap(), 0, len); } +static inline void *heap_alloc_zero(size_t len) +{ + return HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, len); +} + static inline BOOL heap_free(void *mem) { return HeapFree(GetProcessHeap(), 0, mem);

1 0

Austin English : wbemprox: Add a missing break (Coverity).
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: dc9de59f7f1ead0b79ec24a05920ddab1ca97990 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dc9de59f7f1ead0b79ec24a05… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 7 16:42:49 2014 -0800 wbemprox: Add a missing break (Coverity). --- dlls/wbemprox/query.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/wbemprox/query.c b/dlls/wbemprox/query.c index bd68560..6933da2 100644 --- a/dlls/wbemprox/query.c +++ b/dlls/wbemprox/query.c @@ -170,6 +170,7 @@ static UINT resolve_type( UINT left, UINT right ) return CIM_UINT64; default: break; } + break; case CIM_STRING: if (right == CIM_STRING) return CIM_STRING;

1 0

Marcus Meissner : oleaut32: Incorrect sizeof() to MultiByteToWideChar ( Coverity).
by Alexandre Julliard 10 Feb '14

10 Feb '14

Module: wine Branch: master Commit: 5690127b2e9bad351c674b63381236725e91f5a2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5690127b2e9bad351c674b633… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Sat Feb 8 08:23:03 2014 +0100 oleaut32: Incorrect sizeof() to MultiByteToWideChar (Coverity). --- dlls/oleaut32/tests/typelib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/oleaut32/tests/typelib.c b/dlls/oleaut32/tests/typelib.c index 84b9023..73ec679 100644 --- a/dlls/oleaut32/tests/typelib.c +++ b/dlls/oleaut32/tests/typelib.c @@ -3982,12 +3982,12 @@ static void test_dump_typelib(const char *name) /* compare type uuid */ if (ti->uuid && *ti->uuid) { - WCHAR guidW[39] = {0}; + WCHAR guidW[39]; ITypeInfo *typeinfo2; HRESULT hr; GUID guid; - MultiByteToWideChar(CP_ACP, 0, ti->uuid, -1, guidW, 40); + MultiByteToWideChar(CP_ACP, 0, ti->uuid, -1, guidW, sizeof(guidW)/sizeof(guidW[0])); IIDFromString(guidW, &guid); expect_guid(&guid, &typeattr->guid);

1 0