February 2009 - Wine-commits

Aric Stewart : msctf: Implement ITfInputProcessorProfiles::Register.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: 649d12b1702a5bf17d169d83ef221468b49453e5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=649d12b1702a5bf17d169d83e… Author: Aric Stewart <aric(a)codeweavers.com> Date: Tue Feb 3 13:17:41 2009 -0600 msctf: Implement ITfInputProcessorProfiles::Register. --- dlls/msctf/inputprocessor.c | 19 +++++++++++++++++-- dlls/msctf/msctf.c | 2 ++ dlls/msctf/msctf_internal.h | 1 + 3 files changed, 20 insertions(+), 2 deletions(-) diff --git a/dlls/msctf/inputprocessor.c b/dlls/msctf/inputprocessor.c index b501b47..7ece851 100644 --- a/dlls/msctf/inputprocessor.c +++ b/dlls/msctf/inputprocessor.c @@ -97,8 +97,23 @@ static HRESULT WINAPI InputProcessorProfiles_Register( ITfInputProcessorProfiles *iface, REFCLSID rclsid) { InputProcessorProfiles *This = (InputProcessorProfiles*)iface; - FIXME("STUB:(%p)\n",This); - return E_NOTIMPL; + HKEY tipkey; + WCHAR buf[39]; + static const WCHAR fmt[] = {'%','s','\\','%','s',0}; + WCHAR fullkey[68]; + + TRACE("(%p) %s\n",This,debugstr_guid(rclsid)); + + StringFromGUID2(rclsid, buf, 39); + sprintfW(fullkey,fmt,szwSystemTIPKey,buf); + + if (RegCreateKeyExW(HKEY_LOCAL_MACHINE,fullkey, 0, NULL, 0, + KEY_READ | KEY_WRITE, NULL, &tipkey, NULL) != ERROR_SUCCESS) + return E_FAIL; + + RegCloseKey(tipkey); + + return S_OK; } static HRESULT WINAPI InputProcessorProfiles_Unregister( diff --git a/dlls/msctf/msctf.c b/dlls/msctf/msctf.c index 224ce99..e34a9c8 100644 --- a/dlls/msctf/msctf.c +++ b/dlls/msctf/msctf.c @@ -45,6 +45,8 @@ static HINSTANCE MSCTF_hinstance; DWORD tlsIndex = 0; +const WCHAR szwSystemTIPKey[] = {'S','O','F','T','W','A','R','E','\\','M','i','c','r','o','s','o','f','t','\\','C','T','F','\\','T','I','P',0}; + typedef HRESULT (*LPFNCONSTRUCTOR)(IUnknown *pUnkOuter, IUnknown **ppvOut); static const struct { diff --git a/dlls/msctf/msctf_internal.h b/dlls/msctf/msctf_internal.h index 1cb4924..cea1723 100644 --- a/dlls/msctf/msctf_internal.h +++ b/dlls/msctf/msctf_internal.h @@ -27,4 +27,5 @@ extern HRESULT DocumentMgr_Constructor(ITfDocumentMgr **ppOut); extern HRESULT Context_Constructor(TfClientId tidOwner, IUnknown *punk, ITfContext **ppOut, TfEditCookie *pecTextStore); extern HRESULT InputProcessorProfiles_Constructor(IUnknown *pUnkOuter, IUnknown **ppOut); +extern const WCHAR szwSystemTIPKey[]; #endif /* __WINE_MSCTF_I_H */

1 0

Aric Stewart : msctf: Implement ITfInputProcessorProfiles:: AddLanguageProfile.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: 3680bdccc24fa75b895f4519f79a96f49ac920a5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3680bdccc24fa75b895f4519f… Author: Aric Stewart <aric(a)codeweavers.com> Date: Tue Feb 3 13:17:50 2009 -0600 msctf: Implement ITfInputProcessorProfiles::AddLanguageProfile. --- dlls/msctf/inputprocessor.c | 45 +++++++++++++++++++++++++++++++++++++++++- 1 files changed, 43 insertions(+), 2 deletions(-) diff --git a/dlls/msctf/inputprocessor.c b/dlls/msctf/inputprocessor.c index 7ece851..893a88f 100644 --- a/dlls/msctf/inputprocessor.c +++ b/dlls/msctf/inputprocessor.c @@ -130,9 +130,50 @@ static HRESULT WINAPI InputProcessorProfiles_AddLanguageProfile( ULONG cchDesc, const WCHAR *pchIconFile, ULONG cchFile, ULONG uIconIndex) { + HKEY tipkey,fmtkey; + WCHAR buf[39]; + WCHAR fullkey[100]; + ULONG res; + + static const WCHAR fmt[] = {'%','s','\\','%','s',0}; + static const WCHAR fmt2[] = {'%','s','\\','0','x','%','0','8','x','\\','%','s',0}; + static const WCHAR lngp[] = {'L','a','n','g','u','a','g','e','P','r','o','f','i','l','e',0}; + static const WCHAR desc[] = {'D','e','s','c','r','i','p','t','i','o','n',0}; + static const WCHAR icnf[] = {'I','c','o','n','F','i','l','e',0}; + static const WCHAR icni[] = {'I','c','o','n','I','n','d','e','x',0}; + InputProcessorProfiles *This = (InputProcessorProfiles*)iface; - FIXME("STUB:(%p)\n",This); - return E_NOTIMPL; + + TRACE("(%p) %s %x %s %s %s %i\n",This,debugstr_guid(rclsid), langid, + debugstr_guid(guidProfile), debugstr_wn(pchDesc,cchDesc), + debugstr_wn(pchIconFile,cchFile),uIconIndex); + + StringFromGUID2(rclsid, buf, 39); + sprintfW(fullkey,fmt,szwSystemTIPKey,buf); + + if (RegOpenKeyExW(HKEY_LOCAL_MACHINE,fullkey, 0, KEY_READ | KEY_WRITE, + &tipkey ) != ERROR_SUCCESS) + return E_FAIL; + + StringFromGUID2(guidProfile, buf, 39); + sprintfW(fullkey,fmt2,lngp,langid,buf); + + res = RegCreateKeyExW(tipkey,fullkey, 0, NULL, 0, KEY_READ | KEY_WRITE, + NULL, &fmtkey, NULL); + + if (!res) + { + RegSetValueExW(fmtkey, desc, 0, REG_SZ, (LPBYTE)pchDesc, cchDesc * sizeof(WCHAR)); + RegSetValueExW(fmtkey, icnf, 0, REG_SZ, (LPBYTE)pchIconFile, cchFile * sizeof(WCHAR)); + RegSetValueExW(fmtkey, icni, 0, REG_DWORD, (LPBYTE)&uIconIndex, sizeof(DWORD)); + RegCloseKey(fmtkey); + } + RegCloseKey(tipkey); + + if (!res) + return S_OK; + else + return E_FAIL; } static HRESULT WINAPI InputProcessorProfiles_RemoveLanguageProfile(

1 0

Aric Stewart : msctf: Stub implementation of ITfCategoryMgr.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: 2fe3b64fb459414e6ef8a0eb98e254541b74134b URL: http://source.winehq.org/git/wine.git/?a=commit;h=2fe3b64fb459414e6ef8a0eb9… Author: Aric Stewart <aric(a)codeweavers.com> Date: Tue Feb 3 13:18:02 2009 -0600 msctf: Stub implementation of ITfCategoryMgr. --- dlls/msctf/Makefile.in | 1 + dlls/msctf/categorymgr.c | 248 +++++++++++++++++++++++++++++++++++++++++++ dlls/msctf/msctf.c | 1 + dlls/msctf/msctf_internal.h | 1 + dlls/msctf/regsvr.c | 7 ++ dlls/uuid/uuid.c | 1 + include/msctf.idl | 61 +++++++++++ 7 files changed, 320 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2fe3b64fb459414e6ef8a…

1 0

Alexandre Julliard : include: Define LANGID in wtypes.idl.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: 4e29ceeb77c4eb3a7d66dd1217d0fb375505ee4f URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e29ceeb77c4eb3a7d66dd121… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 4 15:06:25 2009 +0100 include: Define LANGID in wtypes.idl. --- dlls/msi/msiserver.idl | 1 - include/dimm.idl | 2 -- include/mlang.idl | 5 ----- include/wtypes.idl | 1 + 4 files changed, 1 insertions(+), 8 deletions(-) diff --git a/dlls/msi/msiserver.idl b/dlls/msi/msiserver.idl index ac442db..2189fc4 100644 --- a/dlls/msi/msiserver.idl +++ b/dlls/msi/msiserver.idl @@ -29,7 +29,6 @@ typedef int INSTALLMESSAGE; typedef int MSICONDITION; typedef int MSIRUNMODE; typedef int INSTALLSTATE; -typedef WORD LANGID; cpp_quote("#endif") [ diff --git a/include/dimm.idl b/include/dimm.idl index 07e04b2..851ec90 100644 --- a/include/dimm.idl +++ b/include/dimm.idl @@ -24,8 +24,6 @@ cpp_quote("#include <imm.h>") cpp_quote("#if 0") -typedef WORD LANGID; - typedef struct { LPSTR lpReading; LPSTR lpWord; diff --git a/include/mlang.idl b/include/mlang.idl index bcc0917..ca89711 100644 --- a/include/mlang.idl +++ b/include/mlang.idl @@ -22,11 +22,6 @@ import "unknwn.idl"; interface IStream; -/* FIXME: LANGID is defined in winnt.h and mlang.h in the platform SDK */ -cpp_quote("#ifndef _WINNT_") -typedef WORD LANGID; -cpp_quote("#endif") - cpp_quote("#define CPIOD_PEEK 0x40000000") cpp_quote("#define CPIOD_FORCE_PROMPT 0x80000000") diff --git a/include/wtypes.idl b/include/wtypes.idl index 8378012..c817d08 100644 --- a/include/wtypes.idl +++ b/include/wtypes.idl @@ -87,6 +87,7 @@ typedef LONG_PTR LRESULT; typedef LONG HRESULT; typedef DWORD LCID; +typedef USHORT LANGID; typedef unsigned __int64 DWORDLONG; typedef __int64 LONGLONG;

1 0

Aric Stewart : msctf: Add stub implementation of ITfInputProcessorProfiles.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: 6c842a86bb630ec83f30802925312b6d98dab208 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6c842a86bb630ec83f3080292… Author: Aric Stewart <aric(a)codeweavers.com> Date: Tue Feb 3 13:17:04 2009 -0600 msctf: Add stub implementation of ITfInputProcessorProfiles. --- dlls/msctf/Makefile.in | 1 + dlls/msctf/inputprocessor.c | 295 +++++++++++++++++++++++++++++++++++++++++++ dlls/msctf/msctf.c | 1 + dlls/msctf/msctf_internal.h | 1 + dlls/msctf/regsvr.c | 7 + dlls/uuid/uuid.c | 1 + include/msctf.idl | 100 +++++++++++++++ 7 files changed, 406 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=6c842a86bb630ec83f308…

1 0

Paul Vriens : winetest: Make sure we can test .NET dlls on Windows.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: aa20ee6b45cc5258749fcbdcf08a615785771fb8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=aa20ee6b45cc5258749fcbdcf… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Tue Feb 3 20:45:34 2009 +0100 winetest: Make sure we can test .NET dlls on Windows. --- programs/winetest/main.c | 22 +++++++++++++++++++++- 1 files changed, 21 insertions(+), 1 deletions(-) diff --git a/programs/winetest/main.c b/programs/winetest/main.c index fc7a378..211ec2e 100644 --- a/programs/winetest/main.c +++ b/programs/winetest/main.c @@ -55,6 +55,10 @@ static char build_id[64]; static char *filters[64]; static unsigned int nb_filters = 0; +/* Needed to check for .NET dlls */ +static HMODULE hmscoree; +static HRESULT (WINAPI *pLoadLibraryShim)(LPCWSTR, LPCWSTR, LPVOID, HMODULE *); + /* check if test is being filtered out */ static BOOL test_filtered_out( LPCSTR module, LPCSTR testname ) { @@ -494,6 +498,8 @@ extract_test_proc (HMODULE hModule, LPCTSTR lpszType, { const char *tempdir = (const char *)lParam; char dllname[MAX_PATH]; + char filename[MAX_PATH]; + WCHAR dllnameW[MAX_PATH]; HMODULE dll; DWORD err; @@ -505,15 +511,21 @@ extract_test_proc (HMODULE hModule, LPCTSTR lpszType, *strstr(dllname, testexe) = 0; dll = LoadLibraryExA(dllname, NULL, LOAD_LIBRARY_AS_DATAFILE); + if (!dll && pLoadLibraryShim) + { + MultiByteToWideChar(CP_ACP, 0, dllname, -1, dllnameW, MAX_PATH); + if (FAILED( pLoadLibraryShim(dllnameW, NULL, NULL, &dll) )) dll = 0; + } if (!dll) { xprintf (" %s=dll is missing\n", dllname); return TRUE; } + GetModuleFileNameA(dll, filename, MAX_PATH); FreeLibrary(dll); if (!(err = get_subtests( tempdir, &wine_tests[nr_of_files], lpszName ))) { - xprintf (" %s=%s\n", dllname, get_file_version(dllname)); + xprintf (" %s=%s\n", dllname, get_file_version(filename)); nr_of_tests += wine_tests[nr_of_files].subtest_count; nr_of_files++; } @@ -614,6 +626,12 @@ run_tests (char *logname) GetLastError ()); wine_tests = xmalloc (nr_of_files * sizeof wine_tests[0]); + /* Do this only once during extraction (and version checking) */ + hmscoree = LoadLibraryA("mscoree.dll"); + pLoadLibraryShim = NULL; + if (hmscoree) + pLoadLibraryShim = (void *)GetProcAddress(hmscoree, "LoadLibraryShim"); + report (R_STATUS, "Extracting tests"); report (R_PROGRESS, 0, nr_of_files); nr_of_files = 0; @@ -623,6 +641,8 @@ run_tests (char *logname) report (R_FATAL, "Can't enumerate test files: %d", GetLastError ()); + FreeLibrary(hmscoree); + xprintf ("Test output:\n" ); report (R_DELTA, 0, "Extracting: Done");

1 0

Alexandre Julliard : dbghelp: Moved SymEnumLines to symbol. c to reuse the regex support.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: 601a1364b95ff1e72a9e1fbc08feb14fce9db2eb URL: http://source.winehq.org/git/wine.git/?a=commit;h=601a1364b95ff1e72a9e1fbc0… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 4 13:13:08 2009 +0100 dbghelp: Moved SymEnumLines to symbol.c to reuse the regex support. --- dlls/dbghelp/source.c | 113 ------------------------------------------------- dlls/dbghelp/symbol.c | 109 +++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 109 insertions(+), 113 deletions(-) diff --git a/dlls/dbghelp/source.c b/dlls/dbghelp/source.c index 8eef337..dd8fc6e 100644 --- a/dlls/dbghelp/source.c +++ b/dlls/dbghelp/source.c @@ -26,9 +26,6 @@ #include "dbghelp_private.h" #include "wine/debug.h" -#ifdef HAVE_REGEX_H -# include <regex.h> -#endif WINE_DEFAULT_DEBUG_CHANNEL(dbghelp); @@ -154,116 +151,6 @@ BOOL WINAPI SymEnumSourceFiles(HANDLE hProcess, ULONG64 ModBase, PCSTR Mask, return TRUE; } -static inline void re_append(char** mask, unsigned* len, char ch) -{ - *mask = HeapReAlloc(GetProcessHeap(), 0, *mask, ++(*len)); - (*mask)[*len - 2] = ch; -} - -static BOOL compile_regex(regex_t* re, const char* srcfile) -{ - char* mask; - unsigned len = 1; - - mask = HeapAlloc(GetProcessHeap(), 0, 1); - re_append(&mask, &len, '^'); - if (!srcfile || !*srcfile) re_append(&mask, &len, '*'); - else while (*srcfile) - { - switch (*srcfile) - { - case '\\': - case '/': - re_append(&mask, &len, '['); - re_append(&mask, &len, '\\'); - re_append(&mask, &len, '\\'); - re_append(&mask, &len, '/'); - re_append(&mask, &len, ']'); - break; - case '.': - re_append(&mask, &len, '\\'); - re_append(&mask, &len, '.'); - break; - default: - re_append(&mask, &len, *srcfile); - break; - } - srcfile++; - } - re_append(&mask, &len, '$'); - mask[len - 1] = '\0'; - len = regcomp(re, mask, REG_NOSUB); - HeapFree(GetProcessHeap(), 0, mask); - if (len) - { - FIXME("Couldn't compile %s\n", mask); - SetLastError(ERROR_INVALID_PARAMETER); - return FALSE; - } - return TRUE; -} - -/****************************************************************** - * SymEnumLines (DBGHELP.@) - * - */ -BOOL WINAPI SymEnumLines(HANDLE hProcess, ULONG64 base, PCSTR compiland, - PCSTR srcfile, PSYM_ENUMLINES_CALLBACK cb, PVOID user) -{ - struct module_pair pair; - struct hash_table_iter hti; - struct symt_ht* sym; - regex_t re; - struct line_info* dli; - void* ptr; - SRCCODEINFO sci; - const char* file; - - if (!cb) return FALSE; - if (!(dbghelp_options & SYMOPT_LOAD_LINES)) return TRUE; - - pair.pcs = process_find_by_handle(hProcess); - if (!pair.pcs) return FALSE; - if (compiland) FIXME("Unsupported yet (filtering on compiland %s)\n", compiland); - pair.requested = module_find_by_addr(pair.pcs, base, DMT_UNKNOWN); - if (!module_get_debug(&pair)) return FALSE; - if (!compile_regex(&re, srcfile)) return FALSE; - - sci.SizeOfStruct = sizeof(sci); - sci.ModBase = base; - - hash_table_iter_init(&pair.effective->ht_symbols, &hti, NULL); - while ((ptr = hash_table_iter_up(&hti))) - { - unsigned int i; - - sym = GET_ENTRY(ptr, struct symt_ht, hash_elt); - if (sym->symt.tag != SymTagFunction) continue; - - sci.FileName[0] = '\0'; - for (i=0; i<vector_length(&((struct symt_function*)sym)->vlines); i++) - { - dli = vector_at(&((struct symt_function*)sym)->vlines, i); - if (dli->is_source_file) - { - file = source_get(pair.effective, dli->u.source_file); - if (regexec(&re, file, 0, NULL, 0) != 0) file = ""; - strcpy(sci.FileName, file); - } - else if (sci.FileName[0]) - { - sci.Key = dli; - sci.Obj[0] = '\0'; /* FIXME */ - sci.LineNumber = dli->line_number; - sci.Address = dli->u.pc_offset; - if (!cb(&sci, user)) break; - } - } - } - regfree(&re); - return TRUE; -} - /****************************************************************** * SymGetSourceFileToken (DBGHELP.@) * diff --git a/dlls/dbghelp/symbol.c b/dlls/dbghelp/symbol.c index 304e075..9e66f00 100644 --- a/dlls/dbghelp/symbol.c +++ b/dlls/dbghelp/symbol.c @@ -123,6 +123,48 @@ static void compile_regex(const char* str, int numchar, regex_t* re, BOOL _case) HeapFree(GetProcessHeap(), 0, mask); } +static BOOL compile_file_regex(regex_t* re, const char* srcfile) +{ + char *mask, *p; + BOOL ret; + + p = mask = HeapAlloc(GetProcessHeap(), 0, 5 * strlen(srcfile) + 4); + *p++ = '^'; + if (!srcfile || !*srcfile) *p++ = '*'; + else while (*srcfile) + { + switch (*srcfile) + { + case '\\': + case '/': + *p++ = '['; + *p++ = '\\'; + *p++ = '\\'; + *p++ = '/'; + *p++ = ']'; + break; + case '.': + *p++ = '\\'; + *p++ = '.'; + break; + default: + *p++ = *srcfile; + break; + } + srcfile++; + } + *p++ = '$'; + *p = 0; + ret = !regcomp(re, mask, REG_NOSUB); + HeapFree(GetProcessHeap(), 0, mask); + if (!ret) + { + FIXME("Couldn't compile %s\n", mask); + SetLastError(ERROR_INVALID_PARAMETER); + } + return ret; +} + static int match_regexp( const regex_t *re, const char *str ) { return !regexec( re, str, 0, NULL, 0 ); @@ -148,6 +190,12 @@ static void compile_regex(const char* str, int numchar, regex_t* re, BOOL _case) re->icase = _case; } +static BOOL compile_file_regex(regex_t* re, const char* srcfile) +{ + compile_regex( srcfile, -1, re, FALSE ); + return TRUE; +} + static int match_regexp( const regex_t *re, const char *str ) { if (re->icase) return !lstrcmpiA( re->str, str ); @@ -1797,3 +1845,64 @@ BOOL WINAPI SymSetScopeFromAddr(HANDLE hProcess, ULONG64 addr) if (!(pcs = process_find_by_handle(hProcess))) return FALSE; return TRUE; } + +/****************************************************************** + * SymEnumLines (DBGHELP.@) + * + */ +BOOL WINAPI SymEnumLines(HANDLE hProcess, ULONG64 base, PCSTR compiland, + PCSTR srcfile, PSYM_ENUMLINES_CALLBACK cb, PVOID user) +{ + struct module_pair pair; + struct hash_table_iter hti; + struct symt_ht* sym; + regex_t re; + struct line_info* dli; + void* ptr; + SRCCODEINFO sci; + const char* file; + + if (!cb) return FALSE; + if (!(dbghelp_options & SYMOPT_LOAD_LINES)) return TRUE; + + pair.pcs = process_find_by_handle(hProcess); + if (!pair.pcs) return FALSE; + if (compiland) FIXME("Unsupported yet (filtering on compiland %s)\n", compiland); + pair.requested = module_find_by_addr(pair.pcs, base, DMT_UNKNOWN); + if (!module_get_debug(&pair)) return FALSE; + if (!compile_file_regex(&re, srcfile)) return FALSE; + + sci.SizeOfStruct = sizeof(sci); + sci.ModBase = base; + + hash_table_iter_init(&pair.effective->ht_symbols, &hti, NULL); + while ((ptr = hash_table_iter_up(&hti))) + { + unsigned int i; + + sym = GET_ENTRY(ptr, struct symt_ht, hash_elt); + if (sym->symt.tag != SymTagFunction) continue; + + sci.FileName[0] = '\0'; + for (i=0; i<vector_length(&((struct symt_function*)sym)->vlines); i++) + { + dli = vector_at(&((struct symt_function*)sym)->vlines, i); + if (dli->is_source_file) + { + file = source_get(pair.effective, dli->u.source_file); + if (!match_regexp(&re, file)) file = ""; + strcpy(sci.FileName, file); + } + else if (sci.FileName[0]) + { + sci.Key = dli; + sci.Obj[0] = '\0'; /* FIXME */ + sci.LineNumber = dli->line_number; + sci.Address = dli->u.pc_offset; + if (!cb(&sci, user)) break; + } + } + } + regfree(&re); + return TRUE; +}

1 0

Alistair Leslie-Hughes : shdocvw: Correct OnAmbientPropertyChange regression.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: a14a75b85ebe1a7d15f9a313f9d79a9571037d6c URL: http://source.winehq.org/git/wine.git/?a=commit;h=a14a75b85ebe1a7d15f9a313f… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed Feb 4 11:51:59 2009 +1100 shdocvw: Correct OnAmbientPropertyChange regression. --- dlls/shdocvw/oleobject.c | 6 +++--- dlls/shdocvw/tests/webbrowser.c | 25 ++++++++++++++++--------- 2 files changed, 19 insertions(+), 12 deletions(-) diff --git a/dlls/shdocvw/oleobject.c b/dlls/shdocvw/oleobject.c index cd5e605..a38d03d 100644 --- a/dlls/shdocvw/oleobject.c +++ b/dlls/shdocvw/oleobject.c @@ -719,9 +719,9 @@ static HRESULT WINAPI OleControl_OnAmbientPropertyChange(IOleControl *iface, DIS switch(dispID) { case DISPID_UNKNOWN: - /* Unknown means multiple properties changed, so check them all. */ - on_offlineconnected_change(This); - on_silent_change(This); + /* Unknown means multiple properties changed, so check them all. + * BUT the Webbrowser OleControl object doesnt appear to do this. + */ return S_OK; case DISPID_AMBIENT_OFFLINEIFNOTCONNECTED: return on_offlineconnected_change(This); diff --git a/dlls/shdocvw/tests/webbrowser.c b/dlls/shdocvw/tests/webbrowser.c index 27cd353..64c5bab 100644 --- a/dlls/shdocvw/tests/webbrowser.c +++ b/dlls/shdocvw/tests/webbrowser.c @@ -69,6 +69,9 @@ DEFINE_GUID(GUID_NULL,0,0,0,0,0,0,0,0,0,0,0); expect_ ## func = called_ ## func = FALSE; \ }while(0) +#define CLEAR_CALLED(func) \ + expect_ ## func = called_ ## func = FALSE + DEFINE_EXPECT(GetContainer); DEFINE_EXPECT(Site_GetWindow); DEFINE_EXPECT(ShowObject); @@ -1787,15 +1790,19 @@ static void test_ambient_unknown(IWebBrowser2 *wb, IOleControl *control, BOOL is hres = IOleControl_OnAmbientPropertyChange(control, DISPID_UNKNOWN); ok(hres == S_OK, "OnAmbientPropertyChange failed %08x\n", hres); - if(is_clientsite) - { - CHECK_CALLED(Invoke_AMBIENT_OFFLINEIFNOTCONNECTED); - CHECK_CALLED(Invoke_AMBIENT_SILENT); - todo_wine CHECK_CALLED(Invoke_AMBIENT_USERMODE); - todo_wine CHECK_CALLED(Invoke_AMBIENT_DLCONTROL); - todo_wine CHECK_CALLED(Invoke_AMBIENT_USERAGENT); - todo_wine CHECK_CALLED(Invoke_AMBIENT_PALETTE); - } + CHECK_EXPECT(Invoke_AMBIENT_OFFLINEIFNOTCONNECTED); + CHECK_EXPECT(Invoke_AMBIENT_SILENT); + CHECK_EXPECT(Invoke_AMBIENT_USERMODE); + CHECK_EXPECT(Invoke_AMBIENT_DLCONTROL); + CHECK_EXPECT(Invoke_AMBIENT_USERAGENT); + CHECK_EXPECT(Invoke_AMBIENT_PALETTE); + + CLEAR_CALLED(Invoke_AMBIENT_OFFLINEIFNOTCONNECTED); + CLEAR_CALLED(Invoke_AMBIENT_SILENT); + CLEAR_CALLED(Invoke_AMBIENT_USERMODE); + CLEAR_CALLED(Invoke_AMBIENT_DLCONTROL); + CLEAR_CALLED(Invoke_AMBIENT_USERAGENT); + CLEAR_CALLED(Invoke_AMBIENT_PALETTE); } static void test_wb_funcs(IUnknown *unk, BOOL is_clientsite)

1 0

Detlef Riekenberg : comctl32/tests: Make the test dpi and theme aware by using SM_CYSIZE.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: f78f4d89b70c31db0a5b95c623d500bd057926f2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f78f4d89b70c31db0a5b95c62… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Wed Feb 4 00:35:57 2009 +0100 comctl32/tests: Make the test dpi and theme aware by using SM_CYSIZE. --- dlls/comctl32/tests/status.c | 25 ++++++++++++++++++++----- 1 files changed, 20 insertions(+), 5 deletions(-) diff --git a/dlls/comctl32/tests/status.c b/dlls/comctl32/tests/status.c index 8b8facf..f47d3ea 100644 --- a/dlls/comctl32/tests/status.c +++ b/dlls/comctl32/tests/status.c @@ -40,6 +40,7 @@ static WNDPROC g_status_wndproc; static RECT g_rcCreated; static HWND g_hMainWnd; static int g_wmsize_count = 0; +static DWORD g_height; static HWND create_status_control(DWORD style, DWORD exstyle) { @@ -113,10 +114,16 @@ static int CALLBACK check_height_font_enumproc(ENUMLOGFONTEX *enumlf, NEWTEXTMET { HWND hwndStatus = (HWND)lParam; HDC hdc = GetDC(NULL); - static const int sizes[] = {8, 9, 10, 12, 16, 22, 28, 36, 48, 72}; - int i; + static const int sizes[] = { 6, 7, 8, 9, 10, 11, 12, 13, 15, 16, + 20, 22, 28, 36, 48, 72}; + DWORD i; + DWORD y; + LPSTR facename = (CHAR *)enumlf->elfFullName; + + /* on win9x, enumlf->elfFullName is only valid for truetype fonts */ + if (type != TRUETYPE_FONTTYPE) + facename = enumlf->elfLogFont.lfFaceName; - trace("Font %s\n", enumlf->elfFullName); for (i = 0; i < sizeof(sizes)/sizeof(sizes[0]); i++) { HFONT hFont; @@ -132,7 +139,11 @@ static int CALLBACK check_height_font_enumproc(ENUMLOGFONTEX *enumlf, NEWTEXTMET GetClientRect(hwndStatus, &rcCtrl); GetTextMetrics(hdc, &tm); - expect(max(tm.tmHeight + (tm.tmInternalLeading ? tm.tmInternalLeading : 2) + 4, 20), rcCtrl.bottom); + y = tm.tmHeight + (tm.tmInternalLeading ? tm.tmInternalLeading : 2) + 4; + + ok( rcCtrl.bottom == max(y, g_height), + "got %d (expected %d) for %s #%d\n", + rcCtrl.bottom, max(y, g_height), facename, sizes[i]); SelectObject(hdc, hOldFont); SendMessage(hwndStatus, WM_SETFONT, (WPARAM)hCtrlFont, TRUE); @@ -213,7 +224,11 @@ static void test_height(void) ZeroMemory(&lf, sizeof(lf)); SendMessage(hwndStatus, SB_SETMINHEIGHT, 0, 0); hdc = GetDC(NULL); - trace("dpi=%d\n", GetDeviceCaps(hdc, LOGPIXELSY)); + + g_height = GetSystemMetrics(SM_CYSIZE) + 2; + if (g_height & 1) g_height--; /* The height is always even */ + + trace("dpi=%d (min height: %d)\n", GetDeviceCaps(hdc, LOGPIXELSY), g_height); EnumFontFamiliesEx(hdc, &lf, (FONTENUMPROC)check_height_family_enumproc, (LPARAM)hwndStatus, 0); ReleaseDC(NULL, hdc);

1 0

David Adam : d3dx9_36: Remove useless traces.
by Alexandre Julliard 04 Feb '09

04 Feb '09

Module: wine Branch: master Commit: 55d6f07e3199cfe6c5145adbda7aa021a7dd7ae0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=55d6f07e3199cfe6c5145adbd… Author: David Adam <david.adam.cnrs(a)gmail.com> Date: Wed Feb 4 08:52:52 2009 +0100 d3dx9_36: Remove useless traces. --- dlls/d3dx9_36/math.c | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/d3dx9_36/math.c b/dlls/d3dx9_36/math.c index b8416c6..2bf301b 100644 --- a/dlls/d3dx9_36/math.c +++ b/dlls/d3dx9_36/math.c @@ -222,7 +222,7 @@ D3DXPLANE* WINAPI D3DXPlaneTransformArray( CONST D3DXMATRIX* matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXPlaneTransform( (D3DXPLANE*)((char*)out + outstride * i), @@ -242,7 +242,7 @@ D3DXVECTOR4* WINAPI D3DXVec2TransformArray( CONST D3DXMATRIX* matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec2Transform( (D3DXVECTOR4*)((char*)out + outstride * i), @@ -260,7 +260,7 @@ D3DXVECTOR2* WINAPI D3DXVec2TransformCoordArray( CONST D3DXMATRIX* matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec2TransformCoord( (D3DXVECTOR2*)((char*)out + outstride * i), @@ -278,7 +278,7 @@ D3DXVECTOR2* WINAPI D3DXVec2TransformNormalArray( CONST D3DXMATRIX *matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec2TransformNormal( (D3DXVECTOR2*)((char*)out + outstride * i), @@ -299,7 +299,7 @@ D3DXVECTOR3* WINAPI D3DXVec3ProjectArray( CONST D3DXMATRIX* view, CONST D3DXMATRIX* world, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec3Project( (D3DXVECTOR3*)((char*)out + outstride * i), @@ -317,7 +317,7 @@ D3DXVECTOR4* WINAPI D3DXVec3TransformArray( CONST D3DXMATRIX* matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec3Transform( (D3DXVECTOR4*)((char*)out + outstride * i), @@ -335,7 +335,7 @@ D3DXVECTOR3* WINAPI D3DXVec3TransformCoordArray( CONST D3DXMATRIX* matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec3TransformCoord( (D3DXVECTOR3*)((char*)out + outstride * i), @@ -353,7 +353,7 @@ D3DXVECTOR3* WINAPI D3DXVec3TransformNormalArray( CONST D3DXMATRIX* matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec3TransformNormal( (D3DXVECTOR3*)((char*)out + outstride * i), @@ -372,7 +372,7 @@ D3DXVECTOR3* WINAPI D3DXVec3UnprojectArray( CONST D3DXMATRIX* view, CONST D3DXMATRIX* world, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec3Unproject( (D3DXVECTOR3*)((char*)out + outstride * i), @@ -390,7 +390,7 @@ D3DXVECTOR4* WINAPI D3DXVec4TransformArray( CONST D3DXMATRIX* matrix, UINT elements) { UINT i; - TRACE("\n"); + for (i = 0; i < elements; ++i) { D3DXVec4Transform( (D3DXVECTOR4*)((char*)out + outstride * i),

1 0