[PATCH v2 1/3] msvcrt/tests: Test wcstoi64 with various Unicode digits.

Piotr Caban piotr.caban at gmail.com
Thu Dec 15 05:01:25 CST 2016


Hi,

On 12/14/16 18:18, Lauri Kenttä wrote:
> +    for (i = 0; i < sizeof(zeros) / sizeof(zeros[0]); ++i) {
> +        WCHAR tmp[] = {zeros[i] + 4, zeros[i], zeros[i] + 5, 0};
> +        res = p_wcstoi64(tmp, NULL, 0);
> +        todo_wine ok(res == 405, "with zero = U+%04X: got %d, expected 405\n", zeros[i], (int)res);
> +        tmp[1] = zeros[i] + 10;
> +        res = p_wcstoi64(tmp, NULL, 0);
Please change the line to:
res = p_wcstoi64(tmp, NULL, 16);
Currently you don't test if zeros[i]+10 is recognized as valid digit.

Except of that the patches are looking good for me. Are these patches 
fixing any bugs? Is any application working thanks to them (I'm asking 
because of the code freeze)?

Thank you,
Piotr



More information about the wine-devel mailing list