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.
pull/5020/head
Brian Silverman 7 years ago
parent 9bbc4b1fab
commit 2f656087f0
  1. 2
      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);
}

Loading…
Cancel
Save