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

[Akihiro Sagawa (＠sgwaki)]

Akihiro Sagawa (@sgwaki) commented about dlls/user32/tests/resource.c:
> - 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);
> -
>    ok(!!p, "[%d]: can't load resource %#lx.\n", i, GetLastError());
> -
> -
>    num = string_table_tests[i].id & 0x000f;
> -
>    for (j = 0; j < num; j++)
> -
>        p += *p + 1;
> -
>    lenght = *p;
> -
>    ok(lenght == wcslen(string_table_tests[i].string), "[%d]: got wrong lenght %d.\n", i, lenght);
> -
>    p++;
> -
>    ok(!wcsncmp(p, string_table_tests[i].string, lenght), "[%d]: got wrong string %s.\n", i, debugstr_w(p));

I prefer `winetest_push_context()` and `winetest_pop_context()` pair instead of every `[%d]: `.