Re: [PATCH v2 0/3] MR1070: Add more tests for \Device\NamedPipe, \Device\NamedPipe\, and exotic pipe names

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/ntdll/tests/pipe.c:
> - static const struct pipe_name_test tests[] = {
> -
>    { NULL              , STATUS_OBJECT_NAME_INVALID },
> -
>    { L""               , STATUS_OBJECT_NAME_INVALID },
> -
>    { L"\\"             , STATUS_SUCCESS },
> -
>    { L"wine-test\\"    , STATUS_SUCCESS, L"wine-test" },
> -
>    { L"wine/test"      , STATUS_SUCCESS, L"wine\\test" },
> -
>    { L"wine:test"      , STATUS_SUCCESS },
> -
>    { L"wine\\.\\test"  , STATUS_SUCCESS, L"wine\\test" },
> -
>    { L"wine\\..\\test" , STATUS_SUCCESS, L"test" },
> -
>    { L"..\\wine-test"  , STATUS_SUCCESS },
> -
>    { L"!\"#$%&'()*+,-./:;<=>?@[\\]^_`{|}~", STATUS_SUCCESS },
> - };
> - UNICODE_STRING name_prefix;
> - size_t i, j;
> -
> - RtlInitUnicodeString(&name_prefix, L"\Device\NamedPipe");

It seems like it would be simpler and more readable just to put this in the test structure itself.