September 2022 - Wine-commits

Rémi Bernon : win32u: Rename get_full_mode variables and avoid using devmode directly.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: 9cbc9386b940792142ba7b2640ec8aa57ba41360 URL: https://gitlab.winehq.org/wine/wine/-/commit/9cbc9386b940792142ba7b2640ec8a… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Sep 9 10:46:53 2022 +0200 win32u: Rename get_full_mode variables and avoid using devmode directly. --- dlls/win32u/sysparams.c | 39 ++++++++++++++++++++------------------- 1 file changed, 20 insertions(+), 19 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 981a11ab05f..4b6a2316c56 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -2140,9 +2140,11 @@ static DEVMODEW *find_display_mode( DEVMODEW *modes, DEVMODEW *devmode ) return NULL; } -static DEVMODEW *get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, DEVMODEW *modes, - DEVMODEW *devmode, DEVMODEW *temp_mode ) +static DEVMODEW *get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, DEVMODEW *adapter_modes, + DEVMODEW *devmode, DEVMODEW *full_mode ) { + DEVMODEW *adapter_mode; + if (devmode) { trace_devmode( devmode ); @@ -2156,40 +2158,39 @@ static DEVMODEW *get_full_mode( const WCHAR *adapter_path, const WCHAR *device_n devmode = NULL; } - if (devmode) memcpy( temp_mode, devmode, devmode->dmSize ); + if (devmode) memcpy( full_mode, devmode, devmode->dmSize ); else { - if (!read_registry_settings( adapter_path, temp_mode )) return NULL; + if (!read_registry_settings( adapter_path, full_mode )) return NULL; TRACE( "Return to original display mode\n" ); } - devmode = temp_mode; - if ((devmode->dmFields & (DM_PELSWIDTH | DM_PELSHEIGHT)) != (DM_PELSWIDTH | DM_PELSHEIGHT)) + if ((full_mode->dmFields & (DM_PELSWIDTH | DM_PELSHEIGHT)) != (DM_PELSWIDTH | DM_PELSHEIGHT)) { - WARN( "devmode doesn't specify the resolution: %#x\n", devmode->dmFields ); + WARN( "devmode doesn't specify the resolution: %#x\n", full_mode->dmFields ); return NULL; } - if (!is_detached_mode( devmode ) && (!devmode->dmPelsWidth || !devmode->dmPelsHeight || !(devmode->dmFields & DM_POSITION))) + if (!is_detached_mode( full_mode ) && (!full_mode->dmPelsWidth || !full_mode->dmPelsHeight || !(full_mode->dmFields & DM_POSITION))) { DEVMODEW current_mode = {.dmSize = sizeof(DEVMODEW)}; if (!user_driver->pGetCurrentDisplaySettings( device_name, &current_mode )) return NULL; - if (!devmode->dmPelsWidth) devmode->dmPelsWidth = current_mode.dmPelsWidth; - if (!devmode->dmPelsHeight) devmode->dmPelsHeight = current_mode.dmPelsHeight; - if (!(devmode->dmFields & DM_POSITION)) + if (!full_mode->dmPelsWidth) full_mode->dmPelsWidth = current_mode.dmPelsWidth; + if (!full_mode->dmPelsHeight) full_mode->dmPelsHeight = current_mode.dmPelsHeight; + if (!(full_mode->dmFields & DM_POSITION)) { - devmode->dmPosition = current_mode.dmPosition; - devmode->dmFields |= DM_POSITION; + full_mode->dmPosition = current_mode.dmPosition; + full_mode->dmFields |= DM_POSITION; } } - if ((devmode = find_display_mode( modes, devmode )) && devmode != temp_mode) + if ((adapter_mode = find_display_mode( adapter_modes, full_mode )) && adapter_mode != full_mode) { - devmode->dmFields |= DM_POSITION; - devmode->dmPosition = temp_mode->dmPosition; + adapter_mode->dmFields |= DM_POSITION; + adapter_mode->dmPosition = full_mode->dmPosition; } - return devmode; + return adapter_mode; } static DEVMODEW *get_display_settings( const WCHAR *devname, const DEVMODEW *devmode ) @@ -2446,7 +2447,7 @@ static LONG apply_display_settings( const WCHAR *devname, const DEVMODEW *devmod LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devmode, HWND hwnd, DWORD flags, void *lparam ) { - DEVMODEW *modes, temp_mode = {.dmSize = sizeof(DEVMODEW)}; + DEVMODEW *modes, full_mode = {.dmSize = sizeof(DEVMODEW)}; WCHAR device_name[CCHDEVICENAME], adapter_path[MAX_PATH]; LONG ret = DISP_CHANGE_SUCCESSFUL; struct adapter *adapter; @@ -2472,7 +2473,7 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm return DISP_CHANGE_BADPARAM; } - if (!(devmode = get_full_mode( adapter_path, device_name, modes, devmode, &temp_mode ))) ret = DISP_CHANGE_BADMODE; + if (!(devmode = get_full_mode( adapter_path, device_name, modes, devmode, &full_mode ))) ret = DISP_CHANGE_BADMODE; else if ((flags & CDS_UPDATEREGISTRY) && !write_registry_settings( adapter_path, devmode )) ret = DISP_CHANGE_NOTUPDATED; else if (flags & (CDS_TEST | CDS_NORESET)) ret = DISP_CHANGE_SUCCESSFUL; else ret = apply_display_settings( device_name, devmode, hwnd, flags, lparam );

1 0

Rémi Bernon : win32u: Fill the local full_mode variable instead of using the modes array.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: b197bd3bf8f0b4623a71a39c7a4b846245cca4e5 URL: https://gitlab.winehq.org/wine/wine/-/commit/b197bd3bf8f0b4623a71a39c7a4b84… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Sep 9 10:51:53 2022 +0200 win32u: Fill the local full_mode variable instead of using the modes array. --- dlls/win32u/sysparams.c | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 4b6a2316c56..2502232dd50 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -2108,9 +2108,9 @@ static BOOL is_detached_mode( const DEVMODEW *mode ) mode->dmPelsHeight == 0; } -static DEVMODEW *find_display_mode( DEVMODEW *modes, DEVMODEW *devmode ) +static const DEVMODEW *find_display_mode( const DEVMODEW *modes, DEVMODEW *devmode ) { - DEVMODEW *mode; + const DEVMODEW *mode; if (is_detached_mode( devmode )) return devmode; @@ -2140,16 +2140,16 @@ static DEVMODEW *find_display_mode( DEVMODEW *modes, DEVMODEW *devmode ) return NULL; } -static DEVMODEW *get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, DEVMODEW *adapter_modes, - DEVMODEW *devmode, DEVMODEW *full_mode ) +static BOOL get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, const DEVMODEW *adapter_modes, + const DEVMODEW *devmode, DEVMODEW *full_mode ) { - DEVMODEW *adapter_mode; + const DEVMODEW *adapter_mode; if (devmode) { trace_devmode( devmode ); - if (devmode->dmSize < offsetof(DEVMODEW, dmICMMethod)) return NULL; + if (devmode->dmSize < offsetof(DEVMODEW, dmICMMethod)) return FALSE; if (!is_detached_mode( devmode ) && (!(devmode->dmFields & DM_BITSPERPEL) || !devmode->dmBitsPerPel) && (!(devmode->dmFields & DM_PELSWIDTH) || !devmode->dmPelsWidth) && @@ -2161,20 +2161,20 @@ static DEVMODEW *get_full_mode( const WCHAR *adapter_path, const WCHAR *device_n if (devmode) memcpy( full_mode, devmode, devmode->dmSize ); else { - if (!read_registry_settings( adapter_path, full_mode )) return NULL; + if (!read_registry_settings( adapter_path, full_mode )) return FALSE; TRACE( "Return to original display mode\n" ); } if ((full_mode->dmFields & (DM_PELSWIDTH | DM_PELSHEIGHT)) != (DM_PELSWIDTH | DM_PELSHEIGHT)) { WARN( "devmode doesn't specify the resolution: %#x\n", full_mode->dmFields ); - return NULL; + return FALSE; } if (!is_detached_mode( full_mode ) && (!full_mode->dmPelsWidth || !full_mode->dmPelsHeight || !(full_mode->dmFields & DM_POSITION))) { DEVMODEW current_mode = {.dmSize = sizeof(DEVMODEW)}; - if (!user_driver->pGetCurrentDisplaySettings( device_name, &current_mode )) return NULL; + if (!user_driver->pGetCurrentDisplaySettings( device_name, &current_mode )) return FALSE; if (!full_mode->dmPelsWidth) full_mode->dmPelsWidth = current_mode.dmPelsWidth; if (!full_mode->dmPelsHeight) full_mode->dmPelsHeight = current_mode.dmPelsHeight; if (!(full_mode->dmFields & DM_POSITION)) @@ -2186,11 +2186,13 @@ static DEVMODEW *get_full_mode( const WCHAR *adapter_path, const WCHAR *device_n if ((adapter_mode = find_display_mode( adapter_modes, full_mode )) && adapter_mode != full_mode) { - adapter_mode->dmFields |= DM_POSITION; - adapter_mode->dmPosition = full_mode->dmPosition; + POINTL position = full_mode->dmPosition; + *full_mode = *adapter_mode; + full_mode->dmFields |= DM_POSITION; + full_mode->dmPosition = position; } - return adapter_mode; + return adapter_mode != NULL; } static DEVMODEW *get_display_settings( const WCHAR *devname, const DEVMODEW *devmode ) @@ -2473,10 +2475,10 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm return DISP_CHANGE_BADPARAM; } - if (!(devmode = get_full_mode( adapter_path, device_name, modes, devmode, &full_mode ))) ret = DISP_CHANGE_BADMODE; - else if ((flags & CDS_UPDATEREGISTRY) && !write_registry_settings( adapter_path, devmode )) ret = DISP_CHANGE_NOTUPDATED; + if (!get_full_mode( adapter_path, device_name, modes, devmode, &full_mode )) ret = DISP_CHANGE_BADMODE; + else if ((flags & CDS_UPDATEREGISTRY) && !write_registry_settings( adapter_path, &full_mode )) ret = DISP_CHANGE_NOTUPDATED; else if (flags & (CDS_TEST | CDS_NORESET)) ret = DISP_CHANGE_SUCCESSFUL; - else ret = apply_display_settings( device_name, devmode, hwnd, flags, lparam ); + else ret = apply_display_settings( device_name, &full_mode, hwnd, flags, lparam ); free( modes ); if (ret) ERR( "Changing %s display settings returned %d.\n", debugstr_us(devname), ret );

1 0

Rémi Bernon : win32u: Introduce reference counting to struct adapter.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: 75e62eb720c0672d39b6dcc08089ab44231795ee URL: https://gitlab.winehq.org/wine/wine/-/commit/75e62eb720c0672d39b6dcc08089ab… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Sep 9 11:24:45 2022 +0200 win32u: Introduce reference counting to struct adapter. --- dlls/win32u/sysparams.c | 41 ++++++++++++++++++++++++++++++++--------- 1 file changed, 32 insertions(+), 9 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 2502232dd50..007e7ab2473 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -217,6 +217,7 @@ struct display_device struct adapter { + LONG refcount; struct list entry; struct display_device dev; unsigned int id; @@ -425,6 +426,21 @@ static void release_display_device_init_mutex( HANDLE mutex ) NtClose( mutex ); } +static struct adapter *adapter_acquire( struct adapter *adapter ) +{ + InterlockedIncrement( &adapter->refcount ); + return adapter; +} + +static void adapter_release( struct adapter *adapter ) +{ + if (!InterlockedDecrement( &adapter->refcount )) + { + free( adapter->modes ); + free( adapter ); + } +} + static BOOL write_adapter_mode( HKEY adapter_key, DWORD index, const DEVMODEW *mode ) { WCHAR bufferW[MAX_PATH]; @@ -1450,6 +1466,7 @@ static void clear_display_devices(void) while (!list_empty( &monitors )) { monitor = LIST_ENTRY( list_head( &monitors ), struct monitor, entry ); + adapter_release( monitor->adapter ); list_remove( &monitor->entry ); free( monitor ); } @@ -1458,8 +1475,7 @@ static void clear_display_devices(void) { adapter = LIST_ENTRY( list_head( &adapters ), struct adapter, entry ); list_remove( &adapter->entry ); - free( adapter->modes ); - free( adapter ); + adapter_release( adapter ); } } @@ -1493,12 +1509,12 @@ static BOOL update_display_cache_from_registry(void) for (adapter_id = 0;; adapter_id++) { if (!(adapter = calloc( 1, sizeof(*adapter) ))) break; + adapter->refcount = 1; adapter->id = adapter_id; if (!read_display_adapter_settings( adapter_id, adapter )) { - free( adapter->modes ); - free( adapter ); + adapter_release( adapter ); break; } @@ -1506,15 +1522,15 @@ static BOOL update_display_cache_from_registry(void) for (monitor_id = 0;; monitor_id++) { if (!(monitor = calloc( 1, sizeof(*monitor) ))) break; - monitor->id = monitor_id; - monitor->adapter = adapter; - if (!read_monitor_settings( adapter, monitor_id, monitor )) { free( monitor ); break; } + monitor->id = monitor_id; + monitor->adapter = adapter_acquire( adapter ); + LIST_FOR_EACH_ENTRY(monitor2, &monitors, struct monitor, entry) { if (EqualRect(&monitor2->rc_monitor, &monitor->rc_monitor)) @@ -1957,16 +1973,19 @@ LONG WINAPI NtUserGetDisplayConfigBufferSizes( UINT32 flags, UINT32 *num_path_in return ERROR_SUCCESS; } +/* Find and acquire the adapter matching name, or primary adapter if name is NULL. + * If not NULL, the returned adapter needs to be released with adapter_release. + */ static struct adapter *find_adapter( UNICODE_STRING *name ) { struct adapter *adapter; LIST_FOR_EACH_ENTRY(adapter, &adapters, struct adapter, entry) { - if (!name || !name->Length) return adapter; /* use primary adapter */ + if (!name || !name->Length) return adapter_acquire( adapter ); /* use primary adapter */ if (!wcsnicmp( name->Buffer, adapter->dev.device_name, name->Length / sizeof(WCHAR) ) && !adapter->dev.device_name[name->Length / sizeof(WCHAR)]) - return adapter; + return adapter_acquire( adapter ); } return NULL; } @@ -2010,6 +2029,7 @@ NTSTATUS WINAPI NtUserEnumDisplayDevices( UNICODE_STRING *device, DWORD index, break; } } + adapter_release( adapter ); } if (found) @@ -2472,6 +2492,7 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm if (!adapter || !modes) { WARN( "Invalid device name %s.\n", debugstr_us(devname) ); + if (adapter) adapter_release( adapter ); return DISP_CHANGE_BADPARAM; } @@ -2480,6 +2501,7 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm else if (flags & (CDS_TEST | CDS_NORESET)) ret = DISP_CHANGE_SUCCESSFUL; else ret = apply_display_settings( device_name, &full_mode, hwnd, flags, lparam ); free( modes ); + adapter_release( adapter ); if (ret) ERR( "Changing %s display settings returned %d.\n", debugstr_us(devname), ret ); return ret; @@ -2519,6 +2541,7 @@ BOOL WINAPI NtUserEnumDisplaySettings( UNICODE_STRING *device, DWORD index, DEVM if (index == ENUM_REGISTRY_SETTINGS) ret = read_registry_settings( adapter_path, devmode ); else if (index != ENUM_CURRENT_SETTINGS) ret = user_driver->pEnumDisplaySettingsEx( device_name, index, devmode, flags ); else ret = user_driver->pGetCurrentDisplaySettings( device_name, devmode ); + adapter_release( adapter ); if (!ret) WARN( "Failed to query %s display settings.\n", debugstr_w(device_name) ); else TRACE( "position %dx%d, resolution %ux%u, frequency %u, depth %u, orientation %#x.\n",

1 0

Rémi Bernon : win32u: Avoid copying the modes array in NtUserChangeDisplaySettings.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: 43434f8dae819d3eee8967819fbc7fc8b11d8e6d URL: https://gitlab.winehq.org/wine/wine/-/commit/43434f8dae819d3eee8967819fbc7f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Sep 9 15:14:33 2022 +0200 win32u: Avoid copying the modes array in NtUserChangeDisplaySettings. --- dlls/win32u/sysparams.c | 20 +++++++------------- 1 file changed, 7 insertions(+), 13 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 007e7ab2473..0a6a55d77a3 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -2160,8 +2160,7 @@ static const DEVMODEW *find_display_mode( const DEVMODEW *modes, DEVMODEW *devmo return NULL; } -static BOOL get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, const DEVMODEW *adapter_modes, - const DEVMODEW *devmode, DEVMODEW *full_mode ) +static BOOL adapter_get_full_mode( const struct adapter *adapter, const DEVMODEW *devmode, DEVMODEW *full_mode ) { const DEVMODEW *adapter_mode; @@ -2181,7 +2180,7 @@ static BOOL get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, if (devmode) memcpy( full_mode, devmode, devmode->dmSize ); else { - if (!read_registry_settings( adapter_path, full_mode )) return FALSE; + if (!read_registry_settings( adapter->config_key, full_mode )) return FALSE; TRACE( "Return to original display mode\n" ); } @@ -2194,7 +2193,7 @@ static BOOL get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, if (!is_detached_mode( full_mode ) && (!full_mode->dmPelsWidth || !full_mode->dmPelsHeight || !(full_mode->dmFields & DM_POSITION))) { DEVMODEW current_mode = {.dmSize = sizeof(DEVMODEW)}; - if (!user_driver->pGetCurrentDisplaySettings( device_name, &current_mode )) return FALSE; + if (!user_driver->pGetCurrentDisplaySettings( adapter->dev.device_name, &current_mode )) return FALSE; if (!full_mode->dmPelsWidth) full_mode->dmPelsWidth = current_mode.dmPelsWidth; if (!full_mode->dmPelsHeight) full_mode->dmPelsHeight = current_mode.dmPelsHeight; if (!(full_mode->dmFields & DM_POSITION)) @@ -2204,7 +2203,7 @@ static BOOL get_full_mode( const WCHAR *adapter_path, const WCHAR *device_name, } } - if ((adapter_mode = find_display_mode( adapter_modes, full_mode )) && adapter_mode != full_mode) + if ((adapter_mode = find_display_mode( adapter->modes, full_mode )) && adapter_mode != full_mode) { POINTL position = full_mode->dmPosition; *full_mode = *adapter_mode; @@ -2469,8 +2468,8 @@ static LONG apply_display_settings( const WCHAR *devname, const DEVMODEW *devmod LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devmode, HWND hwnd, DWORD flags, void *lparam ) { - DEVMODEW *modes, full_mode = {.dmSize = sizeof(DEVMODEW)}; WCHAR device_name[CCHDEVICENAME], adapter_path[MAX_PATH]; + DEVMODEW full_mode = {.dmSize = sizeof(DEVMODEW)}; LONG ret = DISP_CHANGE_SUCCESSFUL; struct adapter *adapter; @@ -2484,23 +2483,18 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm { lstrcpyW( device_name, adapter->dev.device_name ); lstrcpyW( adapter_path, adapter->config_key ); - /* allocate an extra mode to make iteration easier */ - modes = calloc( adapter->mode_count + 1, sizeof(DEVMODEW) ); - if (modes) memcpy( modes, adapter->modes, adapter->mode_count * sizeof(DEVMODEW) ); } unlock_display_devices(); - if (!adapter || !modes) + if (!adapter) { WARN( "Invalid device name %s.\n", debugstr_us(devname) ); - if (adapter) adapter_release( adapter ); return DISP_CHANGE_BADPARAM; } - if (!get_full_mode( adapter_path, device_name, modes, devmode, &full_mode )) ret = DISP_CHANGE_BADMODE; + if (!adapter_get_full_mode( adapter, devmode, &full_mode )) ret = DISP_CHANGE_BADMODE; else if ((flags & CDS_UPDATEREGISTRY) && !write_registry_settings( adapter_path, &full_mode )) ret = DISP_CHANGE_NOTUPDATED; else if (flags & (CDS_TEST | CDS_NORESET)) ret = DISP_CHANGE_SUCCESSFUL; else ret = apply_display_settings( device_name, &full_mode, hwnd, flags, lparam ); - free( modes ); adapter_release( adapter ); if (ret) ERR( "Changing %s display settings returned %d.\n", debugstr_us(devname), ret );

1 0

Rémi Bernon : win32u: Avoid unnecessary copy of adapter path and device name.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: 68e461a6c53db461971901fca843903760a831dc URL: https://gitlab.winehq.org/wine/wine/-/commit/68e461a6c53db461971901fca84390… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Sep 9 15:14:48 2022 +0200 win32u: Avoid unnecessary copy of adapter path and device name. --- dlls/win32u/sysparams.c | 26 ++++++++------------------ 1 file changed, 8 insertions(+), 18 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 0a6a55d77a3..6c447325de4 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -2468,7 +2468,6 @@ static LONG apply_display_settings( const WCHAR *devname, const DEVMODEW *devmod LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devmode, HWND hwnd, DWORD flags, void *lparam ) { - WCHAR device_name[CCHDEVICENAME], adapter_path[MAX_PATH]; DEVMODEW full_mode = {.dmSize = sizeof(DEVMODEW)}; LONG ret = DISP_CHANGE_SUCCESSFUL; struct adapter *adapter; @@ -2479,11 +2478,7 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm if ((!devname || !devname->Length) && !devmode) return apply_display_settings( NULL, NULL, hwnd, flags, lparam ); if (!lock_display_devices()) return DISP_CHANGE_FAILED; - if ((adapter = find_adapter( devname ))) - { - lstrcpyW( device_name, adapter->dev.device_name ); - lstrcpyW( adapter_path, adapter->config_key ); - } + adapter = find_adapter( devname ); unlock_display_devices(); if (!adapter) { @@ -2492,9 +2487,9 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm } if (!adapter_get_full_mode( adapter, devmode, &full_mode )) ret = DISP_CHANGE_BADMODE; - else if ((flags & CDS_UPDATEREGISTRY) && !write_registry_settings( adapter_path, &full_mode )) ret = DISP_CHANGE_NOTUPDATED; + else if ((flags & CDS_UPDATEREGISTRY) && !write_registry_settings( adapter->config_key, &full_mode )) ret = DISP_CHANGE_NOTUPDATED; else if (flags & (CDS_TEST | CDS_NORESET)) ret = DISP_CHANGE_SUCCESSFUL; - else ret = apply_display_settings( device_name, &full_mode, hwnd, flags, lparam ); + else ret = apply_display_settings( adapter->dev.device_name, &full_mode, hwnd, flags, lparam ); adapter_release( adapter ); if (ret) ERR( "Changing %s display settings returned %d.\n", debugstr_us(devname), ret ); @@ -2507,18 +2502,13 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm BOOL WINAPI NtUserEnumDisplaySettings( UNICODE_STRING *device, DWORD index, DEVMODEW *devmode, DWORD flags ) { static const WCHAR wine_display_driverW[] = {'W','i','n','e',' ','D','i','s','p','l','a','y',' ','D','r','i','v','e','r',0}; - WCHAR device_name[CCHDEVICENAME], adapter_path[MAX_PATH]; struct adapter *adapter; BOOL ret; TRACE( "device %s, index %#x, devmode %p, flags %#x\n", debugstr_us(device), index, devmode, flags ); if (!lock_display_devices()) return FALSE; - if ((adapter = find_adapter( device ))) - { - lstrcpyW( device_name, adapter->dev.device_name ); - lstrcpyW( adapter_path, adapter->config_key ); - } + adapter = find_adapter( device ); unlock_display_devices(); if (!adapter) { @@ -2532,12 +2522,12 @@ BOOL WINAPI NtUserEnumDisplaySettings( UNICODE_STRING *device, DWORD index, DEVM devmode->dmSize = offsetof(DEVMODEW, dmICMMethod); memset( &devmode->dmDriverExtra, 0, devmode->dmSize - offsetof(DEVMODEW, dmDriverExtra) ); - if (index == ENUM_REGISTRY_SETTINGS) ret = read_registry_settings( adapter_path, devmode ); - else if (index != ENUM_CURRENT_SETTINGS) ret = user_driver->pEnumDisplaySettingsEx( device_name, index, devmode, flags ); - else ret = user_driver->pGetCurrentDisplaySettings( device_name, devmode ); + if (index == ENUM_REGISTRY_SETTINGS) ret = read_registry_settings( adapter->config_key, devmode ); + else if (index != ENUM_CURRENT_SETTINGS) ret = user_driver->pEnumDisplaySettingsEx( adapter->dev.device_name, index, devmode, flags ); + else ret = user_driver->pGetCurrentDisplaySettings( adapter->dev.device_name, devmode ); adapter_release( adapter ); - if (!ret) WARN( "Failed to query %s display settings.\n", debugstr_w(device_name) ); + if (!ret) WARN( "Failed to query %s display settings.\n", debugstr_us(device) ); else TRACE( "position %dx%d, resolution %ux%u, frequency %u, depth %u, orientation %#x.\n", devmode->dmPosition.x, devmode->dmPosition.y, devmode->dmPelsWidth, devmode->dmPelsHeight, devmode->dmDisplayFrequency, devmode->dmBitsPerPel, devmode->dmDisplayOrientation );

1 0

Rémi Bernon : win32u: Pass adapter parameter to (read|write)_registry_settings.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: a225ad4efd4657486c57a44946eaa9e9ced9c268 URL: https://gitlab.winehq.org/wine/wine/-/commit/a225ad4efd4657486c57a44946eaa9… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Sep 9 15:15:05 2022 +0200 win32u: Pass adapter parameter to (read|write)_registry_settings. And rename them to adapter_(set|get)_registry_settings. --- dlls/win32u/sysparams.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 6c447325de4..c3f01a80321 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -522,7 +522,7 @@ static BOOL read_adapter_mode( HKEY adapter_key, DWORD index, DEVMODEW *mode ) return TRUE; } -static BOOL read_registry_settings( const WCHAR *adapter_path, DEVMODEW *mode ) +static BOOL adapter_get_registry_settings( const struct adapter *adapter, DEVMODEW *mode ) { BOOL ret = FALSE; HANDLE mutex; @@ -531,7 +531,7 @@ static BOOL read_registry_settings( const WCHAR *adapter_path, DEVMODEW *mode ) mutex = get_display_device_init_mutex(); if (!config_key && !(config_key = reg_open_key( NULL, config_keyW, sizeof(config_keyW) ))) ret = FALSE; - else if (!(hkey = reg_open_key( config_key, adapter_path, lstrlenW( adapter_path ) * sizeof(WCHAR) ))) ret = FALSE; + else if (!(hkey = reg_open_key( config_key, adapter->config_key, lstrlenW( adapter->config_key ) * sizeof(WCHAR) ))) ret = FALSE; else { ret = read_adapter_mode( hkey, ENUM_REGISTRY_SETTINGS, mode ); @@ -542,7 +542,7 @@ static BOOL read_registry_settings( const WCHAR *adapter_path, DEVMODEW *mode ) return ret; } -static BOOL write_registry_settings( const WCHAR *adapter_path, const DEVMODEW *mode ) +static BOOL adapter_set_registry_settings( const struct adapter *adapter, const DEVMODEW *mode ) { HANDLE mutex; HKEY hkey; @@ -551,7 +551,7 @@ static BOOL write_registry_settings( const WCHAR *adapter_path, const DEVMODEW * mutex = get_display_device_init_mutex(); if (!config_key && !(config_key = reg_open_key( NULL, config_keyW, sizeof(config_keyW) ))) ret = FALSE; - if (!(hkey = reg_open_key( config_key, adapter_path, lstrlenW( adapter_path ) * sizeof(WCHAR) ))) ret = FALSE; + if (!(hkey = reg_open_key( config_key, adapter->config_key, lstrlenW( adapter->config_key ) * sizeof(WCHAR) ))) ret = FALSE; else { ret = write_adapter_mode( hkey, ENUM_REGISTRY_SETTINGS, mode ); @@ -2180,7 +2180,7 @@ static BOOL adapter_get_full_mode( const struct adapter *adapter, const DEVMODEW if (devmode) memcpy( full_mode, devmode, devmode->dmSize ); else { - if (!read_registry_settings( adapter->config_key, full_mode )) return FALSE; + if (!adapter_get_registry_settings( adapter, full_mode )) return FALSE; TRACE( "Return to original display mode\n" ); } @@ -2233,7 +2233,7 @@ static DEVMODEW *get_display_settings( const WCHAR *devname, const DEVMODEW *dev memcpy( &mode->dmFields, &devmode->dmFields, devmode->dmSize - offsetof(DEVMODEW, dmFields) ); else { - if (!devname) ret = read_registry_settings( adapter->config_key, mode ); + if (!devname) ret = adapter_get_registry_settings( adapter, mode ); else ret = user_driver->pGetCurrentDisplaySettings( adapter->dev.device_name, mode ); if (!ret) goto done; } @@ -2487,7 +2487,7 @@ LONG WINAPI NtUserChangeDisplaySettings( UNICODE_STRING *devname, DEVMODEW *devm } if (!adapter_get_full_mode( adapter, devmode, &full_mode )) ret = DISP_CHANGE_BADMODE; - else if ((flags & CDS_UPDATEREGISTRY) && !write_registry_settings( adapter->config_key, &full_mode )) ret = DISP_CHANGE_NOTUPDATED; + else if ((flags & CDS_UPDATEREGISTRY) && !adapter_set_registry_settings( adapter, &full_mode )) ret = DISP_CHANGE_NOTUPDATED; else if (flags & (CDS_TEST | CDS_NORESET)) ret = DISP_CHANGE_SUCCESSFUL; else ret = apply_display_settings( adapter->dev.device_name, &full_mode, hwnd, flags, lparam ); adapter_release( adapter ); @@ -2522,7 +2522,7 @@ BOOL WINAPI NtUserEnumDisplaySettings( UNICODE_STRING *device, DWORD index, DEVM devmode->dmSize = offsetof(DEVMODEW, dmICMMethod); memset( &devmode->dmDriverExtra, 0, devmode->dmSize - offsetof(DEVMODEW, dmDriverExtra) ); - if (index == ENUM_REGISTRY_SETTINGS) ret = read_registry_settings( adapter->config_key, devmode ); + if (index == ENUM_REGISTRY_SETTINGS) ret = adapter_get_registry_settings( adapter, devmode ); else if (index != ENUM_CURRENT_SETTINGS) ret = user_driver->pEnumDisplaySettingsEx( adapter->dev.device_name, index, devmode, flags ); else ret = user_driver->pGetCurrentDisplaySettings( adapter->dev.device_name, devmode ); adapter_release( adapter );

1 0

Rémi Bernon : win32u: Introduce a new internal WINE_DM_UNSUPPORTED display flag.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: bd8de2a882b2a4d84460d09fe014c64c9da271bb URL: https://gitlab.winehq.org/wine/wine/-/commit/bd8de2a882b2a4d84460d09fe014c6… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Sep 9 11:39:13 2022 +0200 win32u: Introduce a new internal WINE_DM_UNSUPPORTED display flag. --- dlls/win32u/sysparams.c | 4 ++++ dlls/winemac.drv/display.c | 16 +++++++++------- include/wine/gdi_driver.h | 2 ++ 3 files changed, 15 insertions(+), 7 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index c3f01a80321..f06be99f936 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -2136,6 +2136,8 @@ static const DEVMODEW *find_display_mode( const DEVMODEW *modes, DEVMODEW *devmo for (mode = modes; mode && mode->dmSize; mode = NEXT_DEVMODEW(mode)) { + if ((mode->dmFields & DM_DISPLAYFLAGS) && (mode->dmDisplayFlags & WINE_DM_UNSUPPORTED)) + continue; if ((devmode->dmFields & DM_BITSPERPEL) && devmode->dmBitsPerPel && devmode->dmBitsPerPel != mode->dmBitsPerPel) continue; if ((devmode->dmFields & DM_PELSWIDTH) && devmode->dmPelsWidth != mode->dmPelsWidth) @@ -2527,6 +2529,8 @@ BOOL WINAPI NtUserEnumDisplaySettings( UNICODE_STRING *device, DWORD index, DEVM else ret = user_driver->pGetCurrentDisplaySettings( adapter->dev.device_name, devmode ); adapter_release( adapter ); + devmode->dmDisplayFlags &= ~WINE_DM_UNSUPPORTED; + if (!ret) WARN( "Failed to query %s display settings.\n", debugstr_us(device) ); else TRACE( "position %dx%d, resolution %ux%u, frequency %u, depth %u, orientation %#x.\n", devmode->dmPosition.x, devmode->dmPosition.y, devmode->dmPelsWidth, devmode->dmPelsHeight, diff --git a/dlls/winemac.drv/display.c b/dlls/winemac.drv/display.c index 1cebcdfce2f..def291508be 100644 --- a/dlls/winemac.drv/display.c +++ b/dlls/winemac.drv/display.c @@ -97,6 +97,13 @@ static int display_mode_bits_per_pixel(CGDisplayModeRef display_mode) } +static BOOL display_mode_is_supported(CGDisplayModeRef display_mode) +{ + uint32_t io_flags = CGDisplayModeGetIOFlags(display_mode); + return (io_flags & kDisplayModeValidFlag) && (io_flags & kDisplayModeSafeFlag); +} + + static void display_mode_to_devmode(CGDirectDisplayID display_id, CGDisplayModeRef display_mode, DEVMODEW *devmode) { uint32_t io_flags; @@ -124,6 +131,8 @@ static void display_mode_to_devmode(CGDirectDisplayID display_id, CGDisplayModeR devmode->dmDisplayFlags = 0; if (io_flags & kDisplayModeInterlacedFlag) devmode->dmDisplayFlags |= DM_INTERLACED; + if (!display_mode_is_supported(display_mode)) + devmode->dmDisplayFlags |= WINE_DM_UNSUPPORTED; devmode->dmFields |= DM_DISPLAYFLAGS; devmode->dmDisplayFrequency = CGDisplayModeGetRefreshRate(display_mode); @@ -417,13 +426,6 @@ static int get_default_bpp(void) } -static BOOL display_mode_is_supported(CGDisplayModeRef display_mode) -{ - uint32_t io_flags = CGDisplayModeGetIOFlags(display_mode); - return (io_flags & kDisplayModeValidFlag) && (io_flags & kDisplayModeSafeFlag); -} - - #if defined(MAC_OS_X_VERSION_10_8) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_8 static CFDictionaryRef create_mode_dict(CGDisplayModeRef display_mode, BOOL is_original) { diff --git a/include/wine/gdi_driver.h b/include/wine/gdi_driver.h index e1d8998d812..9863a4a36ea 100644 --- a/include/wine/gdi_driver.h +++ b/include/wine/gdi_driver.h @@ -269,6 +269,8 @@ struct gdi_device_manager void (*add_mode)( const DEVMODEW *mode, void *param ); }; +#define WINE_DM_UNSUPPORTED 0x80000000 + struct tagUPDATELAYEREDWINDOWINFO; struct user_driver_funcs

1 0

Rémi Bernon : mf/tests: Split IMFTransform tests to transform.c.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: db56d2f2a30d6a10dc78a937cd133fb895a9d0d2 URL: https://gitlab.winehq.org/wine/wine/-/commit/db56d2f2a30d6a10dc78a937cd133f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Sep 8 09:01:32 2022 +0200 mf/tests: Split IMFTransform tests to transform.c. --- dlls/mf/tests/Makefile.in | 5 +- dlls/mf/tests/mf.c | 4717 +++------------------------------------------ dlls/mf/tests/mf_test.h | 58 + dlls/mf/tests/transform.c | 4321 +++++++++++++++++++++++++++++++++++++++++ 4 files changed, 4622 insertions(+), 4479 deletions(-)

1 0

Rémi Bernon : mf/tests: Add some winetest context to MFT tests.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: 66ad83bd55013845757645f87111cda61aa2ca97 URL: https://gitlab.winehq.org/wine/wine/-/commit/66ad83bd55013845757645f87111cd… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Sep 6 08:06:58 2022 +0200 mf/tests: Add some winetest context to MFT tests. --- dlls/mf/tests/transform.c | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/dlls/mf/tests/transform.c b/dlls/mf/tests/transform.c index 72e44635992..85c36756631 100644 --- a/dlls/mf/tests/transform.c +++ b/dlls/mf/tests/transform.c @@ -351,6 +351,8 @@ static void test_sample_copier(void) return; } + winetest_push_context("copier"); + hr = pMFCreateSampleCopierMFT(&copier); ok(hr == S_OK, "Failed to create sample copier, hr %#lx.\n", hr); @@ -602,6 +604,8 @@ static void test_sample_copier(void) ok(ref == 0, "Release returned %ld\n", ref); ref = IMFMediaType_Release(mediatype); ok(ref == 0, "Release returned %ld\n", ref); + + winetest_pop_context(); } struct sample_metadata @@ -958,6 +962,8 @@ static void test_wma_encoder(void) hr = CoInitialize(NULL); ok(hr == S_OK, "Failed to initialize, hr %#lx.\n", hr); + winetest_push_context("wmaenc"); + if (!create_transform(MFT_CATEGORY_AUDIO_ENCODER, &input_type, &output_type, L"WMAudio Encoder MFT", &MFMediaType_Audio, transform_inputs, ARRAY_SIZE(transform_inputs), transform_outputs, ARRAY_SIZE(transform_outputs), &transform, &CLSID_CWMAEncMediaObject, &class_id)) @@ -1080,6 +1086,7 @@ static void test_wma_encoder(void) ok(ret == 0, "Release returned %lu\n", ret); failed: + winetest_pop_context(); CoUninitialize(); } @@ -1210,6 +1217,8 @@ static void test_wma_decoder(void) hr = CoInitialize(NULL); ok(hr == S_OK, "Failed to initialize, hr %#lx.\n", hr); + winetest_push_context("wmadec"); + if (!create_transform(MFT_CATEGORY_AUDIO_DECODER, &input_type, &output_type, L"WMAudio Decoder MFT", &MFMediaType_Audio, transform_inputs, ARRAY_SIZE(transform_inputs), transform_outputs, ARRAY_SIZE(transform_outputs), &transform, &CLSID_CWMADecMediaObject, &class_id)) @@ -1592,6 +1601,7 @@ static void test_wma_decoder(void) ok( ref == 0, "Release returned %lu\n", ref ); failed: + winetest_pop_context(); CoUninitialize(); } @@ -1849,6 +1859,8 @@ static void test_h264_decoder(void) hr = CoInitialize(NULL); ok(hr == S_OK, "Failed to initialize, hr %#lx.\n", hr); + winetest_push_context("h264dec"); + if (!create_transform(MFT_CATEGORY_VIDEO_DECODER, &input_type, &output_type, L"Microsoft H264 Video Decoder MFT", &MFMediaType_Video, transform_inputs, ARRAY_SIZE(transform_inputs), transform_outputs, ARRAY_SIZE(transform_outputs), &transform, &CLSID_MSH264DecoderMFT, &class_id)) @@ -2353,6 +2365,7 @@ static void test_h264_decoder(void) ok(ret == 0, "Release returned %lu\n", ret); failed: + winetest_pop_context(); CoUninitialize(); } @@ -2464,6 +2477,8 @@ static void test_audio_convert(void) hr = CoInitialize(NULL); ok(hr == S_OK, "Failed to initialize, hr %#lx.\n", hr); + winetest_push_context("resampler"); + if (!create_transform(MFT_CATEGORY_AUDIO_EFFECT, &input_type, &output_type, L"Resampler MFT", &MFMediaType_Audio, transform_inputs, ARRAY_SIZE(transform_inputs), transform_outputs, ARRAY_SIZE(transform_outputs), &transform, &CLSID_CResamplerMediaObject, &class_id)) @@ -2742,6 +2757,7 @@ static void test_audio_convert(void) ok(ret == 0, "Release returned %lu\n", ret); failed: + winetest_pop_context(); CoUninitialize(); } @@ -2921,6 +2937,8 @@ static void test_color_convert(void) hr = CoInitialize(NULL); ok(hr == S_OK, "Failed to initialize, hr %#lx.\n", hr); + winetest_push_context("colorconv"); + if (!create_transform(MFT_CATEGORY_VIDEO_EFFECT, &input_type, &output_type, L"Color Converter MFT", &MFMediaType_Video, transform_inputs, ARRAY_SIZE(transform_inputs), transform_outputs, ARRAY_SIZE(transform_outputs), &transform, &CLSID_CColorConvertDMO, &class_id)) @@ -3117,6 +3135,7 @@ static void test_color_convert(void) ok(ret == 0, "Release returned %ld\n", ret); failed: + winetest_pop_context(); CoUninitialize(); } @@ -3309,6 +3328,8 @@ static void test_video_processor(void) hr = CoInitialize(NULL); ok(hr == S_OK, "Failed to initialize, hr %#lx.\n", hr); + winetest_push_context("videoproc"); + if (!create_transform(MFT_CATEGORY_VIDEO_PROCESSOR, &input_type, &output_type, L"Microsoft Video Processor MFT", &MFMediaType_Video, transform_inputs, ARRAY_SIZE(transform_inputs), transform_outputs, ARRAY_SIZE(transform_outputs), &transform, &CLSID_VideoProcessorMFT, &class_id)) @@ -3867,6 +3888,7 @@ skip_output: ok(ret == 0, "Release returned %ld\n", ret); failed: + winetest_pop_context(); CoUninitialize(); } @@ -4004,6 +4026,8 @@ static void test_mp3_decoder(void) hr = CoInitialize(NULL); ok(hr == S_OK, "Failed to initialize, hr %#lx.\n", hr); + winetest_push_context("mp3dec"); + if (!create_transform(MFT_CATEGORY_AUDIO_DECODER, &input_type, &output_type, L"MP3 Decoder MFT", &MFMediaType_Audio, transform_inputs, ARRAY_SIZE(transform_inputs), transform_outputs, ARRAY_SIZE(transform_outputs), &transform, &CLSID_CMP3DecMediaObject, &class_id)) @@ -4302,6 +4326,7 @@ static void test_mp3_decoder(void) ok(ret == 0, "Release returned %lu\n", ret); failed: + winetest_pop_context(); CoUninitialize(); }

1 0

Rémi Bernon : widl: Allow parameterized types in runtimeclass interfaces.
by Alexandre Julliard 13 Sep '22

13 Sep '22

Module: wine Branch: master Commit: de95ee6c662e99c7d37737609aa5874c6016aadc URL: https://gitlab.winehq.org/wine/wine/-/commit/de95ee6c662e99c7d37737609aa587… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Sep 8 19:35:46 2022 +0200 widl: Allow parameterized types in runtimeclass interfaces. --- tools/widl/parser.y | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 4000f37032c..c483eb06ff0 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -1086,6 +1086,7 @@ interfacedef: attributes interface { if ($2->type_type == TYPE_PARAMETERIZED_TY interfaceref: tINTERFACE typename { $$ = get_type(TYPE_INTERFACE, $2, current_namespace, 0); } | tINTERFACE namespace_pfx typename { $$ = get_type(TYPE_INTERFACE, $3, $2, 0); } + | tINTERFACE parameterized_type { if (type_get_type(($$ = $2)) != TYPE_INTERFACE) error_loc("%s is not an interface\n", $$->name); } ; dispinterfaceref:

1 0