December 2021 - Wine-commits

Jacek Caban : win32u: Add partial wow64 support.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: ac9119badbd2d600430d93670219e6b9de0af5b5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ac9119badbd2d600430d9367… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:07:35 2021 +0100 win32u: Add partial wow64 support. 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/win32u/syscall.c | 18 ++++++ dlls/win32u/wrappers.c | 145 +++++++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 157 insertions(+), 6 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=ac9119badbd2d600430d…

1 0

Jacek Caban : user32: Provide partial null driver.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: bc12c971c3c61c617b0332d1bce8c6404d43e364 URL: https://source.winehq.org/git/wine.git/?a=commit;h=bc12c971c3c61c617b0332d1… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:08:07 2021 +0100 user32: Provide partial null driver. Provide functions that we still call from user32 to avoid crashes on wow64. 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/driver.c | 106 +++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 106 insertions(+) diff --git a/dlls/user32/driver.c b/dlls/user32/driver.c index 295a7ee2cd7..f0150b16643 100644 --- a/dlls/user32/driver.c +++ b/dlls/user32/driver.c @@ -119,10 +119,46 @@ void USER_unload_driver(void) * These are fallbacks for entry points that are not implemented in the real driver. */ +static void CDECL nulldrv_Beep(void) +{ +} + +static BOOL CDECL nulldrv_RegisterHotKey( HWND hwnd, UINT modifiers, UINT vk ) +{ + return TRUE; +} + +static void CDECL nulldrv_UnregisterHotKey( HWND hwnd, UINT modifiers, UINT vk ) +{ +} + static void CDECL nulldrv_DestroyCursorIcon( HCURSOR cursor ) { } +static void CDECL nulldrv_SetCursor( HCURSOR cursor ) +{ +} + +static BOOL CDECL nulldrv_GetCursorPos( LPPOINT pt ) +{ + return TRUE; +} + +static BOOL CDECL nulldrv_SetCursorPos( INT x, INT y ) +{ + return TRUE; +} + +static BOOL CDECL nulldrv_ClipCursor( LPCRECT clip ) +{ + return TRUE; +} + +static void CDECL nulldrv_UpdateClipboard(void) +{ +} + static BOOL CDECL nodrv_CreateWindow( HWND hwnd ) { static int warned; @@ -137,10 +173,29 @@ static BOOL CDECL nodrv_CreateWindow( HWND hwnd ) return FALSE; } +static BOOL CDECL nulldrv_CreateDesktopWindow( HWND hwnd ) +{ + return TRUE; +} + +static BOOL CDECL nulldrv_CreateWindow( HWND hwnd ) +{ + return TRUE; +} + static void CDECL nulldrv_DestroyWindow( HWND hwnd ) { } +static void CDECL nulldrv_FlashWindowEx( FLASHWINFO *info ) +{ +} + +static void CDECL nulldrv_GetDC( HDC hdc, HWND hwnd, HWND top_win, const RECT *win_rect, + const RECT *top_rect, DWORD flags ) +{ +} + static DWORD CDECL nulldrv_MsgWaitForMultipleObjectsEx( DWORD count, const HANDLE *handles, DWORD timeout, DWORD mask, DWORD flags ) { @@ -161,10 +216,18 @@ static void CDECL nulldrv_SetFocus( HWND hwnd ) { } +static void CDECL nulldrv_SetLayeredWindowAttributes( HWND hwnd, COLORREF key, BYTE alpha, DWORD flags ) +{ +} + static void CDECL nulldrv_SetParent( HWND hwnd, HWND parent, HWND old_parent ) { } +static void CDECL nulldrv_SetWindowRgn( HWND hwnd, HRGN hrgn, BOOL redraw ) +{ +} + static void CDECL nulldrv_SetWindowIcon( HWND hwnd, UINT type, HICON icon ) { } @@ -187,6 +250,12 @@ static LRESULT CDECL nulldrv_SysCommand( HWND hwnd, WPARAM wparam, LPARAM lparam return -1; } +static BOOL CDECL nulldrv_UpdateLayeredWindow( HWND hwnd, const UPDATELAYEREDWINDOWINFO *info, + const RECT *window_rect ) +{ + return TRUE; +} + static LRESULT CDECL nulldrv_WindowMessage( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam ) { return 0; @@ -367,6 +436,43 @@ void CDECL __wine_set_user_driver( const struct user_driver_funcs *funcs, UINT v driver = HeapAlloc( GetProcessHeap(), 0, sizeof(*driver) ); *driver = *funcs; +#define SET_USER_FUNC(name) \ + do { if (!driver->p##name) driver->p##name = nulldrv_##name; } while(0) + + SET_USER_FUNC(Beep); + SET_USER_FUNC(RegisterHotKey); + SET_USER_FUNC(UnregisterHotKey); + SET_USER_FUNC(DestroyCursorIcon); + SET_USER_FUNC(SetCursor); + SET_USER_FUNC(GetCursorPos); + SET_USER_FUNC(SetCursorPos); + SET_USER_FUNC(ClipCursor); + SET_USER_FUNC(UpdateClipboard); + SET_USER_FUNC(CreateDesktopWindow); + SET_USER_FUNC(CreateWindow); + SET_USER_FUNC(DestroyWindow); + SET_USER_FUNC(FlashWindowEx); + SET_USER_FUNC(GetDC); + SET_USER_FUNC(MsgWaitForMultipleObjectsEx); + SET_USER_FUNC(ReleaseDC); + SET_USER_FUNC(SetCapture); + SET_USER_FUNC(SetFocus); + SET_USER_FUNC(SetLayeredWindowAttributes); + SET_USER_FUNC(SetParent); + SET_USER_FUNC(SetWindowRgn); + SET_USER_FUNC(SetWindowIcon); + SET_USER_FUNC(SetWindowStyle); + SET_USER_FUNC(SetWindowText); + SET_USER_FUNC(ShowWindow); + SET_USER_FUNC(SysCommand); + SET_USER_FUNC(UpdateLayeredWindow); + SET_USER_FUNC(WindowMessage); + SET_USER_FUNC(WindowPosChanging); + SET_USER_FUNC(WindowPosChanged); + SET_USER_FUNC(SystemParametersInfo); + SET_USER_FUNC(ThreadDetach); +#undef SET_USER_FUNC + prev = InterlockedCompareExchangePointer( (void **)&USER_Driver, driver, &lazy_load_driver ); if (prev != &lazy_load_driver) {

1 0

Jacek Caban : win32u: Implement NtUserGetProcessDpiAwarenessContext and NtUserSetProcessDpiAwarenessContext.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: aab62429c2aac5dbd6539c0aa7b0b6767ad95540 URL: https://source.winehq.org/git/wine.git/?a=commit;h=aab62429c2aac5dbd6539c0a… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:08:21 2021 +0100 win32u: Implement NtUserGetProcessDpiAwarenessContext and NtUserSetProcessDpiAwarenessContext. And use them in 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/sysparams.c | 45 ++++++++++++++++++++++----------------------- dlls/win32u/syscall.c | 2 ++ dlls/win32u/sysparams.c | 38 ++++++++++++++++++++++++++++++++++++++ dlls/win32u/win32u.spec | 4 ++-- dlls/wow64win/syscall.h | 2 ++ dlls/wow64win/user.c | 15 +++++++++++++++ include/ntuser.h | 9 +++++++++ 7 files changed, 90 insertions(+), 25 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=aab62429c2aac5dbd653…

1 0

Jacek Caban : win32u: Implement NtUserGetSystemDpiForProcess.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: d08808d8f3fefc3e558e9b3d349428719a5ada47 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d08808d8f3fefc3e558e9b3d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:08:45 2021 +0100 win32u: Implement NtUserGetSystemDpiForProcess. 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/sysparams.c | 15 +-- dlls/win32u/font.c | 6 +- dlls/win32u/syscall.c | 1 + dlls/win32u/sysparams.c | 306 ++++++++++++++++++++++++++++++++++++++++++- dlls/win32u/win32u.spec | 2 +- dlls/win32u/win32u_private.h | 3 +- dlls/wow64win/syscall.h | 1 + dlls/wow64win/user.c | 7 + include/ntuser.h | 1 + 9 files changed, 319 insertions(+), 23 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=d08808d8f3fefc3e558e…

1 0

Jacek Caban : win32u: Move NtUserGetDpiForMonitor implementation from user32.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: b877c78ec53b072c94211198ba08140b5b04489f URL: https://source.winehq.org/git/wine.git/?a=commit;h=b877c78ec53b072c94211198… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:09:07 2021 +0100 win32u: Move NtUserGetDpiForMonitor 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/sysparams.c | 24 ------------------- dlls/user32/user32.spec | 2 +- dlls/win32u/syscall.c | 1 + dlls/win32u/sysparams.c | 61 +++++++++++++++++++++++++++++++++++++++++++++++++ dlls/win32u/win32u.spec | 2 +- dlls/wow64win/syscall.h | 1 + dlls/wow64win/user.c | 10 ++++++++ include/ntuser.h | 1 + 8 files changed, 76 insertions(+), 26 deletions(-) diff --git a/dlls/user32/sysparams.c b/dlls/user32/sysparams.c index 36375dabd42..11a2532411d 100644 --- a/dlls/user32/sysparams.c +++ b/dlls/user32/sysparams.c @@ -3328,30 +3328,6 @@ UINT WINAPI GetDpiForSystem(void) return system_dpi; } -/*********************************************************************** - * GetDpiForMonitorInternal (USER32.@) - */ -BOOL WINAPI GetDpiForMonitorInternal( HMONITOR monitor, UINT type, UINT *x, UINT *y ) -{ - if (type > 2) - { - SetLastError( ERROR_BAD_ARGUMENTS ); - return FALSE; - } - if (!x || !y) - { - SetLastError( ERROR_INVALID_ADDRESS ); - return FALSE; - } - switch (GetAwarenessFromDpiAwarenessContext( GetThreadDpiAwarenessContext() )) - { - case DPI_AWARENESS_UNAWARE: *x = *y = USER_DEFAULT_SCREEN_DPI; break; - case DPI_AWARENESS_SYSTEM_AWARE: *x = *y = system_dpi; break; - default: *x = *y = get_monitor_dpi( monitor ); break; - } - return TRUE; -} - /********************************************************************** * GetThreadDpiAwarenessContext (USER32.@) */ diff --git a/dlls/user32/user32.spec b/dlls/user32/user32.spec index 20d89c14a84..0f5e880263f 100644 --- a/dlls/user32/user32.spec +++ b/dlls/user32/user32.spec @@ -298,7 +298,7 @@ @ stdcall GetDlgItemTextA(long long ptr long) @ stdcall GetDlgItemTextW(long long ptr long) @ stdcall GetDoubleClickTime() -@ stdcall GetDpiForMonitorInternal(long long ptr ptr) +@ stdcall GetDpiForMonitorInternal(long long ptr ptr) NtUserGetDpiForMonitor @ stdcall GetDpiForSystem() @ stdcall GetDpiForWindow(long) @ stdcall GetFocus() diff --git a/dlls/win32u/syscall.c b/dlls/win32u/syscall.c index e85633b041c..2e8b2408f31 100644 --- a/dlls/win32u/syscall.c +++ b/dlls/win32u/syscall.c @@ -112,6 +112,7 @@ static void * const syscalls[] = NtUserGetClipboardSequenceNumber, NtUserGetClipboardViewer, NtUserGetCursor, + NtUserGetDpiForMonitor, NtUserGetKeyState, NtUserGetKeyboardLayout, NtUserGetKeyboardLayoutName, diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index edc37e7e80f..74ab94aa818 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -1345,6 +1345,43 @@ RECT get_virtual_screen_rect(void) return rect; } +/********************************************************************** + * get_monitor_dpi + */ +static UINT get_monitor_dpi( HMONITOR monitor ) +{ + /* FIXME: use the monitor DPI instead */ + return system_dpi; +} + +/********************************************************************** + * get_thread_dpi_awareness + */ +static DPI_AWARENESS get_thread_dpi_awareness(void) +{ + struct user_thread_info *info = get_user_thread_info(); + ULONG_PTR context = info->dpi_awareness; + + if (!context) context = NtUserGetProcessDpiAwarenessContext( NULL ); + + switch (context) + { + case 0x10: + case 0x11: + case 0x12: + case 0x80000010: + case 0x80000011: + case 0x80000012: + return context & 3; + case (ULONG_PTR)DPI_AWARENESS_CONTEXT_UNAWARE: + case (ULONG_PTR)DPI_AWARENESS_CONTEXT_SYSTEM_AWARE: + case (ULONG_PTR)DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE: + return ~context; + default: + return DPI_AWARENESS_INVALID; + } +} + /********************************************************************** * NtUserGetDisplayConfigBufferSizes (win32u.@) */ @@ -1764,6 +1801,30 @@ ULONG WINAPI NtUserGetSystemDpiForProcess( HANDLE process ) return system_dpi; } +/*********************************************************************** + * NtUserGetDpiForMonitor (win32u.@) + */ +BOOL WINAPI NtUserGetDpiForMonitor( HMONITOR monitor, UINT type, UINT *x, UINT *y ) +{ + if (type > 2) + { + SetLastError( ERROR_BAD_ARGUMENTS ); + return FALSE; + } + if (!x || !y) + { + SetLastError( ERROR_INVALID_ADDRESS ); + return FALSE; + } + switch (get_thread_dpi_awareness()) + { + case DPI_AWARENESS_UNAWARE: *x = *y = USER_DEFAULT_SCREEN_DPI; break; + case DPI_AWARENESS_SYSTEM_AWARE: *x = *y = system_dpi; break; + default: *x = *y = get_monitor_dpi( monitor ); break; + } + return TRUE; +} + /* retrieve the cached base keys for a given entry */ static BOOL get_base_keys( enum parameter_key index, HKEY *base_key, HKEY *volatile_key ) { diff --git a/dlls/win32u/win32u.spec b/dlls/win32u/win32u.spec index a95d6eca565..c073f14896b 100644 --- a/dlls/win32u/win32u.spec +++ b/dlls/win32u/win32u.spec @@ -924,7 +924,7 @@ @ stdcall NtUserGetDisplayConfigBufferSizes(long ptr ptr) @ stub NtUserGetDoubleClickTime @ stub NtUserGetDpiForCurrentProcess -@ stub NtUserGetDpiForMonitor +@ stdcall -syscall NtUserGetDpiForMonitor(long long ptr ptr) @ stub NtUserGetExtendedPointerDeviceProperty @ stub NtUserGetForegroundWindow @ stub NtUserGetGUIThreadInfo diff --git a/dlls/wow64win/syscall.h b/dlls/wow64win/syscall.h index 95a3df42805..bea1b00ebed 100644 --- a/dlls/wow64win/syscall.h +++ b/dlls/wow64win/syscall.h @@ -99,6 +99,7 @@ SYSCALL_ENTRY( NtUserGetClipboardSequenceNumber ) \ SYSCALL_ENTRY( NtUserGetClipboardViewer ) \ SYSCALL_ENTRY( NtUserGetCursor ) \ + SYSCALL_ENTRY( NtUserGetDpiForMonitor ) \ SYSCALL_ENTRY( NtUserGetKeyState ) \ SYSCALL_ENTRY( NtUserGetKeyboardLayout ) \ SYSCALL_ENTRY( NtUserGetKeyboardLayoutName ) \ diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index 1026bc94843..eb739181812 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -315,3 +315,13 @@ NTSTATUS WINAPI wow64_NtUserGetSystemDpiForProcess( UINT *args ) return NtUserGetSystemDpiForProcess( process ); } + +NTSTATUS WINAPI wow64_NtUserGetDpiForMonitor( UINT *args ) +{ + HMONITOR monitor = get_handle( &args ); + UINT type = get_ulong( &args ); + UINT *x = get_ptr( &args ); + UINT *y = get_ptr( &args ); + + return NtUserGetDpiForMonitor( monitor, type, x, y ); +} diff --git a/include/ntuser.h b/include/ntuser.h index 2b747a856e8..5049926aa78 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -115,6 +115,7 @@ HWND WINAPI NtUserGetClipboardViewer(void); HCURSOR WINAPI NtUserGetCursor(void); LONG WINAPI NtUserGetDisplayConfigBufferSizes( UINT32 flags, UINT32 *num_path_info, UINT32 *num_mode_info ); +BOOL WINAPI NtUserGetDpiForMonitor( HMONITOR monitor, UINT type, UINT *x, UINT *y ); INT WINAPI NtUserGetKeyNameText( LONG lparam, WCHAR *buffer, INT size ); SHORT WINAPI NtUserGetKeyState( INT vkey ); HKL WINAPI NtUserGetKeyboardLayout( DWORD thread_id );

1 0

Jacek Caban : win32u: Move DPI mapping from GetMonitorInfoW.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: 7aab49d736449d9f8e72a588111df934bb822f23 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7aab49d736449d9f8e72a588… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:09:37 2021 +0100 win32u: Move DPI mapping from GetMonitorInfoW. 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/sysparams.c | 17 +---------------- dlls/win32u/sysparams.c | 38 ++++++++++++++++++++++++++++++++++++++ 2 files changed, 39 insertions(+), 16 deletions(-) diff --git a/dlls/user32/sysparams.c b/dlls/user32/sysparams.c index 11a2532411d..56f1c444072 100644 --- a/dlls/user32/sysparams.c +++ b/dlls/user32/sysparams.c @@ -3543,22 +3543,7 @@ BOOL WINAPI GetMonitorInfoA( HMONITOR monitor, LPMONITORINFO info ) */ BOOL WINAPI GetMonitorInfoW( HMONITOR monitor, LPMONITORINFO info ) { - BOOL ret; - UINT dpi_from, dpi_to; - - ret = NtUserCallTwoParam( HandleToUlong(monitor), (ULONG_PTR)info, NtUserGetMonitorInfo ); - if (ret) - { - if ((dpi_to = get_thread_dpi())) - { - dpi_from = get_monitor_dpi( monitor ); - info->rcMonitor = map_dpi_rect( info->rcMonitor, dpi_from, dpi_to ); - info->rcWork = map_dpi_rect( info->rcWork, dpi_from, dpi_to ); - } - TRACE( "flags %04x, monitor %s, work %s\n", info->dwFlags, - wine_dbgstr_rect(&info->rcMonitor), wine_dbgstr_rect(&info->rcWork)); - } - return ret; + return NtUserCallTwoParam( HandleToUlong(monitor), (ULONG_PTR)info, NtUserGetMonitorInfo ); } struct enum_mon_data diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 74ab94aa818..4ff6f81846a 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -1382,6 +1382,34 @@ static DPI_AWARENESS get_thread_dpi_awareness(void) } } +/********************************************************************** + * get_thread_dpi + */ +static UINT get_thread_dpi(void) +{ + switch (get_thread_dpi_awareness()) + { + case DPI_AWARENESS_UNAWARE: return USER_DEFAULT_SCREEN_DPI; + case DPI_AWARENESS_SYSTEM_AWARE: return system_dpi; + default: return 0; /* no scaling */ + } +} + +/********************************************************************** + * map_dpi_rect + */ +RECT map_dpi_rect( RECT rect, UINT dpi_from, UINT dpi_to ) +{ + if (dpi_from && dpi_to && dpi_from != dpi_to) + { + rect.left = muldiv( rect.left, dpi_to, dpi_from ); + rect.top = muldiv( rect.top, dpi_to, dpi_from ); + rect.right = muldiv( rect.right, dpi_to, dpi_from ); + rect.bottom = muldiv( rect.bottom, dpi_to, dpi_from ); + } + return rect; +} + /********************************************************************** * NtUserGetDisplayConfigBufferSizes (win32u.@) */ @@ -1756,6 +1784,7 @@ BOOL WINAPI NtUserEnumDisplayMonitors( HDC hdc, RECT *rect, MONITORENUMPROC proc static BOOL get_monitor_info( HMONITOR handle, MONITORINFO *info ) { struct monitor *monitor; + UINT dpi_from, dpi_to; if (info->cbSize != sizeof(MONITORINFOEXW) && info->cbSize != sizeof(MONITORINFO)) return FALSE; @@ -1778,6 +1807,15 @@ static BOOL get_monitor_info( HMONITOR handle, MONITORINFO *info ) asciiz_to_unicode( ((MONITORINFOEXW *)info)->szDevice, "WinDisc" ); } unlock_display_devices(); + + if ((dpi_to = get_thread_dpi())) + { + dpi_from = get_monitor_dpi( handle ); + info->rcMonitor = map_dpi_rect( info->rcMonitor, dpi_from, dpi_to ); + info->rcWork = map_dpi_rect( info->rcWork, dpi_from, dpi_to ); + } + TRACE( "flags %04x, monitor %s, work %s\n", info->dwFlags, + wine_dbgstr_rect(&info->rcMonitor), wine_dbgstr_rect(&info->rcWork)); return TRUE; }

1 0

Jacek Caban : win32u: Move DPI mapping from EnumDisplayMonitors.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: 3570b0913340047a246eb0ad9af68a29b631f106 URL: https://source.winehq.org/git/wine.git/?a=commit;h=3570b0913340047a246eb0ad… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:10:01 2021 +0100 win32u: Move DPI mapping from EnumDisplayMonitors. 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/sysparams.c | 51 +++++++------------------------------------------ dlls/user32/user32.spec | 2 +- dlls/win32u/sysparams.c | 29 +++++++++++++++++++++++++++- 3 files changed, 36 insertions(+), 46 deletions(-) diff --git a/dlls/user32/sysparams.c b/dlls/user32/sysparams.c index 56f1c444072..bbeea5317b6 100644 --- a/dlls/user32/sysparams.c +++ b/dlls/user32/sysparams.c @@ -327,7 +327,7 @@ RECT get_virtual_screen_rect(void) { RECT rect = {0}; - EnumDisplayMonitors( 0, NULL, get_virtual_screen_proc, (LPARAM)&rect ); + NtUserEnumDisplayMonitors( 0, NULL, get_virtual_screen_proc, (LPARAM)&rect ); return rect; } @@ -348,7 +348,7 @@ RECT get_primary_monitor_rect(void) { RECT rect = {0}; - EnumDisplayMonitors( 0, NULL, get_primary_monitor_proc, (LPARAM)&rect ); + NtUserEnumDisplayMonitors( 0, NULL, get_primary_monitor_proc, (LPARAM)&rect ); return rect; } @@ -364,7 +364,7 @@ static INT get_monitor_count(void) { INT count = 0; - EnumDisplayMonitors( 0, NULL, get_monitor_count_proc, (LPARAM)&count ); + NtUserEnumDisplayMonitors( 0, NULL, get_monitor_count_proc, (LPARAM)&count ); return count; } @@ -1747,7 +1747,7 @@ BOOL WINAPI SystemParametersInfoW( UINT uiAction, UINT uiParam, if (!spi_loaded[spi_idx]) { work_area = get_primary_monitor_rect(); - EnumDisplayMonitors( 0, NULL, enum_monitors, (LPARAM)&work_area ); + NtUserEnumDisplayMonitors( 0, NULL, enum_monitors, (LPARAM)&work_area ); spi_loaded[spi_idx] = TRUE; } *(RECT*)pvParam = work_area; @@ -3469,7 +3469,7 @@ HMONITOR WINAPI MonitorFromRect( const RECT *rect, DWORD flags ) info.rect.bottom = info.rect.top + 1; } - if (!EnumDisplayMonitors( 0, NULL, monitor_enum, (LPARAM)&info )) return 0; + if (!NtUserEnumDisplayMonitors( 0, NULL, monitor_enum, (LPARAM)&info )) return 0; if (!info.ret) { if (flags & MONITOR_DEFAULTTOPRIMARY) info.ret = info.primary; @@ -3546,13 +3546,6 @@ BOOL WINAPI GetMonitorInfoW( HMONITOR monitor, LPMONITORINFO info ) return NtUserCallTwoParam( HandleToUlong(monitor), (ULONG_PTR)info, NtUserGetMonitorInfo ); } -struct enum_mon_data -{ - LPARAM lparam; - POINT origin; - RECT limit; -}; - #ifdef __i386__ /* Some apps pass a non-stdcall callback to EnumDisplayMonitors, * so we need a small assembly wrapper to call it. @@ -3580,44 +3573,14 @@ __ASM_GLOBAL_FUNC( enum_mon_callback_wrapper, BOOL WINAPI User32CallEnumDisplayMonitor( struct enum_display_monitor_params *params, ULONG size ) { - struct enum_mon_data *data = (struct enum_mon_data *)params->lparam; - RECT monrect = map_dpi_rect( params->rect, get_monitor_dpi( params->monitor ), get_thread_dpi() ); - - /* FIXME: move DPI conversion and rect filtering to win32u */ - OffsetRect( &monrect, -data->origin.x, -data->origin.y ); - if (!IntersectRect( &monrect, &monrect, &data->limit )) return TRUE; #ifdef __i386__ return enum_mon_callback_wrapper( params->proc, params->monitor, params->hdc, - &monrect, data->lparam ); + &params->rect, params->lparam ); #else - return params->proc( params->monitor, params->hdc, &monrect, data->lparam ); + return params->proc( params->monitor, params->hdc, &params->rect, params->lparam ); #endif } -/*********************************************************************** - * EnumDisplayMonitors (USER32.@) - */ -BOOL WINAPI EnumDisplayMonitors( HDC hdc, LPRECT rect, MONITORENUMPROC proc, LPARAM lp ) -{ - struct enum_mon_data data; - - data.lparam = lp; - - if (hdc) - { - if (!GetDCOrgEx( hdc, &data.origin )) return FALSE; - if (GetClipBox( hdc, &data.limit ) == ERROR) return FALSE; - } - else - { - data.origin.x = data.origin.y = 0; - data.limit.left = data.limit.top = INT_MIN; - data.limit.right = data.limit.bottom = INT_MAX; - } - if (rect && !IntersectRect( &data.limit, &data.limit, rect )) return TRUE; - return NtUserEnumDisplayMonitors( hdc, rect, proc, (LPARAM)&data ); -} - /*********************************************************************** * EnumDisplayDevicesA (USER32.@) */ diff --git a/dlls/user32/user32.spec b/dlls/user32/user32.spec index 0f5e880263f..d82c4949c68 100644 --- a/dlls/user32/user32.spec +++ b/dlls/user32/user32.spec @@ -223,7 +223,7 @@ @ stub EnumDisplayDeviceModesW @ stdcall EnumDisplayDevicesA(str long ptr long) @ stdcall EnumDisplayDevicesW(wstr long ptr long) -@ stdcall EnumDisplayMonitors(long ptr ptr long) +@ stdcall EnumDisplayMonitors(long ptr ptr long) NtUserEnumDisplayMonitors @ stdcall EnumDisplaySettingsA(str long ptr) @ stdcall EnumDisplaySettingsExA(str long ptr long) @ stdcall EnumDisplaySettingsExW(wstr long ptr long) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 4ff6f81846a..8efe17aed3c 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -1741,8 +1741,27 @@ BOOL WINAPI NtUserEnumDisplayMonitors( HDC hdc, RECT *rect, MONITORENUMPROC proc struct enum_display_monitor_params params; struct monitor *monitor; unsigned int count = 0, i; + POINT origin; + RECT limit; BOOL ret = TRUE; + if (hdc) + { + DC *dc; + if (!(dc = get_dc_ptr( hdc ))) return FALSE; + origin.x = dc->attr->vis_rect.left; + origin.y = dc->attr->vis_rect.top; + release_dc_ptr( dc ); + if (NtGdiGetAppClipBox( hdc, &limit ) == ERROR) return FALSE; + } + else + { + origin.x = origin.y = 0; + limit.left = limit.top = INT_MIN; + limit.right = limit.bottom = INT_MAX; + } + if (rect && !intersect_rect( &limit, &limit, rect )) return TRUE; + if (!lock_display_devices()) return FALSE; count = list_count( &monitors ); @@ -1756,9 +1775,17 @@ BOOL WINAPI NtUserEnumDisplayMonitors( HDC hdc, RECT *rect, MONITORENUMPROC proc count = 0; LIST_FOR_EACH_ENTRY(monitor, &monitors, struct monitor, entry) { + RECT monrect; + if (!(monitor->dev.state_flags & DISPLAY_DEVICE_ACTIVE)) continue; + + monrect = map_dpi_rect( monitor->rc_monitor, get_monitor_dpi( monitor->handle ), + get_thread_dpi() ); + offset_rect( &monrect, -origin.x, -origin.y ); + if (!intersect_rect( &monrect, &monrect, &limit )) continue; + enum_info[count].handle = monitor->handle; - enum_info[count].rect = monitor->rc_monitor; + enum_info[count].rect = monrect; count++; }

1 0

Jacek Caban : win32u: Directly access system DPI in ntgdi.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: 338179ae34029ecfea2d4705fa164f258567c6c7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=338179ae34029ecfea2d4705… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:10:15 2021 +0100 win32u: Directly access system DPI in ntgdi. 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/win32u/gdiobj.c | 10 ---------- dlls/win32u/sysparams.c | 7 +++++++ dlls/win32u/win32u_private.h | 2 +- dlls/win32u/wrappers.c | 9 --------- 4 files changed, 8 insertions(+), 20 deletions(-) diff --git a/dlls/win32u/gdiobj.c b/dlls/win32u/gdiobj.c index 796ae460a17..768bf3a89db 100644 --- a/dlls/win32u/gdiobj.c +++ b/dlls/win32u/gdiobj.c @@ -572,16 +572,6 @@ BOOL GDI_dec_ref_count( HGDIOBJ handle ) } -/****************************************************************************** - * get_system_dpi - * - * Get the system DPI, based on the DPI awareness mode. - */ -DWORD get_system_dpi(void) -{ - return user_callbacks ? user_callbacks->pGetDpiForSystem() : 96; -} - static HFONT create_font( const LOGFONTW *deffont ) { ENUMLOGFONTEXDVW lf; diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 8efe17aed3c..f8e462ed2a8 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -1395,6 +1395,13 @@ static UINT get_thread_dpi(void) } } +/* see GetDpiForSystem */ +UINT get_system_dpi(void) +{ + if (get_thread_dpi_awareness() == DPI_AWARENESS_UNAWARE) return USER_DEFAULT_SCREEN_DPI; + return system_dpi; +} + /********************************************************************** * map_dpi_rect */ diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index 1182902921f..ac560d54b9e 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -33,7 +33,6 @@ struct user_callbacks { HWND (WINAPI *pGetDesktopWindow)(void); - UINT (WINAPI *pGetDpiForSystem)(void); BOOL (WINAPI *pGetMonitorInfoW)( HMONITOR, LPMONITORINFO ); INT (WINAPI *pGetSystemMetrics)(INT); BOOL (WINAPI *pGetWindowRect)( HWND hwnd, LPRECT rect ); @@ -241,6 +240,7 @@ struct unix_funcs struct window_surface *surface ); }; +extern UINT get_system_dpi(void) DECLSPEC_HIDDEN; extern RECT get_virtual_screen_rect(void) DECLSPEC_HIDDEN; extern void wrappers_init( unixlib_handle_t handle ) DECLSPEC_HIDDEN; diff --git a/dlls/win32u/wrappers.c b/dlls/win32u/wrappers.c index d482c71375d..559044d69ff 100644 --- a/dlls/win32u/wrappers.c +++ b/dlls/win32u/wrappers.c @@ -940,14 +940,6 @@ static HWND WINAPI call_GetDesktopWindow(void) return pGetDesktopWindow ? pGetDesktopWindow() : NULL; } -static UINT WINAPI call_GetDpiForSystem(void) -{ - static UINT (WINAPI *pGetDpiForSystem)(void); - if (!pGetDpiForSystem) - pGetDpiForSystem = get_user_proc( "GetDpiForSystem", FALSE ); - return pGetDpiForSystem ? pGetDpiForSystem() : 96; -} - static BOOL WINAPI call_GetMonitorInfoW( HMONITOR monitor, MONITORINFO *info ) { static BOOL (WINAPI *pGetMonitorInfoW)( HMONITOR, LPMONITORINFO ); @@ -1000,7 +992,6 @@ static HWND WINAPI call_WindowFromDC( HDC hdc ) static const struct user_callbacks user_funcs = { call_GetDesktopWindow, - call_GetDpiForSystem, call_GetMonitorInfoW, call_GetSystemMetrics, call_GetWindowRect,

1 0

Jacek Caban : win32u: Move system parameters implementation from user32.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: fdf3015816168678c919ab282cf4e88704e931bc URL: https://source.winehq.org/git/wine.git/?a=commit;h=fdf3015816168678c919ab28… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:15:59 2021 +0100 win32u: Move system parameters 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/driver.c | 8 +- dlls/user32/message.c | 2 +- dlls/user32/sysparams.c | 2549 ++---------------------------------------- dlls/user32/user32.spec | 4 +- dlls/win32u/driver.c | 5 +- dlls/win32u/gdiobj.c | 3 + dlls/win32u/syscall.c | 1 + dlls/win32u/sysparams.c | 2338 +++++++++++++++++++++++++++++++++++++- dlls/win32u/win32u.spec | 8 +- dlls/win32u/win32u_private.h | 7 +- dlls/win32u/wrappers.c | 28 + dlls/wow64win/syscall.h | 1 + dlls/wow64win/user.c | 5 + include/ntuser.h | 11 + 14 files changed, 2440 insertions(+), 2530 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=fdf3015816168678c919…

1 0

Jacek Caban : win32u: Directly use sysparams in nulldrv_GetDeviceCaps.
by Alexandre Julliard 06 Dec '21

06 Dec '21

Module: wine Branch: master Commit: 4ca8d676fb15dddceb6e286dfaa8bc7109d77cd7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=4ca8d676fb15dddceb6e286d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 6 03:16:10 2021 +0100 win32u: Directly use sysparams in nulldrv_GetDeviceCaps. 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/win32u/driver.c | 50 ++++++++------------------------------------ dlls/win32u/ntgdi_private.h | 1 - dlls/win32u/sysparams.c | 20 +++++++++++++++++- dlls/win32u/win32u_private.h | 5 ++--- dlls/win32u/wrappers.c | 28 ------------------------- 5 files changed, 30 insertions(+), 74 deletions(-) diff --git a/dlls/win32u/driver.c b/dlls/win32u/driver.c index 158e844cccb..5fa1a2ca236 100644 --- a/dlls/win32u/driver.c +++ b/dlls/win32u/driver.c @@ -83,28 +83,6 @@ const struct gdi_dc_funcs *get_display_driver(void) return &user_driver->dc_funcs; } -struct monitor_info -{ - const WCHAR *name; - RECT rect; -}; - -static BOOL CALLBACK monitor_enum_proc( HMONITOR monitor, HDC hdc, LPRECT rect, LPARAM lparam ) -{ - struct monitor_info *info = (struct monitor_info *)lparam; - MONITORINFOEXW mi; - - mi.cbSize = sizeof(mi); - user_callbacks->pGetMonitorInfoW( monitor, (MONITORINFO *)&mi ); - if (!wcsicmp( info->name, mi.szDevice )) - { - info->rect = mi.rcMonitor; - return FALSE; - } - - return TRUE; -} - static INT CDECL nulldrv_AbortDoc( PHYSDEV dev ) { return 0; @@ -224,41 +202,31 @@ static INT CDECL nulldrv_GetDeviceCaps( PHYSDEV dev, INT cap ) case HORZRES: { DC *dc = get_nulldrv_dc( dev ); - struct monitor_info info; + RECT rect; int ret; - if (!user_callbacks) return 640; - if (dc->display[0]) { - info.name = dc->display; - SetRectEmpty( &info.rect ); - user_callbacks->pEnumDisplayMonitors( NULL, NULL, monitor_enum_proc, (LPARAM)&info ); - if (!IsRectEmpty( &info.rect )) - return info.rect.right - info.rect.left; + rect = get_display_rect( dc->display ); + if (!IsRectEmpty( &rect )) return rect.right - rect.left; } - ret = user_callbacks->pGetSystemMetrics( SM_CXSCREEN ); + ret = get_system_metrics( SM_CXSCREEN ); return ret ? ret : 640; } case VERTRES: { DC *dc = get_nulldrv_dc( dev ); - struct monitor_info info; + RECT rect; int ret; - if (!user_callbacks) return 480; - - if (dc->display[0] && user_callbacks) + if (dc->display[0]) { - info.name = dc->display; - SetRectEmpty( &info.rect ); - user_callbacks->pEnumDisplayMonitors( NULL, NULL, monitor_enum_proc, (LPARAM)&info ); - if (!IsRectEmpty( &info.rect )) - return info.rect.bottom - info.rect.top; + rect = get_display_rect( dc->display ); + if (!IsRectEmpty( &rect )) return rect.bottom - rect.top; } - ret = user_callbacks->pGetSystemMetrics( SM_CYSCREEN ); + ret = get_system_metrics( SM_CYSCREEN ); return ret ? ret : 480; } case BITSPIXEL: diff --git a/dlls/win32u/ntgdi_private.h b/dlls/win32u/ntgdi_private.h index 534687cb60b..13c365c075d 100644 --- a/dlls/win32u/ntgdi_private.h +++ b/dlls/win32u/ntgdi_private.h @@ -372,7 +372,6 @@ extern void GDI_ReleaseObj( HGDIOBJ ) DECLSPEC_HIDDEN; extern UINT GDI_get_ref_count( HGDIOBJ handle ) DECLSPEC_HIDDEN; extern HGDIOBJ GDI_inc_ref_count( HGDIOBJ handle ) DECLSPEC_HIDDEN; extern BOOL GDI_dec_ref_count( HGDIOBJ handle ) DECLSPEC_HIDDEN; -extern DWORD get_system_dpi(void) DECLSPEC_HIDDEN; extern HGDIOBJ get_stock_object( INT obj ) DECLSPEC_HIDDEN; extern DWORD get_gdi_object_type( HGDIOBJ obj ) DECLSPEC_HIDDEN; diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index e1f52461ca9..2072597a968 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -1462,6 +1462,24 @@ RECT get_virtual_screen_rect( UINT dpi ) return rect; } +RECT get_display_rect( const WCHAR *display ) +{ + struct monitor *monitor; + RECT rect = {0}; + + if (!lock_display_devices()) return rect; + + LIST_FOR_EACH_ENTRY( monitor, &monitors, struct monitor, entry ) + { + if (!monitor->adapter || wcsicmp( monitor->adapter->dev.device_name, display )) continue; + rect = monitor->rc_monitor; + break; + } + + unlock_display_devices(); + return map_dpi_rect( rect, system_dpi, get_thread_dpi() ); +} + static RECT get_primary_monitor_rect(void) { struct monitor *monitor; @@ -3907,7 +3925,7 @@ BOOL WINAPI NtUserSystemParametersInfo( UINT action, UINT val, void *ptr, UINT w #undef WINE_SPI_WARN } -static int get_system_metrics( int index ) +int get_system_metrics( int index ) { NONCLIENTMETRICSW ncm; MINIMIZEDMETRICS mm; diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index 3728c3b4065..7e867d4f736 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -33,10 +33,7 @@ struct user_callbacks { HWND (WINAPI *pGetDesktopWindow)(void); - BOOL (WINAPI *pGetMonitorInfoW)( HMONITOR, LPMONITORINFO ); - INT (WINAPI *pGetSystemMetrics)(INT); BOOL (WINAPI *pGetWindowRect)( HWND hwnd, LPRECT rect ); - BOOL (WINAPI *pEnumDisplayMonitors)( HDC, LPRECT, MONITORENUMPROC, LPARAM ); BOOL (WINAPI *pRedrawWindow)( HWND, const RECT*, HRGN, UINT ); LRESULT (WINAPI *pSendMessageTimeoutW)( HWND, UINT, WPARAM, LPARAM, UINT, UINT, PDWORD_PTR ); HWND (WINAPI *pWindowFromDC)( HDC ); @@ -245,7 +242,9 @@ struct unix_funcs struct window_surface *surface ); }; +extern RECT get_display_rect( const WCHAR *display ) DECLSPEC_HIDDEN; extern UINT get_system_dpi(void) DECLSPEC_HIDDEN; +extern int get_system_metrics( int index ) DECLSPEC_HIDDEN; extern RECT get_virtual_screen_rect( UINT dpi ) DECLSPEC_HIDDEN; extern void wrappers_init( unixlib_handle_t handle ) DECLSPEC_HIDDEN; diff --git a/dlls/win32u/wrappers.c b/dlls/win32u/wrappers.c index e81b202fc78..9bdd3ec3634 100644 --- a/dlls/win32u/wrappers.c +++ b/dlls/win32u/wrappers.c @@ -958,22 +958,6 @@ static HWND WINAPI call_GetDesktopWindow(void) return pGetDesktopWindow ? pGetDesktopWindow() : NULL; } -static BOOL WINAPI call_GetMonitorInfoW( HMONITOR monitor, MONITORINFO *info ) -{ - static BOOL (WINAPI *pGetMonitorInfoW)( HMONITOR, LPMONITORINFO ); - if (!pGetMonitorInfoW) - pGetMonitorInfoW = get_user_proc( "GetMonitorInfoW", FALSE ); - return pGetMonitorInfoW && pGetMonitorInfoW( monitor, info ); -} - -static INT WINAPI call_GetSystemMetrics( INT metric ) -{ - static INT (WINAPI *pGetSystemMetrics)(INT); - if (!pGetSystemMetrics) - pGetSystemMetrics = get_user_proc( "GetSystemMetrics", FALSE ); - return pGetSystemMetrics ? pGetSystemMetrics( metric ) : 0; -} - static BOOL WINAPI call_GetWindowRect( HWND hwnd, LPRECT rect ) { static BOOL (WINAPI *pGetWindowRect)( HWND hwnd, LPRECT rect ); @@ -982,15 +966,6 @@ static BOOL WINAPI call_GetWindowRect( HWND hwnd, LPRECT rect ) return pGetWindowRect && pGetWindowRect( hwnd, rect ); } -static BOOL WINAPI call_EnumDisplayMonitors( HDC hdc, RECT *rect, MONITORENUMPROC proc, - LPARAM lparam ) -{ - static BOOL (WINAPI *pEnumDisplayMonitors)( HDC, LPRECT, MONITORENUMPROC, LPARAM ); - if (!pEnumDisplayMonitors) - pEnumDisplayMonitors = get_user_proc( "EnumDisplayMonitors", FALSE ); - return pEnumDisplayMonitors && pEnumDisplayMonitors( hdc, rect, proc, lparam ); -} - static BOOL WINAPI call_RedrawWindow( HWND hwnd, const RECT *rect, HRGN rgn, UINT flags ) { static BOOL (WINAPI *pRedrawWindow)( HWND, const RECT*, HRGN, UINT ); @@ -1019,10 +994,7 @@ static HWND WINAPI call_WindowFromDC( HDC hdc ) static const struct user_callbacks user_funcs = { call_GetDesktopWindow, - call_GetMonitorInfoW, - call_GetSystemMetrics, call_GetWindowRect, - call_EnumDisplayMonitors, call_RedrawWindow, call_SendMessageTimeoutW, call_WindowFromDC,

1 0