Re: [PATCH 0/5] MR6703: msvcrt: Improve DBCS support for _strnicmp_l().

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msvcrt/tests/string.c:
> +    ok(ret == 185, "Got %d.\n", ret);
> +    ret = _tolower_l('a', 0);
> +    ok(ret == 97, "Got %d.\n", ret);
> +
> +    if (!setlocale(LC_ALL, ".936"))
> +    {
> +        win_skip("Skip _tolower_l tests.\n");
> +        return;
> +    }
> +
> +    ret = _tolower_l('\xa5', 0);
> +    ok(ret == -91, "Got %d.\n", ret);
> +    ret = _tolower_l('\xb9', 0);
> +    ok(ret == -71, "Got %d.\n", ret);
> +    ret = _tolower_l('a', 0);
> +    ok(ret == 97, "Got %d.\n", ret);
> I think it's _tolower_l/_toupper_l that needs to be fixed (and C locale handling in strnicmp). Please also add tests with unsigned char cast:
    ret = _tolower_l((unsigned char)'\xa5', 0);
    ok(ret == 165, "Got %d.\n", ret);
    ret = _tolower_l((unsigned char)'\xb9', 0);
    ok(ret == 185, "Got %d.\n", ret);

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6703#note_85527