From 5c77c9248cd429dfd07db290fc1caeadd1b76dc5 Mon Sep 17 00:00:00 2001
From: Eli Fidler <efidler@topologyinc.com>
Date: Tue, 31 May 2016 11:37:39 -0400
Subject: [PATCH] with recent clang, this triggers -Wunevaluated-expression

specifically, "expression with side effects has no effect in an unevaluated context"
---
 test/unittest/valuetest.cpp | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/test/unittest/valuetest.cpp b/test/unittest/valuetest.cpp
index feec049..619a6a5 100644
--- a/test/unittest/valuetest.cpp
+++ b/test/unittest/valuetest.cpp
@@ -1119,14 +1119,18 @@ TEST(Value, ArrayHelperRangeFor) {
 
     {
         int i = 0;
-        for (auto& v : x.GetArray())
-            EXPECT_EQ(i++, v.GetInt());
+        for (auto& v : x.GetArray()) {
+            EXPECT_EQ(i, v.GetInt());
+            i++;
+        }
         EXPECT_EQ(i, 10);
     }
     {
         int i = 0;
-        for (const auto& v : const_cast<const Value&>(x).GetArray())
-            EXPECT_EQ(i++, v.GetInt());
+        for (const auto& v : const_cast<const Value&>(x).GetArray()) {
+            EXPECT_EQ(i, v.GetInt());
+            i++;
+        }
         EXPECT_EQ(i, 10);
     }