[PATCH v6 0/3] MR5060: win32u, explorer, winex11.drv: Restore display settings on process exit.

[Anton Baskanov (＠baskanov)]

--
  v6: explorer: Restore display settings on process exit.
      winex11.drv: Process RRNotify events in xrandr14_get_id.
      user32/tests: Test that display settings are restored on process exit.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060

[PATCH v6 1/3] user32/tests: Test that display settings are restored on process exit.

[Anton Baskanov]

From: Anton Baskanov <baskanov(a)gmail.com>

---
 dlls/user32/tests/monitor.c | 317 ++++++++++++++++++++++++++++++++++--
 1 file changed, 299 insertions(+), 18 deletions(-)

diff --git a/dlls/user32/tests/monitor.c b/dlls/user32/tests/monitor.c
index e93a84242c4..611a107954d 100644
--- a/dlls/user32/tests/monitor.c
+++ b/dlls/user32/tests/monitor.c
@@ -291,8 +291,8 @@ struct device_info
     DEVMODEA original_mode;
 };
 
-#define expect_dm(a, b, c) _expect_dm(__LINE__, a, b, c)
-static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, DWORD test)
+#define expect_dm(a, b, c, d) _expect_dm(__LINE__, a, b, c, d)
+static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, DWORD test, BOOL todo)
 {
     DEVMODEA dm;
     BOOL ret;
@@ -307,9 +307,9 @@ static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, D
             "Device %s test %ld expect dmFields to contain %#lx, got %#lx\n", device, test, expected->dmFields, dm.dmFields);
     ok_(__FILE__, line)(!(expected->dmFields & DM_BITSPERPEL) || dm.dmBitsPerPel == expected->dmBitsPerPel,
             "Device %s test %ld expect dmBitsPerPel %lu, got %lu\n", device, test, expected->dmBitsPerPel, dm.dmBitsPerPel);
-    ok_(__FILE__, line)(!(expected->dmFields & DM_PELSWIDTH) || dm.dmPelsWidth == expected->dmPelsWidth,
+    todo_wine_if(todo) ok_(__FILE__, line)(!(expected->dmFields & DM_PELSWIDTH) || dm.dmPelsWidth == expected->dmPelsWidth,
             "Device %s test %ld expect dmPelsWidth %lu, got %lu\n", device, test, expected->dmPelsWidth, dm.dmPelsWidth);
-    ok_(__FILE__, line)(!(expected->dmFields & DM_PELSHEIGHT) || dm.dmPelsHeight == expected->dmPelsHeight,
+    todo_wine_if(todo) ok_(__FILE__, line)(!(expected->dmFields & DM_PELSHEIGHT) || dm.dmPelsHeight == expected->dmPelsHeight,
             "Device %s test %ld expect dmPelsHeight %lu, got %lu\n", device, test, expected->dmPelsHeight, dm.dmPelsHeight);
     ok_(__FILE__, line)(!(expected->dmFields & DM_POSITION) || dm.dmPosition.x == expected->dmPosition.x,
             "Device %s test %ld expect dmPosition.x %ld, got %ld\n", device, test, expected->dmPosition.x, dm.dmPosition.x);
@@ -325,18 +325,79 @@ static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, D
             dm.dmDisplayOrientation);
 }
 
-static void test_ChangeDisplaySettingsEx(void)
+#define wait_for_dm(a, b, c, d) wait_for_dm_(__LINE__, a, b, c, d)
+static void wait_for_dm_(int line, const char *device, DWORD expected_width, DWORD expected_height, BOOL todo)
+{
+    DEVMODEA dm;
+    BOOL ret;
+    int i;
+
+    for (i = 0; i < 50; ++i)
+    {
+        memset(&dm, 0, sizeof(dm));
+        dm.dmSize = sizeof(dm);
+        SetLastError(0xdeadbeef);
+        ret = EnumDisplaySettingsA(device, ENUM_CURRENT_SETTINGS, &dm);
+        ok_(__FILE__, line)(ret, "Device %s EnumDisplaySettingsA failed, error %#lx\n", device, GetLastError());
+
+        if (dm.dmPelsWidth == expected_width && dm.dmPelsHeight == expected_height)
+            break;
+
+        Sleep(100);
+    }
+
+    todo_wine_if(todo) ok_(__FILE__, line)(dm.dmPelsWidth == expected_width,
+            "Device %s expect dmPelsWidth %lu, got %lu\n", device, expected_width, dm.dmPelsWidth);
+    todo_wine_if(todo) ok_(__FILE__, line)(dm.dmPelsHeight == expected_height,
+            "Device %s expect dmPelsHeight %lu, got %lu\n", device, expected_height, dm.dmPelsHeight);
+}
+
+static HANDLE test_child_process_ChangeDisplaySettingsEx(const char *argv0, const char *device, DWORD flags, const char *exit_event_name)
+{
+    static const char *cds_event_name = "test_child_process_cds_event";
+    PROCESS_INFORMATION info;
+    char buffer[MAX_PATH];
+    STARTUPINFOA startup;
+    DWORD wait_result;
+    HANDLE cds_event;
+    LONG res;
+
+    cds_event = CreateEventA(NULL, FALSE, FALSE, cds_event_name);
+    ok(!!cds_event, "CreateEventA failed, error %#lx\n", GetLastError());
+
+    memset(&startup, 0, sizeof(startup));
+    startup.cb = sizeof(startup);
+
+    snprintf(buffer, sizeof(buffer), "%s monitor fullscreen %s %#lx %s %s", argv0, device, flags,
+            cds_event_name, exit_event_name);
+    res = CreateProcessA(NULL, buffer, NULL, NULL, FALSE, 0, NULL, NULL, &startup, &info);
+    ok(res, "CreateProcessA returned unexpected %ld\n", res);
+    wait_result = WaitForSingleObject(cds_event, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    CloseHandle(cds_event);
+    CloseHandle(info.hThread);
+
+    return info.hProcess;
+}
+
+static void test_ChangeDisplaySettingsEx(int argc, char **argv)
 {
     static const DWORD registry_fields = DM_DISPLAYORIENTATION | DM_BITSPERPEL | DM_PELSWIDTH | DM_PELSHEIGHT |
             DM_DISPLAYFLAGS | DM_DISPLAYFREQUENCY | DM_POSITION;
+    static const char *exit_event0_name = "test_cds_exit_event0";
+    static const char *exit_event1_name = "test_cds_exit_event1";
     static const DWORD depths[] = {8, 16, 32};
     DPI_AWARENESS_CONTEXT context = NULL;
     UINT primary, device, test, mode;
+    HANDLE exit_event0, exit_event1;
     UINT device_size, device_count;
     struct device_info *devices;
+    HANDLE process0, process1;
     DEVMODEA dm, dm2, dm3;
     INT count, old_count;
     DISPLAY_DEVICEA dd;
+    DWORD wait_result;
     POINTL position;
     DEVMODEW dmW;
     BOOL found;
@@ -668,7 +729,7 @@ static void test_ChangeDisplaySettingsEx(void)
                 continue;
             }
             flush_events();
-            expect_dm(&dm3, devices[device].name, test);
+            expect_dm(&dm3, devices[device].name, test, FALSE);
 
             /* Change the registry mode to the second mode */
             res = ChangeDisplaySettingsExA(devices[device].name, &dm2, NULL, CDS_UPDATEREGISTRY | CDS_NORESET, NULL);
@@ -802,7 +863,7 @@ static void test_ChangeDisplaySettingsEx(void)
             }
 
             flush_events();
-            expect_dm(&dm, devices[device].name, mode);
+            expect_dm(&dm, devices[device].name, mode, FALSE);
         }
 
         /* Restore settings */
@@ -875,7 +936,7 @@ static void test_ChangeDisplaySettingsEx(void)
         }
 
         flush_events();
-        expect_dm(&dm, devices[device].name, 0);
+        expect_dm(&dm, devices[device].name, 0, FALSE);
 
         /* Test specifying only position, width and height */
         memset(&dm, 0, sizeof(dm));
@@ -920,7 +981,7 @@ static void test_ChangeDisplaySettingsEx(void)
         ok(dm.dmBitsPerPel, "Expected dmBitsPerPel not zero.\n");
         ok(dm.dmDisplayFrequency, "Expected dmDisplayFrequency not zero.\n");
 
-        expect_dm(&dm, devices[device].name, 0);
+        expect_dm(&dm, devices[device].name, 0, FALSE);
     }
 
     /* Test dmPosition */
@@ -992,7 +1053,7 @@ static void test_ChangeDisplaySettingsEx(void)
             ok(res == DISP_CHANGE_SUCCESSFUL, "ChangeDisplaySettingsExA %s returned unexpected %ld\n", devices[1].name, res);
 
             dm2.dmPosition.x = dm.dmPosition.x + dm.dmPelsWidth;
-            expect_dm(&dm2, devices[1].name, 0);
+            expect_dm(&dm2, devices[1].name, 0, FALSE);
 
             /* Test placing the secondary adapter to all sides of the primary adapter */
             for (test = 0; test < 8; ++test)
@@ -1051,7 +1112,7 @@ static void test_ChangeDisplaySettingsEx(void)
                 }
 
                 flush_events();
-                expect_dm(&dm2, devices[1].name, test);
+                expect_dm(&dm2, devices[1].name, test, FALSE);
             }
 
             /* Test automatic position update when other adapters change resolution */
@@ -1116,7 +1177,7 @@ static void test_ChangeDisplaySettingsEx(void)
             ok(res == DISP_CHANGE_SUCCESSFUL, "ChangeDisplaySettingsExA %s mode %d returned unexpected %ld.\n",
                     devices[device].name, mode, res);
             flush_events();
-            expect_dm(&dm2, devices[device].name, mode);
+            expect_dm(&dm2, devices[device].name, mode, FALSE);
 
             /* EnumDisplaySettingsEx without EDS_ROTATEDMODE reports modes with current orientation */
             memset(&dm3, 0, sizeof(dm3));
@@ -1148,6 +1209,179 @@ static void test_ChangeDisplaySettingsEx(void)
         ok(mode > 0, "Expected at least one display mode found.\n");
     }
 
+    exit_event0 = CreateEventA(NULL, FALSE, FALSE, exit_event0_name);
+    ok(!!exit_event0, "CreateEventA failed, error %#lx\n", GetLastError());
+    exit_event1 = CreateEventA(NULL, FALSE, FALSE, exit_event1_name);
+    ok(!!exit_event1, "CreateEventA failed, error %#lx\n", GetLastError());
+
+    /* Test that if the most recent ChangeDisplaySettingsEx call had
+     * CDS_FULLSCREEN set, then the settings are restored when the caller
+     * process exits */
+
+    process0 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, CDS_FULLSCREEN, exit_event0_name);
+    process1 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, CDS_FULLSCREEN, exit_event1_name);
+
+    /* Verify that the settings are restored to the current registry settings */
+    for (device = 0; device < device_count; ++device)
+    {
+        memset(&dm, 0, sizeof(dm));
+        dm.dmSize = sizeof(dm);
+        dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+        dm.dmPelsWidth = 800;
+        dm.dmPelsHeight = 600;
+        res = ChangeDisplaySettingsExA(devices[device].name, &dm, NULL, CDS_UPDATEREGISTRY | CDS_NORESET, NULL);
+        ok(res == DISP_CHANGE_SUCCESSFUL, "ChangeDisplaySettingsExA %s returned %ld.\n", devices[device].name, res);
+    }
+
+    SetEvent(exit_event0);
+    wait_result = WaitForSingleObject(process0, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    Sleep(100);
+
+    memset(&dm, 0, sizeof(dm));
+    dm.dmSize = sizeof(dm);
+    dm.dmPelsWidth = 640;
+    dm.dmPelsHeight = 480;
+    dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+    expect_dm(&dm, devices[0].name, 0, TRUE);
+
+    SetEvent(exit_event1);
+    wait_result = WaitForSingleObject(process1, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    wait_for_dm(devices[0].name, 800, 600, TRUE);
+
+    CloseHandle(process1);
+    CloseHandle(process0);
+
+    /* Test processes exiting in reverse order */
+
+    process0 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, CDS_FULLSCREEN, exit_event0_name);
+    process1 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, CDS_FULLSCREEN, exit_event1_name);
+
+    SetEvent(exit_event1);
+    wait_result = WaitForSingleObject(process1, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    wait_for_dm(devices[0].name, 800, 600, TRUE);
+
+    SetEvent(exit_event0);
+    wait_result = WaitForSingleObject(process0, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    Sleep(100);
+
+    memset(&dm, 0, sizeof(dm));
+    dm.dmSize = sizeof(dm);
+    dm.dmPelsWidth = 800;
+    dm.dmPelsHeight = 600;
+    dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+    expect_dm(&dm, devices[0].name, 0, TRUE);
+
+    CloseHandle(process1);
+    CloseHandle(process0);
+
+    /* Test that ChangeDisplaySettingsEx without CDS_FULLSCREEN cancels the restoration */
+
+    process0 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, CDS_FULLSCREEN, exit_event0_name);
+    process1 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, 0, exit_event1_name);
+
+    SetEvent(exit_event0);
+    wait_result = WaitForSingleObject(process0, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    Sleep(100);
+
+    memset(&dm, 0, sizeof(dm));
+    dm.dmSize = sizeof(dm);
+    dm.dmPelsWidth = 640;
+    dm.dmPelsHeight = 480;
+    dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+    expect_dm(&dm, devices[0].name, 0, TRUE);
+
+    SetEvent(exit_event1);
+    wait_result = WaitForSingleObject(process1, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    Sleep(100);
+
+    memset(&dm, 0, sizeof(dm));
+    dm.dmSize = sizeof(dm);
+    dm.dmPelsWidth = 640;
+    dm.dmPelsHeight = 480;
+    dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+    expect_dm(&dm, devices[0].name, 0, TRUE);
+
+    CloseHandle(process1);
+    CloseHandle(process0);
+
+    /* Test processes exiting in reverse order */
+
+    process0 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, CDS_FULLSCREEN, exit_event0_name);
+    process1 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, 0, exit_event1_name);
+
+    SetEvent(exit_event1);
+    wait_result = WaitForSingleObject(process1, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    Sleep(100);
+
+    memset(&dm, 0, sizeof(dm));
+    dm.dmSize = sizeof(dm);
+    dm.dmPelsWidth = 640;
+    dm.dmPelsHeight = 480;
+    dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+    expect_dm(&dm, devices[0].name, 0, TRUE);
+
+    SetEvent(exit_event0);
+    wait_result = WaitForSingleObject(process0, 5000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+    Sleep(100);
+
+    memset(&dm, 0, sizeof(dm));
+    dm.dmSize = sizeof(dm);
+    dm.dmPelsWidth = 640;
+    dm.dmPelsHeight = 480;
+    dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+    expect_dm(&dm, devices[0].name, 0, TRUE);
+
+    CloseHandle(process1);
+    CloseHandle(process0);
+
+    if (device_count < 2)
+    {
+        skip("Only one device found.\n");
+    }
+    else
+    {
+        /* Test that the settings are restored for all devices, regardless of
+         * the process that changed them */
+
+        memset(&dm, 0, sizeof(dm));
+        dm.dmSize = sizeof(dm);
+        dm.dmPelsWidth = 640;
+        dm.dmPelsHeight = 480;
+        dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+        res = ChangeDisplaySettingsExA(devices[1].name, &dm, NULL, CDS_FULLSCREEN, NULL);
+        ok(res == DISP_CHANGE_SUCCESSFUL, "ChangeDisplaySettingsExA %s returned %ld.\n", devices[1].name, res);
+
+        process0 = test_child_process_ChangeDisplaySettingsEx(argv[0], devices[0].name, CDS_FULLSCREEN, exit_event0_name);
+
+        SetEvent(exit_event0);
+        wait_result = WaitForSingleObject(process0, 5000);
+        ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+
+        wait_for_dm(devices[0].name, 800, 600, TRUE);
+        wait_for_dm(devices[1].name, 800, 600, TRUE);
+
+        CloseHandle(process0);
+    }
+
+    CloseHandle(exit_event1);
+    CloseHandle(exit_event0);
+
     /* Restore all adapters to their original settings */
     for (device = 0; device < device_count; ++device)
     {
@@ -1162,7 +1396,7 @@ static void test_ChangeDisplaySettingsEx(void)
             broken(res == DISP_CHANGE_FAILED), /* win8 TestBot */
             "ChangeDisplaySettingsExA returned unexpected %ld\n", res);
     for (device = 0; device < device_count; ++device)
-        expect_dm(&devices[device].original_mode, devices[device].name, 0);
+        expect_dm(&devices[device].original_mode, devices[device].name, 0, FALSE);
 
     free(devices);
 }
@@ -2663,6 +2897,45 @@ BOOL CALLBACK MonitorEnumProc(HMONITOR hMonitor, HDC hdcMonitor,
     return TRUE;
 }
 
+static void test_fullscreen(int argc, char **argv)
+{
+    HANDLE event0, event1;
+    DWORD wait_result;
+    DWORD flags = 0;
+    DEVMODEA dm;
+    LONG res;
+
+    if (argc < 7)
+    {
+        ok(0, "too few arguments.\n");
+        return;
+    }
+
+    res = sscanf(argv[4], "%lx", &flags);
+    ok(res == 1, "sscanf returned unexpected %ld.\n", res);
+
+    event0 = OpenEventA(EVENT_MODIFY_STATE, FALSE, argv[5]);
+    ok(!!event0, "OpenEventA failed, error %#lx\n", GetLastError());
+    event1 = OpenEventA(SYNCHRONIZE, FALSE, argv[6]);
+    ok(!!event1, "OpenEventA failed, error %#lx\n", GetLastError());
+
+    memset(&dm, 0, sizeof(dm));
+    dm.dmSize = sizeof(dm);
+    dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
+    dm.dmPelsWidth = 640;
+    dm.dmPelsHeight = 480;
+    res = ChangeDisplaySettingsExA(argv[3], &dm, NULL, flags, NULL);
+    ok(res == DISP_CHANGE_SUCCESSFUL,
+            "ChangeDisplaySettingsExA %s returned unexpected %ld.\n", argv[3], res);
+
+    SetEvent(event0);
+    CloseHandle(event0);
+
+    wait_result = WaitForSingleObject(event1, 7000);
+    ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
+    CloseHandle(event1);
+}
+
 START_TEST(monitor)
 {
     char** myARGV;
@@ -2670,15 +2943,23 @@ START_TEST(monitor)
 
     init_function_pointers();
 
-    if (myARGC >= 3 && strcmp(myARGV[2], "info") == 0)
+    if (myARGC >= 3)
     {
-        printf("Monitor information:\n");
-        EnumDisplayMonitors(NULL, NULL, MonitorEnumProc, 0);
-        return;
+        if (strcmp(myARGV[2], "info") == 0)
+        {
+            printf("Monitor information:\n");
+            EnumDisplayMonitors(NULL, NULL, MonitorEnumProc, 0);
+            return;
+        }
+        else if (strcmp(myARGV[2], "fullscreen") == 0)
+        {
+            test_fullscreen(myARGC, myARGV);
+            return;
+        }
     }
 
     test_enumdisplaydevices();
-    test_ChangeDisplaySettingsEx();
+    test_ChangeDisplaySettingsEx(myARGC, myARGV);
     test_DisplayConfigSetDeviceInfo();
     test_EnumDisplayMonitors();
     test_monitors();
-- 
GitLab


https://gitlab.winehq.org/wine/wine/-/merge_requests/5060

[PATCH v6 2/3] winex11.drv: Process RRNotify events in xrandr14_get_id.

[Anton Baskanov]

From: Anton Baskanov <baskanov(a)gmail.com>

We have to invalidate the current mode cache if there are pending
RRNotify events. The performance hit on EnumDisplaySettingsExW is
around 7%.

Also call X11DRV_DisplayDevices_RegisterEventHandlers in x11drv_init.
Otherwise, RRNotify events will only be handled in the explorer process.
---
 dlls/user32/tests/monitor.c    | 10 +++++-----
 dlls/winex11.drv/event.c       |  2 +-
 dlls/winex11.drv/window.c      |  1 -
 dlls/winex11.drv/x11drv.h      |  1 +
 dlls/winex11.drv/x11drv_main.c |  1 +
 dlls/winex11.drv/xrandr.c      | 28 ++++++++++++++++++++++++++++
 6 files changed, 36 insertions(+), 7 deletions(-)

diff --git a/dlls/user32/tests/monitor.c b/dlls/user32/tests/monitor.c
index 611a107954d..2a42a1e2970 100644
--- a/dlls/user32/tests/monitor.c
+++ b/dlls/user32/tests/monitor.c
@@ -1244,7 +1244,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, TRUE);
+    expect_dm(&dm, devices[0].name, 0, FALSE);
 
     SetEvent(exit_event1);
     wait_result = WaitForSingleObject(process1, 5000);
@@ -1298,7 +1298,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, TRUE);
+    expect_dm(&dm, devices[0].name, 0, FALSE);
 
     SetEvent(exit_event1);
     wait_result = WaitForSingleObject(process1, 5000);
@@ -1311,7 +1311,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, TRUE);
+    expect_dm(&dm, devices[0].name, 0, FALSE);
 
     CloseHandle(process1);
     CloseHandle(process0);
@@ -1332,7 +1332,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, TRUE);
+    expect_dm(&dm, devices[0].name, 0, FALSE);
 
     SetEvent(exit_event0);
     wait_result = WaitForSingleObject(process0, 5000);
@@ -1345,7 +1345,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, TRUE);
+    expect_dm(&dm, devices[0].name, 0, FALSE);
 
     CloseHandle(process1);
     CloseHandle(process0);
diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c
index 8b02361aaff..b1c8dd778d9 100644
--- a/dlls/winex11.drv/event.c
+++ b/dlls/winex11.drv/event.c
@@ -405,7 +405,7 @@ static inline BOOL call_event_handler( Display *display, XEvent *event )
 /***********************************************************************
  *           process_events
  */
-static BOOL process_events( Display *display, Bool (*filter)(Display*, XEvent*,XPointer), ULONG_PTR arg )
+BOOL process_events( Display *display, Bool (*filter)(Display*, XEvent*,XPointer), ULONG_PTR arg )
 {
     XEvent event, prev_event;
     int count = 0;
diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c
index 5a014c9080d..19daac04203 100644
--- a/dlls/winex11.drv/window.c
+++ b/dlls/winex11.drv/window.c
@@ -2048,7 +2048,6 @@ BOOL X11DRV_CreateWindow( HWND hwnd )
                                            CWOverrideRedirect | CWEventMask, &attr );
         XFlush( data->display );
         NtUserSetProp( hwnd, clip_window_prop, (HANDLE)data->clip_window );
-        X11DRV_DisplayDevices_RegisterEventHandlers();
     }
     return TRUE;
 }
diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h
index a525f0dd499..411d8a643e6 100644
--- a/dlls/winex11.drv/x11drv.h
+++ b/dlls/winex11.drv/x11drv.h
@@ -682,6 +682,7 @@ extern void retry_grab_clipping_window(void);
 extern void ungrab_clipping_window(void);
 extern void move_resize_window( HWND hwnd, int dir );
 extern void X11DRV_InitKeyboard( Display *display );
+extern BOOL process_events( Display *display, Bool (*filter)(Display*, XEvent*, XPointer), ULONG_PTR arg );
 extern BOOL X11DRV_ProcessEvents( DWORD mask );
 extern HWND *build_hwnd_list(void);
 
diff --git a/dlls/winex11.drv/x11drv_main.c b/dlls/winex11.drv/x11drv_main.c
index bbb0386bd69..23871e6a2f0 100644
--- a/dlls/winex11.drv/x11drv_main.c
+++ b/dlls/winex11.drv/x11drv_main.c
@@ -694,6 +694,7 @@ static NTSTATUS x11drv_init( void *arg )
 
     init_user_driver();
     X11DRV_DisplayDevices_Init(FALSE);
+    X11DRV_DisplayDevices_RegisterEventHandlers();
     return STATUS_SUCCESS;
 }
 
diff --git a/dlls/winex11.drv/xrandr.c b/dlls/winex11.drv/xrandr.c
index d77eb1a0163..bc4ae0209ac 100644
--- a/dlls/winex11.drv/xrandr.c
+++ b/dlls/winex11.drv/xrandr.c
@@ -1213,6 +1213,32 @@ static void xrandr14_register_event_handlers(void)
                                    "XRandR ProviderChange" );
 }
 
+static Bool filter_rrnotify_event( Display *display, XEvent *event, char *arg )
+{
+    ULONG_PTR event_base = (ULONG_PTR)arg;
+
+    if (event->type == event_base + RRNotify_CrtcChange
+            || event->type == event_base + RRNotify_OutputChange
+            || event->type == event_base + RRNotify_ProviderChange)
+        return 1;
+
+    return 0;
+}
+
+static void process_rrnotify_events(void)
+{
+    struct x11drv_thread_data *data = x11drv_thread_data();
+    int event_base, error_base;
+
+    if (!data) return;
+    if (data->current_event) return; /* don't process nested events */
+
+    if (!pXRRQueryExtension( data->display, &event_base, &error_base ))
+        return;
+
+    process_events( data->display, filter_rrnotify_event, event_base );
+}
+
 /* XRandR 1.4 display settings handler */
 static BOOL xrandr14_get_id( const WCHAR *device_name, BOOL is_primary, x11drv_settings_id *id )
 {
@@ -1228,6 +1254,8 @@ static BOOL xrandr14_get_id( const WCHAR *device_name, BOOL is_primary, x11drv_s
     if (*end)
         return FALSE;
 
+    process_rrnotify_events();
+
     /* Update cache */
     pthread_mutex_lock( &xrandr_mutex );
     if (!current_modes)
-- 
GitLab


https://gitlab.winehq.org/wine/wine/-/merge_requests/5060

[PATCH v6 3/3] explorer: Restore display settings on process exit.

[Anton Baskanov]

From: Anton Baskanov <baskanov(a)gmail.com>

Restore display settings to the ones in the registry when CDS_FULLSCREEN
is used in ChangeDisplaySettings().

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=49674
---
 dlls/user32/tests/monitor.c |  52 +++++++++---------
 dlls/win32u/sysparams.c     |  15 ++++++
 programs/explorer/desktop.c | 105 +++++++++++++++++++++++++++++++++---
 3 files changed, 139 insertions(+), 33 deletions(-)

diff --git a/dlls/user32/tests/monitor.c b/dlls/user32/tests/monitor.c
index 2a42a1e2970..e2b8796a832 100644
--- a/dlls/user32/tests/monitor.c
+++ b/dlls/user32/tests/monitor.c
@@ -291,8 +291,8 @@ struct device_info
     DEVMODEA original_mode;
 };
 
-#define expect_dm(a, b, c, d) _expect_dm(__LINE__, a, b, c, d)
-static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, DWORD test, BOOL todo)
+#define expect_dm(a, b, c) _expect_dm(__LINE__, a, b, c)
+static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, DWORD test)
 {
     DEVMODEA dm;
     BOOL ret;
@@ -307,9 +307,9 @@ static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, D
             "Device %s test %ld expect dmFields to contain %#lx, got %#lx\n", device, test, expected->dmFields, dm.dmFields);
     ok_(__FILE__, line)(!(expected->dmFields & DM_BITSPERPEL) || dm.dmBitsPerPel == expected->dmBitsPerPel,
             "Device %s test %ld expect dmBitsPerPel %lu, got %lu\n", device, test, expected->dmBitsPerPel, dm.dmBitsPerPel);
-    todo_wine_if(todo) ok_(__FILE__, line)(!(expected->dmFields & DM_PELSWIDTH) || dm.dmPelsWidth == expected->dmPelsWidth,
+    ok_(__FILE__, line)(!(expected->dmFields & DM_PELSWIDTH) || dm.dmPelsWidth == expected->dmPelsWidth,
             "Device %s test %ld expect dmPelsWidth %lu, got %lu\n", device, test, expected->dmPelsWidth, dm.dmPelsWidth);
-    todo_wine_if(todo) ok_(__FILE__, line)(!(expected->dmFields & DM_PELSHEIGHT) || dm.dmPelsHeight == expected->dmPelsHeight,
+    ok_(__FILE__, line)(!(expected->dmFields & DM_PELSHEIGHT) || dm.dmPelsHeight == expected->dmPelsHeight,
             "Device %s test %ld expect dmPelsHeight %lu, got %lu\n", device, test, expected->dmPelsHeight, dm.dmPelsHeight);
     ok_(__FILE__, line)(!(expected->dmFields & DM_POSITION) || dm.dmPosition.x == expected->dmPosition.x,
             "Device %s test %ld expect dmPosition.x %ld, got %ld\n", device, test, expected->dmPosition.x, dm.dmPosition.x);
@@ -325,8 +325,8 @@ static void _expect_dm(INT line, const DEVMODEA *expected, const CHAR *device, D
             dm.dmDisplayOrientation);
 }
 
-#define wait_for_dm(a, b, c, d) wait_for_dm_(__LINE__, a, b, c, d)
-static void wait_for_dm_(int line, const char *device, DWORD expected_width, DWORD expected_height, BOOL todo)
+#define wait_for_dm(a, b, c) wait_for_dm_(__LINE__, a, b, c)
+static void wait_for_dm_(int line, const char *device, DWORD expected_width, DWORD expected_height)
 {
     DEVMODEA dm;
     BOOL ret;
@@ -346,9 +346,9 @@ static void wait_for_dm_(int line, const char *device, DWORD expected_width, DWO
         Sleep(100);
     }
 
-    todo_wine_if(todo) ok_(__FILE__, line)(dm.dmPelsWidth == expected_width,
+    ok_(__FILE__, line)(dm.dmPelsWidth == expected_width,
             "Device %s expect dmPelsWidth %lu, got %lu\n", device, expected_width, dm.dmPelsWidth);
-    todo_wine_if(todo) ok_(__FILE__, line)(dm.dmPelsHeight == expected_height,
+    ok_(__FILE__, line)(dm.dmPelsHeight == expected_height,
             "Device %s expect dmPelsHeight %lu, got %lu\n", device, expected_height, dm.dmPelsHeight);
 }
 
@@ -729,7 +729,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
                 continue;
             }
             flush_events();
-            expect_dm(&dm3, devices[device].name, test, FALSE);
+            expect_dm(&dm3, devices[device].name, test);
 
             /* Change the registry mode to the second mode */
             res = ChangeDisplaySettingsExA(devices[device].name, &dm2, NULL, CDS_UPDATEREGISTRY | CDS_NORESET, NULL);
@@ -863,7 +863,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
             }
 
             flush_events();
-            expect_dm(&dm, devices[device].name, mode, FALSE);
+            expect_dm(&dm, devices[device].name, mode);
         }
 
         /* Restore settings */
@@ -936,7 +936,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
         }
 
         flush_events();
-        expect_dm(&dm, devices[device].name, 0, FALSE);
+        expect_dm(&dm, devices[device].name, 0);
 
         /* Test specifying only position, width and height */
         memset(&dm, 0, sizeof(dm));
@@ -981,7 +981,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
         ok(dm.dmBitsPerPel, "Expected dmBitsPerPel not zero.\n");
         ok(dm.dmDisplayFrequency, "Expected dmDisplayFrequency not zero.\n");
 
-        expect_dm(&dm, devices[device].name, 0, FALSE);
+        expect_dm(&dm, devices[device].name, 0);
     }
 
     /* Test dmPosition */
@@ -1053,7 +1053,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
             ok(res == DISP_CHANGE_SUCCESSFUL, "ChangeDisplaySettingsExA %s returned unexpected %ld\n", devices[1].name, res);
 
             dm2.dmPosition.x = dm.dmPosition.x + dm.dmPelsWidth;
-            expect_dm(&dm2, devices[1].name, 0, FALSE);
+            expect_dm(&dm2, devices[1].name, 0);
 
             /* Test placing the secondary adapter to all sides of the primary adapter */
             for (test = 0; test < 8; ++test)
@@ -1112,7 +1112,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
                 }
 
                 flush_events();
-                expect_dm(&dm2, devices[1].name, test, FALSE);
+                expect_dm(&dm2, devices[1].name, test);
             }
 
             /* Test automatic position update when other adapters change resolution */
@@ -1177,7 +1177,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
             ok(res == DISP_CHANGE_SUCCESSFUL, "ChangeDisplaySettingsExA %s mode %d returned unexpected %ld.\n",
                     devices[device].name, mode, res);
             flush_events();
-            expect_dm(&dm2, devices[device].name, mode, FALSE);
+            expect_dm(&dm2, devices[device].name, mode);
 
             /* EnumDisplaySettingsEx without EDS_ROTATEDMODE reports modes with current orientation */
             memset(&dm3, 0, sizeof(dm3));
@@ -1244,13 +1244,13 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, FALSE);
+    expect_dm(&dm, devices[0].name, 0);
 
     SetEvent(exit_event1);
     wait_result = WaitForSingleObject(process1, 5000);
     ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
 
-    wait_for_dm(devices[0].name, 800, 600, TRUE);
+    wait_for_dm(devices[0].name, 800, 600);
 
     CloseHandle(process1);
     CloseHandle(process0);
@@ -1264,7 +1264,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     wait_result = WaitForSingleObject(process1, 5000);
     ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
 
-    wait_for_dm(devices[0].name, 800, 600, TRUE);
+    wait_for_dm(devices[0].name, 800, 600);
 
     SetEvent(exit_event0);
     wait_result = WaitForSingleObject(process0, 5000);
@@ -1277,7 +1277,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 800;
     dm.dmPelsHeight = 600;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, TRUE);
+    expect_dm(&dm, devices[0].name, 0);
 
     CloseHandle(process1);
     CloseHandle(process0);
@@ -1298,7 +1298,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, FALSE);
+    expect_dm(&dm, devices[0].name, 0);
 
     SetEvent(exit_event1);
     wait_result = WaitForSingleObject(process1, 5000);
@@ -1311,7 +1311,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, FALSE);
+    expect_dm(&dm, devices[0].name, 0);
 
     CloseHandle(process1);
     CloseHandle(process0);
@@ -1332,7 +1332,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, FALSE);
+    expect_dm(&dm, devices[0].name, 0);
 
     SetEvent(exit_event0);
     wait_result = WaitForSingleObject(process0, 5000);
@@ -1345,7 +1345,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
     dm.dmPelsWidth = 640;
     dm.dmPelsHeight = 480;
     dm.dmFields = DM_PELSWIDTH | DM_PELSHEIGHT;
-    expect_dm(&dm, devices[0].name, 0, FALSE);
+    expect_dm(&dm, devices[0].name, 0);
 
     CloseHandle(process1);
     CloseHandle(process0);
@@ -1373,8 +1373,8 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
         wait_result = WaitForSingleObject(process0, 5000);
         ok(wait_result == WAIT_OBJECT_0, "WaitForSingleObject returned %lx.\n", wait_result);
 
-        wait_for_dm(devices[0].name, 800, 600, TRUE);
-        wait_for_dm(devices[1].name, 800, 600, TRUE);
+        wait_for_dm(devices[0].name, 800, 600);
+        wait_for_dm(devices[1].name, 800, 600);
 
         CloseHandle(process0);
     }
@@ -1396,7 +1396,7 @@ static void test_ChangeDisplaySettingsEx(int argc, char **argv)
             broken(res == DISP_CHANGE_FAILED), /* win8 TestBot */
             "ChangeDisplaySettingsExA returned unexpected %ld\n", res);
     for (device = 0; device < device_count; ++device)
-        expect_dm(&devices[device].original_mode, devices[device].name, 0, FALSE);
+        expect_dm(&devices[device].original_mode, devices[device].name, 0);
 
     free(devices);
 }
diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c
index 07b9d46d9eb..fa44e6eaeb0 100644
--- a/dlls/win32u/sysparams.c
+++ b/dlls/win32u/sysparams.c
@@ -3176,11 +3176,20 @@ static BOOL all_detached_settings( const DEVMODEW *displays )
 static LONG apply_display_settings( struct adapter *target, const DEVMODEW *devmode,
                                     HWND hwnd, DWORD flags, void *lparam )
 {
+    static const WCHAR restorerW[] = {'_','_','w','i','n','e','_','d','i','s','p','l','a','y','_',
+                                      's','e','t','t','i','n','g','s','_','r','e','s','t','o','r','e','r',0};
+    UNICODE_STRING restoter_str = RTL_CONSTANT_STRING( restorerW );
     WCHAR primary_name[CCHDEVICENAME];
     struct adapter *primary, *adapter;
+    DWORD fullscreen_process_id;
     DEVMODEW *mode, *displays;
+    DWORD restorer_thread_id;
+    HWND restorer_window;
     LONG ret;
 
+    restorer_window = NtUserFindWindowEx( NULL, NULL, &restoter_str, NULL, 0 );
+    restorer_thread_id = NtUserGetWindowThread( restorer_window, NULL );
+
     if (!lock_display_devices()) return DISP_CHANGE_FAILED;
     if (!(displays = get_display_settings( target, devmode )))
     {
@@ -3227,6 +3236,12 @@ static LONG apply_display_settings( struct adapter *target, const DEVMODEW *devm
     free( displays );
     if (ret) return ret;
 
+    if ( restorer_thread_id != GetCurrentThreadId() )
+    {
+        fullscreen_process_id = (flags & CDS_FULLSCREEN) ? GetCurrentProcessId() : 0;
+        send_message( restorer_window, WM_USER + 0, 0, fullscreen_process_id );
+    }
+
     if (!update_display_cache( TRUE ))
         WARN( "Failed to update display cache after mode change.\n" );
 
diff --git a/programs/explorer/desktop.c b/programs/explorer/desktop.c
index 25bba7cb7cb..ad00751537d 100644
--- a/programs/explorer/desktop.c
+++ b/programs/explorer/desktop.c
@@ -611,6 +611,18 @@ static void initialize_launchers( HWND hwnd )
     }
 }
 
+static void wait_named_mutex( const WCHAR *name )
+{
+    HANDLE mutex;
+
+    mutex = CreateMutexW( NULL, TRUE, name );
+    if (GetLastError() == ERROR_ALREADY_EXISTS)
+    {
+        TRACE( "waiting for mutex %s\n", debugstr_w( name ));
+        WaitForSingleObject( mutex, INFINITE );
+    }
+}
+
 /**************************************************************************
  *		wait_clipboard_mutex
  *
@@ -620,7 +632,6 @@ static BOOL wait_clipboard_mutex(void)
 {
     static const WCHAR prefix[] = L"__wine_clipboard_";
     WCHAR buffer[MAX_PATH + ARRAY_SIZE( prefix )];
-    HANDLE mutex;
 
     memcpy( buffer, prefix, sizeof(prefix) );
     if (!GetUserObjectInformationW( GetProcessWindowStation(), UOI_NAME,
@@ -630,12 +641,7 @@ static BOOL wait_clipboard_mutex(void)
         ERR( "failed to get winstation name\n" );
         return FALSE;
     }
-    mutex = CreateMutexW( NULL, TRUE, buffer );
-    if (GetLastError() == ERROR_ALREADY_EXISTS)
-    {
-        TRACE( "waiting for mutex %s\n", debugstr_w( buffer ));
-        WaitForSingleObject( mutex, INFINITE );
-    }
+    wait_named_mutex( buffer );
     return TRUE;
 }
 
@@ -696,6 +702,87 @@ static DWORD WINAPI clipboard_thread( void *arg )
     return 0;
 }
 
+static HANDLE fullscreen_process;
+
+static LRESULT WINAPI display_settings_restorer_wndproc( HWND hwnd, UINT message, WPARAM wp, LPARAM lp )
+{
+    TRACE( "got msg %04x wp %Ix lp %Ix\n", message, wp, lp );
+
+    switch(message)
+    {
+    case WM_USER + 0:
+        TRACE( "fullscreen process id %Iu.\n", lp );
+
+        if (fullscreen_process)
+        {
+            CloseHandle( fullscreen_process );
+            fullscreen_process = NULL;
+        }
+
+        if (lp)
+            fullscreen_process = OpenProcess( SYNCHRONIZE, FALSE, lp );
+
+        return 0;
+    }
+
+    return DefWindowProcW( hwnd, message, wp, lp );
+}
+
+static DWORD WINAPI display_settings_restorer_thread( void *param )
+{
+    static const WCHAR *display_settings_restorer_classname = L"__wine_display_settings_restorer";
+    DWORD wait_result;
+    WNDCLASSW class;
+    MSG msg;
+
+    SetThreadDescription( GetCurrentThread(), L"wine_explorer_display_settings_restorer" );
+
+    wait_named_mutex( L"__wine_display_settings_restorer_mutex" );
+
+    memset( &class, 0, sizeof(class) );
+    class.lpfnWndProc   = display_settings_restorer_wndproc;
+    class.lpszClassName = display_settings_restorer_classname;
+
+    if (!RegisterClassW( &class ) && GetLastError() != ERROR_CLASS_ALREADY_EXISTS)
+    {
+        ERR( "could not register display settings restorer window class err %lu\n", GetLastError() );
+        return 0;
+    }
+    if (!CreateWindowW( display_settings_restorer_classname, NULL, 0, 0, 0, 0, 0, HWND_MESSAGE, 0, 0, NULL ))
+    {
+        TRACE( "failed to create display settings restorer window err %lu\n", GetLastError() );
+        UnregisterClassW( display_settings_restorer_classname, NULL );
+        return 0;
+    }
+
+    for (;;)
+    {
+        if (PeekMessageW( &msg, NULL, 0, 0, PM_REMOVE ))
+        {
+            if (msg.message == WM_QUIT)
+                break;
+            DispatchMessageW( &msg );
+            continue;
+        }
+
+        wait_result = MsgWaitForMultipleObjects( fullscreen_process ? 1 : 0, &fullscreen_process,
+                                                 FALSE, INFINITE, QS_ALLINPUT );
+        if (wait_result == WAIT_FAILED)
+            break;
+        if (!fullscreen_process || wait_result != WAIT_OBJECT_0)
+            continue;
+
+        WARN( "restoring display settings on process exit\n" );
+
+        ChangeDisplaySettingsExW( NULL, NULL, NULL, 0, NULL );
+
+        CloseHandle( fullscreen_process );
+        fullscreen_process = NULL;
+    }
+
+    return 0;
+}
+
 static WNDPROC desktop_orig_wndproc;
 
 /* window procedure for the desktop window */
@@ -968,6 +1055,7 @@ static void initialize_display_settings( unsigned int width, unsigned int height
 {
     DISPLAY_DEVICEW device = {.cb = sizeof(DISPLAY_DEVICEW)};
     DWORD i = 0, flags = CDS_GLOBAL | CDS_UPDATEREGISTRY;
+    HANDLE thread;
 
     /* Store current display mode in the registry */
     while (EnumDisplayDevicesW( NULL, i++, &device, 0 ))
@@ -1000,6 +1088,9 @@ static void initialize_display_settings( unsigned int width, unsigned int height
         if (ChangeDisplaySettingsExW( NULL, &devmode, 0, flags, NULL ))
             ERR( "Failed to set primary display settings.\n" );
     }
+
+    thread = CreateThread( NULL, 0, display_settings_restorer_thread, NULL, 0, NULL );
+    if (thread) CloseHandle( thread );
 }
 
 static void set_desktop_window_title( HWND hwnd, const WCHAR *name )
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/5060

[PATCH v6 3/3] explorer: Restore display settings on process exit.

[Marvin]

Hi,

It looks like your patch introduced the new failures shown below.
Please investigate and fix them before resubmitting your patch.
If they are not new, fixing them anyway would help a lot. Otherwise
please ask for the known failures list to be updated.

The tests also ran into some preexisting test failures. If you know how
to fix them that would be helpful. See the TestBot job for the details:

The full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=144131

Your paranoid android.


=== w8 (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -1.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 1024
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768

=== w8adm (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -1.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 1024
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768

=== w864 (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -1.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 1024
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768

=== w1064v1507 (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w1064v1809 (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w1064_tsign (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w10pro64 (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w10pro64_en_AE_u8 (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w11pro64 (32 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w864 (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -1.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 1024
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 768
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 1024
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -1.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 1024
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 768

=== w1064v1507 (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w1064v1809 (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w1064_2qxl (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY2 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 600
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 800
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 600
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 800
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 600
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 800
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 600
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 800
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 600
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 800
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 600
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 800
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 600
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 800
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 600
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 800
monitor.c:1368: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY2 returned -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1376: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 600
monitor.c:1376: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 800
monitor.c:1377: Test failed: Device \\.\DISPLAY2 expect dmPelsWidth 800, got 480
monitor.c:1377: Test failed: Device \\.\DISPLAY2 expect dmPelsHeight 600, got 640

=== w1064_adm (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w1064_tsign (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w10pro64 (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w10pro64_ar (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w10pro64_ja (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w10pro64_zh_CN (64 bit report) ===

user32:
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1233: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned -2.
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1247: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1253: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsWidth 800, got 480
monitor.c:1267: Test failed: Device \\.\DISPLAY1 expect dmPelsHeight 600, got 640
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 800, got 480
monitor.c:1280: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 600, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1301: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1314: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:2928: Test failed: ChangeDisplaySettingsExA \\.\DISPLAY1 returned unexpected -2.
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1335: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsWidth 640, got 480
monitor.c:1348: Test failed: Device \\.\DISPLAY1 test 0 expect dmPelsHeight 480, got 640

=== w11pro64_amd (64 bit report) ===

user32:
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:1247: Test failed: Device  test 0 EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1247: Test failed: Device  test 0 expect dmFields to contain 0x180000, got 0
monitor.c:1247: Test failed: Device  test 0 expect dmPelsWidth 640, got 0
monitor.c:1247: Test failed: Device  test 0 expect dmPelsHeight 480, got 0
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1253: Test failed: Device  expect dmPelsWidth 800, got 0
monitor.c:1253: Test failed: Device  expect dmPelsHeight 600, got 0
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1267: Test failed: Device  expect dmPelsWidth 800, got 0
monitor.c:1267: Test failed: Device  expect dmPelsHeight 600, got 0
monitor.c:1280: Test failed: Device  test 0 EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1280: Test failed: Device  test 0 expect dmFields to contain 0x180000, got 0
monitor.c:1280: Test failed: Device  test 0 expect dmPelsWidth 800, got 0
monitor.c:1280: Test failed: Device  test 0 expect dmPelsHeight 600, got 0
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:1301: Test failed: Device  test 0 EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1301: Test failed: Device  test 0 expect dmFields to contain 0x180000, got 0
monitor.c:1301: Test failed: Device  test 0 expect dmPelsWidth 640, got 0
monitor.c:1301: Test failed: Device  test 0 expect dmPelsHeight 480, got 0
monitor.c:1314: Test failed: Device  test 0 EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1314: Test failed: Device  test 0 expect dmFields to contain 0x180000, got 0
monitor.c:1314: Test failed: Device  test 0 expect dmPelsWidth 640, got 0
monitor.c:1314: Test failed: Device  test 0 expect dmPelsHeight 480, got 0
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:2910: Test failed: too few arguments.
monitor.c:376: Test failed: WaitForSingleObject returned 102.
monitor.c:1335: Test failed: Device  test 0 EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1335: Test failed: Device  test 0 expect dmFields to contain 0x180000, got 0
monitor.c:1335: Test failed: Device  test 0 expect dmPelsWidth 640, got 0
monitor.c:1335: Test failed: Device  test 0 expect dmPelsHeight 480, got 0
monitor.c:1348: Test failed: Device  test 0 EnumDisplaySettingsA failed, error 0xdeadbeef
monitor.c:1348: Test failed: Device  test 0 expect dmFields to contain 0x180000, got 0
monitor.c:1348: Test failed: Device  test 0 expect dmPelsWidth 640, got 0
monitor.c:1348: Test failed: Device  test 0 expect dmPelsHeight 480, got 0

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:55:32 2024 +0000, Zhiyi Zhang wrote:
> Let's use expected_width instead expectedWidth. Same for expectedHeight.
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65107

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:12 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_342_342)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65108

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:14 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_359_357)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65109

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:14 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_390_388)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65110

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:17 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_386_384)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65111

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:55:48 2024 +0000, Zhiyi Zhang wrote:
> Change the commit subject to "explorer: Restore display settings on
> process exit." and explain it restores the display settings to the ones
> in the registry when CDS_FULLSCREEN is used in ChangeDisplaySettings().
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65112

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:21 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_745_746)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65113

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:22 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_1212_1217)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65114

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:22 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_624_624)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65115

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:58:24 2024 +0000, Zhiyi Zhang wrote:
> Let's use expected_width instead expectedWidth. Same for expectedHeight.
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65116

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:24 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_342_342)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65117

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:26 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_359_357)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65118

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:26 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_390_388)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65119

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:29 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_386_384)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65120

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:58:37 2024 +0000, Zhiyi Zhang wrote:
> Change the commit subject to "explorer: Restore display settings on
> process exit." and explain it restores the display settings to the ones
> in the registry when CDS_FULLSCREEN is used in ChangeDisplaySettings().
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65121

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:33 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_745_746)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65122

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:34 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_1212_1217)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65123

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:34 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_624_624)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65124

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:18 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_2880_2956)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65125

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:30 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_2880_2956)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65126

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:55:46 2024 +0000, Zhiyi Zhang wrote:
> This part is duplicated. Could you move the tests up so that you don't
> have to restore them again?
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65127

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:58:35 2024 +0000, Zhiyi Zhang wrote:
> This part is duplicated. Could you move the tests up so that you don't
> have to restore them again?
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65128

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:33 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_733_733)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65129

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:21 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#ce22c7a18cb22fd4032a00c8133a393e55554648_733_733)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65130

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:55:46 2024 +0000, Zhiyi Zhang wrote:
> You've already called wait_for_dm(). Calling expect_dm seems redundant.
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65131

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:58:35 2024 +0000, Zhiyi Zhang wrote:
> You've already called wait_for_dm(). Calling expect_dm seems redundant.
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65132

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:58:33 2024 +0000, Zhiyi Zhang wrote:
> This test doesn't need two processes.
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65133

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:55:42 2024 +0000, Zhiyi Zhang wrote:
> This test doesn't need two processes.
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65134

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:32 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_2896_2956)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65135

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:20 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_2896_2956)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65136

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:19 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_2903_2956)
> Changed to 7s.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65137

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:13 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_377_375)
> It times out with 2s. Changed to 5s.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65138

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:15 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_1255_1237)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65139

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:55:35 2024 +0000, Zhiyi Zhang wrote:
> Why do you need this here?
> The mode restoration happens asynchronously, this sleep gives it a chance to complete.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65140

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:16 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_1268_1250)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65141

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:23 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_335_335)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65142

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:27 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_1255_1237)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65143

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:28 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_1268_1250)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65144

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:31 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_2903_2956)
> Changed to 7s.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65145

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:25 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_377_375)
> It times out with 2s. Changed to 5s.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65146

[Anton Baskanov (＠baskanov)]

On Wed Mar 13 09:58:27 2024 +0000, Zhiyi Zhang wrote:
> Why do you need this here?
> The mode restoration happens asynchronously, this sleep gives it a chance to complete.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65147

[Anton Baskanov (＠baskanov)]

On Mon Mar 18 03:40:12 2024 +0000, Anton Baskanov wrote:
> changed this line in [version 6 of the diff](/wine/wine/-/merge_requests/5060/diffs?diff_id=105629&start_sha=f0d0b1ebe4994b37a87ca14383bc9d1259ca6fe3#1a2fc38375b48458ba2fb34a2716bf76579d9527_335_335)
> Done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65148

[Anton Baskanov (＠baskanov)]

v5:
- Use snake_case for wait_for_dm() parameters.
- Lower the test timeouts.
- Remove dmFields check from wait_for_dm().
- Use shorter names for the test events.
- Rename test_ChangeDisplaySettingsEx() parameters.
- Remove the second process in multiple device test.
- Remove redundant expect_dm() calls.
- Remove redundant adapter setting restoration in test_ChangeDisplaySettingsEx().
- Move test child process code to a helper function.
- Change the commit 3 subject and description.
- Add tests for ChangeDisplaySettingsExA() without CDS_FULLSCREEN.
- Use a pointer for display_settings_restorer_classname.
- Pass class name instead of atom to UnregisterClassW().
- Move restorer thread creation to initialize_display_settings().
- Change wait_named_mutex() return type to void.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65149

[Anton Baskanov (＠baskanov)]

v6:
- Add back the removed adapter setting restoration.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060#note_65161