----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41881/#review112609 -----------------------------------------------------------
Looks good. How did you choose the particular functions to make constexpr. Why not the units for example? or others? - Joris Van Remoortere On Jan. 4, 2016, 11:28 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41881/ > ----------------------------------------------------------- > > (Updated Jan. 4, 2016, 11:28 a.m.) > > > Review request for mesos, Joris Van Remoortere, Till Toenshoff, and Jiang Yan > Xu. > > > Bugs: MESOS-4277 > https://issues.apache.org/jira/browse/MESOS-4277 > > > Repository: mesos > > > Description > ------- > > For that we need to provide a constexpr constructor for Nanoseconds at > least; make all constructors of Duration types constexpr if possible. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/duration.hpp > dc610e697032f0d5d9c7a6524aeb34984041dec7 > > Diff: https://reviews.apache.org/r/41881/diff/ > > > Testing > ------- > > > Thanks, > > Benjamin Bannier > >
