-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58126/#review170804
-----------------------------------------------------------
3rdparty/stout/include/stout/os/windows/shell.hpp
Line 179 (original), 179 (patched)
<https://reviews.apache.org/r/58126/#comment243682>
Wow. I'd really like to see unit tests on that, so that we can make sure it works in cases
we care about, and so that if we find special cases down the road, we can make sure we don't
break existing cases.
Can you please add a unit test to call this with some basic tests as well as the specific
cases that you found to fail?
- Jeff Coffler
On March 31, 2017, 11:49 p.m., Andrew Schwartzmeyer wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58126/
> -----------------------------------------------------------
>
> (Updated March 31, 2017, 11:49 p.m.)
>
>
> Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, Li Li, and Michael Park.
>
>
> Bugs: MESOS-5418 and MESOS-5522
> https://issues.apache.org/jira/browse/MESOS-5418
> https://issues.apache.org/jira/browse/MESOS-5522
>
>
> Repository: mesos
>
>
> Description
> -------
>
> This was an unused function that ended up being the correct place to
> implement proper `argv` concatenation and escaping. It returns a
> `std::wstring` for use (speifically) by `::CreateProcessW`. This brings
> us a bit closer to Unicode support within Mesos.
>
>
> Diffs
> -----
>
> 3rdparty/stout/include/stout/os/windows/shell.hpp fdce93c2146ddec6117577b538dca77c416e0c01
>
>
> Diff: https://reviews.apache.org/r/58126/diff/1/
>
>
> Testing
> -------
>
> Testing done later in chain.
>
>
> Thanks,
>
> Andrew Schwartzmeyer
>
>
|