diff options
author | Brian Silverman <brian@peloton-tech.com> | 2018-08-08 15:27:00 -0700 |
---|---|---|
committer | Brian Silverman <brian@peloton-tech.com> | 2018-08-08 15:27:00 -0700 |
commit | 2f656087f0e6432a4f2f243283585c34b0d6c801 (patch) | |
tree | 3e3dabcaad7f4ef3d83ac720422b80992975afe2 /src | |
parent | 9bbc4b1fab8262958c172f2d8e87b486c950a051 (diff) | |
download | protobuf-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.cc | 2 |
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); } |