user32/tests: Fix the DBCS WM_CHAR tests in mixed locales.

WM_CHAR interprets the character codes using a code page tied to the keyboard layout, not CP_ACP. Also skip the test if WideCharToMultiByte() used the default character since that breaks the round-trip. Trace the code page when skipping the tests. Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=54424

user32/tests: Fix the DBCS WM_CHAR tests in mixed locales.
8b8d787d · Francois Gouget · Alexandre Julliard · 6fb47669 · 8b8d787d
Commit 8b8d787d authored Apr 17, 2023 by Francois Gouget Committed by Alexandre Julliard Apr 17, 2023
Show whitespace changes
Inline Side-by-side

Showing with 11 additions and 3 deletions

edit.c dlls/user32/tests/edit.c +11 -3

No files found.
--- a/dlls/user32/tests/edit.c
+++ b/dlls/user32/tests/edit.c
@@ -3368,15 +3368,22 @@ static void test_wordbreak_proc(void)
 static void test_dbcs_WM_CHAR(void)
 {
+    HKL hkl;
+    UINT cp;
    WCHAR textW[] = { 0x4e00, 0x4e8c, 0x4e09, 0 }; /* one, two, three */
    unsigned char bytes[7];
+    BOOL hasdefault;
    HWND hwnd[2];
    int i;
-    WideCharToMultiByte(CP_ACP, 0, textW, -1, (char *)bytes, ARRAY_SIZE(bytes), NULL, NULL);
+    hkl = GetKeyboardLayout( 0 );
-    if (!IsDBCSLeadByte(bytes[0]))
+    if (!GetLocaleInfoW(LOWORD(hkl), LOCALE_IDEFAULTANSICODEPAGE | LOCALE_RETURN_NUMBER, (WCHAR *)&cp, sizeof(cp) / sizeof(WCHAR)))
+        cp = CP_ACP;
+    WideCharToMultiByte(cp, WC_NO_BEST_FIT_CHARS, textW, -1, (char *)bytes, ARRAY_SIZE(bytes), NULL, &hasdefault);
+    if (hasdefault || !IsDBCSLeadByteEx(cp, bytes[0]))
    {
-        skip("Skipping DBCS WM_CHAR test in this codepage\n");
+        skip("Skipping DBCS WM_CHAR test in the %d codepage\n", cp);
        return;
    }
    hwnd[0] = create_editcontrol(ES_AUTOHSCROLL | ES_AUTOVSCROLL, 0);
@@ -3396,6 +3403,7 @@ static void test_dbcs_WM_CHAR(void)
        r = SetWindowTextA(hwnd[i], "");
        ok(r, "SetWindowText failed\n");
+        /* sending the lead byte separately works for DBCS locales */
        for (p = bytes; *p; p++)
            PostMessageA(hwnd[i], WM_CHAR, *p, 1);