> On Jan. 3, 2013, 7:39 p.m., Vinod Kone wrote: > > third_party/libprocess/src/encoder.hpp, line 202 > > <https://reviews.apache.org/r/8798/diff/1/?file=244245#file244245line202> > > > > Looks like moving it inside Request is better.
Going to ignore this comment on this review, since it was introduced in the previous review: https://reviews.apache.org/r/8797/ > On Jan. 3, 2013, 7:39 p.m., Vinod Kone wrote: > > third_party/libprocess/src/tests/decoder_tests.cpp, line 72 > > <https://reviews.apache.org/r/8798/diff/1/?file=244247#file244247line72> > > > > its hard to tell whether the spaces match. its probably better to just > > split and check the contents of "Accept-Encoding"? Dropping since the test indicates whether the spaces match or not ;) Also, this may end up changing when my previous review gets updated: https://reviews.apache.org/r/8797/ - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8798/#review15047 ----------------------------------------------------------- On Jan. 2, 2013, 2:54 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8798/ > ----------------------------------------------------------- > > (Updated Jan. 2, 2013, 2:54 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > See above. > > > Diffs > ----- > > third_party/libprocess/Makefile.am 11f8b36d19313f65f1b191a781c6d5ed90130099 > third_party/libprocess/include/process/http.hpp > 40248dab3df59b322ad9c5a4858961ef471779fd > third_party/libprocess/src/decoder.hpp > 253824f964845c8b7c9c29f31301b5b6c2c4a2f1 > third_party/libprocess/src/encoder.hpp > 9664f38b7b972f196921ef13adb8c9caa180ca65 > third_party/libprocess/src/process.cpp > 72e437862ee0b35126c16d32bec79ef76a4e2b23 > third_party/libprocess/src/tests/decoder_tests.cpp PRE-CREATION > third_party/libprocess/src/tests/main.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/8798/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ben Mahler > >
