March 2022 - Wine-Commits - list.winehq.org

Jacek Caban : user32: Merge WINPROC_call_window into call_window_proc.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: 0cfc7c445810c52622afa2b2635e0a23861df386 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0cfc7c445810c52622afa2b2… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 23 14:02:09 2022 +0100 user32: Merge WINPROC_call_window into call_window_proc. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/message.c | 62 ++++++++++++++++++++++++++++++++++++++-------- dlls/user32/user_private.h | 3 +-- dlls/user32/winproc.c | 55 +++++----------------------------------- 3 files changed, 59 insertions(+), 61 deletions(-) diff --git a/dlls/user32/message.c b/dlls/user32/message.c index 1e7ec2b0615..426352da584 100644 --- a/dlls/user32/message.c +++ b/dlls/user32/message.c @@ -53,6 +53,8 @@ WINE_DECLARE_DEBUG_CHANNEL(key); #define MAX_PACK_COUNT 4 +#define MAX_WINPROC_RECURSION 64 + /* the various structures that can be sent in messages, in platform-independent layout */ struct packed_CREATESTRUCTW { @@ -2162,6 +2164,38 @@ static BOOL unpack_dde_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM return TRUE; } +static BOOL init_window_call_params( struct win_proc_params *params, HWND hwnd, UINT msg, WPARAM wParam, + LPARAM lParam, LRESULT *result, BOOL ansi, + enum wm_char_mapping mapping ) +{ + WND *win; + + USER_CheckNotLock(); + + if (!(win = WIN_GetPtr( hwnd ))) return FALSE; + if (win == WND_OTHER_PROCESS || win == WND_DESKTOP) return FALSE; + if (win->tid != GetCurrentThreadId()) + { + WIN_ReleasePtr( win ); + return FALSE; + } + params->func = win->winproc; + params->ansi_dst = !(win->flags & WIN_ISUNICODE); + params->is_dialog = win->dlgInfo != NULL; + WIN_ReleasePtr( win ); + + params->hwnd = WIN_GetFullHandle( hwnd ); + get_winproc_params( params ); + params->msg = msg; + params->wparam = wParam; + params->lparam = lParam; + params->result = result; + params->ansi = ansi; + params->mapping = mapping; + params->dpi_awareness = GetWindowDpiAwarenessContext( params->hwnd ); + return TRUE; +} + /*********************************************************************** * call_window_proc * @@ -2170,35 +2204,43 @@ static BOOL unpack_dde_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM static LRESULT call_window_proc( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam, BOOL unicode, BOOL same_thread, enum wm_char_mapping mapping ) { + struct user_thread_info *thread_info = get_user_thread_info(); + struct win_proc_params params; LRESULT result = 0; CWPSTRUCT cwp; CWPRETSTRUCT cwpret; if (msg & 0x80000000) - { - result = handle_internal_message( hwnd, msg, wparam, lparam ); - goto done; - } + return handle_internal_message( hwnd, msg, wparam, lparam ); + + if (!WIN_IsCurrentThread( hwnd )) return 0; + if (!init_window_call_params( &params, hwnd, msg, wparam, lparam, &result, !unicode, mapping )) + return 0; /* first the WH_CALLWNDPROC hook */ - hwnd = WIN_GetFullHandle( hwnd ); cwp.lParam = lparam; cwp.wParam = wparam; cwp.message = msg; - cwp.hwnd = hwnd; + cwp.hwnd = params.hwnd; HOOK_CallHooks( WH_CALLWNDPROC, HC_ACTION, same_thread, (LPARAM)&cwp, unicode ); - /* now call the window procedure */ - if (!WINPROC_call_window( hwnd, msg, wparam, lparam, &result, unicode, mapping )) goto done; + if (thread_info->recursion_count <= MAX_WINPROC_RECURSION) + { + thread_info->recursion_count++; + + /* now call the window procedure */ + User32CallWindowProc( &params, sizeof(params) ); + + thread_info->recursion_count--; + } /* and finally the WH_CALLWNDPROCRET hook */ cwpret.lResult = result; cwpret.lParam = lparam; cwpret.wParam = wparam; cwpret.message = msg; - cwpret.hwnd = hwnd; + cwpret.hwnd = params.hwnd; HOOK_CallHooks( WH_CALLWNDPROCRET, HC_ACTION, same_thread, (LPARAM)&cwpret, unicode ); - done: return result; } diff --git a/dlls/user32/user_private.h b/dlls/user32/user_private.h index bf798457fc6..4e114a31fa7 100644 --- a/dlls/user32/user_private.h +++ b/dlls/user32/user_private.h @@ -115,9 +115,8 @@ extern LRESULT WINPROC_CallProcAtoW( winproc_callback_t callback, HWND hwnd, UIN extern INT_PTR WINPROC_CallDlgProcA( DLGPROC func, HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam ) DECLSPEC_HIDDEN; extern INT_PTR WINPROC_CallDlgProcW( DLGPROC func, HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam ) DECLSPEC_HIDDEN; -extern BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, - LRESULT *result, BOOL unicode, enum wm_char_mapping mapping ) DECLSPEC_HIDDEN; extern void winproc_init(void) DECLSPEC_HIDDEN; +extern void get_winproc_params( struct win_proc_params *params ) DECLSPEC_HIDDEN; extern ATOM get_class_info( HINSTANCE instance, const WCHAR *name, WNDCLASSEXW *info, UNICODE_STRING *name_str, BOOL ansi ) DECLSPEC_HIDDEN; diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index 46b5fedde3f..c26a765f689 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -36,7 +36,6 @@ WINE_DECLARE_DEBUG_CHANNEL(msg); WINE_DECLARE_DEBUG_CHANNEL(relay); -#define MAX_WINPROC_RECURSION 64 #define WINPROC_PROC16 ((void *)1) /* placeholder for 16-bit window procs */ static inline void *get_buffer( void *static_buffer, size_t size, size_t need ) @@ -795,67 +794,25 @@ BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ) return TRUE; } -/********************************************************************** - * WINPROC_call_window - * - * Call the window procedure of the specified window. - */ -BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, - LRESULT *result, BOOL unicode, enum wm_char_mapping mapping ) +void get_winproc_params( struct win_proc_params *params ) { - struct user_thread_info *thread_info = get_user_thread_info(); - struct win_proc_params params; - WINDOWPROC *proc; - WND *wndPtr; - - USER_CheckNotLock(); - - if (!(wndPtr = WIN_GetPtr( hwnd ))) return FALSE; - if (wndPtr == WND_OTHER_PROCESS || wndPtr == WND_DESKTOP) return FALSE; - if (wndPtr->tid != GetCurrentThreadId()) - { - WIN_ReleasePtr( wndPtr ); - return FALSE; - } - params.func = wndPtr->winproc; - proc = handle_to_proc( wndPtr->winproc ); - params.ansi_dst = !(wndPtr->flags & WIN_ISUNICODE); - params.is_dialog = wndPtr->dlgInfo != NULL; - WIN_ReleasePtr( wndPtr ); - - params.hwnd = WIN_GetFullHandle( hwnd ); - params.msg = msg; - params.wparam = wParam; - params.lparam = lParam; - params.result = result; - params.ansi = !unicode; - params.mapping = mapping; - params.dpi_awareness = GetWindowDpiAwarenessContext( params.hwnd ); + WINDOWPROC *proc = handle_to_proc( params->func ); if (!proc) { - params.procW = params.procA = NULL; + params->procW = params->procA = NULL; } else if (proc == WINPROC_PROC16) { - params.procW = params.procA = WINPROC_PROC16; + params->procW = params->procA = WINPROC_PROC16; } else { - params.procA = proc->procA; - params.procW = proc->procW; + params->procA = proc->procA; + params->procW = proc->procW; } - - if (thread_info->recursion_count > MAX_WINPROC_RECURSION) return FALSE; - thread_info->recursion_count++; - - dispatch_win_proc_params( &params ); - - thread_info->recursion_count--; - return TRUE; } - /********************************************************************** * CallWindowProcA (USER32.@) *

1 0

Jacek Caban : user32: Use User32CallWinEventHook in peek_message.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: 9ced5b0509b829833f818e6c39c3bc8ddde01341 URL: https://source.winehq.org/git/wine.git/?a=commit;h=9ced5b0509b829833f818e6c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 23 14:01:42 2022 +0100 user32: Use User32CallWinEventHook in peek_message. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/hook.c | 6 +++--- dlls/user32/message.c | 43 ++++++++++++++++--------------------------- dlls/win32u/hook.c | 2 ++ include/ntuser.h | 2 ++ 4 files changed, 23 insertions(+), 30 deletions(-) diff --git a/dlls/user32/hook.c b/dlls/user32/hook.c index 783ded6c12a..6dc8c571c0c 100644 --- a/dlls/user32/hook.c +++ b/dlls/user32/hook.c @@ -451,14 +451,14 @@ BOOL WINAPI User32CallWinEventHook( const struct win_event_hook_params *params, TRACE_(relay)( "\1Call winevent hook proc %p (hhook=%p,event=%x,hwnd=%p,object_id=%x,child_id=%x,tid=%04x,time=%x)\n", proc, params->handle, params->event, params->hwnd, params->object_id, - params->child_id, GetCurrentThreadId(), GetCurrentTime() ); + params->child_id, params->tid, params->time ); proc( params->handle, params->event, params->hwnd, params->object_id, params->child_id, - GetCurrentThreadId(), GetCurrentTime() ); + params->tid, params->time ); TRACE_(relay)( "\1Ret winevent hook proc %p (hhook=%p,event=%x,hwnd=%p,object_id=%x,child_id=%x,tid=%04x,time=%x)\n", proc, params->handle, params->event, params->hwnd, params->object_id, - params->child_id, GetCurrentThreadId(), GetCurrentTime() ); + params->child_id, params->tid, params->time ); if (free_module) FreeLibrary( free_module ); return TRUE; diff --git a/dlls/user32/message.c b/dlls/user32/message.c index 8100ee63fc3..1e7ec2b0615 100644 --- a/dlls/user32/message.c +++ b/dlls/user32/message.c @@ -2716,38 +2716,27 @@ static int peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags, case MSG_WINEVENT: if (size >= sizeof(msg_data->winevent)) { - WINEVENTPROC hook_proc; - HMODULE free_module = 0; + struct win_event_hook_params params; - hook_proc = wine_server_get_ptr( msg_data->winevent.hook_proc ); + params.proc = wine_server_get_ptr( msg_data->winevent.hook_proc ); size -= sizeof(msg_data->winevent); if (size) { - WCHAR module[MAX_PATH]; - - size = min( size, (MAX_PATH - 1) * sizeof(WCHAR) ); - memcpy( module, &msg_data->winevent + 1, size ); - module[size / sizeof(WCHAR)] = 0; - if (!(hook_proc = get_hook_proc( hook_proc, module, &free_module ))) - { - ERR( "invalid winevent hook module name %s\n", debugstr_w(module) ); - continue; - } + size = min( size, sizeof(params.module) - sizeof(WCHAR) ); + memcpy( params.module, &msg_data->winevent + 1, size ); } - - TRACE_(relay)( "\1Call winevent proc %p (hook=%04x,event=%x,hwnd=%p,object_id=%lx,child_id=%lx,tid=%04x,time=%x)\n", - hook_proc, msg_data->winevent.hook, info.msg.message, info.msg.hwnd, - info.msg.wParam, info.msg.lParam, msg_data->winevent.tid, info.msg.time); - - hook_proc( wine_server_ptr_handle( msg_data->winevent.hook ), info.msg.message, - info.msg.hwnd, info.msg.wParam, info.msg.lParam, - msg_data->winevent.tid, info.msg.time ); - - TRACE_(relay)( "\1Ret winevent proc %p (hook=%04x,event=%x,hwnd=%p,object_id=%lx,child_id=%lx,tid=%04x,time=%x)\n", - hook_proc, msg_data->winevent.hook, info.msg.message, info.msg.hwnd, - info.msg.wParam, info.msg.lParam, msg_data->winevent.tid, info.msg.time); - - if (free_module) FreeLibrary(free_module); + params.module[size / sizeof(WCHAR)] = 0; + size = FIELD_OFFSET( struct win_hook_params, module[size / sizeof(WCHAR) + 1] ); + + params.handle = wine_server_ptr_handle( msg_data->winevent.hook ); + params.event = info.msg.message; + params.hwnd = info.msg.hwnd; + params.object_id = info.msg.wParam; + params.child_id = info.msg.lParam; + params.tid = msg_data->winevent.tid; + params.time = info.msg.time; + + User32CallWinEventHook( &params, size ); } continue; case MSG_HOOK_LL: diff --git a/dlls/win32u/hook.c b/dlls/win32u/hook.c index e1851aa896a..21e00d1ff3c 100644 --- a/dlls/win32u/hook.c +++ b/dlls/win32u/hook.c @@ -476,6 +476,7 @@ void WINAPI NtUserNotifyWinEvent( DWORD event, HWND hwnd, LONG object_id, LONG c info.hwnd = hwnd; info.object_id = object_id; info.child_id = child_id; + info.tid = GetCurrentThreadId(); SERVER_START_REQ( start_hook_chain ) { @@ -502,6 +503,7 @@ void WINAPI NtUserNotifyWinEvent( DWORD event, HWND hwnd, LONG object_id, LONG c TRACE( "calling WH_WINEVENT hook %p event %x hwnd %p %x %x module %s\n", info.proc, event, hwnd, object_id, child_id, debugstr_w(info.module) ); + info.time = NtGetTickCount(); KeUserModeCallback( NtUserCallWinEventHook, &info, FIELD_OFFSET( struct win_hook_params, module[lstrlenW(info.module) + 1] ), &ret_ptr, &ret_len ); diff --git a/include/ntuser.h b/include/ntuser.h index 746549e4b28..5b7cc0be597 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -58,6 +58,8 @@ struct win_event_hook_params LONG object_id; LONG child_id; void *handle; + DWORD tid; + DWORD time; WINEVENTPROC proc; WCHAR module[MAX_PATH]; };

1 0

Jacek Caban : win32u: Move init_win_proc_params implementation from user32.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: c1a9b0b6a98faddf5b64ac99f3fed759b0199f46 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c1a9b0b6a98faddf5b64ac99… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 23 14:00:56 2022 +0100 win32u: Move init_win_proc_params implementation from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/winproc.c | 46 ++++++++------------------------------------ dlls/win32u/class.c | 21 +++++++++++++++++++- dlls/win32u/message.c | 22 +++++++++++++++++++++ dlls/win32u/ntuser_private.h | 1 + include/ntuser.h | 1 + 5 files changed, 52 insertions(+), 39 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index c46cd373b4e..f3d5d913ac5 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -851,40 +851,6 @@ BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, } -static void init_win_proc_params( struct win_proc_params *info, WNDPROC func, HWND hwnd, UINT msg, - WPARAM wparam, LPARAM lparam, LRESULT *result, BOOL ansi ) -{ - WINDOWPROC *proc; - - USER_CheckNotLock(); - - info->hwnd = WIN_GetFullHandle( hwnd ); - info->msg = msg; - info->wparam = wparam; - info->lparam = lparam; - info->result = result; - info->ansi = info->ansi_dst = ansi; - info->is_dialog = FALSE; - info->mapping = WMCHAR_MAP_CALLWINDOWPROC; - info->dpi_awareness = GetWindowDpiAwarenessContext( info->hwnd ); - info->func = func; - - if (!(proc = handle_to_proc( func ))) - { - info->procW = info->procA = NULL; - } - else if (proc == WINPROC_PROC16) - { - info->procW = info->procA = WINPROC_PROC16; - } - else - { - info->procA = proc->procA; - info->procW = proc->procW; - } -} - - /********************************************************************** * CallWindowProcA (USER32.@) * @@ -914,8 +880,10 @@ LRESULT WINAPI CallWindowProcA( WNDPROC func, HWND hwnd, UINT msg, WPARAM wParam struct win_proc_params params; LRESULT result; - if (!func) return 0; - init_win_proc_params( &params, func, hwnd, msg, wParam, lParam, &result, TRUE ); + params.func = func; + params.result = &result; + if (!NtUserMessageCall( hwnd, msg, wParam, lParam, (ULONG_PTR)&params, FNID_CALLWNDPROC, TRUE )) + return 0; dispatch_win_proc_params( &params ); return result; } @@ -931,8 +899,10 @@ LRESULT WINAPI CallWindowProcW( WNDPROC func, HWND hwnd, UINT msg, WPARAM wParam struct win_proc_params params; LRESULT result; - if (!func) return 0; - init_win_proc_params( &params, func, hwnd, msg, wParam, lParam, &result, FALSE ); + params.func = func; + params.result = &result; + if (!NtUserMessageCall( hwnd, msg, wParam, lParam, (ULONG_PTR)&params, FNID_CALLWNDPROC, FALSE )) + return 0; dispatch_win_proc_params( &params ); return result; } diff --git a/dlls/win32u/class.c b/dlls/win32u/class.c index 9ac402025b0..a030f5d6c0d 100644 --- a/dlls/win32u/class.c +++ b/dlls/win32u/class.c @@ -37,7 +37,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(class); WINE_DECLARE_DEBUG_CHANNEL(win); #define MAX_WINPROCS 4096 -#define WINPROC_PROC16 ((WINDOWPROC *)1) /* placeholder for 16-bit window procs */ +#define WINPROC_PROC16 ((void *)1) /* placeholder for 16-bit window procs */ typedef struct tagCLASS { @@ -184,6 +184,25 @@ BOOL is_winproc_unicode( WNDPROC proc, BOOL def_val ) return ptr->procW != NULL; } +void get_winproc_params( struct win_proc_params *params ) +{ + WINDOWPROC *proc = get_winproc_ptr( params->func ); + + if (!proc) + { + params->procW = params->procA = NULL; + } + else if (proc == WINPROC_PROC16) + { + params->procW = params->procA = WINPROC_PROC16; + } + else + { + params->procA = proc->procA; + params->procW = proc->procW; + } +} + /*********************************************************************** * NtUserInitializeClientPfnArrays (win32u.@) */ diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index 7730de22fef..b9f9f22378e 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -32,6 +32,25 @@ WINE_DEFAULT_DEBUG_CHANNEL(msg); +static BOOL init_win_proc_params( struct win_proc_params *params, HWND hwnd, UINT msg, + WPARAM wparam, LPARAM lparam, BOOL ansi ) +{ + if (!params->func) return FALSE; + + user_check_not_lock(); + + params->hwnd = get_full_window_handle( hwnd ); + params->msg = msg; + params->wparam = wparam; + params->lparam = lparam; + params->ansi = params->ansi_dst = ansi; + params->is_dialog = FALSE; + params->mapping = WMCHAR_MAP_CALLWINDOWPROC; + params->dpi_awareness = get_window_dpi_awareness_context( params->hwnd ); + get_winproc_params( params ); + return TRUE; +} + /*********************************************************************** * handle_internal_message * @@ -269,6 +288,9 @@ BOOL WINAPI NtUserMessageCall( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam { switch (type) { + case FNID_CALLWNDPROC: + return init_win_proc_params( (struct win_proc_params *)result_info, hwnd, msg, + wparam, lparam, ansi ); case FNID_SENDMESSAGE: return send_window_message( hwnd, msg, wparam, lparam, (LRESULT *)result_info, ansi ); case FNID_SENDNOTIFYMESSAGE: diff --git a/dlls/win32u/ntuser_private.h b/dlls/win32u/ntuser_private.h index 14bd98a307b..7c03474286d 100644 --- a/dlls/win32u/ntuser_private.h +++ b/dlls/win32u/ntuser_private.h @@ -243,6 +243,7 @@ ULONG_PTR get_class_long_ptr( HWND hwnd, INT offset, BOOL ansi ) DECLSPEC_HIDDEN WORD get_class_word( HWND hwnd, INT offset ) DECLSPEC_HIDDEN; ATOM get_int_atom_value( UNICODE_STRING *name ) DECLSPEC_HIDDEN; WNDPROC get_winproc( WNDPROC proc, BOOL ansi ) DECLSPEC_HIDDEN; +void get_winproc_params( struct win_proc_params *params ) DECLSPEC_HIDDEN; struct dce *get_class_dce( struct tagCLASS *class ) DECLSPEC_HIDDEN; struct dce *set_class_dce( struct tagCLASS *class, struct dce *dce ) DECLSPEC_HIDDEN; diff --git a/include/ntuser.h b/include/ntuser.h index 12e9b5b3025..e50dcf078ad 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -220,6 +220,7 @@ enum /* NtUserMessageCall codes */ enum { + FNID_CALLWNDPROC = 0x02ab, FNID_SENDMESSAGE = 0x02b1, FNID_SENDNOTIFYMESSAGE = 0x02b7, /* Wine-specific exports */

1 0

Jacek Caban : user32: Factor out dispatch_win_proc_params.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: a821155f21425b91e392bc67ace5e5904281f7e5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a821155f21425b91e392bc67… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 23 14:00:21 2022 +0100 user32: Factor out dispatch_win_proc_params. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/user_private.h | 13 ---- dlls/user32/winproc.c | 174 ++++++++++++++++++++++++++++----------------- include/ntuser.h | 30 ++++++++ 3 files changed, 140 insertions(+), 77 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=a821155f21425b91e392…

1 0

Matteo Bruni : d3d10: Do not accept any compute shader in D3D10ReflectShader().
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: c36a25897c23e6353065335bbb900b5010e7ac76 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c36a25897c23e6353065335b… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Wed Mar 23 13:58:17 2022 +0100 d3d10: Do not accept any compute shader in D3D10ReflectShader(). Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dcompiler_43/reflection.c | 11 ++++++++-- dlls/d3dcompiler_43/tests/reflection.c | 39 +++++++++++++++++++++++++++++----- 2 files changed, 43 insertions(+), 7 deletions(-) diff --git a/dlls/d3dcompiler_43/reflection.c b/dlls/d3dcompiler_43/reflection.c index 096efc91bd4..0fa0dc16839 100644 --- a/dlls/d3dcompiler_43/reflection.c +++ b/dlls/d3dcompiler_43/reflection.c @@ -52,6 +52,8 @@ enum D3DCOMPILER_SIGNATURE_ELEMENT_SIZE #define D3DCOMPILER_SHADER_TARGET_SHADERTYPE_MASK 0xffff0000 #define D3DCOMPILER_SHADER_TARGET_SHADERTYPE_SHIFT 16 +#define D3DCOMPILER_SHDR_SHADER_TYPE_CS 0x4353 + enum d3dcompiler_shader_type { D3DCOMPILER_SHADER_TYPE_CS = 5, @@ -1487,6 +1489,9 @@ static HRESULT d3dcompiler_parse_rdef(struct d3dcompiler_shader_reflection *r, c size_t size = data_size >> 2; uint32_t target_version; const char *ptr = data; +#if D3D_COMPILER_VERSION < 47 + uint32_t shader_type; +#endif HRESULT hr; TRACE("Size %Iu.\n", size); @@ -1508,8 +1513,10 @@ static HRESULT d3dcompiler_parse_rdef(struct d3dcompiler_shader_reflection *r, c target_version = r->target & D3DCOMPILER_SHADER_TARGET_VERSION_MASK; #if D3D_COMPILER_VERSION < 47 - if (target_version >= 0x501 && (!D3D_COMPILER_VERSION || ((r->target & D3DCOMPILER_SHADER_TARGET_SHADERTYPE_MASK) - >> D3DCOMPILER_SHADER_TARGET_SHADERTYPE_SHIFT) != 0x4353 /* CS */)) + shader_type = (r->target & D3DCOMPILER_SHADER_TARGET_SHADERTYPE_MASK) + >> D3DCOMPILER_SHADER_TARGET_SHADERTYPE_SHIFT; + if ((target_version >= 0x501 && shader_type != D3DCOMPILER_SHDR_SHADER_TYPE_CS) + || (!D3D_COMPILER_VERSION && shader_type == D3DCOMPILER_SHDR_SHADER_TYPE_CS)) { WARN("Target version %#x is not supported in d3dcompiler %u.\n", target_version, D3D_COMPILER_VERSION); return E_INVALIDARG; diff --git a/dlls/d3dcompiler_43/tests/reflection.c b/dlls/d3dcompiler_43/tests/reflection.c index bfd79d04618..70f3a51683a 100644 --- a/dlls/d3dcompiler_43/tests/reflection.c +++ b/dlls/d3dcompiler_43/tests/reflection.c @@ -1198,9 +1198,32 @@ static const D3D12_SHADER_INPUT_BIND_DESC test_reflection_bound_resources_result {"c2", D3D_SIT_CBUFFER, 1, 1, 0, 0, D3D_SRV_DIMENSION_UNKNOWN, 0, 0, 1}, }; -#if D3D_COMPILER_VERSION static void test_reflection_cs(void) { + /* + * fxc.exe /T cs_4_0 /Fo + */ +#if 0 + [numthreads(16, 8, 4)] + void main( uint3 DTid : SV_DispatchThreadID ) + { + } +#endif + static const DWORD test_blob_cs_4_0[] = + { + 0x43425844, 0x698a31ca, 0x8c6eee35, 0x2377107a, 0xe1e69066, 0x00000001, 0x00000150, 0x00000005, + 0x00000034, 0x0000008c, 0x0000009c, 0x000000ac, 0x000000d4, 0x46454452, 0x00000050, 0x00000000, + 0x00000000, 0x00000000, 0x0000001c, 0x43530400, 0x00000100, 0x0000001c, 0x7263694d, 0x666f736f, + 0x52282074, 0x4c482029, 0x53204c53, 0x65646168, 0x6f432072, 0x6c69706d, 0x39207265, 0x2e39322e, + 0x2e323539, 0x31313133, 0xababab00, 0x4e475349, 0x00000008, 0x00000000, 0x00000008, 0x4e47534f, + 0x00000008, 0x00000000, 0x00000008, 0x58454853, 0x00000020, 0x00050040, 0x00000008, 0x0100086a, + 0x0400009b, 0x00000010, 0x00000008, 0x00000004, 0x0100003e, 0x54415453, 0x00000074, 0x00000001, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000001, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, + }; + /* * fxc.exe /T cs_5_1 /Fo */ @@ -1228,10 +1251,17 @@ static void test_reflection_cs(void) }; UINT size_x, size_y, size_z, size_total; ID3D11ShaderReflection *ref11; - HRESULT hr; + HRESULT hr, expected; + + expected = D3D_COMPILER_VERSION ? S_OK : E_INVALIDARG; + hr = call_reflect(test_blob_cs_4_0, test_blob_cs_4_0[6], &IID_ID3D11ShaderReflection, (void **)&ref11); + ok(hr == expected, "Unexpected hr %#lx.\n", hr); + + if (SUCCEEDED(hr)) + ref11->lpVtbl->Release(ref11); hr = call_reflect(test_blob, test_blob[6], &IID_ID3D11ShaderReflection, (void **)&ref11); - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + ok(hr == expected, "Unexpected hr %#lx.\n", hr); if (FAILED(hr)) return; size_total = ref11->lpVtbl->GetThreadGroupSize(ref11, &size_x, &size_y, &size_z); @@ -1242,7 +1272,6 @@ static void test_reflection_cs(void) ref11->lpVtbl->Release(ref11); } -#endif static void test_reflection_bound_resources(const DWORD *blob, const D3D12_SHADER_INPUT_BIND_DESC *result, unsigned int result_count, unsigned int target_version) @@ -2208,10 +2237,10 @@ START_TEST(reflection) #if D3D_COMPILER_VERSION test_reflection_references(); test_reflection_interfaces(); - test_reflection_cs(); #else test_d3d10_interfaces(); #endif + test_reflection_cs(); test_reflection_desc_vs(); test_reflection_desc_ps(); test_reflection_desc_ps_output();

1 0

Paul Gofman : d3dcompiler: Implement d3dcompiler_shader_reflection_GetThreadGroupSize().
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: 47b9c450c6cc7f41205ad40cdd6942d4bc782b47 URL: https://source.winehq.org/git/wine.git/?a=commit;h=47b9c450c6cc7f41205ad40c… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Wed Mar 23 13:58:15 2022 +0100 d3dcompiler: Implement d3dcompiler_shader_reflection_GetThreadGroupSize(). Signed-off-by: Paul Gofman <pgofman(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dcompiler_43/reflection.c | 92 ++++++++++++++++++++++++++++++++-- dlls/d3dcompiler_43/tests/reflection.c | 60 ++++++++++++++++++++++ 2 files changed, 148 insertions(+), 4 deletions(-) diff --git a/dlls/d3dcompiler_43/reflection.c b/dlls/d3dcompiler_43/reflection.c index 91cc20cdded..787d0388f93 100644 --- a/dlls/d3dcompiler_43/reflection.c +++ b/dlls/d3dcompiler_43/reflection.c @@ -50,6 +50,12 @@ enum D3DCOMPILER_SIGNATURE_ELEMENT_SIZE #define D3DCOMPILER_SHADER_TARGET_VERSION_MASK 0xffff #define D3DCOMPILER_SHADER_TARGET_SHADERTYPE_MASK 0xffff0000 +#define D3DCOMPILER_SHADER_TARGET_SHADERTYPE_SHIFT 16 + +enum d3dcompiler_shader_type +{ + D3DCOMPILER_SHADER_TYPE_CS = 5, +}; struct d3dcompiler_shader_signature { @@ -158,6 +164,9 @@ struct d3dcompiler_shader_reflection D3D_TESSELLATOR_OUTPUT_PRIMITIVE hs_output_primitive; D3D_TESSELLATOR_PARTITIONING hs_partitioning; D3D_TESSELLATOR_DOMAIN tessellator_domain; + UINT thread_group_size_x; + UINT thread_group_size_y; + UINT thread_group_size_z; struct d3dcompiler_shader_signature *isgn; struct d3dcompiler_shader_signature *osgn; @@ -707,9 +716,21 @@ static HRESULT STDMETHODCALLTYPE d3dcompiler_shader_reflection_GetMinFeatureLeve static UINT STDMETHODCALLTYPE d3dcompiler_shader_reflection_GetThreadGroupSize( ID3D11ShaderReflection *iface, UINT *sizex, UINT *sizey, UINT *sizez) { - FIXME("iface %p, sizex %p, sizey %p, sizez %p stub!\n", iface, sizex, sizey, sizez); + struct d3dcompiler_shader_reflection *reflection = impl_from_ID3D11ShaderReflection(iface); - return 0; + TRACE("iface %p, sizex %p, sizey %p, sizez %p.\n", iface, sizex, sizey, sizez); + + if (!sizex || !sizey || !sizez) + { + WARN("Invalid argument specified.\n"); + return E_INVALIDARG; + } + + *sizex = reflection->thread_group_size_x; + *sizey = reflection->thread_group_size_y; + *sizez = reflection->thread_group_size_z; + + return *sizex * *sizey * *sizez; } static UINT64 STDMETHODCALLTYPE d3dcompiler_shader_reflection_GetRequiresFlags( @@ -1754,15 +1775,79 @@ static HRESULT d3dcompiler_parse_signature(struct d3dcompiler_shader_signature * return S_OK; } +#define SM4_OPCODE_MASK 0xff +#define SM4_INSTRUCTION_LENGTH_SHIFT 24 +#define SM4_INSTRUCTION_LENGTH_MASK (0x1fu << SM4_INSTRUCTION_LENGTH_SHIFT) + +enum sm4_opcode +{ + SM5_OP_DCL_THREAD_GROUP = 0x9b, +}; + static HRESULT d3dcompiler_parse_shdr(struct d3dcompiler_shader_reflection *r, const char *data, size_t data_size) { + uint32_t opcode_token, opcode; + uint32_t size, shader_type; const char *ptr = data; + const uint32_t *u_ptr; + unsigned int len; r->version = read_u32(&ptr); TRACE("Shader version: %u\n", r->version); - /* todo: Check if anything else is needed from the shdr or shex blob. */ + shader_type = (r->version & D3DCOMPILER_SHADER_TARGET_SHADERTYPE_MASK) + >> D3DCOMPILER_SHADER_TARGET_SHADERTYPE_SHIFT; + if (shader_type != D3DCOMPILER_SHADER_TYPE_CS) + { + /* TODO: Check if anything else is needed from the SHDR or SHEX blob. */ + return S_OK; + } + + size = read_u32(&ptr); + TRACE("size %u.\n", size); + if (size * sizeof(uint32_t) != data_size || size < 2) + { + WARN("Invalid size %u.\n", size); + return E_FAIL; + } + size -= 2; + u_ptr = (uint32_t *)ptr; + while (size) + { + opcode_token = *u_ptr; + opcode = opcode_token & SM4_OPCODE_MASK; + len = (opcode_token & SM4_INSTRUCTION_LENGTH_MASK) >> SM4_INSTRUCTION_LENGTH_SHIFT; + if (!len) + { + if (size < 2) + { + WARN("End of byte-code, failed to read length token.\n"); + return E_FAIL; + } + len = u_ptr[1]; + } + if (!len || size < len) + { + WARN("Invalid instruction length %u, size %u.\n", len, size); + return E_FAIL; + } + if (opcode == SM5_OP_DCL_THREAD_GROUP) + { + if (len != 4) + { + WARN("Invalid dcl_thread_group opcode length %u.\n", len); + return E_FAIL; + } + r->thread_group_size_x = u_ptr[1]; + r->thread_group_size_y = u_ptr[2]; + r->thread_group_size_z = u_ptr[3]; + TRACE("Found dcl_thread_group %u, %u, %u.\n", + r->thread_group_size_x, r->thread_group_size_y, r->thread_group_size_z); + } + size -= len; + u_ptr += len; + } return S_OK; } @@ -1972,7 +2057,6 @@ HRESULT WINAPI D3DReflect(const void *data, SIZE_T data_size, REFIID riid, void } *reflector = object; - TRACE("Created ID3D11ShaderReflection %p\n", object); return S_OK; diff --git a/dlls/d3dcompiler_43/tests/reflection.c b/dlls/d3dcompiler_43/tests/reflection.c index f2b7f094de6..e4261ad0d79 100644 --- a/dlls/d3dcompiler_43/tests/reflection.c +++ b/dlls/d3dcompiler_43/tests/reflection.c @@ -777,6 +777,7 @@ static void test_reflection_desc_ps(void) D3D_NAME expected; unsigned int i; #if D3D_COMPILER_VERSION + UINT size_x, size_y, size_z, size_total; UINT ret; #endif @@ -920,6 +921,17 @@ static void test_reflection_desc_ps(void) ok(!desc.Stream, "(%u): got unexpected Stream %u.\n", i, desc.Stream); } +#if D3D_COMPILER_VERSION + size_x = 0xdeadbeef; + size_y = 0xdeadbeef; + size_z = 0xdeadbeef; + size_total = ref11->lpVtbl->GetThreadGroupSize(ref11, &size_x, &size_y, &size_z); + ok(!size_x, "Unexpected size %u.\n", size_x); + ok(!size_y, "Unexpected size %u.\n", size_y); + ok(!size_z, "Unexpected size %u.\n", size_z); + ok(!size_total, "Unexpected size %u.\n", size_total); +#endif + count = ref11->lpVtbl->Release(ref11); ok(!count, "Got unexpected count %lu.\n", count); } @@ -1186,6 +1198,53 @@ static const D3D12_SHADER_INPUT_BIND_DESC test_reflection_bound_resources_result {"c2", D3D_SIT_CBUFFER, 1, 1, 0, 0, D3D_SRV_DIMENSION_UNKNOWN, 0, 0, 1}, }; +#if D3D_COMPILER_VERSION +static void test_reflection_cs(void) +{ + /* + * fxc.exe /T cs_5_1 /Fo + */ +#if 0 + [numthreads(16, 8, 4)] + void main( uint3 DTid : SV_DispatchThreadID ) + { + } +#endif + static const DWORD test_blob[] = + { + 0x43425844, 0x77a220d9, 0xfebd5b9c, 0x14a86e30, 0x57537394, 0x00000001, 0x00000184, 0x00000005, + 0x00000034, 0x000000a0, 0x000000b0, 0x000000c0, 0x000000e8, 0x46454452, 0x00000064, 0x00000000, + 0x00000000, 0x00000000, 0x0000003c, 0x43530501, 0x00000500, 0x0000003c, 0x25441313, 0x0000003c, + 0x00000018, 0x00000028, 0x00000028, 0x00000024, 0x0000000c, 0x00000000, 0x7263694d, 0x666f736f, + 0x52282074, 0x4c482029, 0x53204c53, 0x65646168, 0x6f432072, 0x6c69706d, 0x31207265, 0x00312e30, + 0x4e475349, 0x00000008, 0x00000000, 0x00000008, 0x4e47534f, 0x00000008, 0x00000000, 0x00000008, + 0x58454853, 0x00000020, 0x00050051, 0x00000008, 0x0100086a, 0x0400009b, 0x00000010, 0x00000008, + 0x00000004, 0x0100003e, 0x54415453, 0x00000094, 0x00000001, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000001, 0x00000000, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, + }; + UINT size_x, size_y, size_z, size_total; + ID3D11ShaderReflection *ref11; + HRESULT hr; + + hr = call_reflect(test_blob, test_blob[6], &IID_ID3D11ShaderReflection, (void **)&ref11); + todo_wine_if(D3D_COMPILER_VERSION < 47) + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + if (FAILED(hr)) + return; + size_total = ref11->lpVtbl->GetThreadGroupSize(ref11, &size_x, &size_y, &size_z); + ok(size_x == 16, "Unexpected size %u.\n", size_x); + ok(size_y == 8, "Unexpected size %u.\n", size_y); + ok(size_z == 4, "Unexpected size %u.\n", size_z); + ok(size_total == size_x * size_y * size_z, "Unexpected size %u.\n", size_total); + + ref11->lpVtbl->Release(ref11); +} +#endif + static void test_reflection_bound_resources(const DWORD *blob, const D3D12_SHADER_INPUT_BIND_DESC *result, unsigned int result_count, unsigned int target_version) { @@ -2150,6 +2209,7 @@ START_TEST(reflection) #if D3D_COMPILER_VERSION test_reflection_references(); test_reflection_interfaces(); + test_reflection_cs(); #else test_d3d10_interfaces(); #endif

1 0

Jacek Caban : win32u: Implement NtUserDispatchMessage.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: 1e5dc840f7c36928cdcfb42736283d55afa62def URL: https://source.winehq.org/git/wine.git/?a=commit;h=1e5dc840f7c36928cdcfb427… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 23 14:01:18 2022 +0100 win32u: Implement NtUserDispatchMessage. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/message.c | 50 +-------------------- dlls/user32/user_main.c | 1 + dlls/user32/user_private.h | 1 + dlls/user32/winproc.c | 9 +++- dlls/win32u/gdiobj.c | 1 + dlls/win32u/message.c | 105 +++++++++++++++++++++++++++++++++++++++++++ dlls/win32u/sysparams.c | 2 + dlls/win32u/win32u.spec | 2 +- dlls/win32u/win32u_private.h | 2 + dlls/win32u/wrappers.c | 6 +++ include/ntuser.h | 3 ++ 11 files changed, 131 insertions(+), 51 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=1e5dc840f7c36928cdcf…

1 0

Jacek Caban : user32: Use dispatch_win_proc_params in CallWindowProc.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: afddda2ce6596d864efd0228eb2f69bb0348ae37 URL: https://source.winehq.org/git/wine.git/?a=commit;h=afddda2ce6596d864efd0228… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 23 14:06:49 2022 +0100 user32: Use dispatch_win_proc_params in CallWindowProc. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/winproc.c | 70 +++++++++++++++++++++++++++++---------------------- 1 file changed, 40 insertions(+), 30 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index adbd5aeb8f6..b3da8d26cb3 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -850,6 +850,37 @@ BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, } +static void init_win_proc_params( struct win_proc_params *info, WNDPROC func, HWND hwnd, UINT msg, + WPARAM wparam, LPARAM lparam, LRESULT *result, BOOL ansi ) +{ + WINDOWPROC *proc; + + info->hwnd = hwnd; + info->msg = msg; + info->wparam = wparam; + info->lparam = lparam; + info->result = result; + info->ansi = info->ansi_dst = ansi; + info->is_dialog = FALSE; + info->mapping = WMCHAR_MAP_CALLWINDOWPROC; + info->func = func; + + if (!(proc = handle_to_proc( func ))) + { + info->procW = info->procA = NULL; + } + else if (proc == WINPROC_PROC16) + { + info->procW = info->procA = WINPROC_PROC16; + } + else + { + info->procA = proc->procA; + info->procW = proc->procW; + } +} + + /********************************************************************** * CallWindowProcA (USER32.@) * @@ -874,27 +905,14 @@ BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, * * ECMA-234, Win32 */ -LRESULT WINAPI CallWindowProcA( - WNDPROC func, /* [in] window procedure */ - HWND hwnd, /* [in] target window */ - UINT msg, /* [in] message */ - WPARAM wParam, /* [in] message dependent parameter */ - LPARAM lParam /* [in] message dependent parameter */ -) { - WINDOWPROC *proc; +LRESULT WINAPI CallWindowProcA( WNDPROC func, HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam ) +{ + struct win_proc_params params; LRESULT result; if (!func) return 0; - - if (!(proc = handle_to_proc( func ))) - call_window_proc( hwnd, msg, wParam, lParam, &result, func ); - else if (proc == WINPROC_PROC16) - wow_handlers.call_window_proc( hwnd, msg, wParam, lParam, &result, func ); - else if (proc->procA) - call_window_proc( hwnd, msg, wParam, lParam, &result, proc->procA ); - else - WINPROC_CallProcAtoW( call_window_proc, hwnd, msg, wParam, lParam, &result, - proc->procW, WMCHAR_MAP_CALLWINDOWPROC ); + init_win_proc_params( &params, func, hwnd, msg, wParam, lParam, &result, TRUE ); + dispatch_win_proc_params( &params ); return result; } @@ -904,22 +922,14 @@ LRESULT WINAPI CallWindowProcA( * * See CallWindowProcA. */ -LRESULT WINAPI CallWindowProcW( WNDPROC func, HWND hwnd, UINT msg, - WPARAM wParam, LPARAM lParam ) +LRESULT WINAPI CallWindowProcW( WNDPROC func, HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam ) { - WINDOWPROC *proc; + struct win_proc_params params; LRESULT result; if (!func) return 0; - - if (!(proc = handle_to_proc( func ))) - call_window_proc( hwnd, msg, wParam, lParam, &result, func ); - else if (proc == WINPROC_PROC16) - WINPROC_CallProcWtoA( wow_handlers.call_window_proc, hwnd, msg, wParam, lParam, &result, func ); - else if (proc->procW) - call_window_proc( hwnd, msg, wParam, lParam, &result, proc->procW ); - else - WINPROC_CallProcWtoA( call_window_proc, hwnd, msg, wParam, lParam, &result, proc->procA ); + init_win_proc_params( &params, func, hwnd, msg, wParam, lParam, &result, FALSE ); + dispatch_win_proc_params( &params ); return result; }

1 0

Jacek Caban : user32: Move more code out of call_window_proc.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: b8a805e2f2c2411aa3ed6a4650d6ab3a5a852f78 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b8a805e2f2c2411aa3ed6a46… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 23 14:00:43 2022 +0100 user32: Move more code out of call_window_proc. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/winproc.c | 20 ++++++++++++-------- include/ntuser.h | 1 + 2 files changed, 13 insertions(+), 8 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index b3da8d26cb3..c46cd373b4e 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -119,18 +119,12 @@ static WPARAM map_wparam_char_WtoA( WPARAM wParam, DWORD len ) /* call a 32-bit window procedure */ static LRESULT call_window_proc( HWND hwnd, UINT msg, WPARAM wp, LPARAM lp, LRESULT *result, void *arg ) { - DPI_AWARENESS_CONTEXT context; WNDPROC proc = arg; - USER_CheckNotLock(); - - hwnd = WIN_GetFullHandle( hwnd ); TRACE_(relay)( "\1Call window proc %p (hwnd=%p,msg=%s,wp=%08lx,lp=%08lx)\n", proc, hwnd, SPY_GetMsgName(msg, hwnd), wp, lp ); - context = SetThreadDpiAwarenessContext( GetWindowDpiAwarenessContext( hwnd )); *result = WINPROC_wrapper( proc, hwnd, msg, wp, lp ); - SetThreadDpiAwarenessContext( context ); TRACE_(relay)( "\1Ret window proc %p (hwnd=%p,msg=%s,wp=%08lx,lp=%08lx) retval=%08lx\n", proc, hwnd, SPY_GetMsgName(msg, hwnd), wp, lp, *result ); @@ -711,6 +705,10 @@ static LRESULT WINPROC_CallProcWtoA( winproc_callback_t callback, HWND hwnd, UIN static void dispatch_win_proc_params( struct win_proc_params *params ) { + DPI_AWARENESS_CONTEXT context = SetThreadDpiAwarenessContext( params->dpi_awareness ); + + USER_CheckNotLock(); + if (!params->ansi) { if (params->procW == WINPROC_PROC16) @@ -789,6 +787,8 @@ static void dispatch_win_proc_params( struct win_proc_params *params ) call_window_proc( params->hwnd, params->msg, params->wparam, params->lparam, params->result, params->func ); } + + SetThreadDpiAwarenessContext( context ); } @@ -818,13 +818,14 @@ BOOL WINPROC_call_window( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, params.is_dialog = wndPtr->dlgInfo != NULL; WIN_ReleasePtr( wndPtr ); - params.hwnd = hwnd; + params.hwnd = WIN_GetFullHandle( hwnd ); params.msg = msg; params.wparam = wParam; params.lparam = lParam; params.result = result; params.ansi = !unicode; params.mapping = mapping; + params.dpi_awareness = GetWindowDpiAwarenessContext( params.hwnd ); if (!proc) { @@ -855,7 +856,9 @@ static void init_win_proc_params( struct win_proc_params *info, WNDPROC func, HW { WINDOWPROC *proc; - info->hwnd = hwnd; + USER_CheckNotLock(); + + info->hwnd = WIN_GetFullHandle( hwnd ); info->msg = msg; info->wparam = wparam; info->lparam = lparam; @@ -863,6 +866,7 @@ static void init_win_proc_params( struct win_proc_params *info, WNDPROC func, HW info->ansi = info->ansi_dst = ansi; info->is_dialog = FALSE; info->mapping = WMCHAR_MAP_CALLWINDOWPROC; + info->dpi_awareness = GetWindowDpiAwarenessContext( info->hwnd ); info->func = func; if (!(proc = handle_to_proc( func ))) diff --git a/include/ntuser.h b/include/ntuser.h index 19f303f0dae..12e9b5b3025 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -87,6 +87,7 @@ struct win_proc_params BOOL ansi_dst; BOOL is_dialog; enum wm_char_mapping mapping; + DPI_AWARENESS_CONTEXT dpi_awareness; WNDPROC procA; WNDPROC procW; };

1 0

Eric Pouech : ntdll/tests: Enable compilation with long types in rtl.c.
by Alexandre Julliard March 23, 2022

March 23, 2022

Module: wine Branch: master Commit: d66d642d74846d5d04abe31286c8e424b91ba741 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d66d642d74846d5d04abe312… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Wed Mar 23 08:13:32 2022 +0100 ntdll/tests: Enable compilation with long types in rtl.c. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/rtl.c | 543 +++++++++++++++++++++++++------------------------ 1 file changed, 272 insertions(+), 271 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=d66d642d74846d5d04ab…

1 0