Re: Review Request 44141: Windows: Forked subprocess.cpp, added `Windows` implementation.

> On March 1, 2016, 11:47 p.m., Yi Sun wrote: > > 3rdparty/libprocess/src/windows/subprocess.cpp, line 152 > > > > > > The second param for strcpy_s is the number of chars to be copied. It > > doesn't seem right

Re: Review Request 44141: Windows: Forked subprocess.cpp, added `Windows` implementation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44141/#review121543 --- Ship it! Ship It! - Yi Sun On March 1, 2016, 8:58 p.m.,

Re: Review Request 44141: Windows: Forked subprocess.cpp, added `Windows` implementation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44141/#review121540 --- Bad patch! Reviews applied: [44141, 44140] Failed command: