Re: [PATCH v2 0/1] MR4654: Draft: msvcrt: Protect setlocale against concurrent accesses.

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msvcrt/tests/locale.c:
> +{
> - const char* locale = NULL;
> - if ((intptr_t)arg % 2) locale = "German_Germany.1252";
> - setlocale(LC_NUMERIC, locale);
> - return 0;
>
> +}
> -
>
> +static void test_setlocale_threads(void)
> +{
> - HANDLE threads[64];
> - for (intptr_t i = 0; i < ARRAY_SIZE(threads); i++)
> - {
> -
>    threads[i] = CreateThread(NULL, 0, test_setlocale_threads_proc, (void*)i, 0, NULL);
> -
>    ok(threads[i] != NULL, "CreateThread failed\n");
> - }
> - WaitForMultipleObjects(ARRAY_SIZE(threads), threads, TRUE, INFINITE);

```suggestion:-6+0
    HANDLE threads[64];
    int i;

for (i = 0; i < ARRAY_SIZE(threads); i++)
    {
        threads[i] = CreateThread(NULL, 0, test_setlocale_threads_proc, (void*)i, 0, NULL);
        ok(threads[i] != NULL, "CreateThread failed\n");
    }
    for (i = 0; i < ARRAY_SIZE(threads); i++)
    {
        WaitForSingleObject(threads[i], INFINITE);
        CloseHandle(threads[i]);
    }
```