May 2010 - Wine-commits - list.winehq.org

Henri Verbeet : d3d9/test: Introduce point_match() to simplify testing point sizes.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: 1c12ba44a70b268d02217d1735ea8e45b9d313c7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1c12ba44a70b268d02217d173… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri May 28 16:06:40 2010 +0200 d3d9/test: Introduce point_match() to simplify testing point sizes. --- dlls/d3d9/tests/visual.c | 103 +++++++++++++-------------------------------- 1 files changed, 30 insertions(+), 73 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 1bc1a50..a619574 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -8769,6 +8769,26 @@ static void vpos_register_test(IDirect3DDevice9 *device) IDirect3DSurface9_Release(backbuffer); } +static BOOL point_match(IDirect3DDevice9 *device, UINT x, UINT y, UINT r) +{ + D3DCOLOR color; + + color = D3DCOLOR_ARGB(0x00, 0xff, 0xff, 0xff); + if (!color_match(getPixelColor(device, x + r, y), color, 1)) return FALSE; + if (!color_match(getPixelColor(device, x - r, y), color, 1)) return FALSE; + if (!color_match(getPixelColor(device, x, y + r), color, 1)) return FALSE; + if (!color_match(getPixelColor(device, x, y - r), color, 1)) return FALSE; + + ++r; + color = D3DCOLOR_ARGB(0x00, 0x00, 0x00, 0xff); + if (!color_match(getPixelColor(device, x + r, y), color, 1)) return FALSE; + if (!color_match(getPixelColor(device, x - r, y), color, 1)) return FALSE; + if (!color_match(getPixelColor(device, x, y + r), color, 1)) return FALSE; + if (!color_match(getPixelColor(device, x, y - r), color, 1)) return FALSE; + + return TRUE; +} + static void pointsize_test(IDirect3DDevice9 *device) { HRESULT hr; @@ -8914,86 +8934,23 @@ static void pointsize_test(IDirect3DDevice9 *device) ok(hr == D3D_OK, "IDirect3DDevice9_EndScene failed hr=%08x\n", hr); } - color = getPixelColor(device, 64-7, 64-7); - ok(color == 0x00ffffff, "pSize: Pixel (64-7),(64-7) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 64+7, 64+7); - ok(color == 0x00ffffff, "pSize: Pixel (64+7),(64+7) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 64-8, 64-8); - ok(color == 0x000000ff, "pSize: Pixel (64-8),(64-8) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 64+8, 64+8); - ok(color == 0x000000ff, "pSize: Pixel (64+8),(64+8) has color 0x%08x, expected 0x000000ff\n", color); - - color = getPixelColor(device, 128-15, 64-15); - ok(color == 0x00ffffff, "pSize: Pixel (128-15),(64-15) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 128+15, 64+15); - ok(color == 0x00ffffff, "pSize: Pixel (128+15),(64+15) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 128-16, 64-16); - ok(color == 0x000000ff, "pSize: Pixel (128-16),(64-16) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 128+16, 64+16); - ok(color == 0x000000ff, "pSize: Pixel (128+16),(64+16) has color 0x%08x, expected 0x000000ff\n", color); - - color = getPixelColor(device, 192-15, 64-15); - ok(color == 0x00ffffff, "pSize: Pixel (192-15),(64-15) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 192+15, 64+15); - ok(color == 0x00ffffff, "pSize: Pixel (192+15),(64+15) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 192-16, 64-16); - ok(color == 0x000000ff, "pSize: Pixel (192-16),(64-16) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 192+16, 64+16); - ok(color == 0x000000ff, "pSize: Pixel (192+16),(64+16) has color 0x%08x, expected 0x000000ff\n", color); + ok(point_match(device, 64, 64, 7), "point_match(64, 64, 7) failed, expected point size 15.\n"); + ok(point_match(device, 128, 64, 15), "point_match(128, 64, 15) failed, expected point size 31.\n"); + ok(point_match(device, 192, 64, 15), "point_match(192, 64, 15) failed, expected point size 31.\n"); if (caps.MaxPointSize >= 63.0) { - color = getPixelColor(device, 256-31, 64-31); - ok(color == 0x00ffffff, "pSize: Pixel (256-31),(64-31) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 256+31, 64+31); - ok(color == 0x00ffffff, "pSize: Pixel (256+31),(64+31) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 256-32, 64-32); - ok(color == 0x000000ff, "pSize: Pixel (256-32),(64-32) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 256+32, 64+32); - ok(color == 0x000000ff, "pSize: Pixel (256+32),(64+32) has color 0x%08x, expected 0x000000ff\n", color); - - color = getPixelColor(device, 384-31, 64-31); - ok(color == 0x00ffffff, "pSize: Pixel (384-31),(64-31) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 384+31, 64+31); - ok(color == 0x00ffffff, "pSize: Pixel (384+31),(64+31) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 384-32, 64-32); - ok(color == 0x000000ff, "pSize: Pixel (384-32),(64-32) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 384+32, 64+32); - ok(color == 0x000000ff, "pSize: Pixel (384+32),(64+32) has color 0x%08x, expected 0x000000ff\n", color); - } - - color = getPixelColor(device, 320-0, 64-0); - ok(color == 0x00ffffff, "pSize: Pixel (320-0),(64-0) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 320-1, 64-1); - ok(color == 0x000000ff, "pSize: Pixel (320-1),(64-1) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 320+1, 64+1); - ok(color == 0x000000ff, "pSize: Pixel (320+1),(64+1) has color 0x%08x, expected 0x000000ff\n", color); + ok(point_match(device, 256, 64, 31), "point_match(256, 64, 31) failed, expected point size 63.\n"); + ok(point_match(device, 384, 64, 31), "point_match(384, 64, 31) failed, expected point size 63.\n"); + } + ok(point_match(device, 320, 64, 0), "point_match(320, 64, 0) failed, expected point size 1.\n"); /* ptsize = 15, ptsize_max = 1 --> point has size 1 */ - color = getPixelColor(device, 448-0, 64-0); - ok(color == 0x00ffffff, "pSize: Pixel (448-0),(64-0) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 448-1, 64-1); - ok(color == 0x000000ff, "pSize: Pixel (448-1),(64-1) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 448+1, 64+1); - ok(color == 0x000000ff, "pSize: Pixel (448+1),(64+1) has color 0x%08x, expected 0x000000ff\n", color); - + ok(point_match(device, 448, 64, 0), "point_match(448, 64, 0) failed, expected point size 1.\n"); /* ptsize = 4, ptsize_max = 1, ptsize_min = 15 --> point has size 1 */ - color = getPixelColor(device, 512-0, 64-0); - ok(color == 0x00ffffff, "pSize: Pixel (512-0),(64-0) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 512-1, 64-1); - ok(color == 0x000000ff, "pSize: Pixel (512-1),(64-1) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 512+1, 64+1); - ok(color == 0x000000ff, "pSize: Pixel (512+1),(64+1) has color 0x%08x, expected 0x000000ff\n", color); - + ok(point_match(device, 512, 64, 0), "point_match(512, 64, 0) failed, expected point size 1.\n"); /* ptsize = 1, ptsize_max = default(64), ptsize_min = 15 --> point has size 15 */ - color = getPixelColor(device, 576-7, 64-7); - ok(color == 0x00ffffff, "pSize: Pixel (576-7),(64-7) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 576+7, 64+7); - ok(color == 0x00ffffff, "pSize: Pixel (576+7),(64+7) has color 0x%08x, expected 0x00ffffff\n", color); - color = getPixelColor(device, 576-8, 64-8); - ok(color == 0x000000ff, "pSize: Pixel (576-8),(64-8) has color 0x%08x, expected 0x000000ff\n", color); - color = getPixelColor(device, 576+8, 64+8); - ok(color == 0x000000ff, "pSize: Pixel (576+8),(64+8) has color 0x%08x, expected 0x000000ff\n", color); + ok(point_match(device, 576, 64, 7), "point_match(576, 64, 7) failed, expected point size 15.\n"); IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL);

1 0

Henri Verbeet : d3d9/tests: Remove an invalid point size test.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: c3cda92cf4207d060579efdee0728a1ec427c7c6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c3cda92cf4207d060579efdee… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri May 28 16:06:41 2010 +0200 d3d9/tests: Remove an invalid point size test. This test tests point size clamping behaviour when POINTSIZE_MAX < POINTSIZE_MIN, but that behaviour isn't consistent between D3D drivers. --- dlls/d3d9/tests/visual.c | 18 ++---------------- 1 files changed, 2 insertions(+), 16 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index a619574..b749494 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -8813,7 +8813,6 @@ static void pointsize_test(IDirect3DDevice9 *device) 384, 64, 0.1, 448, 64, 0.1, 512, 64, 0.1, - 576, 64, 0.1, }; /* Transforms the coordinate system [-1.0;1.0]x[-1.0;1.0] to [0.0;0.0]x[640.0;480.0]. Z is untouched */ @@ -8902,17 +8901,6 @@ static void pointsize_test(IDirect3DDevice9 *device) hr = IDirect3DDevice9_DrawPrimitiveUP(device, D3DPT_POINTLIST, 1, &vertices[18], sizeof(float) * 3); ok(hr == D3D_OK, "IDirect3DDevice9_DrawPrimitiveUP failed, hr=%08x\n", hr); - /* What happens if POINTSIZE_MAX < POINTSIZE_MIN? - * ptsize = 3.0, ptsize_max = 1.0, ptsize_min = 15.0 */ - ptsize = 3.0; - hr = IDirect3DDevice9_SetRenderState(device, D3DRS_POINTSIZE, *((DWORD *) (&ptsize))); - ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed, hr=%08x\n", hr); - ptsize = 15.0; - hr = IDirect3DDevice9_SetRenderState(device, D3DRS_POINTSIZE_MIN, *((DWORD *) (&ptsize))); - ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed, hr=%08x\n", hr); - hr = IDirect3DDevice9_DrawPrimitiveUP(device, D3DPT_POINTLIST, 1, &vertices[21], sizeof(float) * 3); - ok(hr == D3D_OK, "IDirect3DDevice9_DrawPrimitiveUP failed, hr=%08x\n", hr); - hr = IDirect3DDevice9_SetRenderState(device, D3DRS_POINTSIZE_MAX, *((DWORD *) (&ptsizemax_orig))); ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed, hr=%08x\n", hr); @@ -8924,7 +8912,7 @@ static void pointsize_test(IDirect3DDevice9 *device) ptsize = 15.0; hr = IDirect3DDevice9_SetRenderState(device, D3DRS_POINTSIZE_MIN, *((DWORD *) (&ptsize))); ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed, hr=%08x\n", hr); - hr = IDirect3DDevice9_DrawPrimitiveUP(device, D3DPT_POINTLIST, 1, &vertices[24], sizeof(float) * 3); + hr = IDirect3DDevice9_DrawPrimitiveUP(device, D3DPT_POINTLIST, 1, &vertices[21], sizeof(float) * 3); ok(hr == D3D_OK, "IDirect3DDevice9_DrawPrimitiveUP failed, hr=%08x\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_POINTSIZE_MIN, *((DWORD *) (&ptsizemin_orig))); @@ -8947,10 +8935,8 @@ static void pointsize_test(IDirect3DDevice9 *device) ok(point_match(device, 320, 64, 0), "point_match(320, 64, 0) failed, expected point size 1.\n"); /* ptsize = 15, ptsize_max = 1 --> point has size 1 */ ok(point_match(device, 448, 64, 0), "point_match(448, 64, 0) failed, expected point size 1.\n"); - /* ptsize = 4, ptsize_max = 1, ptsize_min = 15 --> point has size 1 */ - ok(point_match(device, 512, 64, 0), "point_match(512, 64, 0) failed, expected point size 1.\n"); /* ptsize = 1, ptsize_max = default(64), ptsize_min = 15 --> point has size 15 */ - ok(point_match(device, 576, 64, 7), "point_match(576, 64, 7) failed, expected point size 15.\n"); + ok(point_match(device, 512, 64, 7), "point_match(512, 64, 7) failed, expected point size 15.\n"); IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL);

1 0

Thomas Mullaly : urlmon/tests: Added tests for the IUri_Get* dword property functions.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: 67d642b54036bc63c4194951ad4f7e17f6f33785 URL: http://source.winehq.org/git/wine.git/?a=commit;h=67d642b54036bc63c4194951a… Author: Thomas Mullaly <thomas.mullaly(a)gmail.com> Date: Thu May 27 19:39:28 2010 -0400 urlmon/tests: Added tests for the IUri_Get* dword property functions. --- dlls/urlmon/tests/uri.c | 130 +++++++++++++++++++++++++++++++++++++++++++++++ dlls/urlmon/uri.c | 22 ++++++++- 2 files changed, 151 insertions(+), 1 deletions(-) diff --git a/dlls/urlmon/tests/uri.c b/dlls/urlmon/tests/uri.c index 7e2496e..58f39b9 100644 --- a/dlls/urlmon/tests/uri.c +++ b/dlls/urlmon/tests/uri.c @@ -835,6 +835,133 @@ static void test_IUri_GetStrProperties(void) { } } +static void test_IUri_GetDwordProperties(void) { + IUri *uri = NULL; + HRESULT hr; + DWORD i; + + /* Make sure all the 'Get*' dword property functions handle invalid args correctly. */ + hr = pCreateUri(http_urlW, 0, 0, &uri); + ok(hr == S_OK, "Error: CreateUri returned 0x%08x, expected 0x%08x.\n", hr, S_OK); + if(SUCCEEDED(hr)) { + hr = IUri_GetHostType(uri, NULL); + ok(hr == E_INVALIDARG, "Error: GetHostType returned 0x%08x, expected 0x%08x.\n", hr, E_INVALIDARG); + + hr = IUri_GetPort(uri, NULL); + ok(hr == E_INVALIDARG, "Error: GetPort returned 0x%08x, expected 0x%08x.\n", hr, E_INVALIDARG); + + hr = IUri_GetScheme(uri, NULL); + ok(hr == E_INVALIDARG, "Error: GetScheme returned 0x%08x, expected 0x%08x.\n", hr, E_INVALIDARG); + + hr = IUri_GetZone(uri, NULL); + ok(hr == E_INVALIDARG, "Error: GetZone returned 0x%08x, expected 0x%08x.\n", hr, E_INVALIDARG); + } + if(uri) IUri_Release(uri); + + for(i = 0; i < sizeof(uri_tests)/sizeof(uri_tests[0]); ++i) { + uri_properties test = uri_tests[i]; + LPWSTR uriW; + uri = NULL; + + uriW = a2w(test.uri); + hr = pCreateUri(uriW, test.create_flags, 0, &uri); + if(test.create_todo) { + todo_wine { + ok(hr == test.create_expected, "Error: CreateUri returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, test.create_expected, i); + } + } else { + ok(hr == test.create_expected, "Error: CreateUri returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, test.create_expected, i); + } + + if(SUCCEEDED(hr)) { + uri_dword_property prop; + DWORD received; + + /* Assign an insane value so tests don't accidentally pass when + * they shouldn't! + */ + received = -9999999; + + /* GetHostType() tests. */ + prop = test.dword_props[Uri_PROPERTY_HOST_TYPE-Uri_PROPERTY_DWORD_START]; + hr = IUri_GetHostType(uri, &received); + if(prop.todo) { + todo_wine { + ok(hr == prop.expected, "Error: GetHostType returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + } + todo_wine { + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + } else { + ok(hr == prop.expected, "Error: GetHostType returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + received = -9999999; + + /* GetPort() tests. */ + prop = test.dword_props[Uri_PROPERTY_PORT-Uri_PROPERTY_DWORD_START]; + hr = IUri_GetPort(uri, &received); + if(prop.todo) { + todo_wine { + ok(hr == prop.expected, "Error: GetPort returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + } + todo_wine { + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + } else { + ok(hr == prop.expected, "Error: GetPort returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + received = -9999999; + + /* GetScheme() tests. */ + prop = test.dword_props[Uri_PROPERTY_SCHEME-Uri_PROPERTY_DWORD_START]; + hr = IUri_GetScheme(uri, &received); + if(prop.todo) { + todo_wine { + ok(hr == prop.expected, "Error: GetScheme returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + } + todo_wine { + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + } else { + ok(hr == prop.expected, "Error: GetScheme returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + received = -9999999; + + /* GetZone() tests. */ + prop = test.dword_props[Uri_PROPERTY_ZONE-Uri_PROPERTY_DWORD_START]; + hr = IUri_GetZone(uri, &received); + if(prop.todo) { + todo_wine { + ok(hr == prop.expected, "Error: GetZone returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + } + todo_wine { + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + } else { + ok(hr == prop.expected, "Error: GetZone returned 0x%08x, expected 0x%08x on uri_tests[%d].\n", + hr, prop.expected, i); + ok(received == prop.value, "Error: Expected %d but got %d on uri_tests[%d].\n", prop.value, received, i); + } + } + + if(uri) IUri_Release(uri); + + heap_free(uriW); + } +} + START_TEST(uri) { HMODULE hurlmon; @@ -860,4 +987,7 @@ START_TEST(uri) { trace("test IUri_GetStrProperties...\n"); test_IUri_GetStrProperties(); + + trace("test IUri_GetDwordProperties...\n"); + test_IUri_GetDwordProperties(); } diff --git a/dlls/urlmon/uri.c b/dlls/urlmon/uri.c index 8d8130e..27b0863 100644 --- a/dlls/urlmon/uri.c +++ b/dlls/urlmon/uri.c @@ -104,7 +104,7 @@ static HRESULT WINAPI Uri_GetPropertyDWORD(IUri *iface, Uri_PROPERTY uriProp, DW /* Microsoft's implementation for the ZONE property of a URI seems to be lacking... * From what I can tell, instead of checking which URLZONE the URI belongs to it - * simply assigns URLZONE_INVALID and returns E_NOTIMPL. This also applies the GetZone + * simply assigns URLZONE_INVALID and returns E_NOTIMPL. This also applies to the GetZone * function. */ if(uriProp == Uri_PROPERTY_ZONE) { @@ -287,6 +287,10 @@ static HRESULT WINAPI Uri_GetHostType(IUri *iface, DWORD *pdwHostType) { Uri *This = URI_THIS(iface); FIXME("(%p)->(%p)\n", This, pdwHostType); + + if(!pdwHostType) + return E_INVALIDARG; + return E_NOTIMPL; } @@ -294,6 +298,10 @@ static HRESULT WINAPI Uri_GetPort(IUri *iface, DWORD *pdwPort) { Uri *This = URI_THIS(iface); FIXME("(%p)->(%p)\n", This, pdwPort); + + if(!pdwPort) + return E_INVALIDARG; + return E_NOTIMPL; } @@ -301,6 +309,10 @@ static HRESULT WINAPI Uri_GetScheme(IUri *iface, DWORD *pdwScheme) { Uri *This = URI_THIS(iface); FIXME("(%p)->(%p)\n", This, pdwScheme); + + if(!pdwScheme) + return E_INVALIDARG; + return E_NOTIMPL; } @@ -308,6 +320,14 @@ static HRESULT WINAPI Uri_GetZone(IUri *iface, DWORD *pdwZone) { Uri *This = URI_THIS(iface); FIXME("(%p)->(%p)\n", This, pdwZone); + + if(!pdwZone) + return E_INVALIDARG; + + /* Microsoft doesn't seem to have this implemented yet... See + * the comment in Uri_GetPropertyDWORD for more about this. + */ + *pdwZone = URLZONE_INVALID; return E_NOTIMPL; }

1 0

Hans Leidekker : wuapi: Return success from automatic_updates_Pause.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: c9fd854e04819a06ee10e04fb090810b4e879f10 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c9fd854e04819a06ee10e04fb… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri May 28 08:47:04 2010 +0200 wuapi: Return success from automatic_updates_Pause. --- dlls/wuapi/updates.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wuapi/updates.c b/dlls/wuapi/updates.c index 1bf063b..0366ebc 100644 --- a/dlls/wuapi/updates.c +++ b/dlls/wuapi/updates.c @@ -144,7 +144,7 @@ static HRESULT WINAPI automatic_updates_Pause( IAutomaticUpdates *This ) { FIXME("\n"); - return E_NOTIMPL; + return S_OK; } static HRESULT WINAPI automatic_updates_Resume(

1 0

Dmitry Timoshkov : winex11.drv: Simulate pressing Alt+F4 instead of clicking the caption Close button when handling the WM_DELETE_WINDOW event .
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: 167514ce940447bf301faff4dd4b19978191c8bf URL: http://source.winehq.org/git/wine.git/?a=commit;h=167514ce940447bf301faff4d… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Fri May 28 15:06:14 2010 +0900 winex11.drv: Simulate pressing Alt+F4 instead of clicking the caption Close button when handling the WM_DELETE_WINDOW event. --- dlls/winex11.drv/event.c | 11 ++++++----- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index da45fe2..5bb1ecf 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -522,7 +522,6 @@ static void handle_wm_protocols( HWND hwnd, XClientMessageEvent *event ) if (IsWindowEnabled(hwnd)) { HMENU hSysMenu; - POINT pt; if (GetClassLongW(hwnd, GCL_STYLE) & CS_NOCLOSE) return; hSysMenu = GetSystemMenu(hwnd, FALSE); @@ -553,10 +552,12 @@ static void handle_wm_protocols( HWND hwnd, XClientMessageEvent *event ) break; } } - /* Simulate clicking the caption Close button */ - GetCursorPos( &pt ); - PostMessageW( hwnd, WM_NCLBUTTONDOWN, HTCLOSE, MAKELPARAM( pt.x, pt.y ) ); - PostMessageW( hwnd, WM_LBUTTONUP, HTCLOSE, MAKELPARAM( pt.x, pt.y ) ); + + /* Simulate pressing Alt+F4 */ + keybd_event(VK_MENU, 0, 0, 0); + keybd_event(VK_F4, 0, 0, 0); + keybd_event(VK_F4, 0, KEYEVENTF_KEYUP, 0); + keybd_event(VK_MENU, 0, KEYEVENTF_KEYUP, 0); } } else if (protocol == x11drv_atom(WM_TAKE_FOCUS))

1 0

Kristofer Henriksson : winemp3.acm: Fix the Mac-specific code.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: c58eff621d9c0d3f02f32e8b87f9f539d63a6e3a URL: http://source.winehq.org/git/wine.git/?a=commit;h=c58eff621d9c0d3f02f32e8b8… Author: Kristofer Henriksson <kthenriksson(a)gmail.com> Date: Mon May 24 15:04:42 2010 -0400 winemp3.acm: Fix the Mac-specific code. The previous code does not function on Snow Leopard, and does not conform to Apple's documentation. This fix should function on all versions of OS X later than 10.2. --- configure | 18 +--- configure.ac | 8 +- dlls/winemp3.acm/mpegl3.c | 326 ++++++++++++++++++++++++++++++++------------- include/config.h.in | 10 -- 4 files changed, 235 insertions(+), 127 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c58eff621d9c0d3f02f32…

1 0

Alexandre Julliard : winex11: Set window style to normal for windows with a minimize box.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: 8b84d64075a78c0de920ecd2052fe64506179099 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8b84d64075a78c0de920ecd20… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 28 12:12:32 2010 +0200 winex11: Set window style to normal for windows with a minimize box. Some window managers don't allow minimizing dialog windows. --- dlls/winex11.drv/window.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index ef6c442..f1f2803 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -1181,6 +1181,7 @@ static void set_wm_hints( Display *display, struct x11drv_win_data *data ) /* set the WM_WINDOW_TYPE */ if (style & WS_THICKFRAME) window_type = x11drv_atom(_NET_WM_WINDOW_TYPE_NORMAL); else if (ex_style & WS_EX_APPWINDOW) window_type = x11drv_atom(_NET_WM_WINDOW_TYPE_NORMAL); + else if (style & WS_MINIMIZEBOX) window_type = x11drv_atom(_NET_WM_WINDOW_TYPE_NORMAL); else if (style & WS_DLGFRAME) window_type = x11drv_atom(_NET_WM_WINDOW_TYPE_DIALOG); else if (ex_style & WS_EX_DLGMODALFRAME) window_type = x11drv_atom(_NET_WM_WINDOW_TYPE_DIALOG); else if ((style & WS_POPUP) && owner) window_type = x11drv_atom(_NET_WM_WINDOW_TYPE_DIALOG);

1 0

Alexandre Julliard : winex11: Set size hints to 1x1 for zero-sized windows.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: 81c4b4120d0b2f3af2b3bb2d1fc3071d458a7398 URL: http://source.winehq.org/git/wine.git/?a=commit;h=81c4b4120d0b2f3af2b3bb2d1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 28 12:13:01 2010 +0200 winex11: Set size hints to 1x1 for zero-sized windows. --- dlls/winex11.drv/window.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index f1f2803..b85aff0 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -1018,6 +1018,8 @@ static void set_size_hints( Display *display, struct x11drv_win_data *data, DWOR { size_hints->max_width = data->whole_rect.right - data->whole_rect.left; size_hints->max_height = data->whole_rect.bottom - data->whole_rect.top; + if (size_hints->max_width <= 0 ||size_hints->max_height <= 0) + size_hints->max_width = size_hints->max_height = 1; size_hints->min_width = size_hints->max_width; size_hints->min_height = size_hints->max_height; size_hints->flags |= PMinSize | PMaxSize;

1 0

Alexandre Julliard : winex11: Map zero-size windows and set an empty window region for them.
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: 61e50e15ba45ad54655f98619f5ef33917033165 URL: http://source.winehq.org/git/wine.git/?a=commit;h=61e50e15ba45ad54655f98619… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 28 12:14:43 2010 +0200 winex11: Map zero-size windows and set an empty window region for them. --- dlls/winex11.drv/window.c | 34 +++++++++++++++------------------- 1 files changed, 15 insertions(+), 19 deletions(-) diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index b85aff0..8734315 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -220,9 +220,6 @@ static BOOL is_window_managed( HWND hwnd, UINT swp_flags, const RECT *window_rec */ static BOOL is_window_rect_mapped( const RECT *rect ) { - /* don't map if rect is empty */ - if (IsRectEmpty( rect )) return FALSE; - /* don't map if rect is off-screen */ if (rect->left >= virtual_screen_rect.right || rect->top >= virtual_screen_rect.bottom || @@ -249,19 +246,6 @@ static inline BOOL is_window_resizable( struct x11drv_win_data *data, DWORD styl /*********************************************************************** - * get_window_owner - */ -static HWND get_window_owner( HWND hwnd ) -{ - RECT rect; - HWND owner = GetWindow( hwnd, GW_OWNER ); - /* ignore the zero-size owners used by Delphi apps */ - if (owner && GetWindowRect( owner, &rect ) && IsRectEmpty( &rect )) owner = 0; - return owner; -} - - -/*********************************************************************** * get_mwm_decorations */ static unsigned long get_mwm_decorations( struct x11drv_win_data *data, @@ -456,6 +440,16 @@ static void sync_window_region( Display *display, struct x11drv_win_data *data, if (!data->whole_window) return; data->shaped = FALSE; + if (IsRectEmpty( &data->window_rect )) /* set an empty shape */ + { + static XRectangle empty_rect; + wine_tsx11_lock(); + XShapeCombineRectangles( display, data->whole_window, ShapeBounding, 0, 0, + &empty_rect, 1, ShapeSet, YXBanded ); + wine_tsx11_unlock(); + return; + } + if (hrgn == (HRGN)1) /* hack: win_region == 1 means retrieve region from server */ { if (!(hrgn = CreateRectRgn( 0, 0, 0, 0 ))) return; @@ -1169,7 +1163,7 @@ static void set_wm_hints( Display *display, struct x11drv_win_data *data ) { style = GetWindowLongW( data->hwnd, GWL_STYLE ); ex_style = GetWindowLongW( data->hwnd, GWL_EXSTYLE ); - owner = get_window_owner( data->hwnd ); + owner = GetWindow( data->hwnd, GW_OWNER ); if ((owner_win = get_owner_whole_window( owner, data->managed ))) group_leader = owner_win; } @@ -1260,7 +1254,7 @@ void update_net_wm_states( Display *display, struct x11drv_win_data *data ) new_state |= (1 << NET_WM_STATE_ABOVE); if (ex_style & WS_EX_TOOLWINDOW) new_state |= (1 << NET_WM_STATE_SKIP_TASKBAR) | (1 << NET_WM_STATE_SKIP_PAGER); - if (!(ex_style & WS_EX_APPWINDOW) && get_window_owner( data->hwnd )) + if (!(ex_style & WS_EX_APPWINDOW) && GetWindow( data->hwnd, GW_OWNER )) new_state |= (1 << NET_WM_STATE_SKIP_TASKBAR); if (!data->mapped) /* set the _NET_WM_STATE atom directly */ @@ -1503,6 +1497,8 @@ static void sync_window_position( Display *display, struct x11drv_win_data *data XReconfigureWMWindow( display, data->whole_window, DefaultScreen(display), mask, &changes ); #ifdef HAVE_LIBXSHAPE + if (IsRectEmpty( old_window_rect ) != IsRectEmpty( &data->window_rect )) + sync_window_region( display, data, (HRGN)1 ); if (data->shaped) { int old_x_offset = old_window_rect->left - old_whole_rect->left; @@ -1690,7 +1686,7 @@ static Window create_whole_window( Display *display, struct x11drv_win_data *dat sync_window_text( display, data->whole_window, text ); /* set the window region */ - if (win_rgn) sync_window_region( display, data, win_rgn ); + if (win_rgn || IsRectEmpty( &data->window_rect )) sync_window_region( display, data, win_rgn ); /* set the window opacity */ if (!GetLayeredWindowAttributes( data->hwnd, &key, &alpha, &layered_flags )) layered_flags = 0;

1 0

Alexandre Julliard : winex11: Only unmap windows when the window rect changes to unmapped the first time .
by Alexandre Julliard 28 May '10

28 May '10

Module: wine Branch: master Commit: 2db72e98597a4025873ded6d1e844ec5b37ad0f8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2db72e98597a4025873ded6d1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 28 12:16:17 2010 +0200 winex11: Only unmap windows when the window rect changes to unmapped the first time. --- dlls/winex11.drv/window.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index 8734315..3999054 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -214,7 +214,7 @@ static BOOL is_window_managed( HWND hwnd, UINT swp_flags, const RECT *window_rec /*********************************************************************** - * X11DRV_is_window_rect_mapped + * is_window_rect_mapped * * Check if the X whole window should be mapped based on its rectangle */ @@ -2329,7 +2329,8 @@ void CDECL X11DRV_WindowPosChanged( HWND hwnd, HWND insert_after, UINT swp_flags if (data->mapped) { if (((swp_flags & SWP_HIDEWINDOW) && !(new_style & WS_VISIBLE)) || - (event_type != ConfigureNotify && !is_window_rect_mapped( rectWindow ))) + (event_type != ConfigureNotify && + !is_window_rect_mapped( rectWindow ) && is_window_rect_mapped( &old_window_rect ))) unmap_window( display, data ); }

1 0