Re: [PATCH v2 0/3] MR8792: msvcp140_atomic_wait: A few std::chrono::tzdb semi-stubs.

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msvcp140_atomic_wait/tests/msvcp140_atomic_wait.c:
> -
>    win_skip("__std_tzdb_get_time_zones is not available, skipping tests.\n");
> -
>    return;
> - }
> -
> - z = p___std_tzdb_get_time_zones();
> - ok(!!z, "got NULL.\n");
> - ok(!z->error || broken(z->error == TZDB_ERROR_WIN && !z->ver && !z->count), "got %u.\n", z->error);
> - if (z->error)
> - {
> -
>    win_skip("__std_tzdb_get_time_zones empty result, skipping remaining tests.\n");
> -
>    p___std_tzdb_delete_time_zones(z);
> -
>    return;
> - }
> - ok(!!z->ver, "got NULL.\n");
> - ok(z->count, "got 0.\n");
> - *(volatile char *)z->ver |= 0;

It looks like a leftover after checking if z->ver can be changed (or if the changes are preserved between calls). Please remove it.