[2/2] mlang: Add a test for IMultiLanguage2_IsCodePageInstallable.
Alexandre Julliard
julliard at winehq.org
Wed Jul 1 06:09:52 CDT 2009
Paul Vriens <paul.vriens.wine at gmail.com> writes:
> Hans Leidekker wrote:
>> On Wednesday 01 July 2009 12:12:47 pm Paul Vriens wrote:
>>
>>>> Would using IsValidCodePage() be helpful here?
>>>>
>>> Yep, something like:
>>>
>>> hr = IMultiLanguage2_IsCodePageInstallable(ml2, i);
>>> if (i == 29001 || /* win2k3 */
>>> i == 50930 || i == 50931 || i == 50933 || i == 50935
>>> || i == 50937 || i == 50939 /* vista */)
>>> {
>>> ok(hr == S_FALSE || hr == S_OK ||
>>> broken(!IsValidCodePage(i)), "%u: expected S_FALSE or S_OK, got
>>> 0x%08x\n", i, hr);
>>
>> IsValidCodePage might be helpful yes, although marking it broken is
>> a bit too strong I think, it's just a difference in codepage databases.
>>
>> -Hans
> Yes, broken() seems strong but it's actually meant as
> behaviour_we_dont_want_to_replicate() in this case.
I'm not convinced we don't want to replicate it. Using something like
IsValidCodePage in the implementation would be better than hardcoding
the list.
--
Alexandre Julliard
julliard at winehq.org
More information about the wine-devel
mailing list