> On Aug. 26, 2014, 9:58 p.m., Ben Mahler wrote: > > It was done using macro expansion because variadic templates require C++11. > > > > We're not yet able to assume C++11: > > https://issues.apache.org/jira/browse/MESOS-750 > > Niklas Nielsen wrote: > But as this reduce compile time for folks with C++11 compilers, can we > make this guarded by #ifdef and enable it for good when we make the > transition? > > Michael Park wrote: > +1. It seems like we can handle this version by adding it to the c++11 > directory and guarding it with #ifdef similar to defer.hpp.
Michael Park please make that change. I guess we will just check with: __cplusplus >= 201103L - Patrick ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25079/#review51595 ----------------------------------------------------------- On Aug. 27, 2014, 6:10 p.m., Patrick Reilly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25079/ > ----------------------------------------------------------- > > (Updated Aug. 27, 2014, 6:10 p.m.) > > > Review request for mesos, Adam B and Benjamin Hindman. > > > Bugs: MESOS-1734 > https://issues.apache.org/jira/browse/MESOS-1734 > > > Repository: mesos-git > > > Description > ------- > > Reduce compile time: - replacing a macro expansion with a variadic template > - moving implementation from help.hpp to help.cpp > > > Diffs > ----- > > 3rdparty/libprocess/Makefile.am edbe54b > 3rdparty/libprocess/include/process/help.hpp 4333b5b > 3rdparty/libprocess/src/help.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/25079/diff/ > > > Testing > ------- > > Ran "make check". > > > Thanks, > > Patrick Reilly > >