----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39946/#review105181 -----------------------------------------------------------
src/tests/mesos.cpp (line 447) <https://reviews.apache.org/r/39946/#comment163604> We tend to avoid abbreviations (like ctor, dtor): ``` ? mesos git:(master) ? grep -R destructor src | wc -l 22 ? mesos git:(master) ? grep -R dtor src | wc -l 0 ``` - Ben Mahler On Nov. 4, 2015, 6:39 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39946/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2015, 6:39 p.m.) > > > Review request for mesos and Joris Van Remoortere. > > > Bugs: MESOS-3827 > https://issues.apache.org/jira/browse/MESOS-3827 > > > Repository: mesos > > > Description > ------- > > As discussed in MESOS-3827, this can improve compilation time of classes that > use GMock. On my system, this improves the compilation time of the unit tests > by > about 8%. > > Note that we can't use this technique with TestAllocator, because it is > templatized. > > > Diffs > ----- > > src/Makefile.am d6eb302f0e812a777f51f421deef89140871a1db > src/tests/containerizer/launcher.hpp > 5d34bab789bdafe71d94e8ec263710c50b83e180 > src/tests/containerizer/launcher.cpp PRE-CREATION > src/tests/mesos.hpp f731ac3284a5793b6bf510d3a5b742cbe0938217 > src/tests/mesos.cpp ab2d85b091d121113931e4190a5b496901dcd7a5 > > Diff: https://reviews.apache.org/r/39946/diff/ > > > Testing > ------- > > > Thanks, > > Neil Conway > >