[PATCH v2 2/7] gdi32: Look for a another secondary font family name.

[Rémi Bernon]

On 2020-09-17 18:48, Marvin wrote:
> Hi,
> 
> While running your changed tests, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> 
> Full results can be found at:
> https://testbot.winehq.org/JobDetails.pl?Key=78872
> 
> Your paranoid android.
> 
> 
> === w10pro64_ja (32 bit report) ===
> 
> gdi32:
> font.c:7138: Test failed: 0: EnumFontFamiliesExA unexpected count 1.
> font.c:7156: Test failed: 0: EnumFontFamiliesExW unexpected count 0.
> font.c:7173: Test failed: 0: EnumFontFamiliesExA unexpected count 1.
> font.c:7211: Test failed: 0: EnumFontFamiliesExA unexpected count 0.
> font.c:7138: Test failed: 1: EnumFontFamiliesExA unexpected count 2.
> font.c:7156: Test failed: 1: EnumFontFamiliesExW unexpected count 0.
> font.c:7173: Test failed: 1: EnumFontFamiliesExA unexpected count 1.
> font.c:7183: Test failed: 1: (0) unexpected lfFaceName L"\30d5\30a9\30f3\30c8\540d Cond (ja)"
> font.c:7188: Test failed: 1: (0) unexpected elfStyle L"Reg (ja)"
> font.c:7211: Test failed: 1: EnumFontFamiliesExA unexpected count 0.
> font.c:7138: Test failed: 2: EnumFontFamiliesExA unexpected count 2.
> font.c:7156: Test failed: 2: EnumFontFamiliesExW unexpected count 0.
> font.c:7173: Test failed: 2: EnumFontFamiliesExA unexpected count 1.
> font.c:7183: Test failed: 2: (0) unexpected lfFaceName L"\30d5\30a9\30f3\30c8\540d Cond (ja)"
> font.c:7188: Test failed: 2: (0) unexpected elfStyle L"Reg (ja)"
> font.c:7211: Test failed: 2: EnumFontFamiliesExA unexpected count 0.
> 
> === w10pro64_ja (64 bit report) ===
> 
> gdi32:
> font.c:7138: Test failed: 0: EnumFontFamiliesExA unexpected count 1.
> font.c:7156: Test failed: 0: EnumFontFamiliesExW unexpected count 0.
> font.c:7173: Test failed: 0: EnumFontFamiliesExA unexpected count 1.
> font.c:7211: Test failed: 0: EnumFontFamiliesExA unexpected count 0.
> font.c:7138: Test failed: 1: EnumFontFamiliesExA unexpected count 2.
> font.c:7156: Test failed: 1: EnumFontFamiliesExW unexpected count 0.
> font.c:7173: Test failed: 1: EnumFontFamiliesExA unexpected count 1.
> font.c:7183: Test failed: 1: (0) unexpected lfFaceName L"\30d5\30a9\30f3\30c8\540d Cond (ja)"
> font.c:7188: Test failed: 1: (0) unexpected elfStyle L"Reg (ja)"
> font.c:7211: Test failed: 1: EnumFontFamiliesExA unexpected count 0.
> font.c:7138: Test failed: 2: EnumFontFamiliesExA unexpected count 2.
> font.c:7156: Test failed: 2: EnumFontFamiliesExW unexpected count 0.
> font.c:7173: Test failed: 2: EnumFontFamiliesExA unexpected count 1.
> font.c:7183: Test failed: 2: (0) unexpected lfFaceName L"\30d5\30a9\30f3\30c8\540d Cond (ja)"
> font.c:7188: Test failed: 2: (0) unexpected elfStyle L"Reg (ja)"
> font.c:7211: Test failed: 2: EnumFontFamiliesExA unexpected count 0.
> 

Damn, I've run that test multiple times, but I did it on the 
vm_w1064v1809_ja VM. It seems to be configured differently and this VM 
runs the test that are guarded by:

if (PRIMARYLANGID(GetUserDefaultLangID()) == LANG_ENGLISH)

I guess maybe only the GetSystemDefaultLangID() returns LANG_JAPANESE 
there? I'll fix the tests but is this actually expected?

I'll cancel the whole series anyway, as it doesn't seem to be liked very 
much, I'll resend another one, probably without the controversial unix 
paths thing :).
-- 
Rémi Bernon <rbernon at codeweavers.com>