wine-commits February 2009

wine-commits@winehq.org

2 participants
965 discussions

Ricardo Filipe : setupapi: Make hinstance global for all dialogs.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 7ab7a74171ea59822013c69bc9c23f36e20da7ec URL: http://source.winehq.org/git/wine.git/?a=commit;h=7ab7a74171ea59822013c69bc… Author: Ricardo Filipe <ricardo_barbano(a)hotmail.com> Date: Wed Feb 18 08:39:40 2009 +0000 setupapi: Make hinstance global for all dialogs. --- dlls/setupapi/setupapi_private.h | 2 ++ dlls/setupapi/setupcab.c | 2 ++ dlls/setupapi/virtcopy.c | 9 --------- 3 files changed, 4 insertions(+), 9 deletions(-) diff --git a/dlls/setupapi/setupapi_private.h b/dlls/setupapi/setupapi_private.h index d01edba..428cba4 100644 --- a/dlls/setupapi/setupapi_private.h +++ b/dlls/setupapi/setupapi_private.h @@ -29,6 +29,8 @@ #define REGPART_RENAME "\\Rename" #define REG_VERSIONCONFLICT "Software\\Microsoft\\VersionConflictManager" +extern HINSTANCE SETUPAPI_hInstance; + static inline WCHAR *strdupW( const WCHAR *str ) { WCHAR *ret = NULL; diff --git a/dlls/setupapi/setupcab.c b/dlls/setupapi/setupcab.c index 96050e2..d879f9d 100644 --- a/dlls/setupapi/setupcab.c +++ b/dlls/setupapi/setupcab.c @@ -65,6 +65,7 @@ OSVERSIONINFOW OsVersionInfo; static HINSTANCE CABINET_hInstance = 0; +HINSTANCE SETUPAPI_hInstance = 0; static HFDI (__cdecl *sc_FDICreate)(PFNALLOC, PFNFREE, PFNOPEN, PFNREAD, PFNWRITE, PFNCLOSE, PFNSEEK, int, PERF); @@ -695,6 +696,7 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) OsVersionInfo.dwOSVersionInfoSize = sizeof(OSVERSIONINFOW); if (!GetVersionExW(&OsVersionInfo)) return FALSE; + SETUPAPI_hInstance = hinstDLL; break; case DLL_PROCESS_DETACH: UnloadCABINETDll(); diff --git a/dlls/setupapi/virtcopy.c b/dlls/setupapi/virtcopy.c index 9dddc88..c2fded2 100644 --- a/dlls/setupapi/virtcopy.c +++ b/dlls/setupapi/virtcopy.c @@ -43,8 +43,6 @@ static BOOL VCP_opened = FALSE; static VCPSTATUS vcp_status; -static HINSTANCE SETUPAPI_hInstance; - static WORD VCP_Callback( LPVOID obj, UINT16 msg, WPARAM16 wParam, LPARAM lParam, LPARAM lParamRef ) { WORD args[8]; @@ -308,13 +306,6 @@ RETERR16 WINAPI VcpOpen16(VIFPROC vifproc, LPARAM lparamMsgRef) VCP_Proc = (FARPROC16)vifproc; VCP_MsgRef = lparamMsgRef; - /* load SETUPAPI needed for dialog resources etc. */ - SETUPAPI_hInstance = GetModuleHandleA("setupapi.dll"); - if (!SETUPAPI_hInstance) - { - ERR("Could not load sibling setupapi.dll\n"); - return ERR_VCP_NOMEM; - } VCP_opened = TRUE; return OK; }

1 0

Alexandre Julliard : ntdll: Allocate the stack for all threads, don' t rely on pthread to do it for us.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 1056771b9100a9bf6bb9abf79c8379753129ace4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1056771b9100a9bf6bb9abf79… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 23 14:24:59 2009 +0100 ntdll: Allocate the stack for all threads, don't rely on pthread to do it for us. --- dlls/ntdll/loader.c | 6 +---- dlls/ntdll/ntdll_misc.h | 2 +- dlls/ntdll/thread.c | 55 +++++++++++++--------------------------------- dlls/ntdll/virtual.c | 43 ++++++++++++++++++------------------ 4 files changed, 39 insertions(+), 67 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=1056771b9100a9bf6bb9a…

1 0

Alexandre Julliard : ntdll: Don' t free the thread stack and TEB while exiting, do it from the next thread.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 67e45d66d32a52bd4df8df24a75330bc59d4ec37 URL: http://source.winehq.org/git/wine.git/?a=commit;h=67e45d66d32a52bd4df8df24a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 23 14:02:16 2009 +0100 ntdll: Don't free the thread stack and TEB while exiting, do it from the next thread. --- dlls/ntdll/ntdll_misc.h | 3 +-- dlls/ntdll/thread.c | 42 +++++++++++++++++++++--------------------- 2 files changed, 22 insertions(+), 23 deletions(-) diff --git a/dlls/ntdll/ntdll_misc.h b/dlls/ntdll/ntdll_misc.h index 3dd20d6..55eadd0 100644 --- a/dlls/ntdll/ntdll_misc.h +++ b/dlls/ntdll/ntdll_misc.h @@ -195,8 +195,7 @@ struct ntdll_thread_data int wait_fd[2]; /* 1e8 fd for sleeping server requests */ void *vm86_ptr; /* 1f0 data for vm86 mode */ void *pthread_data; /* 1f4 private data for pthread emulation */ - - void *pad[1]; /* 1f8 change this if you add fields! */ + pthread_t pthread_id; /* 1f8 pthread thread id */ }; static inline struct ntdll_thread_data *ntdll_get_thread_data(void) diff --git a/dlls/ntdll/thread.c b/dlls/ntdll/thread.c index e11976c..40bf507 100644 --- a/dlls/ntdll/thread.c +++ b/dlls/ntdll/thread.c @@ -353,9 +353,9 @@ HANDLE thread_init(void) */ void abort_thread( int status ) { + pthread_sigmask( SIG_BLOCK, &server_block_set, NULL ); if (interlocked_xchg_add( &nb_threads, -1 ) <= 1) _exit( status ); - pthread_sigmask( SIG_BLOCK, &server_block_set, NULL ); close( ntdll_get_thread_data()->wait_fd[0] ); close( ntdll_get_thread_data()->wait_fd[1] ); close( ntdll_get_thread_data()->reply_fd ); @@ -369,11 +369,8 @@ void abort_thread( int status ) */ static void DECLSPEC_NORETURN exit_thread( int status ) { - int fds[4]; - void *teb_addr; - SIZE_T teb_size; - - if (interlocked_xchg_add( &nb_threads, -1 ) <= 1) exit( status ); + static void *prev_teb; + TEB *teb; RtlAcquirePebLock(); RemoveEntryList( &NtCurrentTeb()->TlsLinks ); @@ -381,23 +378,26 @@ static void DECLSPEC_NORETURN exit_thread( int status ) RtlFreeHeap( GetProcessHeap(), 0, NtCurrentTeb()->FlsSlots ); RtlFreeHeap( GetProcessHeap(), 0, NtCurrentTeb()->TlsExpansionSlots ); - fds[0] = ntdll_get_thread_data()->wait_fd[0]; - fds[1] = ntdll_get_thread_data()->wait_fd[1]; - fds[2] = ntdll_get_thread_data()->reply_fd; - fds[3] = ntdll_get_thread_data()->request_fd; pthread_sigmask( SIG_BLOCK, &server_block_set, NULL ); + if (interlocked_xchg_add( &nb_threads, -1 ) <= 1) exit( status ); - virtual_free_system_view( &NtCurrentTeb()->DeallocationStack ); - teb_addr = NtCurrentTeb(); - teb_size = virtual_free_system_view( &teb_addr ); - - close( fds[0] ); - close( fds[1] ); - close( fds[2] ); - close( fds[3] ); + if ((teb = interlocked_xchg_ptr( &prev_teb, NtCurrentTeb() ))) + { + struct ntdll_thread_data *thread_data = (struct ntdll_thread_data *)teb->SystemReserved2; + SIZE_T size; + + pthread_join( thread_data->pthread_id, NULL ); + wine_ldt_free_fs( thread_data->fs ); + size = 0; + NtFreeVirtualMemory( GetCurrentProcess(), &teb->DeallocationStack, &size, MEM_RELEASE ); + size = 0; + NtFreeVirtualMemory( GetCurrentProcess(), (void **)&teb, &size, MEM_RELEASE ); + } - wine_ldt_free_fs( wine_get_fs() ); - if (teb_size) munmap( teb_addr, teb_size ); + close( ntdll_get_thread_data()->wait_fd[0] ); + close( ntdll_get_thread_data()->wait_fd[1] ); + close( ntdll_get_thread_data()->reply_fd ); + close( ntdll_get_thread_data()->request_fd ); pthread_exit( UIntToPtr(status) ); } @@ -479,6 +479,7 @@ static void start_thread( struct wine_pthread_thread_info *info ) debug_info.str_pos = debug_info.strings; debug_info.out_pos = debug_info.output; thread_data->debug_info = &debug_info; + thread_data->pthread_id = pthread_self(); signal_init_thread( teb ); server_init_thread( func ); @@ -632,7 +633,6 @@ NTSTATUS WINAPI RtlCreateUserThread( HANDLE process, const SECURITY_DESCRIPTOR * pthread_attr_init( &attr ); pthread_attr_setstacksize( &attr, stack_reserve ); - pthread_attr_setdetachstate( &attr, PTHREAD_CREATE_DETACHED ); pthread_attr_setscope( &attr, PTHREAD_SCOPE_SYSTEM ); /* force creating a kernel thread */ interlocked_xchg_add( &nb_threads, 1 ); if (pthread_create( &pthread_id, &attr, (void * (*)(void *))start_thread, info ))

1 0

Aric Stewart : msctf: Implement ITfInputProcessorProfiles:: EnumLanguageProfiles.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 7d1952b68908515fde391894f30b6d78a1660302 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d1952b68908515fde391894f… Author: Aric Stewart <aric(a)codeweavers.com> Date: Fri Feb 20 13:53:06 2009 -0600 msctf: Implement ITfInputProcessorProfiles::EnumLanguageProfiles. --- dlls/msctf/inputprocessor.c | 242 ++++++++++++++++++++++++++++++++++++- dlls/msctf/tests/inputprocessor.c | 22 ++++ include/msctf.idl | 21 +++ 3 files changed, 283 insertions(+), 2 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7d1952b68908515fde391…

1 0

Aric Stewart : msctf: Implement ITfInputProcessorProfiles:: EnumInputProcessorInfo.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 4d38c952d14957a0a09b76f47ec80972aa5e2e24 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d38c952d14957a0a09b76f47… Author: Aric Stewart <aric(a)codeweavers.com> Date: Fri Feb 20 13:52:55 2009 -0600 msctf: Implement ITfInputProcessorProfiles::EnumInputProcessorInfo. --- dlls/msctf/inputprocessor.c | 164 ++++++++++++++++++++++++++++++++++++- dlls/msctf/tests/inputprocessor.c | 19 ++++ 2 files changed, 181 insertions(+), 2 deletions(-) diff --git a/dlls/msctf/inputprocessor.c b/dlls/msctf/inputprocessor.c index 3950a94..b098169 100644 --- a/dlls/msctf/inputprocessor.c +++ b/dlls/msctf/inputprocessor.c @@ -52,6 +52,16 @@ typedef struct tagInputProcessorProfiles { LANGID currentLanguage; } InputProcessorProfiles; +typedef struct tagProfilesEnumGuid { + const IEnumGUIDVtbl *Vtbl; + LONG refCount; + + HKEY key; + DWORD next_index; +} ProfilesEnumGuid; + +static HRESULT ProfilesEnumGuid_Constructor(IEnumGUID **ppOut); + static void InputProcessorProfiles_Destructor(InputProcessorProfiles *This) { TRACE("destroying %p\n", This); @@ -236,8 +246,8 @@ static HRESULT WINAPI InputProcessorProfiles_EnumInputProcessorInfo( ITfInputProcessorProfiles *iface, IEnumGUID **ppEnum) { InputProcessorProfiles *This = (InputProcessorProfiles*)iface; - FIXME("STUB:(%p)\n",This); - return E_NOTIMPL; + TRACE("(%p) %p\n",This,ppEnum); + return ProfilesEnumGuid_Constructor(ppEnum); } static HRESULT WINAPI InputProcessorProfiles_GetDefaultLanguageProfile( @@ -485,3 +495,153 @@ HRESULT InputProcessorProfiles_Constructor(IUnknown *pUnkOuter, IUnknown **ppOut *ppOut = (IUnknown *)This; return S_OK; } + +/************************************************** + * IEnumGUID implementaion for ITfInputProcessorProfiles::EnumInputProcessorInfo + **************************************************/ +static void ProfilesEnumGuid_Destructor(ProfilesEnumGuid *This) +{ + TRACE("destroying %p\n", This); + RegCloseKey(This->key); + HeapFree(GetProcessHeap(),0,This); +} + +static HRESULT WINAPI ProfilesEnumGuid_QueryInterface(IEnumGUID *iface, REFIID iid, LPVOID *ppvOut) +{ + ProfilesEnumGuid *This = (ProfilesEnumGuid *)iface; + *ppvOut = NULL; + + if (IsEqualIID(iid, &IID_IUnknown) || IsEqualIID(iid, &IID_IEnumGUID)) + { + *ppvOut = This; + } + + if (*ppvOut) + { + IUnknown_AddRef(iface); + return S_OK; + } + + WARN("unsupported interface: %s\n", debugstr_guid(iid)); + return E_NOINTERFACE; +} + +static ULONG WINAPI ProfilesEnumGuid_AddRef(IEnumGUID *iface) +{ + ProfilesEnumGuid *This = (ProfilesEnumGuid*)iface; + return InterlockedIncrement(&This->refCount); +} + +static ULONG WINAPI ProfilesEnumGuid_Release(IEnumGUID *iface) +{ + ProfilesEnumGuid *This = (ProfilesEnumGuid *)iface; + ULONG ret; + + ret = InterlockedDecrement(&This->refCount); + if (ret == 0) + ProfilesEnumGuid_Destructor(This); + return ret; +} + +/***************************************************** + * IEnumGuid functions + *****************************************************/ +static HRESULT WINAPI ProfilesEnumGuid_Next( LPENUMGUID iface, + ULONG celt, GUID *rgelt, ULONG *pceltFetched) +{ + ProfilesEnumGuid *This = (ProfilesEnumGuid *)iface; + ULONG fetched = 0; + + TRACE("(%p)\n",This); + + if (rgelt == NULL) return E_POINTER; + + if (This->key) while (fetched < celt) + { + LSTATUS res; + HRESULT hr; + WCHAR catid[39]; + DWORD cName = 39; + + res = RegEnumKeyExW(This->key, This->next_index, catid, &cName, + NULL, NULL, NULL, NULL); + if (res != ERROR_SUCCESS && res != ERROR_MORE_DATA) break; + ++(This->next_index); + + hr = CLSIDFromString(catid, rgelt); + if (FAILED(hr)) continue; + + ++fetched; + ++rgelt; + } + + if (pceltFetched) *pceltFetched = fetched; + return fetched == celt ? S_OK : S_FALSE; +} + +static HRESULT WINAPI ProfilesEnumGuid_Skip( LPENUMGUID iface, ULONG celt) +{ + ProfilesEnumGuid *This = (ProfilesEnumGuid *)iface; + TRACE("(%p)\n",This); + + This->next_index += celt; + return S_OK; +} + +static HRESULT WINAPI ProfilesEnumGuid_Reset( LPENUMGUID iface) +{ + ProfilesEnumGuid *This = (ProfilesEnumGuid *)iface; + TRACE("(%p)\n",This); + This->next_index = 0; + return S_OK; +} + +static HRESULT WINAPI ProfilesEnumGuid_Clone( LPENUMGUID iface, + IEnumGUID **ppenum) +{ + ProfilesEnumGuid *This = (ProfilesEnumGuid *)iface; + HRESULT res; + + TRACE("(%p)\n",This); + + if (ppenum == NULL) return E_POINTER; + + res = ProfilesEnumGuid_Constructor(ppenum); + if (SUCCEEDED(res)) + { + ProfilesEnumGuid *new_This = (ProfilesEnumGuid *)*ppenum; + new_This->next_index = This->next_index; + } + return res; +} + +static const IEnumGUIDVtbl IEnumGUID_Vtbl ={ + ProfilesEnumGuid_QueryInterface, + ProfilesEnumGuid_AddRef, + ProfilesEnumGuid_Release, + + ProfilesEnumGuid_Next, + ProfilesEnumGuid_Skip, + ProfilesEnumGuid_Reset, + ProfilesEnumGuid_Clone +}; + +static HRESULT ProfilesEnumGuid_Constructor(IEnumGUID **ppOut) +{ + ProfilesEnumGuid *This; + + This = HeapAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,sizeof(ProfilesEnumGuid)); + if (This == NULL) + return E_OUTOFMEMORY; + + This->Vtbl= &IEnumGUID_Vtbl; + This->refCount = 1; + + if (RegCreateKeyExW(HKEY_LOCAL_MACHINE, szwSystemTIPKey, 0, NULL, 0, + KEY_READ | KEY_WRITE, NULL, &This->key, NULL) != ERROR_SUCCESS) + return E_FAIL; + + TRACE("returning %p\n", This); + *ppOut = (IEnumGUID*)This; + return S_OK; +} diff --git a/dlls/msctf/tests/inputprocessor.c b/dlls/msctf/tests/inputprocessor.c index 34e47e5..0520e49 100644 --- a/dlls/msctf/tests/inputprocessor.c +++ b/dlls/msctf/tests/inputprocessor.c @@ -84,6 +84,24 @@ static void test_Unregister(void) ok(SUCCEEDED(hr),"Unable to unregister text service(%x)\n",hr); } +static void test_EnumInputProcessorInfo(void) +{ + IEnumGUID *ppEnum; + BOOL found = FALSE; + + if (SUCCEEDED(ITfInputProcessorProfiles_EnumInputProcessorInfo(g_ipp, &ppEnum))) + { + ULONG fetched; + GUID g; + while (IEnumGUID_Next(ppEnum, 1, &g, &fetched) == S_OK) + { + if(IsEqualGUID(&g,&CLSID_FakeService)) + found = TRUE; + } + } + ok(found,"Did not find registered text service\n"); +} + static void test_RegisterCategory(void) { HRESULT hr; @@ -105,6 +123,7 @@ START_TEST(inputprocessor) gLangid = GetUserDefaultLCID(); test_Register(); test_RegisterCategory(); + test_EnumInputProcessorInfo(); test_UnregisterCategory(); test_Unregister(); }

1 0

Aric Stewart : msctf: Define TF_LANGUAGEPROFILE.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 606d087199f91d851a4e57ec450b7d0da896e931 URL: http://source.winehq.org/git/wine.git/?a=commit;h=606d087199f91d851a4e57ec4… Author: Aric Stewart <aric(a)codeweavers.com> Date: Fri Feb 20 13:52:37 2009 -0600 msctf: Define TF_LANGUAGEPROFILE. --- include/msctf.idl | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/include/msctf.idl b/include/msctf.idl index 1c3e652..7ffb46c 100644 --- a/include/msctf.idl +++ b/include/msctf.idl @@ -57,6 +57,15 @@ interface IEnumTfLanguageProfiles; interface ITfEditRecord; interface ITfCompositionView; +typedef [uuid(e1b5808d-1e46-4c19-84dc-68c5f5978cc8)] struct TF_LANGUAGEPROFILE +{ + CLSID clsid; + LANGID langid; + GUID catid; + BOOL fActive; + GUID guidProfile; +} TF_LANGUAGEPROFILE; + [ object, uuid(aa80e801-2021-11d2-93e0-0060b067b86e),

1 0

Austin English : dbghelp: Change pointer cast to DWORD_PTR.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 28fe3f76c03553140c57e795ed3cd707ca9561fa URL: http://source.winehq.org/git/wine.git/?a=commit;h=28fe3f76c03553140c57e795e… Author: Austin English <austinenglish(a)gmail.com> Date: Sun Feb 22 15:28:19 2009 -0600 dbghelp: Change pointer cast to DWORD_PTR. --- dlls/dbghelp/msc.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/dbghelp/msc.c b/dlls/dbghelp/msc.c index f293140..99de037 100644 --- a/dlls/dbghelp/msc.c +++ b/dlls/dbghelp/msc.c @@ -2524,7 +2524,7 @@ static BOOL pdb_process_internal(const struct process* pcs, } file_name = (const char*)file + size; file_name += strlen(file_name) + 1; - file = (BYTE*)((DWORD)(file_name + strlen(file_name) + 1 + 3) & ~3); + file = (BYTE*)((DWORD_PTR)(file_name + strlen(file_name) + 1 + 3) & ~3); } /* finish the remaining public and global information */ if (globalimage)

1 0

Alexandre Julliard : winetest: Don't try to unload mshtml, it's broken.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 6b6ca1a801e9e348a1467c7cf6b2036b0e9cef9e URL: http://source.winehq.org/git/wine.git/?a=commit;h=6b6ca1a801e9e348a1467c7cf… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 23 12:34:57 2009 +0100 winetest: Don't try to unload mshtml, it's broken. --- programs/winetest/Makefile.in | 1 + programs/winetest/main.c | 25 ++++++------------------- 2 files changed, 7 insertions(+), 19 deletions(-) diff --git a/programs/winetest/Makefile.in b/programs/winetest/Makefile.in index 140adb5..20fd2ba 100644 --- a/programs/winetest/Makefile.in +++ b/programs/winetest/Makefile.in @@ -5,6 +5,7 @@ VPATH = @srcdir@ MODULE = winetest.exe APPMODE = -mconsole IMPORTS = uuid comctl32 version user32 gdi32 advapi32 wsock32 kernel32 +DELAYIMPORTS = ole32 EXTRARCFLAGS = -DBUILD_SHA1=\"`GIT_DIR=$(TOPSRCDIR)/.git git rev-parse HEAD 2>/dev/null`\" diff --git a/programs/winetest/main.c b/programs/winetest/main.c index 0ab512a..5423ddb 100644 --- a/programs/winetest/main.c +++ b/programs/winetest/main.c @@ -154,28 +154,15 @@ static int running_on_visible_desktop (void) /* check if Gecko is present, trying to trigger the install if not */ static BOOL gecko_check(void) { - HMODULE mshtml; - HRESULT (WINAPI *pDllGetClassObject)(REFCLSID rclsid, REFIID riid, LPVOID *ppv); - IClassFactory *factory = NULL; - IHTMLDocument2 *doc = NULL; + IHTMLDocument2 *doc; IHTMLElement *body; BOOL ret = FALSE; - if (!(mshtml = LoadLibraryA( "mshtml.dll" ))) return FALSE; - if (!(pDllGetClassObject = (void *)GetProcAddress( mshtml, "DllGetClassObject" ))) - goto done; - if (FAILED(pDllGetClassObject( &CLSID_HTMLDocument, &IID_IClassFactory, (void **)&factory ))) - goto done; - if (FAILED(IClassFactory_CreateInstance( factory, NULL, &IID_IHTMLDocument2, (void **)&doc ))) - goto done; - if (FAILED(IHTMLDocument2_get_body( doc, &body ))) - goto done; - IHTMLElement_Release( body ); - ret = TRUE; -done: - if (doc) IHTMLDocument_Release( doc ); - if (factory) IClassFactory_Release( factory ); - FreeLibrary( mshtml ); + CoInitialize( NULL ); + if (FAILED( CoCreateInstance( &CLSID_HTMLDocument, NULL, CLSCTX_INPROC_SERVER, + &IID_IHTMLDocument2, (void **)&doc ))) return FALSE; + if ((ret = SUCCEEDED( IHTMLDocument2_get_body( doc, &body )))) IHTMLElement_Release( body ); + IHTMLDocument_Release( doc ); return ret; }

1 0

Alexandre Julliard : winetest: Process messages while waiting for a process to terminate.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: d9d2f37162179be7dc8c073094d6522149504e15 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d9d2f37162179be7dc8c07309… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 23 12:22:24 2009 +0100 winetest: Process messages while waiting for a process to terminate. --- programs/winetest/main.c | 93 +++++++++++++++++++++++++-------------------- 1 files changed, 52 insertions(+), 41 deletions(-) diff --git a/programs/winetest/main.c b/programs/winetest/main.c index e53502d..0ab512a 100644 --- a/programs/winetest/main.c +++ b/programs/winetest/main.c @@ -339,6 +339,21 @@ extract_test (struct wine_test *test, const char *dir, LPTSTR res_name) CloseHandle(hfile); } +static DWORD wait_process( HANDLE process, DWORD timeout ) +{ + DWORD wait, diff = 0, start = GetTickCount(); + MSG msg; + + while (diff < timeout) + { + wait = MsgWaitForMultipleObjects( 1, &process, FALSE, timeout - diff, QS_ALLINPUT ); + if (wait != WAIT_OBJECT_0 + 1) return wait; + while (PeekMessageW( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessage( &msg ); + diff = GetTickCount() - start; + } + return WAIT_TIMEOUT; +} + /* Run a command for MS milliseconds. If OUT != NULL, also redirect stdout to there. @@ -359,49 +374,45 @@ run_ex (char *cmd, HANDLE out_file, const char *tempdir, DWORD ms) si.hStdError = out_file ? out_file : GetStdHandle( STD_ERROR_HANDLE ); if (!CreateProcessA (NULL, cmd, NULL, NULL, TRUE, CREATE_DEFAULT_ERROR_MODE, - NULL, tempdir, &si, &pi)) { - status = -2; - } else { - CloseHandle (pi.hThread); - wait = WaitForSingleObject (pi.hProcess, ms); - if (wait == WAIT_OBJECT_0) { - GetExitCodeProcess (pi.hProcess, &status); - } else { - switch (wait) { - case WAIT_FAILED: - report (R_ERROR, "Wait for '%s' failed: %d", cmd, - GetLastError ()); - break; - case WAIT_TIMEOUT: - report (R_ERROR, "Process '%s' timed out.", cmd); - break; - default: - report (R_ERROR, "Wait returned %d", wait); - } - status = wait; - if (!TerminateProcess (pi.hProcess, 257)) - report (R_ERROR, "TerminateProcess failed: %d", - GetLastError ()); - wait = WaitForSingleObject (pi.hProcess, 5000); - switch (wait) { - case WAIT_FAILED: - report (R_ERROR, - "Wait for termination of '%s' failed: %d", - cmd, GetLastError ()); - break; - case WAIT_OBJECT_0: - break; - case WAIT_TIMEOUT: - report (R_ERROR, "Can't kill process '%s'", cmd); - break; - default: - report (R_ERROR, "Waiting for termination: %d", - wait); - } - } + NULL, tempdir, &si, &pi)) + return -2; + + CloseHandle (pi.hThread); + status = wait_process( pi.hProcess, ms ); + switch (status) + { + case WAIT_OBJECT_0: + GetExitCodeProcess (pi.hProcess, &status); CloseHandle (pi.hProcess); + return status; + case WAIT_FAILED: + report (R_ERROR, "Wait for '%s' failed: %d", cmd, GetLastError ()); + break; + case WAIT_TIMEOUT: + report (R_ERROR, "Process '%s' timed out.", cmd); + break; + default: + report (R_ERROR, "Wait returned %d", status); + break; } - + if (!TerminateProcess (pi.hProcess, 257)) + report (R_ERROR, "TerminateProcess failed: %d", GetLastError ()); + wait = wait_process( pi.hProcess, 5000 ); + switch (wait) + { + case WAIT_OBJECT_0: + break; + case WAIT_FAILED: + report (R_ERROR, "Wait for termination of '%s' failed: %d", cmd, GetLastError ()); + break; + case WAIT_TIMEOUT: + report (R_ERROR, "Can't kill process '%s'", cmd); + break; + default: + report (R_ERROR, "Waiting for termination: %d", wait); + break; + } + CloseHandle (pi.hProcess); return status; }

1 0

Francois Gouget : netapi32/tests: Use win_skip() to skip over unimplemented functionality.
by Alexandre Julliard 23 Feb '09

23 Feb '09

Module: wine Branch: master Commit: 49de63595abf3e0c906cefbf16780115f25ad79d URL: http://source.winehq.org/git/wine.git/?a=commit;h=49de63595abf3e0c906cefbf1… Author: Francois Gouget <fgouget(a)free.fr> Date: Mon Feb 23 10:42:43 2009 +0100 netapi32/tests: Use win_skip() to skip over unimplemented functionality. --- dlls/netapi32/tests/access.c | 4 ++-- dlls/netapi32/tests/apibuf.c | 2 +- dlls/netapi32/tests/ds.c | 2 +- dlls/netapi32/tests/wksta.c | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/netapi32/tests/access.c b/dlls/netapi32/tests/access.c index 301cd19..8c39a18 100644 --- a/dlls/netapi32/tests/access.c +++ b/dlls/netapi32/tests/access.c @@ -74,7 +74,7 @@ static int init_access_tests(void) rc=GetUserNameW(user_name, &dwSize); if (rc==FALSE && GetLastError()==ERROR_CALL_NOT_IMPLEMENTED) { - skip("GetUserNameW is not available.\n"); + win_skip("GetUserNameW is not available.\n"); return 0; } ok(rc, "User Name Retrieved\n"); @@ -333,7 +333,7 @@ START_TEST(access) * if one is not available, none are. */ if (!pNetApiBufferFree) { - skip("Needed functions are not available\n"); + win_skip("Needed functions are not available\n"); FreeLibrary(hnetapi32); return; } diff --git a/dlls/netapi32/tests/apibuf.c b/dlls/netapi32/tests/apibuf.c index fa29fc4..2438ecd 100644 --- a/dlls/netapi32/tests/apibuf.c +++ b/dlls/netapi32/tests/apibuf.c @@ -98,7 +98,7 @@ START_TEST(apibuf) if (pNetApiBufferAllocate && pNetApiBufferFree && pNetApiBufferReallocate && pNetApiBufferSize) run_apibuf_tests(); else - skip("Needed functions are not available\n"); + win_skip("Needed functions are not available\n"); FreeLibrary(hnetapi32); } diff --git a/dlls/netapi32/tests/ds.c b/dlls/netapi32/tests/ds.c index 277a589..be9597d 100644 --- a/dlls/netapi32/tests/ds.c +++ b/dlls/netapi32/tests/ds.c @@ -87,7 +87,7 @@ START_TEST(ds) test_get(); } else - skip("DsRoleGetPrimaryDomainInformation is not available\n"); + win_skip("DsRoleGetPrimaryDomainInformation is not available\n"); FreeLibrary(hnetapi32); } diff --git a/dlls/netapi32/tests/wksta.c b/dlls/netapi32/tests/wksta.c index b4994f8..e0c16d3 100644 --- a/dlls/netapi32/tests/wksta.c +++ b/dlls/netapi32/tests/wksta.c @@ -51,7 +51,7 @@ static int init_wksta_tests(void) dwSize = sizeof(user_name)/sizeof(user_name[0]); rc=GetUserNameW(user_name, &dwSize); if (rc==FALSE && GetLastError()==ERROR_CALL_NOT_IMPLEMENTED) { - skip("GetUserNameW is not implemented\n"); + win_skip("GetUserNameW is not implemented\n"); return 0; } ok(rc, "User Name Retrieved\n"); @@ -192,7 +192,7 @@ START_TEST(wksta) * if one is not available, none are. */ if (!pNetApiBufferFree) { - skip("Needed functions are not available\n"); + win_skip("Needed functions are not available\n"); FreeLibrary(hnetapi32); return; }

1 0

← Newer
1
...
20
21
22
23
24
25
26
...
97
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 February 2009