February 2016 - Wine-commits

Stefan Dösinger : wined3d: Introduce wined3d_texture_check_block_align().
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: 58719f60d5dca15f0f0de625bc307e04b01fc6d9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=58719f60d5dca15f0f0de625b… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Wed Feb 17 18:35:35 2016 +0100 wined3d: Introduce wined3d_texture_check_block_align(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/surface.c | 31 ++----------------------------- dlls/wined3d/texture.c | 29 +++++++++++++++++++++++++++++ dlls/wined3d/volume.c | 31 ++----------------------------- dlls/wined3d/wined3d_private.h | 2 ++ 4 files changed, 35 insertions(+), 58 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index c1d7b18..7858d15 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -1538,38 +1538,11 @@ void wined3d_surface_upload_data(struct wined3d_surface *surface, const struct w } } -static BOOL surface_check_block_align(struct wined3d_surface *surface, const struct wined3d_box *box) -{ - UINT width_mask, height_mask; - - if (!box->left && !box->top - && box->right == surface->resource.width - && box->bottom == surface->resource.height) - return TRUE; - - if ((box->left >= box->right) - || (box->top >= box->bottom) - || (box->right > surface->resource.width) - || (box->bottom > surface->resource.height)) - return FALSE; - - /* This assumes power of two block sizes, but NPOT block sizes would be - * silly anyway. */ - width_mask = surface->resource.format->block_width - 1; - height_mask = surface->resource.format->block_height - 1; - - if (!(box->left & width_mask) && !(box->top & height_mask) - && !(box->right & width_mask) && !(box->bottom & height_mask)) - return TRUE; - - return FALSE; -} - static BOOL surface_check_block_align_rect(struct wined3d_surface *surface, const RECT *rect) { struct wined3d_box box = {rect->left, rect->top, rect->right, rect->bottom, 0, 1}; - return surface_check_block_align(surface, &box); + return wined3d_texture_check_block_align(surface->container, surface->texture_level, &box); } HRESULT surface_upload_from_surface(struct wined3d_surface *dst_surface, const POINT *dst_point, @@ -2301,7 +2274,7 @@ HRESULT wined3d_surface_map(struct wined3d_surface *surface, struct wined3d_map_ } if ((fmt_flags & WINED3DFMT_FLAG_BLOCKS) && box - && !surface_check_block_align(surface, box)) + && !wined3d_texture_check_block_align(surface->container, surface->texture_level, box)) { WARN("Map box %s is misaligned for %ux%u blocks.\n", debug_box(box), format->block_width, format->block_height); diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index 7a24ff0..804856b 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -1201,6 +1201,35 @@ static const struct wined3d_texture_ops texture3d_ops = texture3d_prepare_texture, }; +BOOL wined3d_texture_check_block_align(const struct wined3d_texture *texture, + unsigned int level, const struct wined3d_box *box) +{ + const struct wined3d_format *format = texture->resource.format; + unsigned int height = max(1, texture->resource.height >> level); + unsigned int width = max(1, texture->resource.width >> level); + unsigned int width_mask, height_mask; + + if ((box->left >= box->right) + || (box->top >= box->bottom) + || (box->right > width) + || (box->bottom > height)) + return FALSE; + + /* This assumes power of two block sizes, but NPOT block sizes would be + * silly anyway. + * + * This also assumes that the format's block depth is 1. */ + width_mask = format->block_width - 1; + height_mask = format->block_height - 1; + + if ((box->left & width_mask) || (box->top & height_mask) + || (box->right & width_mask && box->right != width) + || (box->bottom & height_mask && box->bottom != height)) + return FALSE; + + return TRUE; +} + static HRESULT texture3d_resource_sub_resource_map(struct wined3d_resource *resource, unsigned int sub_resource_idx, struct wined3d_map_desc *map_desc, const struct wined3d_box *box, DWORD flags) { diff --git a/dlls/wined3d/volume.c b/dlls/wined3d/volume.c index 75585d8..6f5de73 100644 --- a/dlls/wined3d/volume.c +++ b/dlls/wined3d/volume.c @@ -433,34 +433,6 @@ static void volume_unload(struct wined3d_resource *resource) resource_unload(resource); } -static BOOL volume_check_block_align(const struct wined3d_volume *volume, - const struct wined3d_box *box) -{ - UINT width_mask, height_mask; - const struct wined3d_format *format = volume->resource.format; - - if (!box) - return TRUE; - - /* This assumes power of two block sizes, but NPOT block sizes would be - * silly anyway. - * - * This also assumes that the format's block depth is 1. */ - width_mask = format->block_width - 1; - height_mask = format->block_height - 1; - - if (box->left & width_mask) - return FALSE; - if (box->top & height_mask) - return FALSE; - if (box->right & width_mask && box->right != volume->resource.width) - return FALSE; - if (box->bottom & height_mask && box->bottom != volume->resource.height) - return FALSE; - - return TRUE; -} - static BOOL wined3d_volume_check_box_dimensions(const struct wined3d_volume *volume, const struct wined3d_box *box) { @@ -512,7 +484,8 @@ HRESULT wined3d_volume_map(struct wined3d_volume *volume, WARN("Map box is invalid.\n"); return WINED3DERR_INVALIDCALL; } - if ((fmt_flags & WINED3DFMT_FLAG_BLOCKS) && !volume_check_block_align(volume, box)) + if ((fmt_flags & WINED3DFMT_FLAG_BLOCKS) && box + && !wined3d_texture_check_block_align(volume->container, volume->texture_level, box)) { WARN("Map box %s is misaligned for %ux%u blocks.\n", debug_box(box), format->block_width, format->block_height); diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index a243727..a20bc65 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2405,6 +2405,8 @@ void wined3d_texture_bind(struct wined3d_texture *texture, struct wined3d_context *context, BOOL srgb) DECLSPEC_HIDDEN; void wined3d_texture_bind_and_dirtify(struct wined3d_texture *texture, struct wined3d_context *context, BOOL srgb) DECLSPEC_HIDDEN; +BOOL wined3d_texture_check_block_align(const struct wined3d_texture *texture, + unsigned int level, const struct wined3d_box *box) DECLSPEC_HIDDEN; void wined3d_texture_force_reload(struct wined3d_texture *texture) DECLSPEC_HIDDEN; void wined3d_texture_load(struct wined3d_texture *texture, struct wined3d_context *context, BOOL srgb) DECLSPEC_HIDDEN;

1 0

Ken Thomases : winemac: Wrap performing requests from background threads in an autorelease pool.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: e5c120893dc164a0ce387ea10228b1a2f8094eaf URL: http://source.winehq.org/git/wine.git/?a=commit;h=e5c120893dc164a0ce387ea10… Author: Ken Thomases <ken(a)codeweavers.com> Date: Wed Feb 17 13:42:54 2016 -0600 winemac: Wrap performing requests from background threads in an autorelease pool. Cocoa manages an autorelease pool on the main thread, but it only drains it when it processes an event. Our requests come through a run loop source, which doesn't count as an event. So, autoreleased objects can accumulate when the app is not being interacted with. Signed-off-by: Ken Thomases <ken(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/cocoa_app.m | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index fb9edc2..176d35c 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -2295,6 +2295,7 @@ static void PerformRequest(void *info) for (;;) { + NSAutoreleasePool* pool = [[NSAutoreleasePool alloc] init]; __block dispatch_block_t block; dispatch_sync(controller->requestsManipQueue, ^{ @@ -2308,10 +2309,14 @@ static void PerformRequest(void *info) }); if (!block) + { + [pool release]; break; + } block(); [block release]; + [pool release]; } }

1 0

Francois Gouget : schedsvc/tests: Recognize Windows 10's RPC version.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: d03c5b483c2465bcc4bfbe451b0cecfce40152b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d03c5b483c2465bcc4bfbe451… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Feb 18 03:14:22 2016 +0100 schedsvc/tests: Recognize Windows 10's RPC version. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/schedsvc/tests/rpcapi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/schedsvc/tests/rpcapi.c b/dlls/schedsvc/tests/rpcapi.c index 0220641..7aeea17 100644 --- a/dlls/schedsvc/tests/rpcapi.c +++ b/dlls/schedsvc/tests/rpcapi.c @@ -114,7 +114,8 @@ START_TEST(rpcapi) version = 0; hr = SchRpcHighestVersion(&version); ok(hr == S_OK, "expected S_OK, got %#x\n", hr); - ok(version == 0x10002 /* vista */ || version == 0x10003 /* win7 */ || version == 0x10004 /* win8 */, + ok(version == 0x10002 /* vista */ || version == 0x10003 /* win7 */ || + version == 0x10004 /* win8 */ || version == 0x10005 /* win10 */, "wrong version %#x\n", version); SetUnhandledExceptionFilter(old_exception_filter);

1 0

Francois Gouget : msvcrt/tests: Fix the Norwegian locale tests on Windows 10.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: 981c16e4e60fd03673ae0ed933b5f7aa7df3bf41 URL: http://source.winehq.org/git/wine.git/?a=commit;h=981c16e4e60fd03673ae0ed93… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Feb 18 03:15:20 2016 +0100 msvcrt/tests: Fix the Norwegian locale tests on Windows 10. Also add comments to indicate which Windows version returns what. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/tests/locale.c | 30 ++++++++++++++++++------------ 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/dlls/msvcrt/tests/locale.c b/dlls/msvcrt/tests/locale.c index f4d86bc..89761dd 100644 --- a/dlls/msvcrt/tests/locale.c +++ b/dlls/msvcrt/tests/locale.c @@ -148,11 +148,11 @@ todo_wine ok(ret != NULL || broken (ret == NULL), "ret == NULL\n"); if(ret) { + trace("Chinese_China.936=%s\n", ret); todo_wine - ok(!strcmp(ret, "Chinese (Simplified)_People's Republic of China.936") - || !strcmp(ret, "Chinese (Simplified)_China.936") + ok(!strcmp(ret, "Chinese (Simplified)_People's Republic of China.936") /* Vista - Win7 */ + || !strcmp(ret, "Chinese (Simplified)_China.936") /* Win8 - Win10 */ || broken(!strcmp(ret, "Chinese_People's Republic of China.936")), "ret = %s\n", ret); - trace("ret is %s\n", ret); } ret = setlocale(LC_ALL, "csy"); @@ -450,33 +450,39 @@ todo_wine ret = setlocale(LC_ALL, "non"); ok(ret != NULL || broken (ret == NULL), "ret == NULL\n"); if(ret) - ok(!strcmp( ret, "Norwegian-Nynorsk_Norway.1252") + ok(!strcmp( ret, "Norwegian-Nynorsk_Norway.1252") /* XP - Win10 */ || !strcmp(ret, "Norwegian (Nynorsk)_Norway.1252") || broken(!strcmp(ret, "Norwegian (Bokm\xe5l)_Norway.1252")) - || broken(!strcmp(ret, "Norwegian_Norway.1252")), "ret = %s\n", ret); + || broken(!strcmp(ret, "Norwegian_Norway.1252")), /* WinME */ + "ret = %s\n", ret); ret = setlocale(LC_ALL, "nor"); ok(ret != NULL || broken (ret == NULL), "ret == NULL\n"); if(ret) - ok(!strcmp(ret, "Norwegian (Bokm\xe5l)_Norway.1252") + ok(!strcmp(ret, "Norwegian (Bokm\xe5l)_Norway.1252") /* XP - Win8 */ + || !strcmp(ret, "Norwegian Bokm\xe5l_Norway.1252") /* Win10 */ || !strcmp(ret, "Norwegian (Bokmal)_Norway.1252") - || broken(!strcmp(ret, "Norwegian_Norway.1252")), "ret = %s\n", ret); + || broken(!strcmp(ret, "Norwegian_Norway.1252")), /* WinME */ + "ret = %s\n", ret); ret = setlocale(LC_ALL, "norwegian-bokmal"); ok(ret != NULL || broken (ret == NULL), "ret == NULL\n"); if(ret) - ok(!strcmp(ret, "Norwegian (Bokm\xe5l)_Norway.1252") + ok(!strcmp(ret, "Norwegian (Bokm\xe5l)_Norway.1252") /* XP - Win8 */ + || !strcmp(ret, "Norwegian Bokm\xe5l_Norway.1252") /* Win10 */ || !strcmp(ret, "Norwegian (Bokmal)_Norway.1252") - || broken(!strcmp(ret, "Norwegian_Norway.1252")), "ret = %s\n", ret); + || broken(!strcmp(ret, "Norwegian_Norway.1252")), /* WinME */ + "ret = %s\n", ret); ret = setlocale(LC_ALL, "norwegian-nynorsk"); ok(ret != NULL || broken (ret == NULL), "ret == NULL\n"); if(ret) - ok(!strcmp(ret, "Norwegian-Nynorsk_Norway.1252") + ok(!strcmp(ret, "Norwegian-Nynorsk_Norway.1252") /* Vista - Win10 */ || !strcmp(ret, "Norwegian (Nynorsk)_Norway.1252") - || broken(!strcmp(ret, "Norwegian_Norway.1252")) + || broken(!strcmp(ret, "Norwegian_Norway.1252")) /* WinME */ || broken(!strcmp(ret, "Norwegian (Bokmal)_Norway.1252")) - || broken(!strcmp(ret, "Norwegian (Bokm\xe5l)_Norway.1252")), "ret = %s\n", ret); + || broken(!strcmp(ret, "Norwegian (Bokm\xe5l)_Norway.1252")) /* XP & 2003 */, + "ret = %s\n", ret); ret = setlocale(LC_ALL, "plk"); ok(ret != NULL || broken (ret == NULL), "ret == NULL\n");

1 0

Francois Gouget : crypt32/tests: Clarify some base64 test failed messages.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: e1f0fb8b2d6d27f7498f432160d805ee3a46e16c URL: http://source.winehq.org/git/wine.git/?a=commit;h=e1f0fb8b2d6d27f7498f43216… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Feb 18 03:17:36 2016 +0100 crypt32/tests: Clarify some base64 test failed messages. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/crypt32/tests/base64.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/crypt32/tests/base64.c b/dlls/crypt32/tests/base64.c index c6073aa..335b9b7 100644 --- a/dlls/crypt32/tests/base64.c +++ b/dlls/crypt32/tests/base64.c @@ -319,20 +319,20 @@ static void testStringToBinaryA(void) ret = pCryptStringToBinaryA(NULL, 0, 0, NULL, NULL, NULL, NULL); ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, - "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); + "Expected ERROR_INVALID_PARAMETER, got ret=%d le=%u\n", ret, GetLastError()); ret = pCryptStringToBinaryA(NULL, 0, 0, NULL, &bufLen, NULL, NULL); ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, - "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); + "Expected ERROR_INVALID_PARAMETER, got ret=%d le=%u\n", ret, GetLastError()); /* Bogus format */ ret = pCryptStringToBinaryA(tests[0].base64, 0, 0, NULL, &bufLen, NULL, NULL); ok(!ret && GetLastError() == ERROR_INVALID_DATA, - "Expected ERROR_INVALID_DATA, got %d\n", GetLastError()); + "Expected ERROR_INVALID_DATA, got ret=%d le=%u\n", ret, GetLastError()); /* Decoding doesn't expect the NOCR flag to be specified */ ret = pCryptStringToBinaryA(tests[0].base64, 1, CRYPT_STRING_BASE64 | CRYPT_STRING_NOCR, NULL, &bufLen, NULL, NULL); ok(!ret && GetLastError() == ERROR_INVALID_DATA, - "Expected ERROR_INVALID_DATA, got %d\n", GetLastError()); + "Expected ERROR_INVALID_DATA, got ret=%d le=%u\n", ret, GetLastError()); /* Bad strings */ for (i = 0; i < sizeof(badStrings) / sizeof(badStrings[0]); i++) { @@ -340,7 +340,7 @@ static void testStringToBinaryA(void) ret = pCryptStringToBinaryA(badStrings[i].str, 0, badStrings[i].format, NULL, &bufLen, NULL, NULL); ok(!ret && GetLastError() == ERROR_INVALID_DATA, - "Expected ERROR_INVALID_DATA, got %d\n", GetLastError()); + "Expected ERROR_INVALID_DATA, got ret=%d le=%u\n", ret, GetLastError()); } /* Good strings */ for (i = 0; i < sizeof(tests) / sizeof(tests[0]); i++)

1 0

Francois Gouget : shell32/tests: Only look for Explorer windows for the progman tests.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: bd111803d289eb6e6b82473a3d8641563237ea5b URL: http://source.winehq.org/git/wine.git/?a=commit;h=bd111803d289eb6e6b82473a3… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Feb 18 03:18:13 2016 +0100 shell32/tests: Only look for Explorer windows for the progman tests. In the Spanish locale the 'Startup' folder is called 'Inicio' which matches another window. Closing that window causes the shutdown dialog to pop up, resulting in a test timeout. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/tests/progman_dde.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/dlls/shell32/tests/progman_dde.c b/dlls/shell32/tests/progman_dde.c index a3faa83..4c9116e 100644 --- a/dlls/shell32/tests/progman_dde.c +++ b/dlls/shell32/tests/progman_dde.c @@ -372,7 +372,13 @@ static HWND CheckWindowCreated(const char *winName, BOOL closeWindow, int testPa for (i = 0; window == NULL && i < PDDE_POLL_NUM; i++) { Sleep(PDDE_POLL_TIME); - window = FindWindowA(NULL, winName); + /* Specify the window class name to make sure what we find is really an + * Explorer window. Explorer used two different window classes so try + * both. + */ + window = FindWindowA("ExplorerWClass", winName); + if (!window) + window = FindWindowA("CabinetWClass", winName); } ok (window != NULL, "Window \"%s\" was not created in %i seconds - assumed failure.%s\n", winName, PDDE_POLL_NUM*PDDE_POLL_TIME/1000, GetStringFromTestParams(testParams));

1 0

Frédéric Delanoy : dwrite/tests: Use todo_wine_if() in tests.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: 4f1a8da1cd6a6cee57351f045aca1bd4177cdbc9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4f1a8da1cd6a6cee57351f045… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Feb 17 20:43:13 2016 +0100 dwrite/tests: Use todo_wine_if() in tests. Signed-off-by: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dwrite/tests/layout.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/dlls/dwrite/tests/layout.c b/dlls/dwrite/tests/layout.c index 3eaca0c..cd2a15f 100644 --- a/dlls/dwrite/tests/layout.c +++ b/dlls/dwrite/tests/layout.c @@ -2057,9 +2057,7 @@ todo_wine if (metrics[i].isSoftHyphen) ok(!metrics[i].isWhitespace, "%u: got %d\n", i, metrics[i].isWhitespace); if (metrics[i].isNewline) { - if (i == 17 || i == 19) - todo_wine ok(metrics[i].width == 0.0f, "%u: got width %f\n", i, metrics[i].width); - else + todo_wine_if (i == 17 || i == 19) ok(metrics[i].width == 0.0f, "%u: got width %f\n", i, metrics[i].width); ok(metrics[i].isWhitespace == 1, "%u: got %d\n", i, metrics[i].isWhitespace); ok(metrics[i].canWrapLineAfter == 1, "%u: got %d\n", i, metrics[i].canWrapLineAfter);

1 0

Frédéric Delanoy : ieframe/tests: Use todo_wine_if() in tests.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: 0d6ed1b6679d32849b8cbb7e219ea7d63bf2cfbb URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d6ed1b6679d32849b8cbb7e2… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Feb 17 21:13:30 2016 +0100 ieframe/tests: Use todo_wine_if() in tests. Signed-off-by: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ieframe/tests/intshcut.c | 5 +---- dlls/ieframe/tests/webbrowser.c | 9 ++------- 2 files changed, 3 insertions(+), 11 deletions(-) diff --git a/dlls/ieframe/tests/intshcut.c b/dlls/ieframe/tests/intshcut.c index 2a5f2df..0959a61 100644 --- a/dlls/ieframe/tests/intshcut.c +++ b/dlls/ieframe/tests/intshcut.c @@ -147,10 +147,7 @@ static void _check_string_transform(unsigned line, IUniformResourceLocatorA *url output = (void*)0xdeadbeef; hr = urlA->lpVtbl->GetURL(urlA, &output); if(expectedOutput) { - if(is_todo) { - todo_wine - ok_(__FILE__,line)(hr == S_OK, "GetUrl failed, hr=0x%x\n", hr); - }else { + todo_wine_if(is_todo) { ok_(__FILE__,line)(hr == S_OK, "GetUrl failed, hr=0x%x\n", hr); } todo_wine diff --git a/dlls/ieframe/tests/webbrowser.c b/dlls/ieframe/tests/webbrowser.c index 3201499..032faef 100644 --- a/dlls/ieframe/tests/webbrowser.c +++ b/dlls/ieframe/tests/webbrowser.c @@ -2806,10 +2806,7 @@ static void test_QueryStatusWB(IWebBrowser2 *webbrowser, BOOL has_document) if (use_container_olecmd) SET_EXPECT(QueryStatus_STOP); hres = IWebBrowser2_QueryStatusWB(webbrowser, OLECMDID_STOP, &status); ok(hres == success_state, "QueryStatusWB failed: %08x %08x\n", hres, success_state); - if (!use_container_olecmd && has_document) - todo_wine ok((has_document && status == success_flags) || (!has_document && status == 0xdeadbeef), - "OLECMDID_STOP not enabled/supported: %08x %08x\n", status, success_flags); - else + todo_wine_if (!use_container_olecmd && has_document) ok((has_document && status == success_flags) || (!has_document && status == 0xdeadbeef), "OLECMDID_STOP not enabled/supported: %08x %08x\n", status, success_flags); status = 0xdeadbeef; @@ -3312,9 +3309,7 @@ static void test_UIActivate(IWebBrowser2 *unk, BOOL activate) } hres = IOleDocumentView_UIActivate(docview, activate); - if(activate) - todo_wine ok(hres == S_OK, "Got 0x%08x\n", hres); - else + todo_wine_if(activate) ok(hres == S_OK, "Got 0x%08x\n", hres); if(activate) {

1 0

Frédéric Delanoy : mshtml/tests: Use todo_wine_if() in tests.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: 7023b20b6cebd2289a1c8d6fb0bb904f2af53c74 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7023b20b6cebd2289a1c8d6fb… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Feb 17 21:55:03 2016 +0100 mshtml/tests: Use todo_wine_if() in tests. Signed-off-by: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/dom.c | 11 ++--------- dlls/mshtml/tests/htmldoc.c | 24 ++++++------------------ dlls/mshtml/tests/xmlhttprequest.c | 4 +--- 3 files changed, 9 insertions(+), 30 deletions(-) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 7b5b395..4b2b825 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -7603,12 +7603,7 @@ static void _test_stylesheet_csstext(unsigned line, IHTMLStyleSheet *stylesheet, hres = IHTMLStyleSheet_get_cssText(stylesheet, &str); ok_(__FILE__,line)(hres == S_OK, "get_cssText failed: %08x\n", hres); - if(!is_todo) { - if(exstr) - ok_(__FILE__,line)(is_prefix_wa(str, exstr), "cssText = %s\n", wine_dbgstr_w(str)); - else - ok_(__FILE__,line)(!str, "cssText = %s\n", wine_dbgstr_w(str)); - }else todo_wine { + todo_wine_if(is_todo) { if(exstr) ok_(__FILE__,line)(is_prefix_wa(str, exstr), "cssText = %s\n", wine_dbgstr_w(str)); else @@ -7625,10 +7620,8 @@ static void _set_stylesheet_csstext(unsigned line, IHTMLStyleSheet *stylesheet, HRESULT hres; hres = IHTMLStyleSheet_put_cssText(stylesheet, str); - if(!is_todo) + todo_wine_if(is_todo) ok_(__FILE__,line)(hres == S_OK, "put_cssText failed: %08x\n", hres); - else - todo_wine ok_(__FILE__,line)(hres == S_OK, "put_cssText failed: %08x\n", hres); SysFreeString(str); } diff --git a/dlls/mshtml/tests/htmldoc.c b/dlls/mshtml/tests/htmldoc.c index 87f17d6..71519ed 100644 --- a/dlls/mshtml/tests/htmldoc.c +++ b/dlls/mshtml/tests/htmldoc.c @@ -427,9 +427,7 @@ static void _test_GetCurMoniker(unsigned line, IUnknown *unk, IMoniker *exmon, c hres = IMoniker_GetDisplayName(mon, NULL, NULL, &url); ok(hres == S_OK, "GetDisplayName failed: %08x\n", hres); - if(is_todo) - todo_wine ok_(__FILE__,line)(!strcmp_wa(url, exurl), "unexpected url %s\n", wine_dbgstr_w(url)); - else + todo_wine_if(is_todo) ok_(__FILE__,line)(!strcmp_wa(url, exurl), "unexpected url %s\n", wine_dbgstr_w(url)); if(!*ptr) ok_(__FILE__,line)(!lstrcmpW(url, doc_url), "url %s != doc_url %s\n", wine_dbgstr_w(url), wine_dbgstr_w(doc_url)); @@ -2613,10 +2611,8 @@ static HRESULT WINAPI DocHostUIHandler_TranslateUrl(IDocHostUIHandler2 *iface, D CHECK_EXPECT(TranslateUrl); ok(iface == expect_uihandler_iface, "called on unexpected iface\n"); ok(!dwTranslate, "dwTranslate = %x\n", dwTranslate); - if(!loading_hash) + todo_wine_if(loading_hash) ok(!strcmp_wa(pchURLIn, nav_serv_url), "pchURLIn = %s, expected %s\n", wine_dbgstr_w(pchURLIn), nav_serv_url); - else - todo_wine ok(!strcmp_wa(pchURLIn, nav_serv_url), "pchURLIn = %s, expected %s\n", wine_dbgstr_w(pchURLIn), nav_serv_url); ok(ppchURLOut != NULL, "ppchURLOut == NULL\n"); ok(!*ppchURLOut, "*ppchURLOut = %p\n", *ppchURLOut); @@ -5793,9 +5789,7 @@ static void test_download(DWORD flags) if(flags & DWL_HTTP) SET_CALLED(Exec_SETPROGRESSMAX); if(flags & DWL_EX_GETHOSTINFO) { - if(nav_url) - todo_wine CHECK_CALLED(GetHostInfo); - else + todo_wine_if(nav_url) CHECK_CALLED(GetHostInfo); } CHECK_CALLED(SetStatusText); @@ -5818,10 +5812,8 @@ static void test_download(DWORD flags) if(flags & DWL_ONREADY_LOADING) CHECK_CALLED(Invoke_OnReadyStateChange_Loading); if(!(flags & (DWL_EMPTY|DWL_JAVASCRIPT))) { - if(!is_extern) + todo_wine_if(is_extern) CHECK_CALLED(Invoke_OnReadyStateChange_Interactive); - else - todo_wine CHECK_CALLED(Invoke_OnReadyStateChange_Interactive); } if(!is_js && !is_extern) CHECK_CALLED(Invoke_OnReadyStateChange_Complete); @@ -5853,10 +5845,8 @@ static void test_download(DWORD flags) todo_wine CHECK_CALLED(Exec_SETPROGRESSPOS); } if(!(flags & DWL_EMPTY)) { - if(!is_extern) + todo_wine_if(is_extern) CHECK_CALLED(Exec_SETDOWNLOADSTATE_0); - else - todo_wine CHECK_CALLED(Exec_SETDOWNLOADSTATE_0); } CLEAR_CALLED(Exec_ShellDocView_103); CLEAR_CALLED(Exec_ShellDocView_105); @@ -5883,10 +5873,8 @@ static void test_download(DWORD flags) if(!is_js && !is_extern) { if(!editmode && !(flags & DWL_REFRESH)) { if(!(flags & DWL_EMPTY)) { - if(support_wbapp) + todo_wine_if(!support_wbapp) CHECK_CALLED(FireNavigateComplete2); - else - todo_wine CHECK_CALLED(FireNavigateComplete2); } CHECK_CALLED(FireDocumentComplete); } diff --git a/dlls/mshtml/tests/xmlhttprequest.c b/dlls/mshtml/tests/xmlhttprequest.c index dd8e797..5d11ec0 100644 --- a/dlls/mshtml/tests/xmlhttprequest.c +++ b/dlls/mshtml/tests/xmlhttprequest.c @@ -850,10 +850,8 @@ static void test_async_xhr(IHTMLDocument2 *doc, const char *xml_url, const char CHECK_CALLED(xmlhttprequest_onreadystatechange_loading); CHECK_CALLED(xmlhttprequest_onreadystatechange_done); /* Workaround for loading large files */ - if(expect_text) + todo_wine_if(!expect_text) ok(loading_cnt == 1, "loading_cnt = %d\n", loading_cnt); - else - todo_wine ok(loading_cnt == 1, "loading_cnt = %d\n", loading_cnt); if(FAILED(hres)) { IHTMLXMLHttpRequest_Release(xhr);

1 0

Alexandre Julliard : libwine: Move the digit folding tables to libwine_port.
by Alexandre Julliard 18 Feb '16

18 Feb '16

Module: wine Branch: master Commit: d59d45e457a714e2dc596b7269d1afaca3fba50f URL: http://source.winehq.org/git/wine.git/?a=commit;h=d59d45e457a714e2dc596b726… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 18 14:02:10 2016 +0900 libwine: Move the digit folding tables to libwine_port. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/port/Makefile.in | 1 + libs/port/digitmap.c | 985 ++++++++++++++++++++++++++++++++++++++++++++++++++ libs/wine/casemap.c | 982 +------------------------------------------------ tools/make_unicode | 33 +- 4 files changed, 1016 insertions(+), 985 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d59d45e457a714e2dc596…

1 0