----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31664/#review75452 -----------------------------------------------------------
src/common/resources.cpp <https://reviews.apache.org/r/31664/#comment122502> Why do we need this include? src/common/resources.cpp <https://reviews.apache.org/r/31664/#comment122501> We can let the compiler do the copy for us. There is an opinion, that this is the preferred way: http://www.boost.org/doc/libs/1_57_0/libs/utility/operators.htm#symmetry - Alexander Rukletsov On March 3, 2015, 8:27 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31664/ > ----------------------------------------------------------- > > (Updated March 3, 2015, 8:27 p.m.) > > > Review request for mesos, Alexander Rukletsov and Ben Mahler. > > > Bugs: MESOS-2373 > https://issues.apache.org/jira/browse/MESOS-2373 > > > Repository: mesos > > > Description > ------- > > Convenience functions to help aggregate `hashmap<SlaveID, Resources>`. Used > in [r31665](https://reviews.apache.org/r/31665). > > > Diffs > ----- > > include/mesos/resources.hpp da6d48871a0061d8bbf5e681dd6e7edac659d812 > src/common/resources.cpp 2c99b6884d7296099e19e2e3182cbe11b5e1e059 > > Diff: https://reviews.apache.org/r/31664/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Michael Park > >