December 2009 - Wine-Commits

Alexandre Julliard : wininet: Fix pointer cast warnings on 64-bit.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 5ded71ff43e5c895c961b91ac52b70a40f80a6f1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5ded71ff43e5c895c961b91ac… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 23 21:25:48 2009 +0100 wininet: Fix pointer cast warnings on 64-bit. --- dlls/wininet/netconnection.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wininet/netconnection.c b/dlls/wininet/netconnection.c index db57e1d..0e7b62a 100644 --- a/dlls/wininet/netconnection.c +++ b/dlls/wininet/netconnection.c @@ -318,7 +318,7 @@ static int netconn_secure_verify(int preverify_ok, X509_STORE_CTX *ctx) if (!endCert) ret = FALSE; if (ret) { - DWORD err = netconn_verify_cert(endCert, store, server); + DWORD_PTR err = netconn_verify_cert(endCert, store, server); if (err) { @@ -660,7 +660,7 @@ DWORD NETCON_secure_connect(WININET_NETCONNECTION *connection, LPWSTR hostname) if (pSSL_connect(connection->ssl_s) <= 0) { - res = (DWORD)pSSL_get_ex_data(connection->ssl_s, error_idx); + res = (DWORD_PTR)pSSL_get_ex_data(connection->ssl_s, error_idx); if (!res) res = ERROR_INTERNET_SECURITY_CHANNEL_ERROR; ERR("SSL_connect failed: %d\n", res);

1 0

Alexandre Julliard : user32: Use the wait message handler for the yield in PeekMessageW too.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 4b4b6a48cb49465319f8c9838d463e34abfa5ce9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4b4b6a48cb49465319f8c9838… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 24 12:03:45 2009 +0100 user32: Use the wait message handler for the yield in PeekMessageW too. --- dlls/user32/message.c | 8 +------- dlls/user32/winproc.c | 4 +++- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/dlls/user32/message.c b/dlls/user32/message.c index 48c8068..d85eabb 100644 --- a/dlls/user32/message.c +++ b/dlls/user32/message.c @@ -2887,13 +2887,7 @@ BOOL WINAPI PeekMessageW( MSG *msg_out, HWND hwnd, UINT first, UINT last, UINT f if (!peek_message( &msg, hwnd, first, last, flags, 0 )) { - if (!(flags & PM_NOYIELD)) - { - DWORD count; - ReleaseThunkLock(&count); - NtYieldExecution(); - if (count) RestoreThunkLock(count); - } + if (!(flags & PM_NOYIELD)) wow_handlers.wait_message( 0, NULL, 0, 0, 0 ); return FALSE; } diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index c73a40e..396ae87 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -1123,7 +1123,9 @@ static LRESULT WINAPI StaticWndProcW( HWND hwnd, UINT msg, WPARAM wParam, LPARAM static DWORD wait_message( DWORD count, CONST HANDLE *handles, DWORD timeout, DWORD mask, DWORD flags ) { - return USER_Driver->pMsgWaitForMultipleObjectsEx( count, handles, timeout, mask, flags ); + DWORD ret = USER_Driver->pMsgWaitForMultipleObjectsEx( count, handles, timeout, mask, flags ); + if (ret == WAIT_TIMEOUT && !count && !timeout) NtYieldExecution(); + return ret; } static HICON alloc_icon_handle( unsigned int size )

1 0

Alexandre Julliard : user32: Move a few 16-bit stubs to user16.c.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: da57abcb02598dae7a121d8d4a3db0043dc3901b URL: http://source.winehq.org/git/wine.git/?a=commit;h=da57abcb02598dae7a121d8d4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 23 19:49:27 2009 +0100 user32: Move a few 16-bit stubs to user16.c. --- dlls/user32/misc.c | 20 -------------------- dlls/user32/user16.c | 40 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 40 insertions(+), 20 deletions(-) diff --git a/dlls/user32/misc.c b/dlls/user32/misc.c index 8e7f48f..1dc2cb8 100644 --- a/dlls/user32/misc.c +++ b/dlls/user32/misc.c @@ -627,26 +627,6 @@ VOID WINAPI LoadLocalFonts(VOID) /*********************************************************************** - * USER_489 (USER.489) - */ -LONG WINAPI stub_USER_489(void) { FIXME("stub\n"); return 0; } - -/*********************************************************************** - * USER_490 (USER.490) - */ -LONG WINAPI stub_USER_490(void) { FIXME("stub\n"); return 0; } - -/*********************************************************************** - * USER_492 (USER.492) - */ -LONG WINAPI stub_USER_492(void) { FIXME("stub\n"); return 0; } - -/*********************************************************************** - * USER_496 (USER.496) - */ -LONG WINAPI stub_USER_496(void) { FIXME("stub\n"); return 0; } - -/*********************************************************************** * User32InitializeImmEntryTable */ BOOL WINAPI User32InitializeImmEntryTable(LPVOID ptr) diff --git a/dlls/user32/user16.c b/dlls/user32/user16.c index 51240a1..299b16e 100644 --- a/dlls/user32/user16.c +++ b/dlls/user32/user16.c @@ -2876,6 +2876,46 @@ BOOL16 WINAPI SystemParametersInfo16( UINT16 uAction, UINT16 uParam, /*********************************************************************** + * USER_489 (USER.489) + */ +LONG WINAPI stub_USER_489(void) +{ + FIXME("stub\n"); + return 0; +} + + +/*********************************************************************** + * USER_490 (USER.490) + */ +LONG WINAPI stub_USER_490(void) +{ + FIXME("stub\n"); + return 0; +} + + +/*********************************************************************** + * USER_492 (USER.492) + */ +LONG WINAPI stub_USER_492(void) +{ + FIXME("stub\n"); + return 0; +} + + +/*********************************************************************** + * USER_496 (USER.496) + */ +LONG WINAPI stub_USER_496(void) +{ + FIXME("stub\n"); + return 0; +} + + +/*********************************************************************** * FormatMessage (USER.606) */ DWORD WINAPI FormatMessage16(

1 0

Alexandre Julliard : user32: Remove some redundant 16-bit message traces.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: a0f9da117a6ae95f8d241a6f219defb35122bd03 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a0f9da117a6ae95f8d241a6f2… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Dec 22 14:50:58 2009 +0100 user32: Remove some redundant 16-bit message traces. --- dlls/user32/msg16.c | 28 ++++---------------------- dlls/user32/spy.c | 45 ++++++++++--------------------------------- dlls/user32/user_private.h | 8 +----- 3 files changed, 18 insertions(+), 63 deletions(-) diff --git a/dlls/user32/msg16.c b/dlls/user32/msg16.c index aef453d..407f0ad 100644 --- a/dlls/user32/msg16.c +++ b/dlls/user32/msg16.c @@ -614,9 +614,6 @@ LRESULT WINPROC_CallProc16To32A( winproc_callback_t callback, HWND16 hwnd, UINT1 LRESULT ret = 0; HWND hwnd32 = WIN_Handle32( hwnd ); - TRACE_(msg)("(hwnd=%p,msg=%s,wp=%08x,lp=%08lx)\n", - hwnd32, SPY_GetMsgName(msg, hwnd32), wParam, lParam); - switch(msg) { case WM_NCCREATE: @@ -938,9 +935,6 @@ LRESULT WINPROC_CallProc32ATo16( winproc_callback16_t callback, HWND hwnd, UINT { LRESULT ret = 0; - TRACE_(msg)("(hwnd=%p,msg=%s,wp=%08lx,lp=%08lx)\n", - hwnd, SPY_GetMsgName(msg, hwnd), wParam, lParam); - switch(msg) { case WM_NCCREATE: @@ -1625,8 +1619,6 @@ LRESULT WINAPI DefWindowProc16( HWND16 hwnd16, UINT16 msg, WPARAM16 wParam, LPAR LRESULT result; HWND hwnd = WIN_Handle32( hwnd16 ); - SPY_EnterMessage( SPY_DEFWNDPROC16, hwnd, msg, wParam, lParam ); - switch(msg) { case WM_NCCREATE: @@ -1646,10 +1638,8 @@ LRESULT WINAPI DefWindowProc16( HWND16 hwnd16, UINT16 msg, WPARAM16 wParam, LPAR cs32.dwExStyle = cs16->dwExStyle; cs32.lpszName = MapSL(cs16->lpszName); cs32.lpszClass = MapSL(cs16->lpszClass); - result = DefWindowProcA( hwnd, msg, wParam, (LPARAM)&cs32 ); + return DefWindowProcA( hwnd, msg, wParam, (LPARAM)&cs32 ); } - break; - case WM_NCCALCSIZE: { RECT16 *rect16 = MapSL(lParam); @@ -1666,9 +1656,8 @@ LRESULT WINAPI DefWindowProc16( HWND16 hwnd16, UINT16 msg, WPARAM16 wParam, LPAR rect16->top = rect32.top; rect16->right = rect32.right; rect16->bottom = rect32.bottom; + return result; } - break; - case WM_WINDOWPOSCHANGING: case WM_WINDOWPOSCHANGED: { @@ -1692,21 +1681,14 @@ LRESULT WINAPI DefWindowProc16( HWND16 hwnd16, UINT16 msg, WPARAM16 wParam, LPAR pos16->cx = pos32.cx; pos16->cy = pos32.cy; pos16->flags = pos32.flags; + return result; } - break; - case WM_GETTEXT: case WM_SETTEXT: - result = DefWindowProcA( hwnd, msg, wParam, (LPARAM)MapSL(lParam) ); - break; - + return DefWindowProcA( hwnd, msg, wParam, (LPARAM)MapSL(lParam) ); default: - result = DefWindowProcA( hwnd, msg, wParam, lParam ); - break; + return DefWindowProcA( hwnd, msg, wParam, lParam ); } - - SPY_ExitMessage( SPY_RESULT_DEFWND16, hwnd, msg, result, wParam, lParam ); - return result; } diff --git a/dlls/user32/spy.c b/dlls/user32/spy.c index eb126e3..ea6c3d7 100644 --- a/dlls/user32/spy.c +++ b/dlls/user32/spy.c @@ -2575,13 +2575,13 @@ void SPY_EnterMessage( INT iFlag, HWND hWnd, UINT msg, switch(iFlag) { case SPY_DISPATCHMESSAGE16: - TRACE("%*s(%04x) %-16s message [%04x] %s dispatched wp=%04lx lp=%08lx\n", + TRACE("%*s(%04x) %-16s [%04x] %s dispatched wp=%04lx lp=%08lx\n", indent, "", HWND_16(hWnd), debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, wParam, lParam); break; case SPY_DISPATCHMESSAGE: - TRACE("%*s(%p) %-16s message [%04x] %s dispatched wp=%08lx lp=%08lx\n", + TRACE("%*s(%p) %-16s [%04x] %s dispatched wp=%08lx lp=%08lx\n", indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, wParam, lParam); break; @@ -2596,11 +2596,11 @@ void SPY_EnterMessage( INT iFlag, HWND hWnd, UINT msg, else sprintf( taskName, "tid %04x", GetCurrentThreadId() ); if (iFlag == SPY_SENDMESSAGE16) - TRACE("%*s(%04x) %-16s message [%04x] %s sent from %s wp=%04lx lp=%08lx\n", + TRACE("%*s(%04x) %-16s [%04x] %s sent from %s wp=%04lx lp=%08lx\n", indent, "", HWND_16(hWnd), debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, taskName, wParam, lParam ); else - { TRACE("%*s(%p) %-16s message [%04x] %s sent from %s wp=%08lx lp=%08lx\n", + { TRACE("%*s(%p) %-16s [%04x] %s sent from %s wp=%08lx lp=%08lx\n", indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, taskName, wParam, lParam ); SPY_DumpStructure(&sp_e, TRUE); @@ -2608,17 +2608,10 @@ void SPY_EnterMessage( INT iFlag, HWND hWnd, UINT msg, } break; - case SPY_DEFWNDPROC16: - if( SPY_ExcludeDWP ) return; - TRACE("%*s(%04x) DefWindowProc16: %s [%04x] wp=%04lx lp=%08lx\n", - indent, "", HWND_16(hWnd), sp_e.msg_name, msg, wParam, lParam ); - break; - case SPY_DEFWNDPROC: if( SPY_ExcludeDWP ) return; - TRACE("%*s(%p) DefWindowProc32: %s [%04x] wp=%08lx lp=%08lx\n", - indent, "", hWnd, sp_e.msg_name, - msg, wParam, lParam ); + TRACE("%*s(%p) DefWindowProc:[%04x] %s wp=%08lx lp=%08lx\n", + indent, "", hWnd, msg, sp_e.msg_name, wParam, lParam ); break; } set_indent_level( indent + SPY_INDENT_UNIT ); @@ -2637,7 +2630,7 @@ void SPY_ExitMessage( INT iFlag, HWND hWnd, UINT msg, LRESULT lReturn, DWORD save_error = GetLastError(); if (!TRACE_ON(message) || SPY_EXCLUDE(msg) || - (SPY_ExcludeDWP && (iFlag == SPY_RESULT_DEFWND16 || iFlag == SPY_RESULT_DEFWND)) ) + (SPY_ExcludeDWP && iFlag == SPY_RESULT_DEFWND)) return; sp_e.msgnum = msg; @@ -2655,39 +2648,23 @@ void SPY_ExitMessage( INT iFlag, HWND hWnd, UINT msg, LRESULT lReturn, switch(iFlag) { - case SPY_RESULT_DEFWND16: - TRACE(" %*s(%04x) DefWindowProc16: %s [%04x] returned %08lx\n", - indent, "", HWND_16(hWnd), sp_e.msg_name, msg, lReturn ); - break; - case SPY_RESULT_DEFWND: - TRACE(" %*s(%p) DefWindowProc32: %s [%04x] returned %08lx\n", - indent, "", hWnd, sp_e.msg_name, msg, lReturn ); + TRACE(" %*s(%p) DefWindowProc: [%04x] %s returned %08lx\n", + indent, "", hWnd, msg, sp_e.msg_name, lReturn ); break; case SPY_RESULT_OK16: - TRACE(" %*s(%04x) %-16s message [%04x] %s returned %08lx\n", + TRACE(" %*s(%04x) %-16s [%04x] %s returned %08lx\n", indent, "", HWND_16(hWnd), debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, lReturn ); break; case SPY_RESULT_OK: - TRACE(" %*s(%p) %-16s message [%04x] %s returned %08lx\n", + TRACE(" %*s(%p) %-16s [%04x] %s returned %08lx\n", indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, lReturn ); SPY_DumpStructure(&sp_e, FALSE); break; - - case SPY_RESULT_INVALIDHWND16: - WARN(" %*s(%04x) %-16s message [%04x] %s HAS INVALID HWND\n", - indent, "", HWND_16(hWnd), debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name ); - break; - - case SPY_RESULT_INVALIDHWND: - WARN(" %*s(%p) %-16s message [%04x] %s HAS INVALID HWND\n", - indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, - sp_e.msg_name ); - break; } SetLastError( save_error ); } diff --git a/dlls/user32/user_private.h b/dlls/user32/user_private.h index 7ca39ff..a54f272 100644 --- a/dlls/user32/user_private.h +++ b/dlls/user32/user_private.h @@ -259,15 +259,11 @@ extern BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lPar #define SPY_DISPATCHMESSAGE 0x0101 #define SPY_SENDMESSAGE16 0x0102 #define SPY_SENDMESSAGE 0x0103 -#define SPY_DEFWNDPROC16 0x0104 -#define SPY_DEFWNDPROC 0x0105 +#define SPY_DEFWNDPROC 0x0104 #define SPY_RESULT_OK16 0x0000 #define SPY_RESULT_OK 0x0001 -#define SPY_RESULT_INVALIDHWND16 0x0002 -#define SPY_RESULT_INVALIDHWND 0x0003 -#define SPY_RESULT_DEFWND16 0x0004 -#define SPY_RESULT_DEFWND 0x0005 +#define SPY_RESULT_DEFWND 0x0002 extern const char *SPY_GetClassLongOffsetName( INT offset ) DECLSPEC_HIDDEN; extern const char *SPY_GetMsgName( UINT msg, HWND hWnd ) DECLSPEC_HIDDEN;

1 0

Alexandre Julliard : user32: Replace spy functions by a simple trace in 16-bit code.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: b63d0e19eda129ea1cd0a42450911f5cb9ff1cc9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b63d0e19eda129ea1cd0a4245… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 23 19:45:03 2009 +0100 user32: Replace spy functions by a simple trace in 16-bit code. --- dlls/user32/msg16.c | 15 ++++++++------- dlls/user32/spy.c | 32 ++++++-------------------------- dlls/user32/user_private.h | 9 +++------ 3 files changed, 17 insertions(+), 39 deletions(-) diff --git a/dlls/user32/msg16.c b/dlls/user32/msg16.c index 407f0ad..085510c 100644 --- a/dlls/user32/msg16.c +++ b/dlls/user32/msg16.c @@ -35,6 +35,7 @@ #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(msg); +WINE_DECLARE_DEBUG_CHANNEL(message); DWORD USER16_AlertableWait = 0; @@ -1498,9 +1499,10 @@ LRESULT WINAPI SendMessage16( HWND16 hwnd16, UINT16 msg, WPARAM16 wparam, LPARAM if (!(winproc = (WNDPROC16)GetWindowLong16( hwnd16, GWLP_WNDPROC ))) return 0; - SPY_EnterMessage( SPY_SENDMESSAGE16, hwnd, msg, wparam, lparam ); + TRACE_(message)("(0x%04x) [%04x] wp=%04x lp=%08lx\n", hwnd16, msg, wparam, lparam ); result = CallWindowProc16( winproc, hwnd16, msg, wparam, lparam ); - SPY_ExitMessage( SPY_RESULT_OK16, hwnd, msg, result, wparam, lparam ); + TRACE_(message)("(0x%04x) [%04x] wp=%04x lp=%08lx returned %08lx\n", + hwnd16, msg, wparam, lparam, result ); } else /* map to 32-bit unicode for inter-thread/process message */ { @@ -1779,8 +1781,7 @@ BOOL16 WINAPI TranslateMessage16( const MSG16 *msg ) LONG WINAPI DispatchMessage16( const MSG16* msg ) { WNDPROC16 winproc; - LONG retval; - HWND hwnd = WIN_Handle32( msg->hwnd ); + LRESULT retval; /* Process timer messages */ if ((msg->message == WM_TIMER) || (msg->message == WM_SYSTIMER)) @@ -1795,10 +1796,10 @@ LONG WINAPI DispatchMessage16( const MSG16* msg ) SetLastError( ERROR_INVALID_WINDOW_HANDLE ); return 0; } - SPY_EnterMessage( SPY_DISPATCHMESSAGE16, hwnd, msg->message, msg->wParam, msg->lParam ); + TRACE_(message)("(0x%04x) [%04x] wp=%04x lp=%08lx\n", msg->hwnd, msg->message, msg->wParam, msg->lParam ); retval = CallWindowProc16( winproc, msg->hwnd, msg->message, msg->wParam, msg->lParam ); - SPY_ExitMessage( SPY_RESULT_OK16, hwnd, msg->message, retval, msg->wParam, msg->lParam ); - + TRACE_(message)("(0x%04x) [%04x] wp=%04x lp=%08lx returned %08lx\n", + msg->hwnd, msg->message, msg->wParam, msg->lParam, retval ); return retval; } diff --git a/dlls/user32/spy.c b/dlls/user32/spy.c index ea6c3d7..a3c7b3c 100644 --- a/dlls/user32/spy.c +++ b/dlls/user32/spy.c @@ -27,7 +27,6 @@ #include "winbase.h" #include "wingdi.h" #include "winreg.h" -#include "wownt32.h" #include "wine/unicode.h" #include "win.h" #include "user_private.h" @@ -1993,8 +1992,8 @@ static const SPY_NOTIFY *end_spnfy_array; /* ptr to last good entry in array #undef SPNFY -static BOOL16 SPY_Exclude[SPY_MAX_MSGNUM+1]; -static BOOL16 SPY_ExcludeDWP = 0; +static unsigned char SPY_Exclude[SPY_MAX_MSGNUM+1]; +static unsigned char SPY_ExcludeDWP = 0; #define SPY_EXCLUDE(msg) \ (SPY_Exclude[(msg) > SPY_MAX_MSGNUM ? SPY_MAX_MSGNUM : (msg)]) @@ -2574,19 +2573,12 @@ void SPY_EnterMessage( INT iFlag, HWND hWnd, UINT msg, /* each SPY_SENDMESSAGE must be complemented by call to SPY_ExitMessage */ switch(iFlag) { - case SPY_DISPATCHMESSAGE16: - TRACE("%*s(%04x) %-16s [%04x] %s dispatched wp=%04lx lp=%08lx\n", - indent, "", HWND_16(hWnd), - debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, wParam, lParam); - break; - case SPY_DISPATCHMESSAGE: TRACE("%*s(%p) %-16s [%04x] %s dispatched wp=%08lx lp=%08lx\n", indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, sp_e.msg_name, wParam, lParam); break; - case SPY_SENDMESSAGE16: case SPY_SENDMESSAGE: { char taskName[20]; @@ -2595,16 +2587,10 @@ void SPY_EnterMessage( INT iFlag, HWND hWnd, UINT msg, if (tid == GetCurrentThreadId()) strcpy( taskName, "self" ); else sprintf( taskName, "tid %04x", GetCurrentThreadId() ); - if (iFlag == SPY_SENDMESSAGE16) - TRACE("%*s(%04x) %-16s [%04x] %s sent from %s wp=%04lx lp=%08lx\n", - indent, "", HWND_16(hWnd), debugstr_w(sp_e.wnd_name), msg, - sp_e.msg_name, taskName, wParam, lParam ); - else - { TRACE("%*s(%p) %-16s [%04x] %s sent from %s wp=%08lx lp=%08lx\n", - indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, - sp_e.msg_name, taskName, wParam, lParam ); - SPY_DumpStructure(&sp_e, TRUE); - } + TRACE("%*s(%p) %-16s [%04x] %s sent from %s wp=%08lx lp=%08lx\n", + indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, + sp_e.msg_name, taskName, wParam, lParam ); + SPY_DumpStructure(&sp_e, TRUE); } break; @@ -2653,12 +2639,6 @@ void SPY_ExitMessage( INT iFlag, HWND hWnd, UINT msg, LRESULT lReturn, indent, "", hWnd, msg, sp_e.msg_name, lReturn ); break; - case SPY_RESULT_OK16: - TRACE(" %*s(%04x) %-16s [%04x] %s returned %08lx\n", - indent, "", HWND_16(hWnd), debugstr_w(sp_e.wnd_name), msg, - sp_e.msg_name, lReturn ); - break; - case SPY_RESULT_OK: TRACE(" %*s(%p) %-16s [%04x] %s returned %08lx\n", indent, "", hWnd, debugstr_w(sp_e.wnd_name), msg, diff --git a/dlls/user32/user_private.h b/dlls/user32/user_private.h index a54f272..10a03db 100644 --- a/dlls/user32/user_private.h +++ b/dlls/user32/user_private.h @@ -255,13 +255,10 @@ extern BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lPar /* message spy definitions */ -#define SPY_DISPATCHMESSAGE16 0x0100 -#define SPY_DISPATCHMESSAGE 0x0101 -#define SPY_SENDMESSAGE16 0x0102 -#define SPY_SENDMESSAGE 0x0103 -#define SPY_DEFWNDPROC 0x0104 +#define SPY_DISPATCHMESSAGE 0x0100 +#define SPY_SENDMESSAGE 0x0101 +#define SPY_DEFWNDPROC 0x0102 -#define SPY_RESULT_OK16 0x0000 #define SPY_RESULT_OK 0x0001 #define SPY_RESULT_DEFWND 0x0002

1 0

Alexandre Julliard : user32: Remove remaining calls to internal 32-bit functions from msg16.c.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: cf81d0f0f9bec8fed38c66851c756b387b384072 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cf81d0f0f9bec8fed38c66851… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 23 19:45:41 2009 +0100 user32: Remove remaining calls to internal 32-bit functions from msg16.c. --- dlls/user32/msg16.c | 13 ++++--------- 1 files changed, 4 insertions(+), 9 deletions(-) diff --git a/dlls/user32/msg16.c b/dlls/user32/msg16.c index 085510c..698fb61 100644 --- a/dlls/user32/msg16.c +++ b/dlls/user32/msg16.c @@ -258,8 +258,6 @@ static LRESULT call_window_proc16( HWND16 hwnd, UINT16 msg, WPARAM16 wParam, LPA } u; } args; - USER_CheckNotLock(); - if (index >= MAX_WINPROCS32) func = winproc16_array[index - MAX_WINPROCS32]; /* Window procedures want ax = hInstance, ds = es = ss */ @@ -1489,7 +1487,8 @@ LRESULT WINAPI SendMessage16( HWND16 hwnd16, UINT16 msg, WPARAM16 wparam, LPARAM LRESULT result; HWND hwnd = WIN_Handle32( hwnd16 ); - if (hwnd != HWND_BROADCAST && WIN_IsCurrentThread(hwnd)) + if (hwnd != HWND_BROADCAST && + GetWindowThreadProcessId( hwnd, NULL ) == GetCurrentThreadId()) { /* call 16-bit window proc directly */ WNDPROC16 winproc; @@ -2495,12 +2494,8 @@ static LRESULT mdiclient_proc16( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lPar if (msg == WM_CREATE) { LPCREATESTRUCTA cs = (LPCREATESTRUCTA)lParam; - WND *win; - BOOL is_win32; - - if (!(win = WIN_GetPtr( hwnd ))) return 0; - is_win32 = (win == WND_OTHER_PROCESS || win == WND_DESKTOP || (win->flags & WIN_ISWIN32)); - WIN_ReleasePtr( win ); + HINSTANCE instance = (HINSTANCE)GetWindowLongPtrW( hwnd, GWLP_HINSTANCE ); + BOOL is_win32 = !instance || ((ULONG_PTR)instance >> 16); /* Translation layer doesn't know what's in the cs->lpCreateParams * so we have to keep track of what environment we're in. */

1 0

Alexandre Julliard : user32: Duplicate DIALOG_Enable/ DisableOwner implementation on the 16-bit side.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 7f11441af9ed6b6bfba188a333e10775f6c63fc6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7f11441af9ed6b6bfba188a33… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Dec 22 17:46:25 2009 +0100 user32: Duplicate DIALOG_Enable/DisableOwner implementation on the 16-bit side. --- dlls/user32/controls.h | 2 -- dlls/user32/dialog.c | 4 ++-- dlls/user32/dialog16.c | 38 ++++++++++++++++++++++++++++++++++++++ 3 files changed, 40 insertions(+), 4 deletions(-) diff --git a/dlls/user32/controls.h b/dlls/user32/controls.h index 8d1869e..279dc4c 100644 --- a/dlls/user32/controls.h +++ b/dlls/user32/controls.h @@ -240,8 +240,6 @@ typedef struct tagDIALOGINFO #define DF_OWNERENABLED 0x0002 extern DIALOGINFO *DIALOG_get_info( HWND hwnd, BOOL create ) DECLSPEC_HIDDEN; -extern void DIALOG_EnableOwner( HWND hOwner ) DECLSPEC_HIDDEN; -extern BOOL DIALOG_DisableOwner( HWND hOwner ) DECLSPEC_HIDDEN; extern INT DIALOG_DoDialogBox( HWND hwnd, HWND owner ) DECLSPEC_HIDDEN; #endif /* __WINE_CONTROLS_H */ diff --git a/dlls/user32/dialog.c b/dlls/user32/dialog.c index f0d194d..bce2684 100644 --- a/dlls/user32/dialog.c +++ b/dlls/user32/dialog.c @@ -109,7 +109,7 @@ const struct builtin_class_descr DIALOG_builtin_class = * Helper function for modal dialogs to enable again the * owner of the dialog box. */ -void DIALOG_EnableOwner( HWND hOwner ) +static void DIALOG_EnableOwner( HWND hOwner ) { /* Owner must be a top-level window */ if (hOwner) @@ -125,7 +125,7 @@ void DIALOG_EnableOwner( HWND hOwner ) * Helper function for modal dialogs to disable the * owner of the dialog box. Returns TRUE if owner was enabled. */ -BOOL DIALOG_DisableOwner( HWND hOwner ) +static BOOL DIALOG_DisableOwner( HWND hOwner ) { /* Owner must be a top-level window */ if (hOwner) diff --git a/dlls/user32/dialog16.c b/dlls/user32/dialog16.c index fc6ad1e..95f009f 100644 --- a/dlls/user32/dialog16.c +++ b/dlls/user32/dialog16.c @@ -66,6 +66,44 @@ typedef struct /*********************************************************************** + * DIALOG_EnableOwner + * + * Helper function for modal dialogs to enable again the + * owner of the dialog box. + */ +static void DIALOG_EnableOwner( HWND hOwner ) +{ + /* Owner must be a top-level window */ + if (hOwner) + hOwner = GetAncestor( hOwner, GA_ROOT ); + if (!hOwner) return; + EnableWindow( hOwner, TRUE ); +} + + +/*********************************************************************** + * DIALOG_DisableOwner + * + * Helper function for modal dialogs to disable the + * owner of the dialog box. Returns TRUE if owner was enabled. + */ +static BOOL DIALOG_DisableOwner( HWND hOwner ) +{ + /* Owner must be a top-level window */ + if (hOwner) + hOwner = GetAncestor( hOwner, GA_ROOT ); + if (!hOwner) return FALSE; + if (IsWindowEnabled( hOwner )) + { + EnableWindow( hOwner, FALSE ); + return TRUE; + } + else + return FALSE; +} + + +/*********************************************************************** * DIALOG_GetControl16 * * Return the class and text of the control pointed to by ptr,

1 0

Alexandre Julliard : user32: Export DIALOG_get_info and DIALOG_DoDialogBox through the WoW handlers.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: eb36bb2e47c77dd12f098d62f471b94c7695c103 URL: http://source.winehq.org/git/wine.git/?a=commit;h=eb36bb2e47c77dd12f098d62f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Dec 22 17:57:42 2009 +0100 user32: Export DIALOG_get_info and DIALOG_DoDialogBox through the WoW handlers. --- dlls/user32/controls.h | 3 +++ dlls/user32/dialog16.c | 8 ++++---- dlls/user32/msg16.c | 2 +- dlls/user32/winproc.c | 20 +++++++++++--------- dlls/user32/wnd16.c | 6 ++++-- 5 files changed, 23 insertions(+), 16 deletions(-) diff --git a/dlls/user32/controls.h b/dlls/user32/controls.h index 279dc4c..65269c5 100644 --- a/dlls/user32/controls.h +++ b/dlls/user32/controls.h @@ -116,6 +116,8 @@ struct wow_handlers32 LRESULT (*static_proc)(HWND,UINT,WPARAM,LPARAM,BOOL); HWND (*create_window)(CREATESTRUCTW*,LPCWSTR,HINSTANCE,UINT); WNDPROC (*alloc_winproc)(WNDPROC,BOOL); + struct tagDIALOGINFO *(*get_dialog_info)(HWND,BOOL); + INT (*dialog_box_loop)(HWND,HWND); }; extern struct wow_handlers16 wow_handlers DECLSPEC_HIDDEN; @@ -129,6 +131,7 @@ extern LRESULT ScrollBarWndProc_common(HWND,UINT,WPARAM,LPARAM,BOOL) DECLSPEC_HI extern LRESULT StaticWndProc_common(HWND,UINT,WPARAM,LPARAM,BOOL) DECLSPEC_HIDDEN; /* 16-bit support */ +extern struct wow_handlers32 wow_handlers32 DECLSPEC_HIDDEN; extern HWND create_window16(CREATESTRUCTW*,LPCWSTR,HINSTANCE,UINT) DECLSPEC_HIDDEN; extern void free_module_classes(HINSTANCE16) DECLSPEC_HIDDEN; extern void register_wow_handlers(void) DECLSPEC_HIDDEN; diff --git a/dlls/user32/dialog16.c b/dlls/user32/dialog16.c index 95f009f..0e86a73 100644 --- a/dlls/user32/dialog16.c +++ b/dlls/user32/dialog16.c @@ -177,7 +177,7 @@ static LPCSTR DIALOG_GetControl16( LPCSTR p, DLG_CONTROL_INFO *info ) static BOOL DIALOG_CreateControls16( HWND hwnd, LPCSTR template, const DLG_TEMPLATE *dlgTemplate, HINSTANCE16 hInst ) { - DIALOGINFO *dlgInfo = DIALOG_get_info( hwnd, TRUE ); + DIALOGINFO *dlgInfo = wow_handlers32.get_dialog_info( hwnd, TRUE ); DLG_CONTROL_INFO info; HWND hwndCtrl, hwndDefButton = 0; INT items = dlgTemplate->nbItems; @@ -441,7 +441,7 @@ static HWND DIALOG_CreateIndirect16( HINSTANCE16 hInst, LPCVOID dlgTemplate, if (modal && (flags & DF_OWNERENABLED)) DIALOG_EnableOwner(owner); return 0; } - dlgInfo = DIALOG_get_info( hwnd, TRUE ); + dlgInfo = wow_handlers32.get_dialog_info( hwnd, TRUE ); dlgInfo->hwndFocus = 0; dlgInfo->hUserFont = hUserFont; dlgInfo->hMenu = HMENU_32( hMenu ); @@ -745,7 +745,7 @@ INT16 WINAPI DialogBoxParam16( HINSTANCE16 hInst, LPCSTR template, { HWND owner = WIN_Handle32(owner16); hwnd = DIALOG_CreateIndirect16( hInst, data, owner, dlgProc, param, TRUE ); - if (hwnd) ret = DIALOG_DoDialogBox( hwnd, owner ); + if (hwnd) ret = wow_handlers32.dialog_box_loop( hwnd, owner ); GlobalUnlock16( hmem ); } FreeResource16( hmem ); @@ -765,7 +765,7 @@ INT16 WINAPI DialogBoxIndirectParam16( HINSTANCE16 hInst, HANDLE16 dlgTemplate, if (!(ptr = GlobalLock16( dlgTemplate ))) return -1; hwnd = DIALOG_CreateIndirect16( hInst, ptr, owner, dlgProc, param, TRUE ); GlobalUnlock16( dlgTemplate ); - if (hwnd) return DIALOG_DoDialogBox( hwnd, owner ); + if (hwnd) return wow_handlers32.dialog_box_loop( hwnd, owner ); return -1; } diff --git a/dlls/user32/msg16.c b/dlls/user32/msg16.c index 1d96738..aef453d 100644 --- a/dlls/user32/msg16.c +++ b/dlls/user32/msg16.c @@ -38,7 +38,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(msg); DWORD USER16_AlertableWait = 0; -static struct wow_handlers32 wow_handlers32; +struct wow_handlers32 wow_handlers32; static LRESULT cwp_hook_callback( HWND hwnd, UINT msg, WPARAM wp, LPARAM lp, LRESULT *result, void *arg ) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index 354efe0..c38c087 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -1160,15 +1160,17 @@ static int free_icon_handle( HICON handle ) */ void WINAPI UserRegisterWowHandlers( const struct wow_handlers16 *new, struct wow_handlers32 *orig ) { - orig->button_proc = ButtonWndProc_common; - orig->combo_proc = ComboWndProc_common; - orig->edit_proc = EditWndProc_common; - orig->listbox_proc = ListBoxWndProc_common; - orig->mdiclient_proc = MDIClientWndProc_common; - orig->scrollbar_proc = ScrollBarWndProc_common; - orig->static_proc = StaticWndProc_common; - orig->create_window = WIN_CreateWindowEx; - orig->alloc_winproc = WINPROC_AllocProc; + orig->button_proc = ButtonWndProc_common; + orig->combo_proc = ComboWndProc_common; + orig->edit_proc = EditWndProc_common; + orig->listbox_proc = ListBoxWndProc_common; + orig->mdiclient_proc = MDIClientWndProc_common; + orig->scrollbar_proc = ScrollBarWndProc_common; + orig->static_proc = StaticWndProc_common; + orig->create_window = WIN_CreateWindowEx; + orig->alloc_winproc = WINPROC_AllocProc; + orig->get_dialog_info = DIALOG_get_info; + orig->dialog_box_loop = DIALOG_DoDialogBox; wow_handlers = *new; } diff --git a/dlls/user32/wnd16.c b/dlls/user32/wnd16.c index b4741ff..9509bd0 100644 --- a/dlls/user32/wnd16.c +++ b/dlls/user32/wnd16.c @@ -848,7 +848,8 @@ LONG WINAPI GetWindowLong16( HWND16 hwnd16, INT16 offset ) return 0; } } - else if (offset == DWLP_DLGPROC) is_winproc = (DIALOG_get_info( hwnd, FALSE ) != NULL); + else if (offset == DWLP_DLGPROC) + is_winproc = (wow_handlers32.get_dialog_info( hwnd, FALSE ) != NULL); } retvalue = GetWindowLongA( hwnd, offset ); if (is_winproc) retvalue = (LONG_PTR)WINPROC_GetProc16( (WNDPROC)retvalue, FALSE ); @@ -864,7 +865,8 @@ LONG WINAPI SetWindowLong16( HWND16 hwnd16, INT16 offset, LONG newval ) HWND hwnd = WIN_Handle32( hwnd16 ); BOOL is_winproc = (offset == GWLP_WNDPROC); - if (offset == DWLP_DLGPROC) is_winproc = (DIALOG_get_info( hwnd, FALSE ) != NULL); + if (offset == DWLP_DLGPROC) + is_winproc = (wow_handlers32.get_dialog_info( hwnd, FALSE ) != NULL); if (is_winproc) {

1 0

Alexandre Julliard : user32: Export the WIN_GetFullHandle function through the WoW handlers.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: b201a41edf29386074f6b1c410d00254344994b8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b201a41edf29386074f6b1c41… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Dec 22 18:09:30 2009 +0100 user32: Export the WIN_GetFullHandle function through the WoW handlers. --- dlls/user32/comm16.c | 2 +- dlls/user32/controls.h | 6 ++++++ dlls/user32/hook16.c | 2 +- dlls/user32/win.c | 12 ++++++------ dlls/user32/win.h | 8 +------- dlls/user32/winproc.c | 1 + 6 files changed, 16 insertions(+), 15 deletions(-) diff --git a/dlls/user32/comm16.c b/dlls/user32/comm16.c index 46f72c6..072b8bd 100644 --- a/dlls/user32/comm16.c +++ b/dlls/user32/comm16.c @@ -63,7 +63,7 @@ #include "windef.h" #include "winbase.h" #include "wine/winuser16.h" -#include "win.h" +#include "controls.h" #include "user_private.h" #include "wine/debug.h" diff --git a/dlls/user32/controls.h b/dlls/user32/controls.h index 65269c5..ae964b8 100644 --- a/dlls/user32/controls.h +++ b/dlls/user32/controls.h @@ -115,6 +115,7 @@ struct wow_handlers32 LRESULT (*scrollbar_proc)(HWND,UINT,WPARAM,LPARAM,BOOL); LRESULT (*static_proc)(HWND,UINT,WPARAM,LPARAM,BOOL); HWND (*create_window)(CREATESTRUCTW*,LPCWSTR,HINSTANCE,UINT); + HWND (*get_win_handle)(HWND); WNDPROC (*alloc_winproc)(WNDPROC,BOOL); struct tagDIALOGINFO *(*get_dialog_info)(HWND,BOOL); INT (*dialog_box_loop)(HWND,HWND); @@ -137,6 +138,11 @@ extern void free_module_classes(HINSTANCE16) DECLSPEC_HIDDEN; extern void register_wow_handlers(void) DECLSPEC_HIDDEN; extern void WINAPI UserRegisterWowHandlers( const struct wow_handlers16 *new, struct wow_handlers32 *orig ); +static inline HWND WIN_Handle32( HWND16 hwnd16 ) +{ + return wow_handlers32.get_win_handle( (HWND)(ULONG_PTR)hwnd16 ); +} + /* Class functions */ struct tagCLASS; /* opaque structure */ diff --git a/dlls/user32/hook16.c b/dlls/user32/hook16.c index 17d187d..cdea60f 100644 --- a/dlls/user32/hook16.c +++ b/dlls/user32/hook16.c @@ -28,7 +28,7 @@ #include "winuser.h" #include "wownt32.h" #include "wine/winuser16.h" -#include "win.h" +#include "controls.h" #include "user_private.h" #include "wine/debug.h" diff --git a/dlls/user32/win.c b/dlls/user32/win.c index 393c562..6ce524c 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -524,18 +524,18 @@ HWND WIN_IsCurrentThread( HWND hwnd ) /*********************************************************************** - * WIN_Handle32 + * WIN_GetFullHandle * - * Convert a 16-bit window handle to a full 32-bit handle. + * Convert a possibly truncated window handle to a full 32-bit handle. */ -HWND WIN_Handle32( HWND16 hwnd16 ) +HWND WIN_GetFullHandle( HWND hwnd ) { WND *ptr; - HWND hwnd = (HWND)(ULONG_PTR)hwnd16; - if (hwnd16 <= 1 || hwnd16 == 0xffff) return hwnd; + if (!hwnd || (ULONG_PTR)hwnd >> 16) return hwnd; + if (LOWORD(hwnd) <= 1 || LOWORD(hwnd) == 0xffff) return hwnd; /* do sign extension for -2 and -3 */ - if (hwnd16 >= (HWND16)-3) return (HWND)(LONG_PTR)(INT16)hwnd16; + if (LOWORD(hwnd) >= (WORD)-3) return (HWND)(LONG_PTR)(INT16)LOWORD(hwnd); if (!(ptr = WIN_GetPtr( hwnd ))) return hwnd; diff --git a/dlls/user32/win.h b/dlls/user32/win.h index ab37e3e..f764d89 100644 --- a/dlls/user32/win.h +++ b/dlls/user32/win.h @@ -79,7 +79,7 @@ typedef struct tagWND extern HWND get_hwnd_message_parent(void) DECLSPEC_HIDDEN; extern BOOL is_desktop_window( HWND hwnd ) DECLSPEC_HIDDEN; extern WND *WIN_GetPtr( HWND hwnd ) DECLSPEC_HIDDEN; -extern HWND WIN_Handle32( HWND16 hwnd16 ) DECLSPEC_HIDDEN; +extern HWND WIN_GetFullHandle( HWND hwnd ) DECLSPEC_HIDDEN; extern HWND WIN_IsCurrentProcess( HWND hwnd ) DECLSPEC_HIDDEN; extern HWND WIN_IsCurrentThread( HWND hwnd ) DECLSPEC_HIDDEN; extern HWND WIN_SetOwner( HWND hwnd, HWND owner ) DECLSPEC_HIDDEN; @@ -98,12 +98,6 @@ extern HDESK open_winstation_desktop( HWINSTA hwinsta, LPCWSTR name, DWORD flags extern void USER_Lock(void) DECLSPEC_HIDDEN; extern void USER_Unlock(void) DECLSPEC_HIDDEN; -static inline HWND WIN_GetFullHandle( HWND hwnd ) -{ - if (!HIWORD(hwnd) && hwnd) hwnd = WIN_Handle32( LOWORD(hwnd) ); - return hwnd; -} - /* to release pointers retrieved by WIN_GetPtr */ static inline void WIN_ReleasePtr( WND *ptr ) { diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index c38c087..b84abb5 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -1168,6 +1168,7 @@ void WINAPI UserRegisterWowHandlers( const struct wow_handlers16 *new, struct wo orig->scrollbar_proc = ScrollBarWndProc_common; orig->static_proc = StaticWndProc_common; orig->create_window = WIN_CreateWindowEx; + orig->get_win_handle = WIN_GetFullHandle; orig->alloc_winproc = WINPROC_AllocProc; orig->get_dialog_info = DIALOG_get_info; orig->dialog_box_loop = DIALOG_DoDialogBox;

1 0

Alexandre Julliard : user32: Get rid of 16-bit handles in the driver clipboard interface.
by Alexandre Julliard Dec. 23, 2009

Dec. 23, 2009

Module: wine Branch: master Commit: 83b66549da32fe6b23ed366da818b37d6b39f259 URL: http://source.winehq.org/git/wine.git/?a=commit;h=83b66549da32fe6b23ed366da… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Dec 22 21:18:40 2009 +0100 user32: Get rid of 16-bit handles in the driver clipboard interface. --- dlls/user32/clipboard.c | 5 +- dlls/user32/driver.c | 14 +- dlls/user32/user_private.h | 4 +- dlls/winex11.drv/clipboard.c | 274 ++++++++++--------------------------- dlls/winex11.drv/winex11.drv.spec | 4 +- 5 files changed, 87 insertions(+), 214 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=83b66549da32fe6b23ed3…

1 0