aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorBrian Silverman <brian@peloton-tech.com>2018-08-08 15:27:00 -0700
committerBrian Silverman <brian@peloton-tech.com>2018-08-08 15:27:00 -0700
commit2f656087f0e6432a4f2f243283585c34b0d6c801 (patch)
tree3e3dabcaad7f4ef3d83ac720422b80992975afe2 /src
parent9bbc4b1fab8262958c172f2d8e87b486c950a051 (diff)
downloadprotobuf-2f656087f0e6432a4f2f243283585c34b0d6c801.tar.gz
protobuf-2f656087f0e6432a4f2f243283585c34b0d6c801.tar.bz2
protobuf-2f656087f0e6432a4f2f243283585c34b0d6c801.zip
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.
Diffstat (limited to 'src')
-rw-r--r--src/google/protobuf/extension_set_unittest.cc2
1 files changed, 1 insertions, 1 deletions
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);
}