----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41622/#review112908 -----------------------------------------------------------
Ship it! src/tests/registrar_tests.cpp (line 19) <https://reviews.apache.org/r/41622/#comment173391> Not used?! src/tests/registrar_tests.cpp (line 65) <https://reviews.apache.org/r/41622/#comment173393> Not used!? src/tests/registrar_tests.cpp (line 67) <https://reviews.apache.org/r/41622/#comment173390> Not used?! src/tests/registrar_tests.cpp (line 909) <https://reviews.apache.org/r/41622/#comment173388> Seems `std::set` also can lose that namespace. - Till Toenshoff On Jan. 5, 2016, 7:12 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41622/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2016, 7:12 p.m.) > > > Review request for mesos and Adam B. > > > Repository: mesos > > > Description > ------- > > Removed unnecessary std namespaces from registrar tests. > > > Diffs > ----- > > src/tests/registrar_tests.cpp 6064621a001d66423f9b2dc7b749b67d9fc4bc13 > > Diff: https://reviews.apache.org/r/41622/diff/ > > > Testing > ------- > > `GTEST_FILTER="*Registrar*" bin/mesos-tests.sh` > > > Thanks, > > Greg Mann > >
