----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15285/ -----------------------------------------------------------
(Updated Dec. 3, 2013, 2:07 a.m.) Review request for mesos, Benjamin Hindman and Ross Allen. Changes ------- Test cases and suggestions from Ben H Repository: mesos-git Description ------- This patch adds a toString member function to the Flag structure which lets us iterate and fetch flag values. During flag initialization, the toString methods are closed over the flag value and option references. Flag values can't be closed over up front as flag values may be changed by user code arbitrarily. So toString takes in the owning flags instance as a FlagsBase pointer: toString(FlagsBase*). Diffs (updated) ----- 3rdparty/libprocess/3rdparty/stout/include/stout/flags/flag.hpp d31c984 3rdparty/libprocess/3rdparty/stout/include/stout/flags/flags.hpp cfe996e 3rdparty/libprocess/3rdparty/stout/include/stout/flags/stringifier.hpp PRE-CREATION 3rdparty/libprocess/3rdparty/stout/tests/flags_tests.cpp 9af2da1 Diff: https://reviews.apache.org/r/15285/diff/ Testing ------- make check Thanks, Niklas Nielsen
