January 2007 - Wine-commits

Rob Shearman : ntdll: Don' t print a fixme in RtlQueueUserWorkItem if WT_EXECUTELONGFUNCTION is specified
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 7e1fead3fbd05b4ff6b61fd119e3dfde9b3d8ea8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7e1fead3fbd05b4ff6b61fd11… Author: Rob Shearman <rob(a)codeweavers.com> Date: Thu Jan 4 18:23:44 2007 +0000 ntdll: Don't print a fixme in RtlQueueUserWorkItem if WT_EXECUTELONGFUNCTION is specified since that is what the current behaviour is tuned for. --- dlls/ntdll/threadpool.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/ntdll/threadpool.c b/dlls/ntdll/threadpool.c index cfe4f65..3d06cb4 100644 --- a/dlls/ntdll/threadpool.c +++ b/dlls/ntdll/threadpool.c @@ -181,11 +181,13 @@ NTSTATUS WINAPI RtlQueueWorkItem(PRTL_WO work_item->function = Function; work_item->context = Context; - if (Flags != WT_EXECUTEDEFAULT) + if (Flags & ~WT_EXECUTELONGFUNCTION) FIXME("Flags 0x%x not supported\n", Flags); status = add_work_item_to_queue(work_item); + /* FIXME: tune this algorithm to not be as aggressive with creating threads + * if WT_EXECUTELONGFUNCTION isn't specified */ if ((status == STATUS_SUCCESS) && ((num_workers == 0) || (num_workers == num_busy_workers))) {

1 0

Andrew Talbot : dmband: Declare some functions static.
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: c8df290e3c05c68a258233d52b2199f20752893b URL: http://source.winehq.org/git/wine.git/?a=commit;h=c8df290e3c05c68a258233d52… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Thu Jan 4 17:04:06 2007 +0000 dmband: Declare some functions static. --- dlls/dmband/dmutils.c | 207 +++++++++++++++++++++++++------------------------ dlls/dmband/dmutils.h | 11 --- 2 files changed, 104 insertions(+), 114 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c8df290e3c05c68a25823…

1 0

Andrew Talbot : d3d9: Declare some functions static.
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 590bca8425f8a7238100fac15eea4c333ab77305 URL: http://source.winehq.org/git/wine.git/?a=commit;h=590bca8425f8a7238100fac15… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Thu Jan 4 17:01:21 2007 +0000 d3d9: Declare some functions static. --- dlls/d3d9/directx.c | 13 +++++++------ 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/dlls/d3d9/directx.c b/dlls/d3d9/directx.c index f49c11c..f360b4e 100644 --- a/dlls/d3d9/directx.c +++ b/dlls/d3d9/directx.c @@ -213,9 +213,9 @@ ULONG WINAPI D3D9CB_DestroyRenderTarget( return IDirect3DSurface9_Release((IDirect3DSurface9*) surfaceParent); } -HRESULT WINAPI D3D9CB_CreateAdditionalSwapChain(IUnknown *device, - WINED3DPRESENT_PARAMETERS* pPresentationParameters, - IWineD3DSwapChain ** ppSwapChain) { +static HRESULT WINAPI D3D9CB_CreateAdditionalSwapChain(IUnknown *device, + WINED3DPRESENT_PARAMETERS* pPresentationParameters, + IWineD3DSwapChain ** ppSwapChain) { HRESULT res = D3D_OK; IDirect3DSwapChain9Impl *d3dSwapChain = NULL; D3DPRESENT_PARAMETERS localParameters; @@ -308,9 +308,10 @@ ULONG WINAPI D3D9CB_DestroyDepthStencilS return IDirect3DSurface9_Release((IDirect3DSurface9*) surfaceParent); } -HRESULT WINAPI IDirect3D9Impl_CreateDevice(LPDIRECT3D9 iface, UINT Adapter, D3DDEVTYPE DeviceType, HWND hFocusWindow, - DWORD BehaviourFlags, D3DPRESENT_PARAMETERS* pPresentationParameters, - IDirect3DDevice9** ppReturnedDeviceInterface) { +static HRESULT WINAPI IDirect3D9Impl_CreateDevice(LPDIRECT3D9 iface, UINT Adapter, D3DDEVTYPE DeviceType, + HWND hFocusWindow, DWORD BehaviourFlags, + D3DPRESENT_PARAMETERS* pPresentationParameters, + IDirect3DDevice9** ppReturnedDeviceInterface) { IDirect3D9Impl *This = (IDirect3D9Impl *)iface; IDirect3DDevice9Impl *object = NULL;

1 0

Andrew Talbot : ddraw: Declare some functions static.
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 386fdd81e96a2111c23bd18fb41496d2b8278b56 URL: http://source.winehq.org/git/wine.git/?a=commit;h=386fdd81e96a2111c23bd18fb… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Thu Jan 4 17:07:43 2007 +0000 ddraw: Declare some functions static. --- dlls/ddraw/ddraw_private.h | 1 - dlls/ddraw/utils.c | 39 ++++++++++++++++++++------------------- 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h index 8828aad..ca3bc76 100644 --- a/dlls/ddraw/ddraw_private.h +++ b/dlls/ddraw/ddraw_private.h @@ -631,7 +631,6 @@ typedef struct } member_info; /* Structure copy */ -#define DDRAW_dump_flags(flags,names,num_names) DDRAW_dump_flags_(flags, names, num_names, 1) #define ME(x,f,e) { x, #x, (void (*)(const void *))(f), offsetof(STRUCT, e) } #define DD_STRUCT_COPY_BYSIZE(to,from) \ diff --git a/dlls/ddraw/utils.c b/dlls/ddraw/utils.c index bf31780..a80688c 100644 --- a/dlls/ddraw/utils.c +++ b/dlls/ddraw/utils.c @@ -30,7 +30,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(ddraw); -void DDRAW_dump_flags_(DWORD flags, const flag_info* names, size_t num_names, int newline); /***************************************************************************** * PixelFormat_WineD3DtoDD @@ -629,6 +628,25 @@ DDRAW_dump_DDCOLORKEY(const DDCOLORKEY * { DPRINTF(" Low : %d - High : %d", ddck->dwColorSpaceLowValue, ddck->dwColorSpaceHighValue); } + +#define DDRAW_dump_flags(flags,names,num_names) DDRAW_dump_flags_(flags, names, num_names, 1) +static void +DDRAW_dump_flags_(DWORD flags, + const flag_info* names, + size_t num_names, + int newline) +{ + unsigned int i; + + for (i=0; i < num_names; i++) + if ((flags & names[i].val) || /* standard flag value */ + ((!flags) && (!names[i].val))) /* zero value only */ + DPRINTF("%s ", names[i].name); + + if (newline) + DPRINTF("\n"); +} + void DDRAW_dump_DDSCAPS2(const DDSCAPS2 *in) { static const flag_info flags[] = { @@ -702,23 +720,6 @@ DDRAW_dump_DDSCAPS(const DDSCAPS *in) DDRAW_dump_DDSCAPS2(&in_bis); } -void -DDRAW_dump_flags_(DWORD flags, - const flag_info* names, - size_t num_names, - int newline) -{ - unsigned int i; - - for (i=0; i < num_names; i++) - if ((flags & names[i].val) || /* standard flag value */ - ((!flags) && (!names[i].val))) /* zero value only */ - DPRINTF("%s ", names[i].name); - - if (newline) - DPRINTF("\n"); -} - static void DDRAW_dump_pixelformat_flag(DWORD flagmask) { @@ -743,7 +744,7 @@ DDRAW_dump_pixelformat_flag(DWORD flagma DDRAW_dump_flags_(flagmask, flags, sizeof(flags)/sizeof(flags[0]), 0); } -void +static void DDRAW_dump_members(DWORD flags, const void* data, const member_info* mems,

1 0

Rob Shearman : ntdll: RtlInitUnicodeString on a string too long to fit in a UNICODE_STRING
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 5398e17d0ac88d3b300b6868c5d2a858c57d1095 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5398e17d0ac88d3b300b6868c… Author: Rob Shearman <rob(a)codeweavers.com> Date: Thu Jan 4 16:55:58 2007 +0000 ntdll: RtlInitUnicodeString on a string too long to fit in a UNICODE_STRING should set the it to have the maximum possible length and size rather than a modulus of the actual length. Fix test failures for RtlInitUnicodeString on Windows XP upwards. --- dlls/ntdll/rtlstr.c | 5 ++++- dlls/ntdll/tests/rtlstr.c | 8 ++++---- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/dlls/ntdll/rtlstr.c b/dlls/ntdll/rtlstr.c index 8e997e7..34336fd 100644 --- a/dlls/ntdll/rtlstr.c +++ b/dlls/ntdll/rtlstr.c @@ -215,7 +215,10 @@ void WINAPI RtlInitUnicodeString( { if ((target->Buffer = (PWSTR) source)) { - target->Length = strlenW(source) * sizeof(WCHAR); + unsigned int length = strlenW(source) * sizeof(WCHAR); + if (length > 0xfffc) + length = 0xfffc; + target->Length = length; target->MaximumLength = target->Length + sizeof(WCHAR); } else target->Length = target->MaximumLength = 0; diff --git a/dlls/ntdll/tests/rtlstr.c b/dlls/ntdll/tests/rtlstr.c index 52f0ee3..cc1606a 100644 --- a/dlls/ntdll/tests/rtlstr.c +++ b/dlls/ntdll/tests/rtlstr.c @@ -246,12 +246,12 @@ static void test_RtlInitUnicodeStringEx( uni.MaximumLength = 12345; uni.Buffer = (void *) 0xdeadbeef; pRtlInitUnicodeString(&uni, teststring2); - ok(uni.Length == 33920, + ok(uni.Length == 33920 /* <= Win2000 */ || uni.Length == 65532 /* >= Win XP */, "pRtlInitUnicodeString(&uni, 0) sets Length to %u, expected %u\n", - uni.Length, 33920); - ok(uni.MaximumLength == 33922, + uni.Length, 65532); + ok(uni.MaximumLength == 33922 /* <= Win2000 */ || uni.MaximumLength == 65534 /* >= Win XP */, "pRtlInitUnicodeString(&uni, 0) sets MaximumLength to %u, expected %u\n", - uni.MaximumLength, 33922); + uni.MaximumLength, 65534); ok(uni.Buffer == teststring2, "pRtlInitUnicodeString(&uni, 0) sets Buffer to %p, expected %p\n", uni.Buffer, teststring2);

1 0

Rob Shearman : ntdll: Fix some test failures of RtlGUIDFromString and RtlStringFromGUID
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 40b39b06d47d2a7feb88929ccbe90319d2ae6a69 URL: http://source.winehq.org/git/wine.git/?a=commit;h=40b39b06d47d2a7feb88929cc… Author: Rob Shearman <rob(a)codeweavers.com> Date: Thu Jan 4 16:56:16 2007 +0000 ntdll: Fix some test failures of RtlGUIDFromString and RtlStringFromGUID by setting the length correctly and by doing a case-insentive compare on the GUID string that is returned. --- dlls/ntdll/tests/rtlstr.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/ntdll/tests/rtlstr.c b/dlls/ntdll/tests/rtlstr.c index cc1606a..5c70eda 100644 --- a/dlls/ntdll/tests/rtlstr.c +++ b/dlls/ntdll/tests/rtlstr.c @@ -1684,14 +1684,14 @@ static void test_RtlGUIDFromString(void) UNICODE_STRING str; NTSTATUS ret; - str.Length = str.MaximumLength = (sizeof(szGuid) - 1) / sizeof(WCHAR); + str.Length = str.MaximumLength = sizeof(szGuid) - sizeof(WCHAR); str.Buffer = (LPWSTR)szGuid; ret = pRtlGUIDFromString(&str, &guid); ok(ret == 0, "expected ret=0, got 0x%0x\n", ret); ok(memcmp(&guid, &IID_Endianess, sizeof(guid)) == 0, "Endianess broken\n"); - str.Length = str.MaximumLength = (sizeof(szGuid2) - 1) / sizeof(WCHAR); + str.Length = str.MaximumLength = sizeof(szGuid2) - sizeof(WCHAR); str.Buffer = (LPWSTR)szGuid2; ret = pRtlGUIDFromString(&str, &guid); @@ -1708,7 +1708,7 @@ static void test_RtlStringFromGUID(void) ret = pRtlStringFromGUID(&IID_Endianess, &str); ok(ret == 0, "expected ret=0, got 0x%0x\n", ret); - ok(str.Buffer && !lstrcmpW(str.Buffer, szGuid), "Endianess broken\n"); + ok(str.Buffer && !lstrcmpiW(str.Buffer, szGuid), "Endianess broken\n"); } START_TEST(rtlstr)

1 0

Rob Shearman : rpcrt4: Pass WT_EXECUTELONGFUNCTION to QueueUserWorkItem since the
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 98c87d0e70861a59b409ea123434882709d966a5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=98c87d0e70861a59b409ea123… Author: Rob Shearman <rob(a)codeweavers.com> Date: Thu Jan 4 18:23:53 2007 +0000 rpcrt4: Pass WT_EXECUTELONGFUNCTION to QueueUserWorkItem since the function being called can sometimes take a long time to return and we almost always want a new thread to be created if there are no free threads available. --- dlls/rpcrt4/rpc_server.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/rpcrt4/rpc_server.c b/dlls/rpcrt4/rpc_server.c index 285ef09..84090aa 100644 --- a/dlls/rpcrt4/rpc_server.c +++ b/dlls/rpcrt4/rpc_server.c @@ -345,7 +345,7 @@ static DWORD CALLBACK RPCRT4_io_thread(L packet->conn = conn; packet->hdr = hdr; packet->msg = msg; - QueueUserWorkItem(RPCRT4_worker_thread, packet, WT_EXECUTEDEFAULT); + QueueUserWorkItem(RPCRT4_worker_thread, packet, WT_EXECUTELONGFUNCTION); #endif msg = NULL; }

1 0

Rob Shearman : wininet: Remove the custom thread pool implementation and use QueueUserWorkItem instead .
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 02c89e5c6f7cf34bdaaa8e83f755c6373bf7dfa2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=02c89e5c6f7cf34bdaaa8e83f… Author: Rob Shearman <rob(a)codeweavers.com> Date: Thu Jan 4 18:26:31 2007 +0000 wininet: Remove the custom thread pool implementation and use QueueUserWorkItem instead. --- dlls/wininet/internet.c | 182 +++++------------------------------------------ dlls/wininet/internet.h | 3 - 2 files changed, 18 insertions(+), 167 deletions(-) diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index 6a155c5..cef4526 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -70,8 +70,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(wininet); -#define MAX_IDLE_WORKER 1000*60*1 -#define MAX_WORKER_THREADS 10 #define RESPONSE_TIMEOUT 30 typedef struct @@ -83,18 +81,8 @@ typedef struct static VOID INTERNET_CloseHandle(LPWININETHANDLEHEADER hdr); HINTERNET WINAPI INTERNET_InternetOpenUrlW(LPWININETAPPINFOW hIC, LPCWSTR lpszUrl, LPCWSTR lpszHeaders, DWORD dwHeadersLength, DWORD dwFlags, DWORD dwContext); -static VOID INTERNET_ExecuteWork(void); static DWORD g_dwTlsErrIndex = TLS_OUT_OF_INDEXES; -static LONG dwNumThreads; -static LONG dwNumIdleThreads; -static LONG dwNumJobs; -static HANDLE hEventArray[2]; -#define hQuitEvent hEventArray[0] -#define hWorkEvent hEventArray[1] -static CRITICAL_SECTION csQueue; -static LPWORKREQUEST lpHeadWorkQueue; -static LPWORKREQUEST lpWorkQueueTail; static HMODULE WININET_hModule; #define HANDLE_CHUNK_SIZE 0x10 @@ -246,16 +234,8 @@ BOOL WINAPI DllMain (HINSTANCE hinstDLL, if (g_dwTlsErrIndex == TLS_OUT_OF_INDEXES) return FALSE; - hQuitEvent = CreateEventW(0, TRUE, FALSE, NULL); - hWorkEvent = CreateEventW(0, FALSE, FALSE, NULL); - InitializeCriticalSection(&csQueue); - URLCacheContainers_CreateDefaults(); - dwNumThreads = 0; - dwNumIdleThreads = 0; - dwNumJobs = 0; - WININET_hModule = (HMODULE)hinstDLL; case DLL_THREAD_ATTACH: @@ -278,12 +258,6 @@ BOOL WINAPI DllMain (HINSTANCE hinstDLL, HeapFree(GetProcessHeap(), 0, TlsGetValue(g_dwTlsErrIndex)); TlsFree(g_dwTlsErrIndex); } - - SetEvent(hQuitEvent); - - CloseHandle(hQuitEvent); - CloseHandle(hWorkEvent); - DeleteCriticalSection(&csQueue); break; } @@ -3084,107 +3058,18 @@ DWORD INTERNET_GetLastError(void) */ static DWORD CALLBACK INTERNET_WorkerThreadFunc(LPVOID lpvParam) { - DWORD dwWaitRes; - - while (1) - { - if(dwNumJobs > 0) { - INTERNET_ExecuteWork(); - continue; - } - dwWaitRes = WaitForMultipleObjects(2, hEventArray, FALSE, MAX_IDLE_WORKER); - - if (dwWaitRes == WAIT_OBJECT_0 + 1) - INTERNET_ExecuteWork(); - else - break; - - InterlockedIncrement(&dwNumIdleThreads); - } - - InterlockedDecrement(&dwNumIdleThreads); - InterlockedDecrement(&dwNumThreads); - TRACE("Worker thread exiting\n"); - return TRUE; -} - - -/*********************************************************************** - * INTERNET_InsertWorkRequest (internal) - * - * Insert work request into queue - * - * RETURNS - * - */ -static BOOL INTERNET_InsertWorkRequest(LPWORKREQUEST lpWorkRequest) -{ - BOOL bSuccess = FALSE; - LPWORKREQUEST lpNewRequest; - - TRACE("\n"); - - lpNewRequest = HeapAlloc(GetProcessHeap(), 0, sizeof(WORKREQUEST)); - if (lpNewRequest) - { - memcpy(lpNewRequest, lpWorkRequest, sizeof(WORKREQUEST)); - lpNewRequest->prev = NULL; - - EnterCriticalSection(&csQueue); - - lpNewRequest->next = lpWorkQueueTail; - if (lpWorkQueueTail) - lpWorkQueueTail->prev = lpNewRequest; - lpWorkQueueTail = lpNewRequest; - if (!lpHeadWorkQueue) - lpHeadWorkQueue = lpWorkQueueTail; - - LeaveCriticalSection(&csQueue); - - bSuccess = TRUE; - InterlockedIncrement(&dwNumJobs); - } - - return bSuccess; -} - - -/*********************************************************************** - * INTERNET_GetWorkRequest (internal) - * - * Retrieves work request from queue - * - * RETURNS - * - */ -static BOOL INTERNET_GetWorkRequest(LPWORKREQUEST lpWorkRequest) -{ - BOOL bSuccess = FALSE; - LPWORKREQUEST lpRequest = NULL; + LPWORKREQUEST lpRequest = lpvParam; + WORKREQUEST workRequest; TRACE("\n"); - EnterCriticalSection(&csQueue); - - if (lpHeadWorkQueue) - { - lpRequest = lpHeadWorkQueue; - lpHeadWorkQueue = lpHeadWorkQueue->prev; - if (lpRequest == lpWorkQueueTail) - lpWorkQueueTail = lpHeadWorkQueue; - } - - LeaveCriticalSection(&csQueue); + memcpy(&workRequest, lpRequest, sizeof(WORKREQUEST)); + HeapFree(GetProcessHeap(), 0, lpRequest); - if (lpRequest) - { - memcpy(lpWorkRequest, lpRequest, sizeof(WORKREQUEST)); - HeapFree(GetProcessHeap(), 0, lpRequest); - bSuccess = TRUE; - InterlockedDecrement(&dwNumJobs); - } + workRequest.asyncproc(&workRequest); - return bSuccess; + WININET_Release( workRequest.hdr ); + return TRUE; } @@ -3198,60 +3083,29 @@ static BOOL INTERNET_GetWorkRequest(LPWO */ BOOL INTERNET_AsyncCall(LPWORKREQUEST lpWorkRequest) { - HANDLE hThread; - DWORD dwTID; - BOOL bSuccess = FALSE; + BOOL bSuccess; + LPWORKREQUEST lpNewRequest; TRACE("\n"); - if (InterlockedDecrement(&dwNumIdleThreads) < 0) - { - InterlockedIncrement(&dwNumIdleThreads); + lpNewRequest = HeapAlloc(GetProcessHeap(), 0, sizeof(WORKREQUEST)); + if (!lpNewRequest) + return FALSE; - if (InterlockedIncrement(&dwNumThreads) > MAX_WORKER_THREADS || - !(hThread = CreateThread(NULL, 0, - INTERNET_WorkerThreadFunc, NULL, 0, &dwTID))) - { - InterlockedDecrement(&dwNumThreads); - INTERNET_SetLastError(ERROR_INTERNET_ASYNC_THREAD_FAILED); - goto lerror; - } + memcpy(lpNewRequest, lpWorkRequest, sizeof(WORKREQUEST)); - TRACE("Created new thread\n"); + bSuccess = QueueUserWorkItem(INTERNET_WorkerThreadFunc, lpNewRequest, WT_EXECUTELONGFUNCTION); + if (!bSuccess) + { + HeapFree(GetProcessHeap(), 0, lpNewRequest); + INTERNET_SetLastError(ERROR_INTERNET_ASYNC_THREAD_FAILED); } - bSuccess = TRUE; - INTERNET_InsertWorkRequest(lpWorkRequest); - SetEvent(hWorkEvent); - -lerror: - return bSuccess; } /*********************************************************************** - * INTERNET_ExecuteWork (internal) - * - * RETURNS - * - */ -static VOID INTERNET_ExecuteWork(void) -{ - WORKREQUEST workRequest; - - TRACE("\n"); - - if (!INTERNET_GetWorkRequest(&workRequest)) - return; - - workRequest.asyncproc(&workRequest); - - WININET_Release( workRequest.hdr ); -} - - -/*********************************************************************** * INTERNET_GetResponseBuffer (internal) * * RETURNS diff --git a/dlls/wininet/internet.h b/dlls/wininet/internet.h index 4015ee0..1524139 100644 --- a/dlls/wininet/internet.h +++ b/dlls/wininet/internet.h @@ -374,9 +374,6 @@ typedef struct WORKREQ struct WORKREQ_INTERNETREADFILEEXA InternetReadFileExA; } u; - struct WORKREQ *next; - struct WORKREQ *prev; - } WORKREQUEST, *LPWORKREQUEST; HINTERNET WININET_AllocHandle( LPWININETHANDLEHEADER info );

1 0

Dmitry Timoshkov : user32: Fix some failures of the ShowWindow test under Windows 2003.
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: 688d9af61d9bc7e0fe099b8919143787c8ed0462 URL: http://source.winehq.org/git/wine.git/?a=commit;h=688d9af61d9bc7e0fe099b891… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Thu Jan 4 22:09:20 2007 +0800 user32: Fix some failures of the ShowWindow test under Windows 2003. --- dlls/user32/tests/msg.c | 20 +++++++++++--------- 1 files changed, 11 insertions(+), 9 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 8179517..a1435da 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -8330,7 +8330,7 @@ static const struct message WmShowNormal { WM_SHOWWINDOW, sent|wparam, 1 }, { WM_WINDOWPOSCHANGING, sent|wparam, SWP_SHOWWINDOW|SWP_NOSIZE|SWP_NOMOVE }, { HCBT_ACTIVATE, hook }, - { WM_WINDOWPOSCHANGING, sent|wparam, SWP_NOSIZE|SWP_NOMOVE }, + { WM_WINDOWPOSCHANGING, sent|wparam|optional, SWP_NOSIZE|SWP_NOMOVE }, /* win2003 doesn't send it */ { HCBT_SETFOCUS, hook }, { WM_WINDOWPOSCHANGED, sent|wparam, SWP_SHOWWINDOW|SWP_NOSIZE|SWP_NOMOVE|SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE }, { 0 } @@ -8359,9 +8359,9 @@ static const struct message WmShowNoActi { WM_MOVE, sent|defwinproc }, { WM_SIZE, sent|wparam|defwinproc, SIZE_RESTORED }, { HCBT_SETFOCUS, hook }, - { HCBT_ACTIVATE, hook }, - { WM_WINDOWPOSCHANGING, sent|wparam, SWP_NOSIZE|SWP_NOMOVE }, - { HCBT_SETFOCUS, hook }, + { HCBT_ACTIVATE, hook|optional }, /* win2003 doesn't send it */ + { WM_WINDOWPOSCHANGING, sent|wparam|optional, SWP_NOSIZE|SWP_NOMOVE }, /* win2003 doesn't send it */ + { HCBT_SETFOCUS, hook|optional }, /* win2003 doesn't send it */ { 0 } }; static const struct message WmShowNA_1[] = { @@ -8397,12 +8397,13 @@ static const struct message WmRestore_3[ { HCBT_MINMAX, hook|lparam, 0, SW_RESTORE }, { WM_GETMINMAXINFO, sent }, { WM_WINDOWPOSCHANGING, sent|wparam, SWP_FRAMECHANGED|SWP_NOCOPYBITS|0x8000 }, - { HCBT_ACTIVATE, hook }, - { WM_WINDOWPOSCHANGING, sent|wparam, SWP_NOSIZE|SWP_NOMOVE }, - { HCBT_SETFOCUS, hook }, + { HCBT_ACTIVATE, hook|optional }, /* win2003 doesn't send it */ + { WM_WINDOWPOSCHANGING, sent|wparam|optional, SWP_NOSIZE|SWP_NOMOVE }, /* win2003 doesn't send it */ + { HCBT_SETFOCUS, hook|optional }, /* win2003 doesn't send it */ { WM_WINDOWPOSCHANGED, sent|wparam, SWP_FRAMECHANGED|SWP_NOCOPYBITS|0x8000 }, { WM_MOVE, sent|defwinproc }, { WM_SIZE, sent|wparam|defwinproc, SIZE_MAXIMIZED }, + { HCBT_SETFOCUS, hook|optional }, /* win2003 sends it */ { 0 } }; static const struct message WmRestore_4[] = { @@ -8508,6 +8509,7 @@ static const struct message WmShowMaximi { WM_WINDOWPOSCHANGED, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED|SWP_NOCOPYBITS|0x8000 }, { WM_MOVE, sent|defwinproc }, { WM_SIZE, sent|wparam|defwinproc, SIZE_MAXIMIZED }, + { HCBT_SETFOCUS, hook|optional }, /* win2003 sends it */ { 0 } }; static const struct message WmShowMaximized_2[] = { @@ -8546,7 +8548,7 @@ static void test_ShowWindow(void) BOOL todo_msg; /* message sequence doesn't match what Wine does */ } sw[] = { -/* 1 */ { SW_SHOWNORMAL, FALSE, WS_VISIBLE, WmShowNormal, TRUE }, +/* 1 */ { SW_SHOWNORMAL, FALSE, WS_VISIBLE, WmShowNormal, FALSE }, /* 2 */ { SW_SHOWNORMAL, TRUE, WS_VISIBLE, WmEmptySeq, FALSE }, /* 3 */ { SW_HIDE, TRUE, 0, WmHide_1, FALSE }, /* 4 */ { SW_HIDE, FALSE, 0, WmEmptySeq, FALSE }, @@ -8591,7 +8593,7 @@ static void test_ShowWindow(void) /* 43 */ { SW_SHOWMAXIMIZED, TRUE, WS_VISIBLE|WS_MAXIMIZE, WmMinMax_2, FALSE }, /* 44 */ { SW_MINIMIZE, TRUE, WS_VISIBLE|WS_MINIMIZE, WmMinimize_1, TRUE }, /* 45 */ { SW_MINIMIZE, TRUE, WS_VISIBLE|WS_MINIMIZE, WmMinMax_3, FALSE }, -/* 46 */ { SW_RESTORE, TRUE, WS_VISIBLE|WS_MAXIMIZE, WmRestore_3, TRUE }, +/* 46 */ { SW_RESTORE, TRUE, WS_VISIBLE|WS_MAXIMIZE, WmRestore_3, FALSE }, /* 47 */ { SW_RESTORE, TRUE, WS_VISIBLE, WmRestore_4, FALSE }, /* 48 */ { SW_SHOWMAXIMIZED, TRUE, WS_VISIBLE|WS_MAXIMIZE, WmShowMaximized_3, FALSE }, /* 49 */ { SW_SHOW, TRUE, WS_VISIBLE|WS_MAXIMIZE, WmEmptySeq, FALSE },

1 0

Dmitry Timoshkov : user32: Fix some failures in the MDI message test.
by Alexandre Julliard 04 Jan '07

04 Jan '07

Module: wine Branch: master Commit: d57dc9da0a88a941204b75b0d5e41e04d547fc2b URL: http://source.winehq.org/git/wine.git/?a=commit;h=d57dc9da0a88a941204b75b0d… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Thu Jan 4 22:32:05 2007 +0800 user32: Fix some failures in the MDI message test. --- dlls/user32/mdi.c | 129 ++++++++++++++++++++++++++++----------------- dlls/user32/tests/msg.c | 21 +++++-- dlls/user32/win.c | 2 +- dlls/winex11.drv/window.c | 2 +- 4 files changed, 97 insertions(+), 57 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d57dc9da0a88a941204b7…

1 0