----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4899/#review7290 -----------------------------------------------------------
Stopped short at utils.hpp for lack of tests. src/common/protobuf_utils.hpp <https://reviews.apache.org/r/4899/#comment16096> const reference for this guy too right? src/common/utils.hpp <https://reviews.apache.org/r/4899/#comment16097> Tests for these seem in order. IIRC there is a decent temp dir utility to setup files and dirs for this. - John On 2012-04-27 01:11:30, Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4899/ > ----------------------------------------------------------- > > (Updated 2012-04-27 01:11:30) > > > Review request for mesos and John Sirois. > > > Summary > ------- > > We can now read/write strings to a file. > > Some common utils for working with protobuf objects. > > > Diffs > ----- > > src/Makefile.am cd503a8 > src/common/protobuf_utils.hpp PRE-CREATION > src/common/utils.hpp 1d81e21 > src/slave/slave.cpp b233b68 > src/tests/protobuf_io_tests.cpp 22f37ac > > Diff: https://reviews.apache.org/r/4899/diff > > > Testing > ------- > > make check > > > Thanks, > > Vinod > >
