wine-commits December 2019

wine-commits@winehq.org

1 participants
635 discussions

Michael Stefaniuc : dmband: Add partial implementation of Band track GetParam/SetParam.
by Alexandre Julliard 10 Dec '19

10 Dec '19

Module: wine Branch: master Commit: d191c5ca181c1abb908dddc1c06ee291a8fee505 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d191c5ca181c1abb908dddc1… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Tue Dec 10 00:15:12 2019 +0100 dmband: Add partial implementation of Band track GetParam/SetParam. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dmband/bandtrack.c | 57 ++++++++++++++++++++++++++++++++++++++-------- dlls/dmband/tests/dmband.c | 22 ++++++++++++++---- 2 files changed, 65 insertions(+), 14 deletions(-) diff --git a/dlls/dmband/bandtrack.c b/dlls/dmband/bandtrack.c index a0305bdb28..14d2c7a571 100644 --- a/dlls/dmband/bandtrack.c +++ b/dlls/dmband/bandtrack.c @@ -132,20 +132,57 @@ static HRESULT WINAPI band_track_Play(IDirectMusicTrack8 *iface, void *state_dat return S_OK; } -static HRESULT WINAPI band_track_GetParam(IDirectMusicTrack8 *iface, REFGUID rguidType, - MUSIC_TIME mtTime, MUSIC_TIME *pmtNext, void *pParam) +static HRESULT WINAPI band_track_GetParam(IDirectMusicTrack8 *iface, REFGUID type, MUSIC_TIME time, + MUSIC_TIME *next, void *param) { - IDirectMusicBandTrack *This = impl_from_IDirectMusicTrack8(iface); - FIXME("(%p, %s, %d, %p, %p): stub\n", This, debugstr_dmguid(rguidType), mtTime, pmtNext, pParam); - return S_OK; + IDirectMusicBandTrack *This = impl_from_IDirectMusicTrack8(iface); + + TRACE("(%p, %s, %d, %p, %p)\n", This, debugstr_dmguid(type), time, next, param); + + if (!type) + return E_POINTER; + if (!IsEqualGUID(type, &GUID_BandParam)) + return DMUS_E_GET_UNSUPPORTED; + + FIXME("GUID_BandParam not handled yet\n"); + + return S_OK; } -static HRESULT WINAPI band_track_SetParam(IDirectMusicTrack8 *iface, REFGUID rguidType, - MUSIC_TIME mtTime, void *pParam) +static HRESULT WINAPI band_track_SetParam(IDirectMusicTrack8 *iface, REFGUID type, MUSIC_TIME time, + void *param) { - IDirectMusicBandTrack *This = impl_from_IDirectMusicTrack8(iface); - FIXME("(%p, %s, %d, %p): stub\n", This, debugstr_dmguid(rguidType), mtTime, pParam); - return S_OK; + IDirectMusicBandTrack *This = impl_from_IDirectMusicTrack8(iface); + + TRACE("(%p, %s, %d, %p)\n", This, debugstr_dmguid(type), time, param); + + if (!type) + return E_POINTER; + if (FAILED(IDirectMusicTrack8_IsParamSupported(iface, type))) + return DMUS_E_TYPE_UNSUPPORTED; + + if (IsEqualGUID(type, &GUID_BandParam)) + FIXME("GUID_BandParam not handled yet\n"); + else if (IsEqualGUID(type, &GUID_Clear_All_Bands)) + FIXME("GUID_Clear_All_Bands not handled yet\n"); + else if (IsEqualGUID(type, &GUID_ConnectToDLSCollection)) + FIXME("GUID_ConnectToDLSCollection not handled yet\n"); + else if (IsEqualGUID(type, &GUID_Disable_Auto_Download)) + FIXME("GUID_Disable_Auto_Download not handled yet\n"); + else if (IsEqualGUID(type, &GUID_Download)) + FIXME("GUID_Download not handled yet\n"); + else if (IsEqualGUID(type, &GUID_DownloadToAudioPath)) + FIXME("GUID_DownloadToAudioPath not handled yet\n"); + else if (IsEqualGUID(type, &GUID_Enable_Auto_Download)) + FIXME("GUID_Enable_Auto_Download not handled yet\n"); + else if (IsEqualGUID(type, &GUID_IDirectMusicBand)) + FIXME("GUID_IDirectMusicBand not handled yet\n"); + else if (IsEqualGUID(type, &GUID_StandardMIDIFile)) + FIXME("GUID_StandardMIDIFile not handled yet\n"); + else if (IsEqualGUID(type, &GUID_UnloadFromAudioPath)) + FIXME("GUID_UnloadFromAudioPath not handled yet\n"); + + return S_OK; } static HRESULT WINAPI band_track_IsParamSupported(IDirectMusicTrack8 *iface, REFGUID rguidType) diff --git a/dlls/dmband/tests/dmband.c b/dlls/dmband/tests/dmband.c index 459d00ed1d..dce8a47ba4 100644 --- a/dlls/dmband/tests/dmband.c +++ b/dlls/dmband/tests/dmband.c @@ -180,6 +180,7 @@ static void test_bandtrack(void) IPersistStream *ps; CLSID class; ULARGE_INTEGER size; + char buf[64] = { 0 }; HRESULT hr; #define X(guid) &guid, #guid const struct { @@ -233,26 +234,39 @@ static void test_bandtrack(void) } hr = IDirectMusicTrack8_EndPlay(dmt8, NULL); ok(hr == S_OK, "IDirectMusicTrack8_EndPlay failed: %08x\n", hr); - todo_wine { hr = IDirectMusicTrack8_Play(dmt8, NULL, 0, 0, 0, 0, NULL, NULL, 0); + todo_wine ok(hr == DMUS_S_END, "IDirectMusicTrack8_Play failed: %08x\n", hr); hr = IDirectMusicTrack8_GetParam(dmt8, NULL, 0, NULL, NULL); ok(hr == E_POINTER, "IDirectMusicTrack8_GetParam failed: %08x\n", hr); hr = IDirectMusicTrack8_SetParam(dmt8, NULL, 0, NULL); ok(hr == E_POINTER, "IDirectMusicTrack8_SetParam failed: %08x\n", hr); - } hr = IDirectMusicTrack8_IsParamSupported(dmt8, NULL); ok(hr == E_POINTER, "IDirectMusicTrack8_IsParamSupported failed: %08x\n", hr); for (i = 0; i < ARRAY_SIZE(param_types); i++) { hr = IDirectMusicTrack8_IsParamSupported(dmt8, param_types[i].type); - if (param_types[i].supported) + if (param_types[i].supported) { ok(hr == S_OK, "IsParamSupported(%s) failed: %08x, expected S_OK\n", param_types[i].name, hr); - else + hr = IDirectMusicTrack8_GetParam(dmt8, param_types[i].type, 0, NULL, buf); + if (param_types[i].type != &GUID_BandParam) + ok(hr == DMUS_E_GET_UNSUPPORTED, + "GetParam(%s) failed: %08x, expected DMUS_E_GET_UNSUPPORTED\n", + param_types[i].name, hr); + } else { ok(hr == DMUS_E_TYPE_UNSUPPORTED, "IsParamSupported(%s) failed: %08x, expected DMUS_E_TYPE_UNSUPPORTED\n", param_types[i].name, hr); + hr = IDirectMusicTrack8_GetParam(dmt8, param_types[i].type, 0, NULL, buf); + ok(hr == DMUS_E_GET_UNSUPPORTED, + "GetParam(%s) failed: %08x, expected DMUS_E_GET_UNSUPPORTED\n", + param_types[i].name, hr); + hr = IDirectMusicTrack8_SetParam(dmt8, param_types[i].type, 0, buf); + ok(hr == DMUS_E_TYPE_UNSUPPORTED, + "SetParam(%s) failed: %08x, expected DMUS_E_TYPE_UNSUPPORTED\n", + param_types[i].name, hr); + } } hr = IDirectMusicTrack8_AddNotificationType(dmt8, NULL);

1 0

Patrick Hibbs : wtsapi32: Make WTSRegisterSessionNotificationEx's return consistent with WTSRegisterSessionNotification.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: f870aa4988fa8e980919c144682da72865364ff0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f870aa4988fa8e980919c144… Author: Patrick Hibbs <hibbsncc1701(a)gmail.com> Date: Sat Dec 7 03:06:14 2019 -0500 wtsapi32: Make WTSRegisterSessionNotificationEx's return consistent with WTSRegisterSessionNotification. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=47433 Signed-off-by: Patrick Hibbs <hibbsncc1701(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wtsapi32/wtsapi32.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wtsapi32/wtsapi32.c b/dlls/wtsapi32/wtsapi32.c index c2b817a6ed..026e7f4369 100644 --- a/dlls/wtsapi32/wtsapi32.c +++ b/dlls/wtsapi32/wtsapi32.c @@ -385,7 +385,7 @@ BOOL WINAPI WTSRegisterSessionNotification(HWND hWnd, DWORD dwFlags) BOOL WINAPI WTSRegisterSessionNotificationEx(HANDLE hServer, HWND hWnd, DWORD dwFlags) { FIXME("Stub %p %p 0x%08x\n", hServer, hWnd, dwFlags); - return FALSE; + return TRUE; }

1 0

Nikolay Sivov : comctl32/listview: Use wcsnicmp() instead of a local helper.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: 88a3a5842ad82b443e5aa084441c51f5a5380e9d URL: https://source.winehq.org/git/wine.git/?a=commit;h=88a3a5842ad82b443e5aa084… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Dec 9 15:42:45 2019 +0300 comctl32/listview: Use wcsnicmp() instead of a local helper. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comctl32/listview.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index eb42526f57..2dd25908db 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -539,12 +539,6 @@ static inline int textcmpWT(LPCWSTR aw, LPCWSTR bt, BOOL isW) return 1; } - -static inline int lstrncmpiW(LPCWSTR s1, LPCWSTR s2, int n) -{ - n = min(min(n, lstrlenW(s1)), lstrlenW(s2)); - return CompareStringW(LOCALE_USER_DEFAULT, NORM_IGNORECASE, s1, n, s2, n) - CSTR_EQUAL; -} /******** Debugging functions *****************************************/ @@ -1953,7 +1947,7 @@ static INT LISTVIEW_ProcessLetterKeys(LISTVIEW_INFO *infoPtr, WPARAM charCode, L item.cchTextMax = MAX_PATH; if (!LISTVIEW_GetItemW(infoPtr, &item)) return 0; - if (!lstrncmpiW(item.pszText, infoPtr->szSearchParam, infoPtr->nSearchParamLength)) + if (!wcsnicmp(item.pszText, infoPtr->szSearchParam, infoPtr->nSearchParamLength)) { nItem = i; break; @@ -1961,7 +1955,7 @@ static INT LISTVIEW_ProcessLetterKeys(LISTVIEW_INFO *infoPtr, WPARAM charCode, L /* this is used to find first char match when search string is not available yet, otherwise every WM_CHAR will search to next item by first char, ignoring that we're already waiting for user to complete a string */ - else if (nItem == -1 && infoPtr->nSearchParamLength == 1 && !lstrncmpiW(item.pszText, infoPtr->szSearchParam, 1)) + else if (nItem == -1 && infoPtr->nSearchParamLength == 1 && !wcsnicmp(item.pszText, infoPtr->szSearchParam, 1)) { /* this would work but we must keep looking for a longer match */ nItem = i;

1 0

Jacek Caban : mshtml: Support loading Gecko from global location.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: 077fd3891a57a3d460688ce20d1b8db50c1ac752 URL: https://source.winehq.org/git/wine.git/?a=commit;h=077fd3891a57a3d460688ce2… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 9 17:04:37 2019 +0100 mshtml: Support loading Gecko from global location. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/Makefile.in | 2 ++ dlls/mshtml/nsembed.c | 70 +++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 72 insertions(+) diff --git a/dlls/mshtml/Makefile.in b/dlls/mshtml/Makefile.in index 7e1d517f07..469667081e 100644 --- a/dlls/mshtml/Makefile.in +++ b/dlls/mshtml/Makefile.in @@ -65,6 +65,8 @@ C_SRCS = \ view.c \ xmlhttprequest.c +nsembed_EXTRADEFS = -DINSTALL_DATADIR="\"${datadir}\"" + RC_SRCS = mshtml.rc IDL_SRCS = \ diff --git a/dlls/mshtml/nsembed.c b/dlls/mshtml/nsembed.c index f47f727630..2122536fc5 100644 --- a/dlls/mshtml/nsembed.c +++ b/dlls/mshtml/nsembed.c @@ -54,6 +54,16 @@ WINE_DECLARE_DEBUG_CHANNEL(gecko); #define NS_STRING_CONTAINER_INIT_DEPEND 0x0002 #define NS_CSTRING_CONTAINER_INIT_DEPEND 0x0002 +#ifdef __i386__ +#define GECKO_ARCH_STRING "x86" +#elif defined(__x86_64__) +#define GECKO_ARCH_STRING "x86_64" +#else +#define GECKO_ARCH_STRING "" +#endif + +#define GECKO_DIR_NAME "wine-gecko-" GECKO_VERSION "-" GECKO_ARCH_STRING + typedef UINT32 PRUint32; static nsresult (CDECL *NS_InitXPCOM2)(nsIServiceManager**,void*,void*); @@ -730,6 +740,56 @@ static WCHAR *find_wine_gecko_reg(void) return check_version(buffer); } +static WCHAR *heap_strcat(const WCHAR *str1, const WCHAR *str2) +{ + size_t len1 = lstrlenW(str1); + size_t len2 = lstrlenW(str2); + WCHAR *ret = heap_alloc((len1 + len2 + 1) * sizeof(WCHAR)); + if(!ret) return NULL; + memcpy(ret, str1, len1 * sizeof(WCHAR)); + memcpy(ret + len1, str2, len2 * sizeof(WCHAR)); + ret[len1 + len2] = 0; + return ret; +} + +static WCHAR *find_wine_gecko_datadir(void) +{ + const WCHAR *data_dir; + WCHAR *path = NULL, *ret; + + if((data_dir = _wgetenv(L"WINEDATADIR"))) + path = heap_strcat(data_dir, L"\\gecko\\" GECKO_DIR_NAME); + else if((data_dir = _wgetenv(L"WINEBUILDDIR"))) + path = heap_strcat(data_dir, L"\\..\\gecko\\" GECKO_DIR_NAME); + if(!path) + return NULL; + + ret = check_version(path); + heap_free(path); + return ret; +} + +static WCHAR *find_wine_gecko_unix(const char *unix_path) +{ + static WCHAR * (CDECL *p_wine_get_dos_file_name)(const char*); + WCHAR *dos_dir, *ret; + + if(!p_wine_get_dos_file_name) { + p_wine_get_dos_file_name = (void*)GetProcAddress(GetModuleHandleA("kernel32"), "wine_get_dos_file_name"); + if(!p_wine_get_dos_file_name) + return FALSE; + } + + dos_dir = p_wine_get_dos_file_name(unix_path); + if(!dos_dir) + return FALSE; + + ret = check_version(dos_dir); + + heap_free(dos_dir); + return ret; +} + static CRITICAL_SECTION cs_load_gecko; static CRITICAL_SECTION_DEBUG cs_load_gecko_dbg = { @@ -747,6 +807,11 @@ BOOL load_gecko(void) TRACE("()\n"); + if(!GECKO_ARCH_STRING[0]) { + FIXME("Wine Gecko is not supported on this architecture.\n"); + return FALSE; + } + /* load_gecko may be called recursively */ if(loading_thread == GetCurrentThreadId()) return pCompMgr != NULL; @@ -759,6 +824,11 @@ BOOL load_gecko(void) loading_thread = GetCurrentThreadId(); if(!(gecko_path = find_wine_gecko_reg()) + && !(gecko_path = find_wine_gecko_datadir()) + && !(gecko_path = find_wine_gecko_unix(INSTALL_DATADIR "/wine/gecko/" GECKO_DIR_NAME)) + && (!strcmp(INSTALL_DATADIR, "/usr/share") || + !(gecko_path = find_wine_gecko_unix("/usr/share/wine/gecko/" GECKO_DIR_NAME))) + && !(gecko_path = find_wine_gecko_unix("/opt/wine/gecko/" GECKO_DIR_NAME)) && install_wine_gecko()) gecko_path = find_wine_gecko_reg();

1 0

Jacek Caban : mshtml: Refactor Gecko loading code.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: ba462a06609b106ecaadce48f3c2544f8f81a9d7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ba462a06609b106ecaadce48… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 9 17:04:22 2019 +0100 mshtml: Refactor Gecko loading code. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/nsembed.c | 271 ++++++++++++++++++++++++-------------------------- 1 file changed, 132 insertions(+), 139 deletions(-) diff --git a/dlls/mshtml/nsembed.c b/dlls/mshtml/nsembed.c index 244341248e..f47f727630 100644 --- a/dlls/mshtml/nsembed.c +++ b/dlls/mshtml/nsembed.c @@ -432,6 +432,8 @@ static BOOL install_wine_gecko(void) CloseHandle(pi.hThread); WaitForSingleObject(pi.hProcess, INFINITE); CloseHandle(pi.hProcess); + }else { + WARN("installation failed\n"); } return ret; @@ -483,139 +485,6 @@ static void set_environment(LPCWSTR gre_path) heap_free(path); } -static BOOL load_xul(const PRUnichar *gre_path) -{ - static const WCHAR xul_dllW[] = {'\\','x','u','l','.','d','l','l',0}; - WCHAR file_name[MAX_PATH]; - - lstrcpyW(file_name, gre_path); - lstrcatW(file_name, xul_dllW); - - TRACE("(%s)\n", debugstr_w(file_name)); - - set_environment(gre_path); - - xul_handle = LoadLibraryExW(file_name, 0, LOAD_LIBRARY_SEARCH_DLL_LOAD_DIR); - if(!xul_handle) { - WARN("Could not load XUL: %d\n", GetLastError()); - return FALSE; - } - -#define NS_DLSYM(func) \ - func = (void *)GetProcAddress(xul_handle, #func); \ - if(!func) \ - ERR("Could not GetProcAddress(" #func ") failed\n") - - NS_DLSYM(NS_InitXPCOM2); - NS_DLSYM(NS_ShutdownXPCOM); - NS_DLSYM(NS_GetComponentRegistrar); - NS_DLSYM(NS_StringContainerInit2); - NS_DLSYM(NS_CStringContainerInit2); - NS_DLSYM(NS_StringContainerFinish); - NS_DLSYM(NS_CStringContainerFinish); - NS_DLSYM(NS_StringSetData); - NS_DLSYM(NS_CStringSetData); - NS_DLSYM(NS_NewLocalFile); - NS_DLSYM(NS_StringGetData); - NS_DLSYM(NS_CStringGetData); - NS_DLSYM(NS_StringGetIsVoid); - NS_DLSYM(NS_Alloc); - NS_DLSYM(NS_Free); - NS_DLSYM(ccref_incr); - NS_DLSYM(ccref_decr); - NS_DLSYM(ccref_init); - NS_DLSYM(ccp_init); - NS_DLSYM(describe_cc_node); - NS_DLSYM(note_cc_edge); - -#undef NS_DLSYM - - return TRUE; -} - -static BOOL check_version(LPCWSTR gre_path, const char *version_string) -{ - WCHAR file_name[MAX_PATH]; - char version[128]; - DWORD read=0; - HANDLE hfile; - - static const WCHAR wszVersion[] = {'\\','V','E','R','S','I','O','N',0}; - - lstrcpyW(file_name, gre_path); - lstrcatW(file_name, wszVersion); - - hfile = CreateFileW(file_name, GENERIC_READ, FILE_SHARE_READ, NULL, - OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); - if(hfile == INVALID_HANDLE_VALUE) { - ERR("Could not open VERSION file\n"); - return FALSE; - } - - ReadFile(hfile, version, sizeof(version), &read, NULL); - version[read] = 0; - CloseHandle(hfile); - - TRACE("%s\n", debugstr_a(version)); - - if(strcmp(version, version_string)) { - ERR("Unexpected version %s, expected %s\n", debugstr_a(version), - debugstr_a(version_string)); - return FALSE; - } - - return TRUE; -} - -static BOOL load_wine_gecko_v(PRUnichar *gre_path, HKEY mshtml_key, - const char *version, const char *version_string) -{ - DWORD res, type, size = MAX_PATH; - HKEY hkey = mshtml_key; - - static const WCHAR wszGeckoPath[] = - {'G','e','c','k','o','P','a','t','h',0}; - - if(version) { - /* @@ Wine registry key: HKLM\Software\Wine\MSHTML\<version> */ - res = RegOpenKeyA(mshtml_key, version, &hkey); - if(res != ERROR_SUCCESS) - return FALSE; - } - - res = RegQueryValueExW(hkey, wszGeckoPath, NULL, &type, (LPBYTE)gre_path, &size); - if(hkey != mshtml_key) - RegCloseKey(hkey); - if(res != ERROR_SUCCESS || type != REG_SZ) - return FALSE; - - if(!check_version(gre_path, version_string)) - return FALSE; - - return load_xul(gre_path); -} - -static BOOL load_wine_gecko(PRUnichar *gre_path) -{ - HKEY hkey; - DWORD res; - BOOL ret; - - static const WCHAR wszMshtmlKey[] = { - 'S','o','f','t','w','a','r','e','\\','W','i','n','e', - '\\','M','S','H','T','M','L',0}; - - /* @@ Wine registry key: HKLM\Software\Wine\MSHTML */ - res = RegOpenKeyW(HKEY_LOCAL_MACHINE, wszMshtmlKey, &hkey); - if(res != ERROR_SUCCESS) - return FALSE; - - ret = load_wine_gecko_v(gre_path, hkey, GECKO_VERSION, GECKO_VERSION_STRING); - - RegCloseKey(hkey); - return ret; -} - static void set_bool_pref(nsIPrefBranch *pref, const char *pref_name, BOOL val) { nsresult nsres; @@ -743,6 +612,124 @@ static BOOL init_xpcom(const PRUnichar *gre_path) return TRUE; } +static BOOL load_xul(WCHAR *gecko_path) +{ + size_t len; + + set_environment(gecko_path); + + TRACE("(%s)\n", debugstr_w(gecko_path)); + + len = wcslen(gecko_path); + wcscpy(gecko_path + len, L"\\xul.dll"); + xul_handle = LoadLibraryExW(gecko_path, 0, LOAD_LIBRARY_SEARCH_DLL_LOAD_DIR); + gecko_path[len] = 0; + if(!xul_handle) { + WARN("Could not load XUL: %d\n", GetLastError()); + return FALSE; + } + +#define NS_DLSYM(func) \ + func = (void *)GetProcAddress(xul_handle, #func); \ + if(!func) \ + ERR("Could not GetProcAddress(" #func ") failed\n") + + NS_DLSYM(NS_InitXPCOM2); + NS_DLSYM(NS_ShutdownXPCOM); + NS_DLSYM(NS_GetComponentRegistrar); + NS_DLSYM(NS_StringContainerInit2); + NS_DLSYM(NS_CStringContainerInit2); + NS_DLSYM(NS_StringContainerFinish); + NS_DLSYM(NS_CStringContainerFinish); + NS_DLSYM(NS_StringSetData); + NS_DLSYM(NS_CStringSetData); + NS_DLSYM(NS_NewLocalFile); + NS_DLSYM(NS_StringGetData); + NS_DLSYM(NS_CStringGetData); + NS_DLSYM(NS_StringGetIsVoid); + NS_DLSYM(NS_Alloc); + NS_DLSYM(NS_Free); + NS_DLSYM(ccref_incr); + NS_DLSYM(ccref_decr); + NS_DLSYM(ccref_init); + NS_DLSYM(ccp_init); + NS_DLSYM(describe_cc_node); + NS_DLSYM(note_cc_edge); + +#undef NS_DLSYM + + return init_xpcom(gecko_path); +} + +static WCHAR *check_version(const WCHAR *path) +{ + WCHAR *file_name; + char version[128]; + DWORD read=0; + size_t len; + HANDLE hfile; + + if(!wcsncmp(path, L"\\??\\", 4)) + path += 4; + if(path[1] != ':') { + TRACE("Skipping %s\n", debugstr_w(path)); + return FALSE; /* Gecko needs to be accessible via dos path */ + } + + len = wcslen(path); + file_name = heap_alloc((len + 12) * sizeof(WCHAR)); + if(!file_name) + return NULL; + + PathCanonicalizeW(file_name, path); + len = lstrlenW(file_name); + wcscpy(file_name + len, L"\\VERSION"); + + hfile = CreateFileW(file_name, GENERIC_READ, FILE_SHARE_READ, NULL, + OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); + file_name[len] = 0; + if(hfile == INVALID_HANDLE_VALUE) { + TRACE("%s not found\n", debugstr_w(file_name)); + heap_free(file_name); + return NULL; + } + + ReadFile(hfile, version, sizeof(version), &read, NULL); + version[read] = 0; + CloseHandle(hfile); + + TRACE("%s: %s\n", debugstr_w(file_name), debugstr_a(version)); + + if(strcmp(version, GECKO_VERSION_STRING)) { + ERR("Unexpected version %s, expected \"%s\"\n", debugstr_a(version), + GECKO_VERSION_STRING); + heap_free(file_name); + return NULL; + } + + return file_name; +} + +static WCHAR *find_wine_gecko_reg(void) +{ + WCHAR buffer[MAX_PATH]; + DWORD res, type, size; + HKEY hkey; + + /* @@ Wine registry key: HKLM\Software\Wine\MSHTML\<version> */ + res = RegOpenKeyW(HKEY_LOCAL_MACHINE, L"Software\\Wine\\MSHTML\\" GECKO_VERSION, &hkey); + if(res != ERROR_SUCCESS) + return NULL; + + size = ARRAY_SIZE(buffer); + res = RegQueryValueExW(hkey, L"GeckoPath", NULL, &type, (LPBYTE)buffer, &size); + RegCloseKey(hkey); + if(res != ERROR_SUCCESS || type != REG_SZ) + return FALSE; + + return check_version(buffer); +} + static CRITICAL_SECTION cs_load_gecko; static CRITICAL_SECTION_DEBUG cs_load_gecko_dbg = { @@ -754,7 +741,6 @@ static CRITICAL_SECTION cs_load_gecko = { &cs_load_gecko_dbg, -1, 0, 0, 0, 0 }; BOOL load_gecko(void) { - PRUnichar gre_path[MAX_PATH]; BOOL ret = FALSE; static DWORD loading_thread; @@ -768,13 +754,20 @@ BOOL load_gecko(void) EnterCriticalSection(&cs_load_gecko); if(!loading_thread) { + WCHAR *gecko_path; + loading_thread = GetCurrentThreadId(); - if(load_wine_gecko(gre_path) - || (install_wine_gecko() && load_wine_gecko(gre_path))) - ret = init_xpcom(gre_path); - else - MESSAGE("Could not load wine-gecko. HTML rendering will be disabled.\n"); + if(!(gecko_path = find_wine_gecko_reg()) + && install_wine_gecko()) + gecko_path = find_wine_gecko_reg(); + + if(gecko_path) { + ret = load_xul(gecko_path); + heap_free(gecko_path); + }else { + MESSAGE("Could not find Wine Gecko. HTML rendering will be disabled.\n"); + } }else { ret = pCompMgr != NULL; }

1 0

Gabriel Ivăncescu : vbscript/tests: Add tests for the script TypeInfo's TypeComp binds.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: d393681e41b59b475d1e91e6f7da7c8243105037 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d393681e41b59b475d1e91e6… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Mon Dec 9 17:29:26 2019 +0200 vbscript/tests: Add tests for the script TypeInfo's TypeComp binds. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vbscript/tests/vbscript.c | 59 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 59 insertions(+) diff --git a/dlls/vbscript/tests/vbscript.c b/dlls/vbscript/tests/vbscript.c index 1d246c50b4..ad3ccf37a0 100644 --- a/dlls/vbscript/tests/vbscript.c +++ b/dlls/vbscript/tests/vbscript.c @@ -1077,6 +1077,65 @@ static void test_script_typeinfo(void) ok(hr == E_INVALIDARG, "Bind returned: %08x\n", hr); hr = ITypeComp_Bind(typecomp, str, 0, 0, &typeinfo2, &desckind, NULL); ok(hr == E_INVALIDARG, "Bind returned: %08x\n", hr); + + hr = ITypeComp_Bind(typecomp, str, 0, 0, &typeinfo2, &desckind, &bindptr); + ok(hr == S_OK, "Bind failed: %08x\n", hr); + ok(desckind == DESCKIND_NONE, "Unexpected desckind %u\n", desckind); + wcscpy(str, L"GLOBAL_VAR"); + hr = ITypeComp_Bind(typecomp, str, 0, INVOKE_FUNC, &typeinfo2, &desckind, &bindptr); + ok(hr == TYPE_E_TYPEMISMATCH, "Bind returned: %08x\n", hr); + ok(!lstrcmpW(str, L"GLOBAL_VAR"), "Unexpected string %s\n", wine_dbgstr_w(str)); + wcscpy(str, L"C"); + hr = ITypeComp_Bind(typecomp, str, 0, 0, &typeinfo2, &desckind, &bindptr); + ok(hr == S_OK, "Bind failed: %08x\n", hr); + ok(desckind == DESCKIND_NONE, "Unexpected desckind %u\n", desckind); + wcscpy(str, L"addRef"); + hr = ITypeComp_Bind(typecomp, str, 0, 0, &typeinfo2, &desckind, &bindptr); + ok(hr == S_OK, "Bind failed: %08x\n", hr); + ok(desckind == DESCKIND_FUNCDESC, "Unexpected desckind %u\n", desckind); + ok(!lstrcmpW(str, L"addRef"), "Unexpected string %s\n", wine_dbgstr_w(str)); + ITypeInfo_ReleaseFuncDesc(typeinfo2, bindptr.lpfuncdesc); + ITypeInfo_Release(typeinfo2); + for (i = 0; i < ARRAY_SIZE(var); i++) + { + wcscpy(str, var[i].name); + hr = ITypeComp_Bind(typecomp, str, 0, INVOKE_PROPERTYGET, &typeinfo2, &desckind, &bindptr); + ok(hr == S_OK, "Bind failed: %08x\n", hr); + ok(desckind == DESCKIND_VARDESC, "Unexpected desckind %u\n", desckind); + ITypeInfo_ReleaseVarDesc(typeinfo2, bindptr.lpvardesc); + ITypeInfo_Release(typeinfo2); + } + for (i = 0; i < ARRAY_SIZE(func); i++) + { + wcscpy(str, func[i].name); + hr = ITypeComp_Bind(typecomp, str, 0, INVOKE_FUNC, &typeinfo2, &desckind, &bindptr); + ok(hr == S_OK, "Bind failed: %08x\n", hr); + ok(desckind == DESCKIND_FUNCDESC, "Unexpected desckind %u\n", desckind); + ITypeInfo_ReleaseFuncDesc(typeinfo2, bindptr.lpfuncdesc); + ITypeInfo_Release(typeinfo2); + } + wcscpy(str, L"VBScriptTypeInfo"); + hr = ITypeComp_BindType(typecomp, NULL, 0, &typeinfo2, &typecomp2); + ok(hr == E_INVALIDARG, "BindType returned: %08x\n", hr); + hr = ITypeComp_BindType(typecomp, str, 0, NULL, &typecomp2); + ok(hr == E_INVALIDARG, "BindType returned: %08x\n", hr); + hr = ITypeComp_BindType(typecomp, str, 0, &typeinfo2, NULL); + ok(hr == E_INVALIDARG, "BindType returned: %08x\n", hr); + hr = ITypeComp_BindType(typecomp, str, 0, &typeinfo2, &typecomp2); + ok(hr == S_OK, "BindType failed: %08x\n", hr); + ok(!typeinfo2, "Unexpected TypeInfo %p (expected null)\n", typeinfo2); + ok(!typecomp2, "Unexpected TypeComp %p (expected null)\n", typecomp2); + wcscpy(str, L"C"); + hr = ITypeComp_BindType(typecomp, str, 0, &typeinfo2, &typecomp2); + ok(hr == S_OK, "BindType failed: %08x\n", hr); + ok(!typeinfo2, "Unexpected TypeInfo %p (expected null)\n", typeinfo2); + ok(!typecomp2, "Unexpected TypeComp %p (expected null)\n", typecomp2); + wcscpy(str, L"IDispatch"); + hr = ITypeComp_BindType(typecomp, str, 0, &typeinfo2, &typecomp2); + ok(hr == S_OK, "BindType failed: %08x\n", hr); + ok(!typeinfo2, "Unexpected TypeInfo %p (expected null)\n", typeinfo2); + ok(!typecomp2, "Unexpected TypeComp %p (expected null)\n", typecomp2); + ITypeComp_Release(typecomp); /* Updating the script won't update the typeinfo obtained before,

1 0

Gabriel Ivăncescu : vbscript: Implement ScriptTypeComp_BindType.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: 422c66a671c444ce5ece6b15c445287a0e720ba8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=422c66a671c444ce5ece6b15… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Mon Dec 9 17:29:25 2019 +0200 vbscript: Implement ScriptTypeComp_BindType. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vbscript/vbdisp.c | 21 +++++++++++++++++++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/dlls/vbscript/vbdisp.c b/dlls/vbscript/vbdisp.c index 2f68c16567..6dc34b32d9 100644 --- a/dlls/vbscript/vbdisp.c +++ b/dlls/vbscript/vbdisp.c @@ -988,8 +988,25 @@ static HRESULT WINAPI ScriptTypeComp_BindType(ITypeComp *iface, LPOLESTR szName, ITypeInfo **ppTInfo, ITypeComp **ppTComp) { ScriptTypeInfo *This = ScriptTypeInfo_from_ITypeComp(iface); - FIXME("(%p)->(%s %08x %p %p)\n", This, debugstr_w(szName), lHashVal, ppTInfo, ppTComp); - return E_NOTIMPL; + ITypeInfo *disp_typeinfo; + ITypeComp *disp_typecomp; + HRESULT hr; + + TRACE("(%p)->(%s %08x %p %p)\n", This, debugstr_w(szName), lHashVal, ppTInfo, ppTComp); + + if (!szName || !ppTInfo || !ppTComp) + return E_INVALIDARG; + + /* Look into the inherited IDispatch */ + hr = get_dispatch_typeinfo(&disp_typeinfo); + if (FAILED(hr)) return hr; + + hr = ITypeInfo_GetTypeComp(disp_typeinfo, &disp_typecomp); + if (FAILED(hr)) return hr; + + hr = ITypeComp_BindType(disp_typecomp, szName, lHashVal, ppTInfo, ppTComp); + ITypeComp_Release(disp_typecomp); + return hr; } static const ITypeCompVtbl ScriptTypeCompVtbl = {

1 0

Gabriel Ivăncescu : vbscript: Don't free procs allocated in the heap pool.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: c47dfe45c73379b5dd6052596f7bca31d527a87d URL: https://source.winehq.org/git/wine.git/?a=commit;h=c47dfe45c73379b5dd605259… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Mon Dec 9 17:27:56 2019 +0200 vbscript: Don't free procs allocated in the heap pool. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vbscript/vbdisp.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/dlls/vbscript/vbdisp.c b/dlls/vbscript/vbdisp.c index 2e5905120e..47ecc15be4 100644 --- a/dlls/vbscript/vbdisp.c +++ b/dlls/vbscript/vbdisp.c @@ -1002,13 +1002,6 @@ static ULONG WINAPI ScriptDisp_Release(IDispatchEx *iface) if(!ref) { assert(!This->ctx); - while (This->procs) - { - class_desc_t *class_desc = This->procs; - This->procs = class_desc->next; - heap_free(class_desc); - } - for (i = 0; i < This->global_vars_cnt; i++) release_dynamic_var(This->global_vars[i]);

1 0

Gabriel Ivăncescu : vbscript: Implement ScriptTypeComp_Bind.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: 87075726696ce6b642807b18221174418a24d83a URL: https://source.winehq.org/git/wine.git/?a=commit;h=87075726696ce6b642807b18… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Mon Dec 9 17:29:24 2019 +0200 vbscript: Implement ScriptTypeComp_Bind. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vbscript/tests/vbscript.c | 4 ---- dlls/vbscript/vbdisp.c | 51 ++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 49 insertions(+), 6 deletions(-) diff --git a/dlls/vbscript/tests/vbscript.c b/dlls/vbscript/tests/vbscript.c index 0c3acd0ca7..1d246c50b4 100644 --- a/dlls/vbscript/tests/vbscript.c +++ b/dlls/vbscript/tests/vbscript.c @@ -1070,16 +1070,12 @@ static void test_script_typeinfo(void) ITypeComp_Release(typecomp2); wcscpy(str, L"not_found"); hr = ITypeComp_Bind(typecomp, NULL, 0, 0, &typeinfo2, &desckind, &bindptr); - todo_wine ok(hr == E_INVALIDARG, "Bind returned: %08x\n", hr); hr = ITypeComp_Bind(typecomp, str, 0, 0, NULL, &desckind, &bindptr); - todo_wine ok(hr == E_INVALIDARG, "Bind returned: %08x\n", hr); hr = ITypeComp_Bind(typecomp, str, 0, 0, &typeinfo2, NULL, &bindptr); - todo_wine ok(hr == E_INVALIDARG, "Bind returned: %08x\n", hr); hr = ITypeComp_Bind(typecomp, str, 0, 0, &typeinfo2, &desckind, NULL); - todo_wine ok(hr == E_INVALIDARG, "Bind returned: %08x\n", hr); ITypeComp_Release(typecomp); diff --git a/dlls/vbscript/vbdisp.c b/dlls/vbscript/vbdisp.c index 47ecc15be4..2f68c16567 100644 --- a/dlls/vbscript/vbdisp.c +++ b/dlls/vbscript/vbdisp.c @@ -932,9 +932,56 @@ static HRESULT WINAPI ScriptTypeComp_Bind(ITypeComp *iface, LPOLESTR szName, ULO ITypeInfo **ppTInfo, DESCKIND *pDescKind, BINDPTR *pBindPtr) { ScriptTypeInfo *This = ScriptTypeInfo_from_ITypeComp(iface); - FIXME("(%p)->(%s %08x %d %p %p %p)\n", This, debugstr_w(szName), lHashVal, + UINT flags = wFlags ? wFlags : ~0; + ITypeInfo *disp_typeinfo; + ITypeComp *disp_typecomp; + HRESULT hr; + UINT i; + + TRACE("(%p)->(%s %08x %d %p %p %p)\n", This, debugstr_w(szName), lHashVal, wFlags, ppTInfo, pDescKind, pBindPtr); - return E_NOTIMPL; + + if (!szName || !ppTInfo || !pDescKind || !pBindPtr) + return E_INVALIDARG; + + for (i = 0; i < This->num_funcs; i++) + { + if (wcsicmp(szName, This->funcs[i].func->name)) continue; + if (!(flags & INVOKE_FUNC)) return TYPE_E_TYPEMISMATCH; + + hr = ITypeInfo_GetFuncDesc(&This->ITypeInfo_iface, i, &pBindPtr->lpfuncdesc); + if (FAILED(hr)) return hr; + + *pDescKind = DESCKIND_FUNCDESC; + *ppTInfo = &This->ITypeInfo_iface; + ITypeInfo_AddRef(*ppTInfo); + return S_OK; + } + + for (i = 0; i < This->num_vars; i++) + { + if (wcsicmp(szName, This->disp->global_vars[i]->name)) continue; + if (!(flags & INVOKE_PROPERTYGET)) return TYPE_E_TYPEMISMATCH; + + hr = ITypeInfo_GetVarDesc(&This->ITypeInfo_iface, i, &pBindPtr->lpvardesc); + if (FAILED(hr)) return hr; + + *pDescKind = DESCKIND_VARDESC; + *ppTInfo = &This->ITypeInfo_iface; + ITypeInfo_AddRef(*ppTInfo); + return S_OK; + } + + /* Look into the inherited IDispatch */ + hr = get_dispatch_typeinfo(&disp_typeinfo); + if (FAILED(hr)) return hr; + + hr = ITypeInfo_GetTypeComp(disp_typeinfo, &disp_typecomp); + if (FAILED(hr)) return hr; + + hr = ITypeComp_Bind(disp_typecomp, szName, lHashVal, wFlags, ppTInfo, pDescKind, pBindPtr); + ITypeComp_Release(disp_typecomp); + return hr; } static HRESULT WINAPI ScriptTypeComp_BindType(ITypeComp *iface, LPOLESTR szName, ULONG lHashVal,

1 0

Jacek Caban : msvcrt: Provide exp2 in importlib.
by Alexandre Julliard 09 Dec '19

09 Dec '19

Module: wine Branch: master Commit: bdf905d661cd3cd61fb2dbaa5ddd9e641f421ffb URL: https://source.winehq.org/git/wine.git/?a=commit;h=bdf905d661cd3cd61fb2dbaa… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Dec 6 14:48:11 2019 +0100 msvcrt: Provide exp2 in importlib. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/mathf.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dlls/msvcrt/mathf.c b/dlls/msvcrt/mathf.c index 9f27e64122..4aff6d7008 100644 --- a/dlls/msvcrt/mathf.c +++ b/dlls/msvcrt/mathf.c @@ -37,3 +37,7 @@ float sqrtf(float x) { return sqrt(x); } float floorf(float x) { return floor(x); } float ceilf(float x) { return ceil(x); } #endif + +#if _MSVCR_VER < 120 +double exp2(double x) { return pow(2.0, x); } +#endif

1 0

← Newer
1
...
34
35
36
37
38
39
40
...
64
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 December 2019