mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Neil Conway <>
Subject Review Request 51967: Changed return type of `os::pagesize` in stout.
Date Fri, 16 Sep 2016 19:02:14 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos and Michael Park.

Repository: mesos


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`.


  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



`make check` on OSX and Linux.


Neil Conway

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message