Repository: mesos Updated Branches: refs/heads/master 65efb3473 -> 2ef6ef1ae
Fixed clang warning [-Wunevaluated-expression] in process tests. Removes `warning: expression with side effects has no effect in an unevaluated context`. Review: https://reviews.apache.org/r/36820 Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/2ef6ef1a Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/2ef6ef1a Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/2ef6ef1a Branch: refs/heads/master Commit: 2ef6ef1ae7077e1fb47d7e82bb5a1408839d172f Parents: 65efb34 Author: Joris Van Remoortere <[email protected]> Authored: Tue Jul 28 14:49:35 2015 -0700 Committer: Michael Park <[email protected]> Committed: Tue Jul 28 14:49:35 2015 -0700 ---------------------------------------------------------------------- 3rdparty/libprocess/src/tests/process_tests.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/2ef6ef1a/3rdparty/libprocess/src/tests/process_tests.cpp ---------------------------------------------------------------------- diff --git a/3rdparty/libprocess/src/tests/process_tests.cpp b/3rdparty/libprocess/src/tests/process_tests.cpp index bc54d29..95e3257 100644 --- a/3rdparty/libprocess/src/tests/process_tests.cpp +++ b/3rdparty/libprocess/src/tests/process_tests.cpp @@ -1246,7 +1246,8 @@ TEST(ProcessTest, Await1) int i = 1; foreach (const Future<int>& result, future.get()) { ASSERT_TRUE(result.isReady()); - ASSERT_EQ(i++, result.get()); + ASSERT_EQ(i, result.get()); + ++i; } }
