wine-commits March 2014

wine-commits@winehq.org

2 participants
530 discussions

Nikolay Sivov : stdole2: Fix usage of 'optional' with 'defaultvalue'.
by Alexandre Julliard 19 Mar '14

19 Mar '14

Module: wine Branch: master Commit: 62ee13c59431551abc0186af4145f33f66d62517 URL: http://source.winehq.org/git/wine.git/?a=commit;h=62ee13c59431551abc0186af4… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Mar 19 10:20:51 2014 +0400 stdole2: Fix usage of 'optional' with 'defaultvalue'. --- include/stdole2.idl | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/stdole2.idl b/include/stdole2.idl index 432314f..81b5463 100644 --- a/include/stdole2.idl +++ b/include/stdole2.idl @@ -405,9 +405,9 @@ library stdole helpcontext(0x2775) ] HRESULT LoadPicture([in, optional] VARIANT filename, - [in, optional, defaultvalue(0)] int widthDesired, - [in, optional, defaultvalue(0)] int heightDesired, - [in, optional, defaultvalue(Default)] enum LoadPictureConstants flags, + [in, defaultvalue(0)] int widthDesired, + [in, defaultvalue(0)] int heightDesired, + [in, defaultvalue(Default)] enum LoadPictureConstants flags, [out, retval] IPictureDisp **retval); [ entry("OleSavePictureFile"),

1 0

Austin English : kernel32: Improve GetNumaHighestNodeNumber stub.
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: 77bd74fa2515e8babb6cda2653a12adad65191dc URL: http://source.winehq.org/git/wine.git/?a=commit;h=77bd74fa2515e8babb6cda265… Author: Austin English <austinenglish(a)gmail.com> Date: Tue Mar 18 10:47:09 2014 -0700 kernel32: Improve GetNumaHighestNodeNumber stub. --- dlls/kernel32/process.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 60156aa..3ff47e1 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -3911,9 +3911,9 @@ HRESULT WINAPI RegisterApplicationRecoveryCallback(APPLICATION_RECOVERY_CALLBACK */ BOOL WINAPI GetNumaHighestNodeNumber(PULONG highestnode) { - FIXME("(%p): stub\n", highestnode); - SetLastError(ERROR_CALL_NOT_IMPLEMENTED); - return FALSE; + *highestnode = 0; + FIXME("(%p): semi-stub\n", highestnode); + return TRUE; } /**********************************************************************

1 0

Jacek Caban : vbscript: Allow creating RegExp object by new expression.
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: f123556f0701f5c1c36e24d4de74886e2deeb0a8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f123556f0701f5c1c36e24d4d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Mar 18 15:52:13 2014 +0100 vbscript: Allow creating RegExp object by new expression. --- dlls/vbscript/interp.c | 11 +++++++++++ dlls/vbscript/tests/lang.vbs | 12 ++++++++++++ dlls/vbscript/tests/regexp.vbs | 6 ++++++ dlls/vbscript/vbregexp.c | 36 ++++++++++++++++++++++++------------ dlls/vbscript/vbscript.h | 2 ++ 5 files changed, 55 insertions(+), 12 deletions(-) diff --git a/dlls/vbscript/interp.c b/dlls/vbscript/interp.c index 066d675..bc3e940 100644 --- a/dlls/vbscript/interp.c +++ b/dlls/vbscript/interp.c @@ -975,8 +975,19 @@ static HRESULT interp_new(exec_ctx_t *ctx) VARIANT v; HRESULT hres; + static const WCHAR regexpW[] = {'r','e','g','e','x','p',0}; + TRACE("%s\n", debugstr_w(arg)); + if(!strcmpiW(arg, regexpW)) { + V_VT(&v) = VT_DISPATCH; + hres = create_regexp(&V_DISPATCH(&v)); + if(FAILED(hres)) + return hres; + + return stack_push(ctx, &v); + } + for(class_desc = ctx->script->classes; class_desc; class_desc = class_desc->next) { if(!strcmpiW(class_desc->name, arg)) break; diff --git a/dlls/vbscript/tests/lang.vbs b/dlls/vbscript/tests/lang.vbs index 612c615..d3e8318 100644 --- a/dlls/vbscript/tests/lang.vbs +++ b/dlls/vbscript/tests/lang.vbs @@ -1132,4 +1132,16 @@ Call testarrarg(1, "VT_I2*") Call testarrarg(false, "VT_BOOL*") Call testarrarg(Empty, "VT_EMPTY*") +' It's allowed to declare non-builtin RegExp class... +class RegExp + public property get Global() + Call ok(false, "Global called") + Global = "fail" + end property +end class + +' ...but there is no way to use it because builtin instance is always created +set x = new RegExp +Call ok(x.Global = false, "x.Global = " & x.Global) + reportSuccess() diff --git a/dlls/vbscript/tests/regexp.vbs b/dlls/vbscript/tests/regexp.vbs index 72fa108..0354fb7 100644 --- a/dlls/vbscript/tests/regexp.vbs +++ b/dlls/vbscript/tests/regexp.vbs @@ -168,4 +168,10 @@ Call ok(submatch.Count = 2, "submatch.Count = " & submatch.Count) Call ok(submatch.Item(0) = "a", "submatch.Item(0) = " & submatch.Item(0)) Call ok(submatch.Item(1) = "b", "submatch.Item(0) = " & submatch.Item(1)) +Set x = new regexp +Call ok(x.Pattern = "", "RegExp.Pattern = " & x.Pattern) +Call ok(x.IgnoreCase = false, "RegExp.IgnoreCase = " & x.IgnoreCase) +Call ok(x.Global = false, "RegExp.Global = " & x.Global) +Call ok(x.Multiline = false, "RegExp.Multiline = " & x.Multiline) + Call reportSuccess() diff --git a/dlls/vbscript/vbregexp.c b/dlls/vbscript/vbregexp.c index ada364f..0a399b9 100644 --- a/dlls/vbscript/vbregexp.c +++ b/dlls/vbscript/vbregexp.c @@ -1599,29 +1599,41 @@ static IRegExpVtbl RegExpVtbl = { RegExp_Replace }; -HRESULT WINAPI VBScriptRegExpFactory_CreateInstance(IClassFactory *iface, IUnknown *pUnkOuter, REFIID riid, void **ppv) +HRESULT create_regexp(IDispatch **ret) { - RegExp2 *ret; + RegExp2 *regexp; HRESULT hres; - TRACE("(%p %s %p)\n", pUnkOuter, debugstr_guid(riid), ppv); - hres = init_regexp_typeinfo(RegExp2_tid); if(FAILED(hres)) return hres; - ret = heap_alloc_zero(sizeof(*ret)); - if(!ret) + regexp = heap_alloc_zero(sizeof(*regexp)); + if(!regexp) return E_OUTOFMEMORY; - ret->IRegExp2_iface.lpVtbl = &RegExp2Vtbl; - ret->IRegExp_iface.lpVtbl = &RegExpVtbl; + regexp->IRegExp2_iface.lpVtbl = &RegExp2Vtbl; + regexp->IRegExp_iface.lpVtbl = &RegExpVtbl; + regexp->ref = 1; + heap_pool_init(&regexp->pool); - ret->ref = 1; - heap_pool_init(&ret->pool); + *ret = (IDispatch*)&regexp->IRegExp2_iface; + return S_OK; +} + +HRESULT WINAPI VBScriptRegExpFactory_CreateInstance(IClassFactory *iface, IUnknown *pUnkOuter, REFIID riid, void **ppv) +{ + IDispatch *regexp; + HRESULT hres; + + TRACE("(%p %s %p)\n", pUnkOuter, debugstr_guid(riid), ppv); + + hres = create_regexp(&regexp); + if(FAILED(hres)) + return hres; - hres = IRegExp2_QueryInterface(&ret->IRegExp2_iface, riid, ppv); - IRegExp2_Release(&ret->IRegExp2_iface); + hres = IDispatch_QueryInterface(regexp, riid, ppv); + IDispatch_Release(regexp); return hres; } diff --git a/dlls/vbscript/vbscript.h b/dlls/vbscript/vbscript.h index 94139bb..bf7a17c 100644 --- a/dlls/vbscript/vbscript.h +++ b/dlls/vbscript/vbscript.h @@ -382,6 +382,8 @@ static inline BOOL is_int32(double d) return INT32_MIN <= d && d <= INT32_MAX && (double)(int)d == d; } +HRESULT create_regexp(IDispatch**) DECLSPEC_HIDDEN; + HRESULT WINAPI VBScriptFactory_CreateInstance(IClassFactory*,IUnknown*,REFIID,void**) DECLSPEC_HIDDEN; HRESULT WINAPI VBScriptRegExpFactory_CreateInstance(IClassFactory*,IUnknown*,REFIID,void**) DECLSPEC_HIDDEN;

1 0

Jacek Caban : urlmon: Fixed sec_mgr.c tests failing on some recent Windows.
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: d7a4f0eb1f6d27dc64cd26d26428c459b133c431 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d7a4f0eb1f6d27dc64cd26d26… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Mar 18 15:51:59 2014 +0100 urlmon: Fixed sec_mgr.c tests failing on some recent Windows. --- dlls/urlmon/tests/sec_mgr.c | 36 +++++++++++++++++++++++++++++++----- 1 file changed, 31 insertions(+), 5 deletions(-) diff --git a/dlls/urlmon/tests/sec_mgr.c b/dlls/urlmon/tests/sec_mgr.c index f952780..91e1a3f 100644 --- a/dlls/urlmon/tests/sec_mgr.c +++ b/dlls/urlmon/tests/sec_mgr.c @@ -65,6 +65,11 @@ expect_ ## func = called_ ## func = FALSE; \ }while(0) +#define SET_CALLED(func) \ + do { \ + expect_ ## func = called_ ## func = FALSE; \ + }while(0) + DEFINE_EXPECT(ParseUrl_SECURITY_URL_input); DEFINE_EXPECT(ParseUrl_SECURITY_URL_input2); DEFINE_EXPECT(ParseUrl_SECURITY_URL_expected); @@ -509,15 +514,26 @@ static void test_url_action(IInternetSecurityManager *secmgr, IInternetZoneManag */ res = RegOpenKeyA(HKEY_CURRENT_USER, "Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3", &hkey); - if(res != ERROR_SUCCESS) { - ok(0, "Could not open zone key\n"); + ok(res == ERROR_SUCCESS, "Could not open zone key\n"); + if(res != ERROR_SUCCESS) return; - } wsprintfA(buf, "%X", action); size = sizeof(DWORD); res = RegQueryValueExA(hkey, buf, NULL, NULL, (BYTE*)&reg_policy, &size); RegCloseKey(hkey); + + /* Try settings from HKEY_LOCAL_MACHINE. */ + if(res != ERROR_SUCCESS || size != sizeof(DWORD)) { + res = RegOpenKeyA(HKEY_LOCAL_MACHINE, + "Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3", &hkey); + ok(res == ERROR_SUCCESS, "Could not open zone key\n"); + + size = sizeof(DWORD); + res = RegQueryValueExA(hkey, buf, NULL, NULL, (BYTE*)&reg_policy, &size); + RegCloseKey(hkey); + } + if(res != ERROR_SUCCESS || size != sizeof(DWORD)) { policy = 0xdeadbeef; hres = IInternetSecurityManager_ProcessUrlAction(secmgr, url9, action, (BYTE*)&policy, @@ -1213,10 +1229,10 @@ static void test_GetZoneAttributes(void) ok((pZA->cbSize == 0xffffffff) || (pZA->cbSize == sizeof(ZONEATTRIBUTES)), "got cbSize = 0x%x (expected 0xffffffff)\n", pZA->cbSize); - /* IE8 no longer fail on invalid zones */ + /* IE8 up to IE10 don't fail on invalid zones */ memset(buffer, -1, sizeof(buffer)); hr = IInternetZoneManager_GetZoneAttributes(zonemgr, 0xdeadbeef, pZA); - ok(hr == S_OK || (hr == E_FAIL), + ok(hr == S_OK || hr == E_FAIL || hr == E_POINTER, "got 0x%x (expected S_OK or E_FAIL)\n", hr); hr = IInternetZoneManager_GetZoneAttributes(zonemgr, 0, NULL); @@ -1958,6 +1974,16 @@ static void test_CoInternetIsFeatureZoneElevationEnabled(void) testcases[i].policy_flags = URLPOLICY_ALLOW; } + /* IE10 does not seem to use passed ISecurityManager */ + SET_EXPECT(ProcessUrlAction); + pCoInternetIsFeatureZoneElevationEnabled(url1, url1, &security_manager, 0); + i = called_ProcessUrlAction; + SET_CALLED(ProcessUrlAction); + if(!i) { + skip("CoInternetIsFeatureZoneElevationEnabled does not use passed ISecurityManager\n"); + return; + } + for(i=0; i<sizeof(testcases)/sizeof(testcases[0]); i++) { url_from = a2w(testcases[i].url_from); url_to = a2w(testcases[i].url_to);

1 0

Piotr Caban : msvcrt: Fix p__p___mb_cur_max tests.
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: c4f935d449c316f0f34665d739b97212f6de4fc8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c4f935d449c316f0f34665d73… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Mar 18 12:56:43 2014 +0100 msvcrt: Fix p__p___mb_cur_max tests. --- dlls/msvcrt/tests/locale.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/dlls/msvcrt/tests/locale.c b/dlls/msvcrt/tests/locale.c index d8a1876..6ef07ec 100644 --- a/dlls/msvcrt/tests/locale.c +++ b/dlls/msvcrt/tests/locale.c @@ -785,10 +785,8 @@ static void test__Gettnames(void) static void test___mb_cur_max_func(void) { int mb_cur_max; - CPINFO cp; setlocale(LC_ALL, "C"); - GetCPInfo(CP_ACP, &cp); /* for newer Windows */ if(!p___mb_cur_max_func) @@ -812,13 +810,7 @@ static void test___mb_cur_max_func(void) win_skip("Skipping __p___mb_cur_max tests\n"); else { mb_cur_max = *p__p___mb_cur_max(); - if (cp.MaxCharSize != 1) { - todo_wine ok(mb_cur_max == cp.MaxCharSize, "mb_cur_max = %d, expected %d\n", - mb_cur_max, cp.MaxCharSize); - } - else { - ok(mb_cur_max == 1, "mb_cur_max = %d, expected 1\n", mb_cur_max); - } + ok(mb_cur_max == 1, "mb_cur_max = %d, expected 1\n", mb_cur_max); /* some old Windows don't set chinese */ if (!setlocale(LC_ALL, "chinese"))

1 0

Frédéric Delanoy : user32/tests: Use BOOL type where appropriate.
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: 9b1a2d8f4fc46c3ac7f832a9ec83f696447f3b04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9b1a2d8f4fc46c3ac7f832a9e… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Tue Mar 18 08:33:31 2014 +0100 user32/tests: Use BOOL type where appropriate. --- dlls/user32/tests/win.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/user32/tests/win.c b/dlls/user32/tests/win.c index d693172..69f3270 100644 --- a/dlls/user32/tests/win.c +++ b/dlls/user32/tests/win.c @@ -5899,7 +5899,7 @@ static void run_NCRedrawLoop(UINT flags) ShowWindow(hwnd, SW_SHOW); UpdateWindow(hwnd); flush_events( FALSE ); - while(PeekMessageA(&msg, hwnd, 0, 0, PM_REMOVE) != 0) + while (PeekMessageA(&msg, hwnd, 0, 0, PM_REMOVE)) { if (msg.message == WM_PAINT) loopcount++; if (loopcount >= 100) break; @@ -7063,7 +7063,7 @@ static void create_window_tree(HWND parent, HWND *window, int size) struct window_attributes { char class_name[128]; - int is_visible, is_enabled, is_groupbox, is_httransparent, is_extransparent; + BOOL is_visible, is_enabled, is_groupbox, is_httransparent, is_extransparent; }; static void get_window_attributes(HWND hwnd, struct window_attributes *attrs)

1 0

Jacek Caban : mshtml: Fixed tests on old IEs and improved results on IE11.
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: a6908473b68a8946a5c7dd0b0cb94193542febc2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a6908473b68a8946a5c7dd0b0… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Mar 18 11:41:56 2014 +0100 mshtml: Fixed tests on old IEs and improved results on IE11. --- dlls/mshtml/tests/htmldoc.c | 27 +++++++++++++++++++-------- 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/dlls/mshtml/tests/htmldoc.c b/dlls/mshtml/tests/htmldoc.c index ed72685..313fb78 100644 --- a/dlls/mshtml/tests/htmldoc.c +++ b/dlls/mshtml/tests/htmldoc.c @@ -1211,7 +1211,8 @@ static HRESULT WINAPI Binding_QueryInterface(IBinding *iface, REFIID riid, void return E_NOINTERFACE; } - ok(0, "unexpected call %s\n", wine_dbgstr_guid(riid)); + trace("Binding::QI(%s)\n", wine_dbgstr_guid(riid)); + *ppv = NULL; return E_NOINTERFACE; } @@ -2666,13 +2667,15 @@ static HRESULT WINAPI CustomDocHostUIHandler_QueryInterface(IDocHostUIHandler2 * return S_OK; } + *ppv = NULL; + if(IsEqualGUID(&IID_IOleCommandTarget, riid)) return E_NOINTERFACE; - else if(IsEqualGUID(&IID_IDocHostShowUI, riid)) + if(IsEqualGUID(&IID_IDocHostShowUI, riid)) return E_NOINTERFACE; /* TODO */ - ok(0, "unexpected call %s\n", wine_dbgstr_guid(riid)); + trace("CustomDocHostUIHandler::QI(%s)\n", wine_dbgstr_guid(riid)); return E_NOINTERFACE; } @@ -2971,11 +2974,15 @@ static HRESULT WINAPI OleCommandTarget_Exec(IOleCommandTarget *iface, const GUID return E_NOTIMPL; + case 83: + case 102: case 134: /* TODO */ + case 135: case 136: /* TODO */ case 139: /* TODO */ case 143: /* TODO */ case 144: /* TODO */ + case 178: return E_NOTIMPL; default: @@ -3036,7 +3043,7 @@ static HRESULT WINAPI OleCommandTarget_Exec(IOleCommandTarget *iface, const GUID ok(ind == 0, "Lower bound = %d\n", ind); hres = SafeArrayGetUBound(sa, 1, &ind); ok(hres == S_OK, "SafeArrayGetUBound failed: %x\n", hres); - ok(ind == 7 || broken(ind == 5), "Upper bound = %d\n", ind); + ok(ind == 7 || ind == 8 /* IE11 */ ||broken(ind == 5), "Upper bound = %d\n", ind); ind = 0; SafeArrayGetElement(sa, &ind, &var); @@ -4869,7 +4876,7 @@ static HRESULT WINAPI WebBrowser_put_TheaterMode(IWebBrowser2 *iface, VARIANT_BO static HRESULT WINAPI WebBrowser_get_AddressBar(IWebBrowser2 *iface, VARIANT_BOOL *Value) { - ok(0, "unexpected call\n"); + trace("get_AddressBar: ignoring\n"); /* Some old IEs call it */ return E_NOTIMPL; } @@ -5242,7 +5249,7 @@ static HRESULT QueryInterface(REFIID riid, void **ppv) else if(IsEqualGUID(&IID_IDocHostUIHandlerPriv, riid)) return E_NOINTERFACE; /* ? */ else - ok(0, "unexpected riid %s\n", wine_dbgstr_guid(riid)); + trace("QI(%s)\n", wine_dbgstr_guid(riid)); if(*ppv) return S_OK; @@ -5804,8 +5811,12 @@ static void test_download(DWORD flags) CLEAR_CALLED(UpdateUI); CLEAR_CALLED(Exec_UPDATECOMMANDS); CLEAR_CALLED(Exec_SETTITLE); - if(flags & DWL_EXPECT_HISTUPDATE) - CHECK_CALLED(Exec_Explorer_38); + if(flags & DWL_EXPECT_HISTUPDATE) { + if(flags & DWL_FROM_HISTORY) + CHECK_CALLED_BROKEN(Exec_Explorer_38); /* Some old IEs don't call it. */ + else + CHECK_CALLED(Exec_Explorer_38); + } todo_wine CHECK_CALLED_BROKEN(UpdateBackForwardState); } if(!is_js && !is_extern) {

1 0

Dmitry Timoshkov : schedsvc: Replace external trace statements by internal versions.
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: 0e802b9c92b83f578f4901c53889d5a30f316957 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0e802b9c92b83f578f4901c53… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Tue Mar 18 17:42:52 2014 +0900 schedsvc: Replace external trace statements by internal versions. --- dlls/schedsvc/schedsvc.c | 48 +++++++++++++++++++++++----------------------- dlls/schedsvc/svc_main.c | 8 ++++---- 2 files changed, 28 insertions(+), 28 deletions(-) diff --git a/dlls/schedsvc/schedsvc.c b/dlls/schedsvc/schedsvc.c index 6ce8064..988430d 100644 --- a/dlls/schedsvc/schedsvc.c +++ b/dlls/schedsvc/schedsvc.c @@ -31,7 +31,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(schedsvc); HRESULT __cdecl SchRpcHighestVersion(DWORD *version) { - WINE_TRACE("%p\n", version); + TRACE("%p\n", version); *version = MAKELONG(3, 1); return S_OK; @@ -170,8 +170,8 @@ HRESULT __cdecl SchRpcRegisterTask(const WCHAR *path, const WCHAR *xml, DWORD fl DWORD disposition; HRESULT hr; - WINE_TRACE("%s,%s,%#x,%s,%u,%u,%p,%p,%p\n", wine_dbgstr_w(path), wine_dbgstr_w(xml), flags, - wine_dbgstr_w(sddl), task_logon_type, n_creds, creds, actual_path, xml_error_info); + TRACE("%s,%s,%#x,%s,%u,%u,%p,%p,%p\n", debugstr_w(path), debugstr_w(xml), flags, + debugstr_w(sddl), task_logon_type, n_creds, creds, actual_path, xml_error_info); *actual_path = NULL; *xml_error_info = NULL; @@ -282,7 +282,7 @@ HRESULT __cdecl SchRpcRetrieveTask(const WCHAR *path, const WCHAR *languages, UL WCHAR *full_name; HRESULT hr; - WINE_TRACE("%s,%s,%p,%p\n", wine_dbgstr_w(path), wine_dbgstr_w(languages), n_languages, xml); + TRACE("%s,%s,%p,%p\n", debugstr_w(path), debugstr_w(languages), n_languages, xml); full_name = get_full_name(path, NULL); if (!full_name) return E_OUTOFMEMORY; @@ -299,7 +299,7 @@ HRESULT __cdecl SchRpcCreateFolder(const WCHAR *path, const WCHAR *sddl, DWORD f WCHAR *full_name; HRESULT hr; - WINE_TRACE("%s,%s,%#x\n", wine_dbgstr_w(path), wine_dbgstr_w(sddl), flags); + TRACE("%s,%s,%#x\n", debugstr_w(path), debugstr_w(sddl), flags); if (flags) return E_INVALIDARG; @@ -314,61 +314,61 @@ HRESULT __cdecl SchRpcCreateFolder(const WCHAR *path, const WCHAR *sddl, DWORD f HRESULT __cdecl SchRpcSetSecurity(const WCHAR *path, const WCHAR *sddl, DWORD flags) { - WINE_FIXME("%s,%s,%#x: stub\n", wine_dbgstr_w(path), wine_dbgstr_w(sddl), flags); + FIXME("%s,%s,%#x: stub\n", debugstr_w(path), debugstr_w(sddl), flags); return E_NOTIMPL; } HRESULT __cdecl SchRpcGetSecurity(const WCHAR *path, DWORD flags, WCHAR **sddl) { - WINE_FIXME("%s,%#x,%p: stub\n", wine_dbgstr_w(path), flags, sddl); + FIXME("%s,%#x,%p: stub\n", debugstr_w(path), flags, sddl); return E_NOTIMPL; } HRESULT __cdecl SchRpcEnumFolders(const WCHAR *path, DWORD flags, DWORD *start_index, DWORD n_requested, DWORD *n_names, TASK_NAMES *names) { - WINE_FIXME("%s,%#x,%p,%u,%p,%p: stub\n", wine_dbgstr_w(path), flags, start_index, n_requested, n_names, names); + FIXME("%s,%#x,%p,%u,%p,%p: stub\n", debugstr_w(path), flags, start_index, n_requested, n_names, names); return E_NOTIMPL; } HRESULT __cdecl SchRpcEnumTasks(const WCHAR *path, DWORD flags, DWORD *start_index, DWORD n_requested, DWORD *n_names, TASK_NAMES *names) { - WINE_FIXME("%s,%#x,%p,%u,%p,%p: stub\n", wine_dbgstr_w(path), flags, start_index, n_requested, n_names, names); + FIXME("%s,%#x,%p,%u,%p,%p: stub\n", debugstr_w(path), flags, start_index, n_requested, n_names, names); return E_NOTIMPL; } HRESULT __cdecl SchRpcEnumInstances(const WCHAR *path, DWORD flags, DWORD *n_guids, GUID **guids) { - WINE_FIXME("%s,%#x,%p,%p: stub\n", wine_dbgstr_w(path), flags, n_guids, guids); + FIXME("%s,%#x,%p,%p: stub\n", debugstr_w(path), flags, n_guids, guids); return E_NOTIMPL; } HRESULT __cdecl SchRpcGetInstanceInfo(GUID guid, WCHAR **path, DWORD *task_state, WCHAR **action, WCHAR **info, DWORD *n_instances, GUID **instances, DWORD *pid) { - WINE_FIXME("%s,%p,%p,%p,%p,%p,%p,%p: stub\n", wine_dbgstr_guid(&guid), path, task_state, action, - info, n_instances, instances, pid); + FIXME("%s,%p,%p,%p,%p,%p,%p,%p: stub\n", wine_dbgstr_guid(&guid), path, task_state, action, + info, n_instances, instances, pid); return E_NOTIMPL; } HRESULT __cdecl SchRpcStopInstance(GUID guid, DWORD flags) { - WINE_FIXME("%s,%#x: stub\n", wine_dbgstr_guid(&guid), flags); + FIXME("%s,%#x: stub\n", wine_dbgstr_guid(&guid), flags); return E_NOTIMPL; } HRESULT __cdecl SchRpcStop(const WCHAR *path, DWORD flags) { - WINE_FIXME("%s,%#x: stub\n", wine_dbgstr_w(path), flags); + FIXME("%s,%#x: stub\n", debugstr_w(path), flags); return E_NOTIMPL; } HRESULT __cdecl SchRpcRun(const WCHAR *path, DWORD n_args, const WCHAR **args, DWORD flags, DWORD session_id, const WCHAR *user, GUID *guid) { - WINE_FIXME("%s,%u,%p,%#x,%#x,%s,%p: stub\n", wine_dbgstr_w(path), n_args, args, flags, - session_id, wine_dbgstr_w(user), guid); + FIXME("%s,%u,%p,%#x,%#x,%s,%p: stub\n", debugstr_w(path), n_args, args, flags, + session_id, debugstr_w(user), guid); return E_NOTIMPL; } @@ -377,7 +377,7 @@ HRESULT __cdecl SchRpcDelete(const WCHAR *path, DWORD flags) WCHAR *full_name; HRESULT hr = S_OK; - WINE_TRACE("%s,%#x\n", wine_dbgstr_w(path), flags); + TRACE("%s,%#x\n", debugstr_w(path), flags); if (flags) return E_INVALIDARG; @@ -400,38 +400,38 @@ HRESULT __cdecl SchRpcDelete(const WCHAR *path, DWORD flags) HRESULT __cdecl SchRpcRename(const WCHAR *path, const WCHAR *name, DWORD flags) { - WINE_FIXME("%s,%s,%#x: stub\n", wine_dbgstr_w(path), wine_dbgstr_w(name), flags); + FIXME("%s,%s,%#x: stub\n", debugstr_w(path), debugstr_w(name), flags); return E_NOTIMPL; } HRESULT __cdecl SchRpcScheduledRuntimes(const WCHAR *path, SYSTEMTIME *start, SYSTEMTIME *end, DWORD flags, DWORD n_requested, DWORD *n_runtimes, SYSTEMTIME **runtimes) { - WINE_FIXME("%s,%p,%p,%#x,%u,%p,%p: stub\n", wine_dbgstr_w(path), start, end, flags, - n_requested, n_runtimes, runtimes); + FIXME("%s,%p,%p,%#x,%u,%p,%p: stub\n", debugstr_w(path), start, end, flags, + n_requested, n_runtimes, runtimes); return E_NOTIMPL; } HRESULT __cdecl SchRpcGetLastRunInfo(const WCHAR *path, SYSTEMTIME *last_runtime, DWORD *last_return_code) { - WINE_FIXME("%s,%p,%p: stub\n", wine_dbgstr_w(path), last_runtime, last_return_code); + FIXME("%s,%p,%p: stub\n", debugstr_w(path), last_runtime, last_return_code); return E_NOTIMPL; } HRESULT __cdecl SchRpcGetTaskInfo(const WCHAR *path, DWORD flags, DWORD *enabled, DWORD *task_state) { - WINE_FIXME("%s,%#x,%p,%p: stub\n", wine_dbgstr_w(path), flags, enabled, task_state); + FIXME("%s,%#x,%p,%p: stub\n", debugstr_w(path), flags, enabled, task_state); return E_NOTIMPL; } HRESULT __cdecl SchRpcGetNumberOfMissedRuns(const WCHAR *path, DWORD *runs) { - WINE_FIXME("%s,%p: stub\n", wine_dbgstr_w(path), runs); + FIXME("%s,%p: stub\n", debugstr_w(path), runs); return E_NOTIMPL; } HRESULT __cdecl SchRpcEnableTask(const WCHAR *path, DWORD enabled) { - WINE_FIXME("%s,%u: stub\n", wine_dbgstr_w(path), enabled); + FIXME("%s,%u: stub\n", debugstr_w(path), enabled); return E_NOTIMPL; } diff --git a/dlls/schedsvc/svc_main.c b/dlls/schedsvc/svc_main.c index 95872cd..8c451c9 100644 --- a/dlls/schedsvc/svc_main.c +++ b/dlls/schedsvc/svc_main.c @@ -101,7 +101,7 @@ static void schedsvc_update_status(DWORD state) static void WINAPI schedsvc_handler(DWORD control) { - WINE_TRACE("%#x\n", control); + TRACE("%#x\n", control); switch (control) { @@ -124,7 +124,7 @@ static RPC_STATUS RPC_init(void) WCHAR transport[] = SCHEDSVC_TRANSPORT; RPC_STATUS status; - WINE_TRACE("using %s\n", wine_dbgstr_w(transport)); + TRACE("using %s\n", debugstr_w(transport)); status = RpcServerUseProtseqEpW(transport, 0, NULL, NULL); if (status != RPC_S_OK) @@ -173,7 +173,7 @@ static void RPC_finish(void) void WINAPI ServiceMain(DWORD argc, LPWSTR *argv) { - WINE_TRACE("starting Task Scheduler Service\n"); + TRACE("starting Task Scheduler Service\n"); if (RPC_init() != RPC_S_OK) return; @@ -193,7 +193,7 @@ void WINAPI ServiceMain(DWORD argc, LPWSTR *argv) RPC_finish(); schedsvc_update_status(SERVICE_STOPPED); - WINE_TRACE("exiting Task Scheduler Service\n"); + TRACE("exiting Task Scheduler Service\n"); } void __RPC_FAR * __RPC_USER MIDL_user_allocate(SIZE_T len)

1 0

Henri Verbeet : d3d9/tests: Use a separate device for dp3_alpha_test().
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: ad81149953f58223a949f192157737a05cc2e006 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ad81149953f58223a949f1921… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Mar 18 09:37:55 2014 +0100 d3d9/tests: Use a separate device for dp3_alpha_test(). --- dlls/d3d9/tests/visual.c | 54 +++++++++++++++++++++++++++++++--------------- 1 file changed, 37 insertions(+), 17 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 44dedba..893d7b1 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -10853,23 +10853,42 @@ static void alphareplicate_test(IDirect3DDevice9 *device) { } -static void dp3_alpha_test(IDirect3DDevice9 *device) { - HRESULT hr; +static void dp3_alpha_test(void) +{ + IDirect3DDevice9 *device; + IDirect3D9 *d3d; + ULONG refcount; D3DCAPS9 caps; DWORD color; - struct vertex quad[] = { - { -1.0, -1.0, 0.1, 0x408080c0 }, - { 1.0, -1.0, 0.1, 0x408080c0 }, - { -1.0, 1.0, 0.1, 0x408080c0 }, - { 1.0, 1.0, 0.1, 0x408080c0 }, + HWND window; + HRESULT hr; + + static const struct vertex quad[] = + { + {-1.0f, -1.0f, 0.1f, 0x408080c0}, + {-1.0f, 1.0f, 0.1f, 0x408080c0}, + { 1.0f, -1.0f, 0.1f, 0x408080c0}, + { 1.0f, 1.0f, 0.1f, 0x408080c0}, }; + window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW | WS_VISIBLE, + 0, 0, 640, 480, NULL, NULL, NULL, NULL); + d3d = Direct3DCreate9(D3D_SDK_VERSION); + ok(!!d3d, "Failed to create a D3D object.\n"); + if (!(device = create_device(d3d, window, window, TRUE))) + { + skip("Failed to create a D3D device, skipping tests.\n"); + goto done; + } + memset(&caps, 0, sizeof(caps)); hr = IDirect3DDevice9_GetDeviceCaps(device, &caps); - ok(SUCCEEDED(hr), "GetDeviceCaps failed with 0x%08x\n", hr); - if (!(caps.TextureOpCaps & D3DTEXOPCAPS_DOTPRODUCT3)) { + ok(SUCCEEDED(hr), "Failed to get device caps, hr %#x.\n", hr); + if (!(caps.TextureOpCaps & D3DTEXOPCAPS_DOTPRODUCT3)) + { skip("D3DTOP_DOTPRODUCT3 not supported\n"); - return; + IDirect3DDevice9_Release(device); + goto done; } hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET | D3DCLEAR_ZBUFFER, 0x00000000, 1.0f, 0); @@ -10904,6 +10923,8 @@ static void dp3_alpha_test(IDirect3DDevice9 *device) { ok(hr == D3D_OK, "IDirect3DDevice9_SetTextureStageState failed with 0x%08x\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_TEXTUREFACTOR, 0xffffffff); ok(SUCCEEDED(hr), "IDirect3DDevice9_SetRenderState failed with 0x%08x\n", hr); + hr = IDirect3DDevice9_SetRenderState(device, D3DRS_LIGHTING, FALSE); + ok(SUCCEEDED(hr), "Failed to set render state, hr %#x.\n", hr); hr = IDirect3DDevice9_BeginScene(device); ok(hr == D3D_OK, "IDirect3DDevice9_BeginScene failed with 0x%08x\n", hr); @@ -10920,12 +10941,11 @@ static void dp3_alpha_test(IDirect3DDevice9 *device) { hr = IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL); ok(SUCCEEDED(hr), "IDirect3DDevice9_Present failed with 0x%08x\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_COLOROP, D3DTOP_DISABLE); - ok(hr == D3D_OK, "IDirect3DDevice9_SetTextureStageState failed with 0x%08x\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_ALPHAOP, D3DTOP_DISABLE); - ok(hr == D3D_OK, "IDirect3DDevice9_SetTextureStageState failed with 0x%08x\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_COLOROP, D3DTOP_DISABLE); - ok(hr == D3D_OK, "IDirect3DDevice9_SetTextureStageState failed with 0x%08x\n", hr); + refcount = IDirect3DDevice9_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); +done: + IDirect3D9_Release(d3d); + DestroyWindow(window); } static void zwriteenable_test(IDirect3DDevice9 *device) { @@ -15687,11 +15707,11 @@ START_TEST(visual) texop_test(device_ptr); texop_range_test(device_ptr); alphareplicate_test(device_ptr); - dp3_alpha_test(device_ptr); cleanup_device(device_ptr); device_ptr = NULL; + dp3_alpha_test(); depth_buffer_test(); depth_buffer2_test(); depth_blit_test();

1 0

Henri Verbeet : d3d9/tests: Use a separate device for depth_buffer_test().
by Alexandre Julliard 18 Mar '14

18 Mar '14

Module: wine Branch: master Commit: 34e9d94d812d19776286583bd72e1398ed32579d URL: http://source.winehq.org/git/wine.git/?a=commit;h=34e9d94d812d19776286583bd… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Mar 18 09:37:54 2014 +0100 d3d9/tests: Use a separate device for depth_buffer_test(). --- dlls/d3d9/tests/visual.c | 23 +++++++++++++++++++++-- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index fd8eb6d..44dedba 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -11736,7 +11736,7 @@ done: DestroyWindow(window); } -static void depth_buffer_test(IDirect3DDevice9 *device) +static void depth_buffer_test(void) { static const struct vertex quad1[] = { @@ -11768,11 +11768,25 @@ static void depth_buffer_test(IDirect3DDevice9 *device) }; IDirect3DSurface9 *backbuffer, *rt1, *rt2, *rt3; + IDirect3DDevice9 *device; unsigned int i, j; D3DVIEWPORT9 vp; + IDirect3D9 *d3d; D3DCOLOR color; + ULONG refcount; + HWND window; HRESULT hr; + window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW | WS_VISIBLE, + 0, 0, 640, 480, NULL, NULL, NULL, NULL); + d3d = Direct3DCreate9(D3D_SDK_VERSION); + ok(!!d3d, "Failed to create a D3D object.\n"); + if (!(device = create_device(d3d, window, window, TRUE))) + { + skip("Failed to create a D3D device, skipping tests.\n"); + goto done; + } + vp.X = 0; vp.Y = 0; vp.Width = 640; @@ -11864,6 +11878,11 @@ static void depth_buffer_test(IDirect3DDevice9 *device) IDirect3DSurface9_Release(rt3); IDirect3DSurface9_Release(rt2); IDirect3DSurface9_Release(rt1); + refcount = IDirect3DDevice9_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); +done: + IDirect3D9_Release(d3d); + DestroyWindow(window); } /* Test that partial depth copies work the way they're supposed to. The clear @@ -15669,11 +15688,11 @@ START_TEST(visual) texop_range_test(device_ptr); alphareplicate_test(device_ptr); dp3_alpha_test(device_ptr); - depth_buffer_test(device_ptr); cleanup_device(device_ptr); device_ptr = NULL; + depth_buffer_test(); depth_buffer2_test(); depth_blit_test(); intz_test();

1 0

← Newer
1
...
19
20
21
22
23
24
25
...
53
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits March 2014