Re: msvcrt:string Avoid size_t in ok()

[Jerome Leclanche]

Not to be pedantic, but "did not return" is what you meant.


J. Leclanche


On Sat, Aug 28, 2010 at 2:38 PM, Alexandre Goujon <ale.goujon(a)gmail.com> wrote:
> ---
>  dlls/msvcrt/tests/string.c |   12 ++++++------
>  1 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/dlls/msvcrt/tests/string.c b/dlls/msvcrt/tests/string.c
> index 94c2fa3..3c143d1 100644
> --- a/dlls/msvcrt/tests/string.c
> +++ b/dlls/msvcrt/tests/string.c
> @@ -1185,22 +1185,22 @@ static void test_mbstowcs(void)
>     mOut[4] = '!'; mOut[5] = '\0';
>
>     ret = mbstowcs(NULL, mSimple, 0);
> -    ok(ret == 4, "ret = %d\n", ret);
> +    ok(ret == 4, "mbstowcs did not returned 4\n");
>
>     ret = mbstowcs(wOut, mSimple, 4);
> -    ok(ret == 4, "ret = %d\n", ret);
> +    ok(ret == 4, "mbstowcs did not returned 4\n");
>     ok(!memcmp(wOut, wSimple, 4*sizeof(wchar_t)), "wOut = %s\n", wine_dbgstr_w(wOut));
>     ok(wOut[4] == '!', "wOut[4] != \'!\'\n");
>
>     ret = wcstombs(NULL, wSimple, 0);
> -    ok(ret == 4, "ret = %d\n", ret);
> +    ok(ret == 4, "wcstombs did not returned 4\n");
>
>     ret = wcstombs(mOut, wSimple, 6);
> -    ok(ret == 4, "ret = %d\n", ret);
> +    ok(ret == 4, "wcstombs did not returned 4\n");
>     ok(!memcmp(mOut, mSimple, 5*sizeof(char)), "mOut = %s\n", mOut);
>
>     ret = wcstombs(mOut, wSimple, 2);
> -    ok(ret == 2, "ret = %d\n", ret);
> +    ok(ret == 2, "wcstombs did not returned 2\n");
>     ok(!memcmp(mOut, mSimple, 5*sizeof(char)), "mOut = %s\n", mOut);
>
>     if(!setlocale(LC_ALL, "Japanese_Japan.932")) {
> @@ -1209,7 +1209,7 @@ static void test_mbstowcs(void)
>     }
>
>     ret = mbstowcs(wOut, mHiragana, 6);
> -    ok(ret == 2, "ret = %d\n", ret);
> +    ok(ret == 2, "mbstowcs did not returned 2\n");
>     ok(!memcmp(wOut, wHiragana, sizeof(wHiragana)), "wOut = %s\n", wine_dbgstr_w(wOut));
>
>     ret = wcstombs(mOut, wHiragana, 6);
> --
> 1.7.0.4