----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37968/#review98366 -----------------------------------------------------------
Ship it! I've committed this for you with the following minor changes! 3rdparty/libprocess/3rdparty/stout/include/stout/format.hpp (line 85) <https://reviews.apache.org/r/37968/#comment154813> `s/for detail/for further details/` 3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/shell.hpp (line 57) <https://reviews.apache.org/r/37968/#comment154812> Removed this newline. - Michael Park On Sept. 10, 2015, 2:38 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37968/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2015, 2:38 a.m.) > > > Review request for mesos and Michael Park. > > > Bugs: MESOS-3344 > https://issues.apache.org/jira/browse/MESOS-3344 > > > Repository: mesos > > > Description > ------- > > Add more comments for why not using const reference for some functions > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/format.hpp > 1cf6dd18aa163688d6c8f3a6d33eacad3918015d > 3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/shell.hpp > 68fc1fd179ee51fc5de0452c0f2ea3d354e0567f > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/shell.hpp > 01e59de466496dec9367ad6f48538327f53a7e18 > > Diff: https://reviews.apache.org/r/37968/diff/ > > > Testing > ------- > > > Thanks, > > Guangya Liu > >
