February 2009 - Wine-Commits

Aric Stewart : msctf: Stub implementation of ITfCategoryMgr.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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

Aric Stewart : msctf: Implement InputProcessorProfiles::GetCurrentLanguage.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

Module: wine Branch: master Commit: fd92f871a32e1575784761cdac40bb0fd280c407 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fd92f871a32e1575784761cda… Author: Aric Stewart <aric(a)codeweavers.com> Date: Tue Feb 3 13:17:19 2009 -0600 msctf: Implement InputProcessorProfiles::GetCurrentLanguage. --- dlls/msctf/inputprocessor.c | 13 +++++++++++-- 1 files changed, 11 insertions(+), 2 deletions(-) diff --git a/dlls/msctf/inputprocessor.c b/dlls/msctf/inputprocessor.c index ca77964..b501b47 100644 --- a/dlls/msctf/inputprocessor.c +++ b/dlls/msctf/inputprocessor.c @@ -43,6 +43,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(msctf); typedef struct tagInputProcessorProfiles { const ITfInputProcessorProfilesVtbl *InputProcessorProfilesVtbl; LONG refCount; + + LANGID currentLanguage; } InputProcessorProfiles; static void InputProcessorProfiles_Destructor(InputProcessorProfiles *This) @@ -184,8 +186,14 @@ static HRESULT WINAPI InputProcessorProfiles_GetCurrentLanguage( ITfInputProcessorProfiles *iface, LANGID *plangid) { InputProcessorProfiles *This = (InputProcessorProfiles*)iface; - FIXME("STUB:(%p)\n",This); - return E_NOTIMPL; + TRACE("(%p) 0x%x\n",This,This->currentLanguage); + + if (!plangid) + return E_INVALIDARG; + + *plangid = This->currentLanguage; + + return S_OK; } static HRESULT WINAPI InputProcessorProfiles_ChangeCurrentLanguage( @@ -288,6 +296,7 @@ HRESULT InputProcessorProfiles_Constructor(IUnknown *pUnkOuter, IUnknown **ppOut This->InputProcessorProfilesVtbl= &InputProcessorProfiles_InputProcessorProfilesVtbl; This->refCount = 1; + This->currentLanguage = GetUserDefaultLCID(); TRACE("returning %p\n", This); *ppOut = (IUnknown *)This;

1 0

Aric Stewart : msctf: Implement ITfInputProcessorProfiles:: AddLanguageProfile.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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: Add stub implementation of ITfInputProcessorProfiles.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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

Alexandre Julliard : include: Define LANGID in wtypes.idl.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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

Paul Vriens : winetest: Make sure we can test .NET dlls on Windows.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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

Detlef Riekenberg : comctl32/tests: Make the test dpi and theme aware by using SM_CYSIZE.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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

Alistair Leslie-Hughes : shdocvw: Correct OnAmbientPropertyChange regression.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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

Alexandre Julliard : dbghelp: Moved SymEnumLines to symbol. c to reuse the regex support.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

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

Alexandre Julliard : dbghelp: Fall back to string comparison if regex support is missing.
by Alexandre Julliard Feb. 4, 2009

Feb. 4, 2009

Module: wine Branch: master Commit: 768954b245c280072c5975de7129c1133760d59b URL: http://source.winehq.org/git/wine.git/?a=commit;h=768954b245c280072c5975de7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 4 13:02:08 2009 +0100 dbghelp: Fall back to string comparison if regex support is missing. --- dlls/dbghelp/symbol.c | 51 +++++++++++++++++++++++++++++++++++++++++++----- 1 files changed, 45 insertions(+), 6 deletions(-) diff --git a/dlls/dbghelp/symbol.c b/dlls/dbghelp/symbol.c index 5a05928..304e075 100644 --- a/dlls/dbghelp/symbol.c +++ b/dlls/dbghelp/symbol.c @@ -67,6 +67,8 @@ int symt_cmp_addr(const void* p1, const void* p2) return cmp_addr(a1, a2); } +#ifdef HAVE_REGEX_H + /* transforms a dbghelp's regular expression into a POSIX one * Here are the valid dbghelp reg ex characters: * * 0 or more characters @@ -121,6 +123,44 @@ static void compile_regex(const char* str, int numchar, regex_t* re, BOOL _case) HeapFree(GetProcessHeap(), 0, mask); } +static int match_regexp( const regex_t *re, const char *str ) +{ + return !regexec( re, str, 0, NULL, 0 ); +} + +#else /* HAVE_REGEX_H */ + +/* if we don't have regexp support, fall back to a simple string comparison */ + +typedef struct +{ + char *str; + BOOL icase; +} regex_t; + +static void compile_regex(const char* str, int numchar, regex_t* re, BOOL _case) +{ + if (numchar == -1) numchar = strlen( str ); + + re->str = HeapAlloc( GetProcessHeap(), 0, numchar + 1 ); + memcpy( re->str, str, numchar ); + re->str[numchar] = 0; + re->icase = _case; +} + +static int match_regexp( const regex_t *re, const char *str ) +{ + if (re->icase) return !lstrcmpiA( re->str, str ); + return !strcmp( re->str, str ); +} + +static void regfree( regex_t *re ) +{ + HeapFree( GetProcessHeap(), 0, re->str ); +} + +#endif /* HAVE_REGEX_H */ + struct symt_compiland* symt_new_compiland(struct module* module, unsigned long address, unsigned src_idx) { @@ -658,8 +698,7 @@ static BOOL symt_enum_module(struct module_pair* pair, const regex_t* regex, while ((ptr = hash_table_iter_up(&hti))) { sym = GET_ENTRY(ptr, struct symt_ht, hash_elt); - if (sym->hash_elt.name && - regexec(regex, sym->hash_elt.name, 0, NULL, 0) == 0) + if (sym->hash_elt.name && match_regexp(regex, sym->hash_elt.name)) { se->sym_info->SizeOfStruct = sizeof(SYMBOL_INFO); se->sym_info->MaxNameLen = sizeof(se->buffer) - sizeof(SYMBOL_INFO); @@ -808,7 +847,7 @@ static BOOL symt_enum_locals_helper(struct module_pair* pair, } break; case SymTagData: - if (regexec(preg, symt_get_name(lsym), 0, NULL, 0) == 0) + if (match_regexp(preg, symt_get_name(lsym))) { if (send_symbol(se, pair, func, lsym)) return FALSE; } @@ -912,7 +951,7 @@ static BOOL sym_enum(HANDLE hProcess, ULONG64 BaseOfDll, PCSTR Mask, { if (pair.requested->type == DMT_PE && module_get_debug(&pair)) { - if (regexec(&mod_regex, pair.requested->module_name, 0, NULL, 0) == 0 && + if (match_regexp(&mod_regex, pair.requested->module_name) && symt_enum_module(&pair, &sym_regex, se)) break; } @@ -927,7 +966,7 @@ static BOOL sym_enum(HANDLE hProcess, ULONG64 BaseOfDll, PCSTR Mask, !module_get_containee(pair.pcs, pair.requested) && module_get_debug(&pair)) { - if (regexec(&mod_regex, pair.requested->module_name, 0, NULL, 0) == 0 && + if (match_regexp(&mod_regex, pair.requested->module_name) && symt_enum_module(&pair, &sym_regex, se)) break; } @@ -1641,7 +1680,7 @@ BOOL WINAPI SymMatchString(PCSTR string, PCSTR re, BOOL _case) TRACE("%s %s %c\n", string, re, _case ? 'Y' : 'N'); compile_regex(re, -1, &preg, _case); - ret = regexec(&preg, string, 0, NULL, 0) == 0; + ret = match_regexp(&preg, string); regfree(&preg); return ret; }

1 0