Re: Review Request 13387: (Stout) Rewrote os::read(string path) because the existing implementation does not work well under the inconsistency of /proc pseudo filesystem.

[Jiang Yan Xu]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13387/
-----------------------------------------------------------

(Updated Aug. 9, 2013, 9:48 p.m.)


Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.


Changes
-------

Comments.


Bugs: MESOS-534
    https://issues.apache.org/jira/browse/MESOS-534


Repository: mesos-git


Description
-------

- Removed os::read(int fd) because it's not used elsewhere.
- Separated os::read()s into os/read.hpp.


Diffs (updated)
-----

  3rdparty/libprocess/3rdparty/stout/Makefile.am 
770378d11d30e1e222e418c1839affd9d4ffe8d3 
  3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp 
448739e313bf5127e7e5469dd4090445dc5c5405 
  3rdparty/libprocess/3rdparty/stout/include/stout/os/read.hpp PRE-CREATION 
  3rdparty/libprocess/3rdparty/stout/tests/os_tests.cpp 
f926056239b9cd86041e497c7c3b8eadc45f44c0 

Diff: https://reviews.apache.org/r/13387/diff/


Testing
-------

make check


Thanks,

Jiang Yan Xu