April 2013 - Wine-commits - list.winehq.org

Alistair Leslie-Hughes : oledb32: Implement DataConvert DBTYPE_I8-> DBTYPE_I8.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: 975f1dcd7a1a5bfa5cb58c5f88d7e372da6793d9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=975f1dcd7a1a5bfa5cb58c5f8… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Tue Apr 16 10:34:52 2013 +1000 oledb32: Implement DataConvert DBTYPE_I8->DBTYPE_I8. --- dlls/oledb32/convert.c | 11 +++++++++++ dlls/oledb32/tests/convert.c | 29 +++++++++++++++++++++++++++++ 2 files changed, 40 insertions(+), 0 deletions(-) diff --git a/dlls/oledb32/convert.c b/dlls/oledb32/convert.c index 1cce07a..9a82956 100644 --- a/dlls/oledb32/convert.c +++ b/dlls/oledb32/convert.c @@ -287,6 +287,17 @@ static HRESULT WINAPI convert_DataConvert(IDataConvert* iface, break; } + case DBTYPE_I8: + { + LONGLONG *d = dst; + switch(src_type) + { + case DBTYPE_I8: *d = *(LONGLONG*)src; hr = S_OK; break; + default: FIXME("Unimplemented conversion %04x -> I8\n", src_type); return E_NOTIMPL; + } + break; + } + case DBTYPE_R4: { FLOAT *d = dst; diff --git a/dlls/oledb32/tests/convert.c b/dlls/oledb32/tests/convert.c index 4623825..ef05c40 100644 --- a/dlls/oledb32/tests/convert.c +++ b/dlls/oledb32/tests/convert.c @@ -917,6 +917,34 @@ todo_wine IDataConvert_Release(convert); } +static void test_converttoi8(void) +{ + IDataConvert *convert; + HRESULT hr; + LARGE_INTEGER dst; + BYTE src[20]; + DBSTATUS dst_status; + DBLENGTH dst_len; + + hr = CoCreateInstance(&CLSID_OLEDB_CONVERSIONLIBRARY, NULL, CLSCTX_INPROC_SERVER, &IID_IDataConvert, (void**)&convert); + if(FAILED(hr)) + { + win_skip("Unable to load oledb conversion library\n"); + return; + } + + dst.QuadPart = 0xcc; + ((ULARGE_INTEGER*)src)->QuadPart = 1234; + dst_len = 0x1234; + hr = IDataConvert_DataConvert(convert, DBTYPE_I8, DBTYPE_I8, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); + ok(hr == S_OK, "got %08x\n", hr); + ok(dst_status == DBSTATUS_S_OK, "got %08x\n", dst_status); + ok(dst_len == sizeof(dst), "got %ld\n", dst_len); + ok(dst.QuadPart == 1234, "got %d\n", (int)dst.QuadPart); + + IDataConvert_Release(convert); +} + static void test_converttobstr(void) { IDataConvert *convert; @@ -2530,6 +2558,7 @@ START_TEST(convert) test_canconvert(); test_converttoi2(); test_converttoi4(); + test_converttoi8(); test_converttostr(); test_converttobstr(); test_converttowstr();

1 0

Alistair Leslie-Hughes : oledb32: Implement DataConvert DBTYPE_BSTR-> DBTYPE_I8.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: e06cdb62071ec74857c9d6dfff97d092b09f6746 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e06cdb62071ec74857c9d6dff… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Tue Apr 16 11:23:59 2013 +1000 oledb32: Implement DataConvert DBTYPE_BSTR->DBTYPE_I8. --- dlls/oledb32/convert.c | 1 + dlls/oledb32/tests/convert.c | 13 +++++++++++++ 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/dlls/oledb32/convert.c b/dlls/oledb32/convert.c index 9a82956..d23c944 100644 --- a/dlls/oledb32/convert.c +++ b/dlls/oledb32/convert.c @@ -292,6 +292,7 @@ static HRESULT WINAPI convert_DataConvert(IDataConvert* iface, LONGLONG *d = dst; switch(src_type) { + case DBTYPE_BSTR: hr = VarI8FromStr(*(WCHAR**)src, LOCALE_USER_DEFAULT, 0, d); break; case DBTYPE_I8: *d = *(LONGLONG*)src; hr = S_OK; break; default: FIXME("Unimplemented conversion %04x -> I8\n", src_type); return E_NOTIMPL; } diff --git a/dlls/oledb32/tests/convert.c b/dlls/oledb32/tests/convert.c index ef05c40..a2b1366 100644 --- a/dlls/oledb32/tests/convert.c +++ b/dlls/oledb32/tests/convert.c @@ -925,6 +925,8 @@ static void test_converttoi8(void) BYTE src[20]; DBSTATUS dst_status; DBLENGTH dst_len; + static const WCHAR ten[] = {'1','0',0}; + BSTR b; hr = CoCreateInstance(&CLSID_OLEDB_CONVERSIONLIBRARY, NULL, CLSCTX_INPROC_SERVER, &IID_IDataConvert, (void**)&convert); if(FAILED(hr)) @@ -942,6 +944,17 @@ static void test_converttoi8(void) ok(dst_len == sizeof(dst), "got %ld\n", dst_len); ok(dst.QuadPart == 1234, "got %d\n", (int)dst.QuadPart); + dst.QuadPart = 0xcc; + ((ULARGE_INTEGER*)src)->QuadPart = 1234; + b = SysAllocString(ten); + *(BSTR *)src = b; + hr = IDataConvert_DataConvert(convert, DBTYPE_BSTR, DBTYPE_I8, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); + ok(hr == S_OK, "got %08x\n", hr); + ok(dst_status == DBSTATUS_S_OK, "got %08x\n", dst_status); + ok(dst_len == sizeof(dst), "got %ld\n", dst_len); + ok(dst.QuadPart == 10, "got %d\n", (int)dst.QuadPart); + SysFreeString(b); + IDataConvert_Release(convert); }

1 0

Gurmail Bassi : dinput: Update existing joystick values after setting the range property.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: 37169522f62f85048900cf3329fb54800cbb8f3e URL: http://source.winehq.org/git/wine.git/?a=commit;h=37169522f62f85048900cf332… Author: Gurmail Bassi <Gurmail.Bassi(a)mottmac.com> Date: Sun Apr 14 22:25:03 2013 +0100 dinput: Update existing joystick values after setting the range property. --- dlls/dinput/joystick.c | 47 +++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 47 insertions(+), 0 deletions(-) diff --git a/dlls/dinput/joystick.c b/dlls/dinput/joystick.c index 73ded55..7da8aac 100644 --- a/dlls/dinput/joystick.c +++ b/dlls/dinput/joystick.c @@ -94,6 +94,7 @@ HRESULT WINAPI JoystickWGenericImpl_SetProperty(LPDIRECTINPUTDEVICE8W iface, REF { JoystickGenericImpl *This = impl_from_IDirectInputDevice8W(iface); DWORD i; + ObjProps remap_props; TRACE("(%p,%s,%p)\n",This,debugstr_guid(rguid),ph); @@ -112,6 +113,28 @@ HRESULT WINAPI JoystickWGenericImpl_SetProperty(LPDIRECTINPUTDEVICE8W iface, REF if (ph->dwHow == DIPH_DEVICE) { TRACE("proprange(%d,%d) all\n", pr->lMin, pr->lMax); for (i = 0; i < This->base.data_format.wine_df->dwNumObjs; i++) { + + remap_props.lDevMin = This->props[i].lMin; + remap_props.lDevMax = This->props[i].lMax; + + remap_props.lDeadZone = This->props[i].lDeadZone; + remap_props.lSaturation = This->props[i].lSaturation; + + remap_props.lMin = pr->lMin; + remap_props.lMax = pr->lMax; + + switch (This->base.data_format.offsets[i]) { + case DIJOFS_X : This->js.lX = joystick_map_axis(&remap_props, This->js.lX); break; + case DIJOFS_Y : This->js.lY = joystick_map_axis(&remap_props, This->js.lY); break; + case DIJOFS_Z : This->js.lZ = joystick_map_axis(&remap_props, This->js.lZ); break; + case DIJOFS_RX : This->js.lRx = joystick_map_axis(&remap_props, This->js.lRx); break; + case DIJOFS_RY : This->js.lRy = joystick_map_axis(&remap_props, This->js.lRy); break; + case DIJOFS_RZ : This->js.lRz = joystick_map_axis(&remap_props, This->js.lRz); break; + case DIJOFS_SLIDER(0): This->js.rglSlider[0] = joystick_map_axis(&remap_props, This->js.rglSlider[0]); break; + case DIJOFS_SLIDER(1): This->js.rglSlider[1] = joystick_map_axis(&remap_props, This->js.rglSlider[1]); break; + default: break; + } + This->props[i].lMin = pr->lMin; This->props[i].lMax = pr->lMax; } @@ -120,6 +143,30 @@ HRESULT WINAPI JoystickWGenericImpl_SetProperty(LPDIRECTINPUTDEVICE8W iface, REF TRACE("proprange(%d,%d) obj=%d\n", pr->lMin, pr->lMax, obj); if (obj >= 0) { + + /*ePSXe polls the joystick immediately after setting the range for calibration purposes, so the old values need to be remapped to the new range before it does so*/ + + remap_props.lDevMin = This->props[obj].lMin; + remap_props.lDevMax = This->props[obj].lMax; + + remap_props.lDeadZone = This->props[obj].lDeadZone; + remap_props.lSaturation = This->props[obj].lSaturation; + + remap_props.lMin = pr->lMin; + remap_props.lMax = pr->lMax; + + switch (ph->dwObj) { + case DIJOFS_X : This->js.lX = joystick_map_axis(&remap_props, This->js.lX); break; + case DIJOFS_Y : This->js.lY = joystick_map_axis(&remap_props, This->js.lY); break; + case DIJOFS_Z : This->js.lZ = joystick_map_axis(&remap_props, This->js.lZ); break; + case DIJOFS_RX : This->js.lRx = joystick_map_axis(&remap_props, This->js.lRx); break; + case DIJOFS_RY : This->js.lRy = joystick_map_axis(&remap_props, This->js.lRy); break; + case DIJOFS_RZ : This->js.lRz = joystick_map_axis(&remap_props, This->js.lRz); break; + case DIJOFS_SLIDER(0): This->js.rglSlider[0] = joystick_map_axis(&remap_props, This->js.rglSlider[0]); break; + case DIJOFS_SLIDER(1): This->js.rglSlider[1] = joystick_map_axis(&remap_props, This->js.rglSlider[1]); break; + default: break; + } + This->props[obj].lMin = pr->lMin; This->props[obj].lMax = pr->lMax; return DI_OK;

1 0

Stefan Dösinger : d3d9/tests: Test NPOT texture creation.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: 39e7248d71df4031d755aa5439aeded7643db842 URL: http://source.winehq.org/git/wine.git/?a=commit;h=39e7248d71df4031d755aa543… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri Apr 12 10:47:59 2013 +0200 d3d9/tests: Test NPOT texture creation. --- dlls/d3d9/tests/device.c | 124 ++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 124 insertions(+), 0 deletions(-) diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c index e1f3c63..310f47a 100644 --- a/dlls/d3d9/tests/device.c +++ b/dlls/d3d9/tests/device.c @@ -6267,6 +6267,129 @@ static void test_rtpatch(void) DestroyWindow(window); } +static void test_npot_textures(void) +{ + IDirect3DDevice9 *device = NULL; + IDirect3D9 *d3d9; + ULONG refcount; + HWND window = NULL; + HRESULT hr; + D3DCAPS9 caps; + IDirect3DTexture9 *texture; + IDirect3DCubeTexture9 *cube_texture; + IDirect3DVolumeTexture9 *volume_texture; + struct + { + D3DPOOL pool; + const char *pool_name; + HRESULT hr; + } + pools[] = + { + { D3DPOOL_DEFAULT, "D3DPOOL_DEFAULT", D3DERR_INVALIDCALL }, + { D3DPOOL_MANAGED, "D3DPOOL_MANAGED", D3DERR_INVALIDCALL }, + { D3DPOOL_SYSTEMMEM, "D3DPOOL_SYSTEMMEM", D3DERR_INVALIDCALL }, + { D3DPOOL_SCRATCH, "D3DPOOL_SCRATCH", D3D_OK }, + }; + unsigned int i, levels; + BOOL tex_pow2, cube_pow2, vol_pow2; + + if (!(d3d9 = pDirect3DCreate9(D3D_SDK_VERSION))) + { + skip("Failed to create IDirect3D9 object, skipping tests.\n"); + return; + } + + window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + if (!(device = create_device(d3d9, window, window, TRUE))) + { + skip("Failed to create a D3D device, skipping tests.\n"); + goto done; + } + + hr = IDirect3DDevice9_GetDeviceCaps(device, &caps); + ok(SUCCEEDED(hr), "Failed to get caps, hr %#x.\n", hr); + tex_pow2 = !!(caps.TextureCaps & D3DPTEXTURECAPS_POW2); + cube_pow2 = !!(caps.TextureCaps & D3DPTEXTURECAPS_CUBEMAP_POW2); + vol_pow2 = !!(caps.TextureCaps & D3DPTEXTURECAPS_VOLUMEMAP_POW2); + ok(cube_pow2 == tex_pow2, "Cube texture and 2d texture pow2 restrictions mismatch.\n"); + ok(vol_pow2 == tex_pow2, "Volume texture and 2d texture pow2 restrictions mismatch.\n"); + + for (i = 0; i < sizeof(pools) / sizeof(*pools); i++) + { + for (levels = 0; levels <= 2; levels++) + { + HRESULT expected; + + hr = IDirect3DDevice9_CreateTexture(device, 10, 10, levels, 0, D3DFMT_X8R8G8B8, + pools[i].pool, &texture, NULL); + if (!tex_pow2) + { + expected = D3D_OK; + } + else if (caps.TextureCaps & D3DPTEXTURECAPS_NONPOW2CONDITIONAL) + { + if (levels == 1) + expected = D3D_OK; + else + expected = pools[i].hr; + } + else + { + expected = pools[i].hr; + } + ok(hr == expected, "CreateTexture(w=h=10, %s, levels=%u) returned hr %#x, expected %#x.\n", + pools[i].pool_name, levels, hr, expected); + + if (SUCCEEDED(hr)) + IDirect3DTexture9_Release(texture); + } + + hr = IDirect3DDevice9_CreateCubeTexture(device, 3, 1, 0, D3DFMT_X8R8G8B8, pools[i].pool, + &cube_texture, NULL); + if (tex_pow2) + { + ok(hr == pools[i].hr, "CreateCubeTexture(EdgeLength=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, pools[i].hr); + } + else + { + ok(SUCCEEDED(hr), "CreateCubeTexture(EdgeLength=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, D3D_OK); + } + + if (SUCCEEDED(hr)) + IDirect3DCubeTexture9_Release(cube_texture); + + hr = IDirect3DDevice9_CreateVolumeTexture(device, 2, 2, 3, 1, 0, D3DFMT_X8R8G8B8, pools[i].pool, + &volume_texture, NULL); + if (tex_pow2) + { + ok(hr == pools[i].hr, "CreateVolumeTextur(Depth=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, pools[i].hr); + } + else + { + ok(SUCCEEDED(hr), "CreateVolumeTextur(Depth=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, D3D_OK); + } + + if (SUCCEEDED(hr)) + IDirect3DVolumeTexture9_Release(volume_texture); + } + +done: + if (device) + { + refcount = IDirect3DDevice9_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); + } + IDirect3D9_Release(d3d9); + DestroyWindow(window); + +} + START_TEST(device) { HMODULE d3d9_handle = LoadLibraryA( "d3d9.dll" ); @@ -6355,6 +6478,7 @@ START_TEST(device) test_set_palette(); test_swvp_buffer(); test_rtpatch(); + test_npot_textures(); } out:

1 0

Stefan Dösinger : wined3d: Refuse to create NPOT textures without GL support.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: c6ed62735d02bd168e435a3911da2b56ea2fa37a URL: http://source.winehq.org/git/wine.git/?a=commit;h=c6ed62735d02bd168e435a391… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Tue Apr 16 14:48:32 2013 +0200 wined3d: Refuse to create NPOT textures without GL support. --- dlls/wined3d/texture.c | 90 +++++++++++++++++++++++++++++++++--------------- 1 files changed, 62 insertions(+), 28 deletions(-) diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index 5b83d4a..dc23e14 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -2,7 +2,7 @@ * Copyright 2002-2005 Jason Edmeades * Copyright 2002-2005 Raphael Junqueira * Copyright 2005 Oliver Stieber - * Copyright 2007-2008 Stefan Dösinger for CodeWeavers + * Copyright 2007-2009, 2013 Stefan Dösinger for CodeWeavers * Copyright 2009-2011 Henri Verbeet for CodeWeavers * * This library is free software; you can redistribute it and/or @@ -760,7 +760,6 @@ static HRESULT cubetexture_init(struct wined3d_texture *texture, UINT edge_lengt { const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; const struct wined3d_format *format = wined3d_get_format(gl_info, format_id); - UINT pow2_edge_length; unsigned int i, j; UINT tmp_w; HRESULT hr; @@ -802,6 +801,27 @@ static HRESULT cubetexture_init(struct wined3d_texture *texture, UINT edge_lengt TRACE("Calculated levels = %u.\n", levels); } + if (!gl_info->supported[ARB_TEXTURE_NON_POWER_OF_TWO]) + { + UINT pow2_edge_length = 1; + while (pow2_edge_length < edge_length) pow2_edge_length <<= 1; + + if (edge_length != pow2_edge_length) + { + if (pool == WINED3D_POOL_SCRATCH) + { + /* SCRATCH textures cannot be used for texturing */ + WARN("Creating a scratch NPOT cube texture despite lack of HW support.\n"); + } + else + { + WARN("Attempted to create a NPOT cube texture (edge_length=%u) without GL support.\n", + edge_length); + return WINED3DERR_INVALIDCALL; + } + } + } + hr = wined3d_texture_init(texture, &texture2d_ops, 6, levels, WINED3D_RTYPE_CUBE_TEXTURE, device, usage, format, pool, parent, parent_ops, &texture2d_resource_ops); @@ -811,27 +831,10 @@ static HRESULT cubetexture_init(struct wined3d_texture *texture, UINT edge_lengt return hr; } - /* Find the nearest pow2 match. */ - pow2_edge_length = 1; - while (pow2_edge_length < edge_length) pow2_edge_length <<= 1; - - if (gl_info->supported[ARB_TEXTURE_NON_POWER_OF_TWO] || (edge_length == pow2_edge_length)) - { - /* Precalculated scaling for 'faked' non power of two texture coords. */ - texture->pow2_matrix[0] = 1.0f; - texture->pow2_matrix[5] = 1.0f; - texture->pow2_matrix[10] = 1.0f; - texture->pow2_matrix[15] = 1.0f; - } - else - { - /* Precalculated scaling for 'faked' non power of two texture coords. */ - texture->pow2_matrix[0] = ((float)edge_length) / ((float)pow2_edge_length); - texture->pow2_matrix[5] = ((float)edge_length) / ((float)pow2_edge_length); - texture->pow2_matrix[10] = ((float)edge_length) / ((float)pow2_edge_length); - texture->pow2_matrix[15] = 1.0f; - texture->flags &= ~WINED3D_TEXTURE_POW2_MAT_IDENT; - } + texture->pow2_matrix[0] = 1.0f; + texture->pow2_matrix[5] = 1.0f; + texture->pow2_matrix[10] = 1.0f; + texture->pow2_matrix[15] = 1.0f; texture->target = GL_TEXTURE_CUBE_MAP_ARB; /* Generate all the surfaces. */ @@ -906,12 +909,19 @@ static HRESULT texture_init(struct wined3d_texture *texture, UINT width, UINT he if (pow2_width != width || pow2_height != height) { - if (levels > 1) + /* levels == 0 returns an error as well */ + if (levels != 1) { - WARN("Attempted to create a mipmapped np2 texture without unconditional np2 support.\n"); - return WINED3DERR_INVALIDCALL; + if (pool == WINED3D_POOL_SCRATCH) + { + WARN("Creating a scratch mipmapped NPOT texture despite lack of HW support.\n"); + } + else + { + WARN("Attempted to create a mipmapped NPOT texture without unconditional NPOT support.\n"); + return WINED3DERR_INVALIDCALL; + } } - levels = 1; } } @@ -1186,6 +1196,31 @@ static HRESULT volumetexture_init(struct wined3d_texture *texture, UINT width, U TRACE("Calculated levels = %u.\n", levels); } + if (!gl_info->supported[ARB_TEXTURE_NON_POWER_OF_TWO]) + { + UINT pow2_w, pow2_h, pow2_d; + pow2_w = 1; + while (pow2_w < width) pow2_w <<= 1; + pow2_h = 1; + while (pow2_h < height) pow2_h <<= 1; + pow2_d = 1; + while (pow2_d < depth) pow2_d <<= 1; + + if (pow2_w != width || pow2_h != height || pow2_d != depth) + { + if (pool == WINED3D_POOL_SCRATCH) + { + WARN("Creating a scratch NPOT volume texture despite lack of HW support.\n"); + } + else + { + WARN("Attempted to create a NPOT volume texture (%u,%u,%u) without GL support.\n", + width, height, depth); + return WINED3DERR_INVALIDCALL; + } + } + } + hr = wined3d_texture_init(texture, &texture3d_ops, 1, levels, WINED3D_RTYPE_VOLUME_TEXTURE, device, usage, format, pool, parent, parent_ops, &texture3d_resource_ops); @@ -1195,7 +1230,6 @@ static HRESULT volumetexture_init(struct wined3d_texture *texture, UINT width, U return hr; } - /* Is NP2 support for volumes needed? */ texture->pow2_matrix[0] = 1.0f; texture->pow2_matrix[5] = 1.0f; texture->pow2_matrix[10] = 1.0f;

1 0

Stefan Dösinger : d3d8/tests: Test NPOT texture creation.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: a18cffb4de203b035ae56d8099ca2e2ab4e5cd06 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a18cffb4de203b035ae56d809… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri Apr 12 10:50:36 2013 +0200 d3d8/tests: Test NPOT texture creation. --- dlls/d3d8/tests/device.c | 124 ++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 124 insertions(+), 0 deletions(-) diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c index 2f281d2..28dce38 100644 --- a/dlls/d3d8/tests/device.c +++ b/dlls/d3d8/tests/device.c @@ -4768,6 +4768,129 @@ static void test_rtpatch(void) DestroyWindow(window); } +static void test_npot_textures(void) +{ + IDirect3DDevice8 *device = NULL; + IDirect3D8 *d3d8; + ULONG refcount; + HWND window = NULL; + HRESULT hr; + D3DCAPS8 caps; + IDirect3DTexture8 *texture; + IDirect3DCubeTexture8 *cube_texture; + IDirect3DVolumeTexture8 *volume_texture; + struct + { + D3DPOOL pool; + const char *pool_name; + HRESULT hr; + } + pools[] = + { + { D3DPOOL_DEFAULT, "D3DPOOL_DEFAULT", D3DERR_INVALIDCALL }, + { D3DPOOL_MANAGED, "D3DPOOL_MANAGED", D3DERR_INVALIDCALL }, + { D3DPOOL_SYSTEMMEM, "D3DPOOL_SYSTEMMEM", D3DERR_INVALIDCALL }, + { D3DPOOL_SCRATCH, "D3DPOOL_SCRATCH", D3D_OK }, + }; + unsigned int i, levels; + BOOL tex_pow2, cube_pow2, vol_pow2; + + if (!(d3d8 = pDirect3DCreate8(D3D_SDK_VERSION))) + { + skip("Failed to create IDirect3D8 object, skipping tests.\n"); + return; + } + + window = CreateWindowA("static", "d3d8_test", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + if (!(device = create_device(d3d8, window, window, TRUE))) + { + skip("Failed to create a D3D device, skipping tests.\n"); + goto done; + } + + hr = IDirect3DDevice8_GetDeviceCaps(device, &caps); + ok(SUCCEEDED(hr), "Failed to get caps, hr %#x.\n", hr); + tex_pow2 = !!(caps.TextureCaps & D3DPTEXTURECAPS_POW2); + cube_pow2 = !!(caps.TextureCaps & D3DPTEXTURECAPS_CUBEMAP_POW2); + vol_pow2 = !!(caps.TextureCaps & D3DPTEXTURECAPS_VOLUMEMAP_POW2); + ok(cube_pow2 == tex_pow2, "Cube texture and 2d texture pow2 restrictions mismatch.\n"); + ok(vol_pow2 == tex_pow2, "Volume texture and 2d texture pow2 restrictions mismatch.\n"); + + for (i = 0; i < sizeof(pools) / sizeof(*pools); i++) + { + for (levels = 0; levels <= 2; levels++) + { + HRESULT expected; + + hr = IDirect3DDevice8_CreateTexture(device, 10, 10, levels, 0, D3DFMT_X8R8G8B8, + pools[i].pool, &texture); + if (!tex_pow2) + { + expected = D3D_OK; + } + else if (caps.TextureCaps & D3DPTEXTURECAPS_NONPOW2CONDITIONAL) + { + if (levels == 1) + expected = D3D_OK; + else + expected = pools[i].hr; + } + else + { + expected = pools[i].hr; + } + ok(hr == expected, "CreateTexture(w=h=10, %s, levels=%u) returned hr %#x, expected %#x.\n", + pools[i].pool_name, levels, hr, expected); + + if (SUCCEEDED(hr)) + IDirect3DTexture8_Release(texture); + } + + hr = IDirect3DDevice8_CreateCubeTexture(device, 3, 1, 0, D3DFMT_X8R8G8B8, + pools[i].pool, &cube_texture); + if (tex_pow2) + { + ok(hr == pools[i].hr, "CreateCubeTexture(EdgeLength=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, pools[i].hr); + } + else + { + ok(SUCCEEDED(hr), "CreateCubeTexture(EdgeLength=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, D3D_OK); + } + + if (SUCCEEDED(hr)) + IDirect3DCubeTexture8_Release(cube_texture); + + hr = IDirect3DDevice8_CreateVolumeTexture(device, 2, 2, 3, 1, 0, D3DFMT_X8R8G8B8, + pools[i].pool, &volume_texture); + if (tex_pow2) + { + ok(hr == pools[i].hr, "CreateVolumeTextur(Depth=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, pools[i].hr); + } + else + { + ok(SUCCEEDED(hr), "CreateVolumeTextur(Depth=3, %s) returned hr %#x, expected %#x.\n", + pools[i].pool_name, hr, D3D_OK); + } + + if (SUCCEEDED(hr)) + IDirect3DVolumeTexture8_Release(volume_texture); + } + +done: + if (device) + { + refcount = IDirect3DDevice8_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); + } + IDirect3D8_Release(d3d8); + DestroyWindow(window); + +} + START_TEST(device) { HMODULE d3d8_handle = LoadLibraryA( "d3d8.dll" ); @@ -4844,6 +4967,7 @@ START_TEST(device) test_set_palette(); test_swvp_buffer(); test_rtpatch(); + test_npot_textures(); } UnregisterClassA("d3d8_test_wc", GetModuleHandleA(NULL)); }

1 0

Stefan Dösinger : d3d9/tests: Fix a test failure on cards that don't support mipmapped cube textures.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: 6c031274c3f2f4db4d1125465b111ec143e6168d URL: http://source.winehq.org/git/wine.git/?a=commit;h=6c031274c3f2f4db4d1125465… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Tue Apr 16 14:48:35 2013 +0200 d3d9/tests: Fix a test failure on cards that don't support mipmapped cube textures. --- dlls/d3d9/tests/device.c | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c index 310f47a..3283a6e 100644 --- a/dlls/d3d9/tests/device.c +++ b/dlls/d3d9/tests/device.c @@ -4653,7 +4653,10 @@ static void test_cube_texture_levels(IDirect3DDevice9 *device) D3DSURFACE_DESC desc; DWORD levels; HRESULT hr; + D3DCAPS9 caps; + hr = IDirect3DDevice9_GetDeviceCaps(device, &caps); + ok(SUCCEEDED(hr), "Failed to get device caps, hr %#x.\n", hr); if (FAILED(IDirect3DDevice9_CreateCubeTexture(device, 64, 0, 0, D3DFMT_X8R8G8B8, D3DPOOL_DEFAULT, &texture, NULL))) { @@ -4662,7 +4665,10 @@ static void test_cube_texture_levels(IDirect3DDevice9 *device) } levels = IDirect3DCubeTexture9_GetLevelCount(texture); - ok(levels == 7, "Got unexpected levels %u.\n", levels); + if (caps.TextureCaps & D3DPTEXTURECAPS_MIPCUBEMAP) + ok(levels == 7, "Got unexpected levels %u.\n", levels); + else + ok(levels == 1, "Got unexpected levels %u.\n", levels); hr = IDirect3DCubeTexture9_GetLevelDesc(texture, levels - 1, &desc); ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr);

1 0

Stefan Dösinger : wined3d: Enable GL_ARB_texture_rectangle in GLSL ffp fragment shaders.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: ea69fa943f18c7fa89a1b85b54f1daeb7c1ce08e URL: http://source.winehq.org/git/wine.git/?a=commit;h=ea69fa943f18c7fa89a1b85b5… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Tue Apr 16 14:48:36 2013 +0200 wined3d: Enable GL_ARB_texture_rectangle in GLSL ffp fragment shaders. --- dlls/wined3d/glsl_shader.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c index 3493ff2..904d6ec 100644 --- a/dlls/wined3d/glsl_shader.c +++ b/dlls/wined3d/glsl_shader.c @@ -5122,6 +5122,9 @@ static GLuint shader_glsl_generate_ffp_fragment_shader(struct wined3d_shader_buf shader_addline(buffer, "#version 120\n"); + if (gl_info->supported[ARB_TEXTURE_RECTANGLE]) + shader_addline(buffer, "#extension GL_ARB_texture_rectangle : enable\n"); + shader_addline(buffer, "vec4 tmp0, tmp1;\n"); shader_addline(buffer, "vec4 ret;\n"); if (tempreg_used || settings->sRGB_write)

1 0

Ken Thomases : loader: On Mac, if Wine is already on secondary thread, don' t reserve it for the system frameworks.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: cf73350d9ff3e689b1966548b49841252ca7d488 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cf73350d9ff3e689b1966548b… Author: Ken Thomases <ken(a)codeweavers.com> Date: Tue Apr 16 00:37:48 2013 -0500 loader: On Mac, if Wine is already on secondary thread, don't reserve it for the system frameworks. --- libs/wine/loader.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/libs/wine/loader.c b/libs/wine/loader.c index 302e9c1..7af1233 100644 --- a/libs/wine/loader.c +++ b/libs/wine/loader.c @@ -761,6 +761,12 @@ static void apple_main_thread( void (*init_func)(void) ) CFRunLoopSourceContext source_context = { 0 }; CFRunLoopSourceRef source; + if (!pthread_main_np()) + { + init_func(); + return; + } + /* Multi-processing Services can get confused about the main thread if the * first time it's used is on a secondary thread. Use it here to make sure * that doesn't happen. */

1 0

Ken Thomases : libwine: Add libwine's directory (@loader_path) to rpath.
by Alexandre Julliard 16 Apr '13

16 Apr '13

Module: wine Branch: master Commit: 304463b4e14472d6ff7e7cccc59b96356a98d794 URL: http://source.winehq.org/git/wine.git/?a=commit;h=304463b4e14472d6ff7e7cccc… Author: Ken Thomases <ken(a)codeweavers.com> Date: Tue Apr 16 00:37:54 2013 -0500 libwine: Add libwine's directory (@loader_path) to rpath. --- libs/wine/Makefile.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libs/wine/Makefile.in b/libs/wine/Makefile.in index b8c713a..3468ee0 100644 --- a/libs/wine/Makefile.in +++ b/libs/wine/Makefile.in @@ -131,7 +131,7 @@ libwine.dll: $(OBJS) wine.def Makefile.in DYLIB_LDFLAGS = -compatibility_version $(SOVERSION) -current_version $(VERSION) libwine.$(VERSION).dylib: $(OBJS) $(RELPATH) Makefile.in - $(LDSHARED) -install_name @rpath/$(DYNAME) $(DYLIB_LDFLAGS) $(OBJS) $(EXTRALIBS) $(LDFLAGS) $(LIBS) -o $@ + $(LDSHARED) -install_name @rpath/$(DYNAME) -Wl,-rpath,@loader_path/ $(DYLIB_LDFLAGS) $(OBJS) $(EXTRALIBS) $(LDFLAGS) $(LIBS) -o $@ libwine.$(SOVERSION).dylib: libwine.$(VERSION).dylib $(RM) $@ && $(LN_S) libwine.$(VERSION).dylib $@

1 0