Re: [PATCH 2/2 v4] ucrtbase: Add _get_narrow_winmain_command_line tests

[Piotr Caban]

On 07/28/16 13:08, Dmitry Timoshkov wrote:
> Piotr Caban <piotr(a)codeweavers.com> wrote:
> > +    sprintf(cmd, "\"\"%c\"\"\"%s\" \t \"misc\" cmd", name[0], name+1);
> > +    memset(&startup, 0, sizeof(startup));
> > +    startup.cb = sizeof(startup);
> > +    CreateProcessA(path, cmd, NULL, NULL, TRUE,
> > +            CREATE_DEFAULT_ERROR_MODE|NORMAL_PRIORITY_CLASS,
> > +            NULL, NULL, &startup, &proc);
> > +    winetest_wait_child_process(proc.hProcess);
>
> Thanks for adding the tests. It shouldn't be too hard to also test other
> white space characters, at least testing '\r' and '\n' would be nice to
> have.

CreateProcess will fail in case of '\r' or '\n' characters.