diff options
author | David Garcia Quintas <dgq@google.com> | 2018-03-21 18:25:22 -0700 |
---|---|---|
committer | David Garcia Quintas <dgq@google.com> | 2018-03-21 18:25:22 -0700 |
commit | caa2605a283c784ebc6f83778c04a9d841c954f9 (patch) | |
tree | 744f0ba309e574bf256ad40af291d787e24a09e1 /test/core/gprpp | |
parent | 5d70023fa3c3282eb1f2ef25c3bc6339b4a6af76 (diff) |
Don't capture unnecessary or unused variables
Diffstat (limited to 'test/core/gprpp')
-rw-r--r-- | test/core/gprpp/inlined_vector_test.cc | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/test/core/gprpp/inlined_vector_test.cc b/test/core/gprpp/inlined_vector_test.cc index b900afaf3d..2a357420f3 100644 --- a/test/core/gprpp/inlined_vector_test.cc +++ b/test/core/gprpp/inlined_vector_test.cc @@ -87,14 +87,17 @@ TEST(InlinedVectorTest, ClearAndRepopulate) { } TEST(InlinedVectorTest, ConstIndexOperator) { - const int kNumElements = 10; + constexpr int kNumElements = 10; InlinedVector<int, 5> v; EXPECT_EQ(0UL, v.size()); for (int i = 0; i < kNumElements; ++i) { v.push_back(i); EXPECT_EQ(i + 1UL, v.size()); } - auto const_func = [kNumElements](const InlinedVector<int, 5>& v) { + // The following lambda function is exceptionally allowed to use an anonymous + // capture due to the erroneous behavior of the MSVC compiler, that refuses to + // capture the kNumElements constexpr, something allowed by the standard. + auto const_func = [&](const InlinedVector<int, 5>& v) { for (int i = 0; i < kNumElements; ++i) { EXPECT_EQ(i, v[i]); } |