[PATCH 1/3] kernel32: Support UTF-7 in MultiByteToWideChar.

Alexandre Julliard julliard at winehq.org
Mon Sep 3 10:26:39 CDT 2012


Alex Henrie <alexhenrie24 at gmail.com> writes:

> 2012/9/3 Alexandre Julliard <julliard at winehq.org>:
>> You don't have a single test with an explicit source length, so
>> obviously you are not going to find problems in that area. Like I said,
>> you need a lot more tests.
>
> example_2 already tests an explicit source length. In all the other
> examples srclen = strlen(src) + 1, see
> http://source.winehq.org/git/wine.git/blob/2097c5ddb6bec24cf4bbd440a6f401cfa67007f0:/dlls/kernel32/locale.c#l1929
>
> So again, what specific additional test, or what specific change to
> the existing tests, do you want to see?

Think about what the code is doing, and how to test it. In particular
spend some time thinking about the source length parameter.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list