Re: [PATCH] msvcrt/tests: Add tests for mbstowcs and wcstombs with a negative count.

[Piotr Caban]

Hi Alex,

On 06/12/16 22:07, Alex Henrie wrote:
> +    ret = mbstowcs(wOut, mSimple, -1);
> +todo_wine
> +    ok(ret == -1 || broken(ret == 4) /* xp */, "mbstowcs returned %i instead of -1\n", ret);
> This will cause compilation warning:
> ../../../../wine_src/dlls/msvcrt/tests/string.c:1863:5: warning: format 
> '%i' expects argument of type 'int', but argument 3 has type 'size_t' 
> [-Wformat=]
>       ok(ret == -1 || broken(ret == 4) /* xp */, "mbstowcs returned %i 
> instead of -1\n", ret);

I'm also not sure if Windows XP behavior should be marked as broken. For 
me newer systems behavior is broken (doesn't handle strings longer then 
INT_MAX). Do you have an application that depends on it?

Thanks,
Piotr