----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34676/#review85417 -----------------------------------------------------------
src/common/resources.cpp <https://reviews.apache.org/r/34676/#comment136978> Instead of a render function, just make it a output stream operator overload for Volume. ostream& operator << (ostream& stream, const Volume& volume) { } src/common/resources.cpp <https://reviews.apache.org/r/34676/#comment136979> per the above comment, this will be stream << ": " << disk.volume(); - Vinod Kone On May 26, 2015, 11:19 p.m., Brian Wickman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34676/ > ----------------------------------------------------------- > > (Updated May 26, 2015, 11:19 p.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-2693 > https://issues.apache.org/jira/browse/MESOS-2693 > > > Repository: mesos > > > Description > ------- > > Add printing of extended attributes to Resource objects. I have no idea what > format is appropriate. I made something that stands some chance of being > parseable in the future. > > > Diffs > ----- > > src/common/resources.cpp d93f38ecdc31605b6a84c8d103ac90e8f0632b01 > src/tests/resources_tests.cpp 459a68bb8e30852becc7d8825a24f984cb027b08 > > Diff: https://reviews.apache.org/r/34676/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Brian Wickman > >
