You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by mp...@apache.org on 2015/07/28 23:51:57 UTC
mesos git commit: Fixed clang warning [-Wunevaluated-expression] in
process tests.
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 <jo...@gmail.com>
Authored: Tue Jul 28 14:49:35 2015 -0700
Committer: Michael Park <mc...@gmail.com>
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;
}
}