From 2f656087f0e6432a4f2f243283585c34b0d6c801 Mon Sep 17 00:00:00 2001 From: Brian Silverman Date: Wed, 8 Aug 2018 15:27:00 -0700 Subject: Fix broken test of const repeated field iterators The old version had a NOP for loop, which doesn't make sense. I'm pretty sure this was the original intention. Newer GCC's -Wunused-but-set-variable flags the old version. --- src/google/protobuf/extension_set_unittest.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/google/protobuf/extension_set_unittest.cc b/src/google/protobuf/extension_set_unittest.cc index bc65d295..f121c79c 100644 --- a/src/google/protobuf/extension_set_unittest.cc +++ b/src/google/protobuf/extension_set_unittest.cc @@ -1055,7 +1055,7 @@ TEST(ExtensionSetTest, RepeatedFields) { unittest::repeated_nested_enum_extension).begin(), enum_const_end = message.GetRepeatedExtension( unittest::repeated_nested_enum_extension).end(); - enum_iter != enum_end; ++enum_iter) { + enum_const_iter != enum_const_end; ++enum_const_iter) { ASSERT_EQ(*enum_const_iter, unittest::TestAllTypes::NestedEnum_MAX); } -- cgit v1.2.3