Re: [PATCH v2 0/2] MR4313: MSVCRT _(w)environ[] has a separate allocation chunk for each entry.

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msvcrt/tests/environ.c:
>          break;
>      }
>  }
> -
> - /* force change in _environ[], seems like native needs a first update to set allocation per entry */
> - _putenv( "__fish=cat" );
> - _putenv( "__fish=" );

Please temporarily add 2 variables to the block so the test doesn't have to be skipped in any case. Also it looks that you're not gaining anything by modifying original environment. After adding new environment variables, you can do the tests on them. I think it will be cleaner.