----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9793/#review17520 -----------------------------------------------------------
Ship it! third_party/libprocess/third_party/stout/include/stout/os.hpp <https://reviews.apache.org/r/9793/#comment37179> s/len/length? third_party/libprocess/third_party/stout/include/stout/os.hpp <https://reviews.apache.org/r/9793/#comment37180> To make 2 more readable I sometimes will pull out a const size_t NAME_SIZE above and use that. - Benjamin Hindman On March 7, 2013, 3:15 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9793/ > ----------------------------------------------------------- > > (Updated March 7, 2013, 3:15 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > See description. > > > Diffs > ----- > > third_party/libprocess/third_party/stout/include/stout/os.hpp > da14a8fb95b69684c899e04ca79a1ad8eb11b2b8 > > Diff: https://reviews.apache.org/r/9793/diff/ > > > Testing > ------- > > Tested manually, I will be splitting apart the monolithic stout tests in > another review chain. > > > Thanks, > > Ben Mahler > >
