----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28959/#review64830 -----------------------------------------------------------
3rdparty/libprocess/include/process/event.hpp <https://reviews.apache.org/r/28959/#comment107616> no using statements in header files please. 3rdparty/libprocess/src/decoder.hpp <https://reviews.apache.org/r/28959/#comment107617> no using statements in header files. 3rdparty/libprocess/src/tests/process_tests.cpp <https://reviews.apache.org/r/28959/#comment107618> please remove this. either enumerate the symbols you're using using process::network::accept; using process::network::connect; or don't use using and just scope the calls explicitly. - Dominic Hamon On Dec. 11, 2014, 11:41 a.m., Evelina Dumitrescu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28959/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2014, 11:41 a.m.) > > > Review request for mesos, Ben Mahler and Dominic Hamon. > > > Repository: mesos-git > > > Description > ------- > > Introduce process::network namespace to encapsulate socket call wrappers and > Socket class > > > Diffs > ----- > > 3rdparty/libprocess/include/process/event.hpp > 6392332d7bf63157d305868ac6bfe95c6921ddaa > 3rdparty/libprocess/include/process/socket.hpp > 91f9084984daa18d323f05946211bea90f94ae11 > 3rdparty/libprocess/src/decoder.hpp > b66f378b4ecbded9e091267e943e721bb153c290 > 3rdparty/libprocess/src/http.cpp 465e4473685a2e441401e454ad1e8c04db2a4e0a > 3rdparty/libprocess/src/process.cpp > 688aebd196ae954441a393c5b702db1b2036453c > 3rdparty/libprocess/src/tests/http_tests.cpp > 3bda76d4b4b958e956b4f5c248190c2fdfcd0d51 > 3rdparty/libprocess/src/tests/process_tests.cpp > 45be2e15d6f57db89ba7ab758d828ffce3be4b61 > > Diff: https://reviews.apache.org/r/28959/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Evelina Dumitrescu > >
