Re: [PATCH 0/1] MR4025: user32: Add more builtin string resources.

[Akihiro Sagawa (＠sgwaki)]

Akihiro Sagawa (@sgwaki) commented about dlls/user32/tests/resource.c:
> -
>    { 807, L"&Close" },
> -
>    { 808, L"Help" },
> -
>    { 809, L"&Try Again" },
> -
>    { 810, L"&Continue" },
> - };
> -
> - module = GetModuleHandleW(L"user32.dll");
> - ok(!!module, "Can't get module %#lx.\n", GetLastError());
> -
> - for (i = 0; i < ARRAYSIZE(string_table_tests); i++)
> - {
> -
>    src = FindResourceExW(module, (LPWSTR)RT_STRING,
> -
>            MAKEINTRESOURCEW((LOWORD(string_table_tests[i].id) >> 4) + 1),
> -
>            MAKELANGID(LANG_ENGLISH, SUBLANG_DEFAULT));
> -
>    ok(!!src, "[%d]: can't get a resource %#lx.\n", i, GetLastError());
> -
>    p = LoadResource(module, src);

I would be interested in others' opinions, but in my opinion it is better to just use `LoadString()` here and skip the test if the locale isn't English.