Repository: mesos
Updated Branches:
  refs/heads/master 67d8518e5 -> d4153e8ca


Simplified usage of `os::pagesize` in mesos.

Review: https://reviews.apache.org/r/51968/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/19d5b5db
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/19d5b5db
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/19d5b5db

Branch: refs/heads/master
Commit: 19d5b5db99f914f3828a59d156946414c74ca433
Parents: f768df3
Author: Neil Conway <neil.con...@gmail.com>
Authored: Sat Sep 17 00:10:24 2016 +0200
Committer: Michael Park <mp...@apache.org>
Committed: Sat Sep 17 03:17:17 2016 +0200

----------------------------------------------------------------------
 src/slave/container_loggers/lib_logrotate.hpp | 2 +-
 src/slave/container_loggers/logrotate.hpp     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/19d5b5db/src/slave/container_loggers/lib_logrotate.hpp
----------------------------------------------------------------------
diff --git a/src/slave/container_loggers/lib_logrotate.hpp 
b/src/slave/container_loggers/lib_logrotate.hpp
index 9f8ec2f..a8653d7 100644
--- a/src/slave/container_loggers/lib_logrotate.hpp
+++ b/src/slave/container_loggers/lib_logrotate.hpp
@@ -86,7 +86,7 @@ struct LoggerFlags : public virtual flags::FlagsBase
 
   static Option<Error> validateSize(const Bytes& value)
   {
-    if (value.bytes() < static_cast<uint64_t>(os::pagesize())) {
+    if (value.bytes() < os::pagesize()) {
       return Error(
           "Expected --max_stdout_size and --max_stderr_size of "
           "at least " + stringify(os::pagesize()) + " bytes");

http://git-wip-us.apache.org/repos/asf/mesos/blob/19d5b5db/src/slave/container_loggers/logrotate.hpp
----------------------------------------------------------------------
diff --git a/src/slave/container_loggers/logrotate.hpp 
b/src/slave/container_loggers/logrotate.hpp
index 96697d4..f906a16 100644
--- a/src/slave/container_loggers/logrotate.hpp
+++ b/src/slave/container_loggers/logrotate.hpp
@@ -57,7 +57,7 @@ struct Flags : public virtual flags::FlagsBase
         "Defaults to 10 MB.  Must be at least 1 (memory) page.",
         Megabytes(10),
         [](const Bytes& value) -> Option<Error> {
-          if (value.bytes() < static_cast<uint64_t>(os::pagesize())) {
+          if (value.bytes() < os::pagesize()) {
             return Error(
                 "Expected --max_size of at least " +
                 stringify(os::pagesize()) + " bytes");

Reply via email to