----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26472/#review55930 -----------------------------------------------------------
3rdparty/libprocess/3rdparty/stout/include/stout/stringify.hpp <https://reviews.apache.org/r/26472/#comment96305> This removes attempting to stringify the item a second time to display it in an error message. I can add it back in, but it seems like that would likely just recurse into erros. - Cody Maloney On Oct. 9, 2014, 1:20 a.m., Cody Maloney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26472/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2014, 1:20 a.m.) > > > Review request for mesos, Adam B and Dominic Hamon. > > > Bugs: MESOS-1870 > https://issues.apache.org/jira/browse/MESOS-1870 > > > Repository: mesos-git > > > Description > ------- > > This makes it so any time there is an abort, we get a line number and at > least a basic message as to why there was an abort. If you want a clean(er) > exit, use <stout/exit>. > > Also adds an overload which takes a standard string and unwraps it to a const > char * automatically, since a lot of the time we are building strings to pass > them to abort. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/abort.hpp 6b5b5d1 > 3rdparty/libprocess/3rdparty/stout/include/stout/flags/flags.hpp 9d244b2 > 3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp 7138bc2 > 3rdparty/libprocess/3rdparty/stout/include/stout/os/fork.hpp 8aa21ed > 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp ccf80a7 > 3rdparty/libprocess/3rdparty/stout/include/stout/result.hpp ce8dd9b > 3rdparty/libprocess/3rdparty/stout/include/stout/stringify.hpp ed0a1ef > 3rdparty/libprocess/3rdparty/stout/include/stout/thread.hpp b1af74f > 3rdparty/libprocess/3rdparty/stout/include/stout/try.hpp 87c5fc8 > 3rdparty/libprocess/3rdparty/stout/tests/os_tests.cpp 9207c55 > > Diff: https://reviews.apache.org/r/26472/diff/ > > > Testing > ------- > > make distcheck > > > Thanks, > > Cody Maloney > >
