----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51967/ -----------------------------------------------------------
Review request for mesos and Michael Park. Repository: mesos Description ------- Previously, `os::pagesize` returned a signed integer (`int`), where a negative value indicated an error. This is problematic: several callers neglected to check for errors and most callers ultimately want an unsigned value anyway. We could make this return `Try<size_t>`, but that seems heavyweight: the Win32 implementation of this function will never fail and the POSIX implementation (using `sysconf(3)`) seems unlikely to fail in practice. Hence, this commit changes `os::pagesize` to return `size_t`. Diffs ----- 3rdparty/stout/include/stout/os/freebsd.hpp 3f5142c81f4ea363238d6b546130e51518fd9daa 3rdparty/stout/include/stout/os/linux.hpp 33825280eb1404bcd89324f8ab5949f735b2d130 3rdparty/stout/include/stout/os/osx.hpp 54d34121d8cde434fca60679e19eae507b4048ba 3rdparty/stout/include/stout/os/posix/pagesize.hpp f3ae69adf096d558e083615dfcf848c94e017e6e 3rdparty/stout/include/stout/os/windows/pagesize.hpp 6112e9781a9d42f7ec1ae0832c0c877d1915b09b Diff: https://reviews.apache.org/r/51967/diff/ Testing ------- `make check` on OSX and Linux. Thanks, Neil Conway