----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25789/ -----------------------------------------------------------
(Updated Sept. 18, 2014, 11:10 p.m.) Review request for mesos and Benjamin Hindman. Changes ------- Dealt with mixed use between (const char *) and (std::string) in a single join() call. Accompanying unit test. Repository: mesos-git Description ------- Add Variadic strings join for c++11 and above. There is a second version of the variadic join which takes a reference to a stringstream as a parameter. This is handy when strings::join is just a part of a larger string manipulation. Diffs (updated) ----- 3rdparty/libprocess/3rdparty/stout/include/stout/strings.hpp a1702cd 3rdparty/libprocess/3rdparty/stout/tests/strings_tests.cpp 51008e5 Diff: https://reviews.apache.org/r/25789/diff/ Testing ------- Ran make check for stout. Added test cases for join as these were missing. Thanks, Joris Van Remoortere