----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39006/#review101924 -----------------------------------------------------------
Patch looks great! Reviews applied: [39005, 39006] All tests passed. - Mesos ReviewBot On Oct. 8, 2015, 2:36 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39006/ > ----------------------------------------------------------- > > (Updated Oct. 8, 2015, 2:36 p.m.) > > > Review request for mesos, Bernd Mathiske, Ben Mahler, and Till Toenshoff. > > > Bugs: MESOS-3551 > https://issues.apache.org/jira/browse/MESOS-3551 > > > Repository: mesos > > > Description > ------- > > Switch call sites to using safe strerror_r wrapper. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/error.hpp > 12ba1ca861114e60f8276c0ee91c543abcfc2519 > 3rdparty/libprocess/3rdparty/stout/include/stout/ip.hpp > d1e2df6151149e03ffb4a76e2c24ff78b891e016 > 3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/fork.hpp > 7eb51e8771e95f57548fc35753e75c6d56cd97cd > 3rdparty/libprocess/3rdparty/stout/tests/os/sendfile_tests.cpp > e740d5bc0f0cc5cf8e99b2064c1e39c08282da67 > 3rdparty/libprocess/3rdparty/stout/tests/os_tests.cpp > e6d36ec1bf414b52d0899f0edf83e0ad8910dd0e > > Diff: https://reviews.apache.org/r/39006/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Bannier > >