-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45310/#review125546
-----------------------------------------------------------




3rdparty/libprocess/3rdparty/stout/include/stout/try.hpp (line 56)
<https://reviews.apache.org/r/45310/#comment188402>

    Without additional context the reviewer may think that this change will 
break the build. Maybe you can change the order between this review and the 
next or eliminate the constructors in the next commit.


- Daniel Pravat


On March 24, 2016, 8:22 p.m., Michael Park wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45310/
> -----------------------------------------------------------
> 
> (Updated March 24, 2016, 8:22 p.m.)
> 
> 
> Review request for mesos, Alex Clemmer and Joris Van Remoortere.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Correctly constrained the templated constructor.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/include/stout/try.hpp 
> c444c0118d39ee6a5da4618d7c62784464377280 
> 
> Diff: https://reviews.apache.org/r/45310/diff/
> 
> 
> Testing
> -------
> 
> `make check`
> 
> 
> Thanks,
> 
> Michael Park
> 
>

Reply via email to