diff options
author | Jon Skeet <jonskeet@google.com> | 2015-07-16 09:26:10 +0100 |
---|---|---|
committer | Jon Skeet <jonskeet@google.com> | 2015-07-16 09:26:10 +0100 |
commit | 78b452b7acf985747d3d2813dedea3d267b08bae (patch) | |
tree | f1f80ed866d477d9b22d13898ec78dc8a21d5269 /csharp/src/ProtocolBuffers.Test | |
parent | 3bf74a91bb936c71bfbe61db5c87d10eaac1bc41 (diff) | |
download | protobuf-78b452b7acf985747d3d2813dedea3d267b08bae.tar.gz protobuf-78b452b7acf985747d3d2813dedea3d267b08bae.tar.bz2 protobuf-78b452b7acf985747d3d2813dedea3d267b08bae.zip |
Remove the struct-based iterator for RepeatedField.
We don't use it in the runtime or generated code anywhere now, so the extra small performance boost isn't as critical, and it has some undesirable consequences.
The tests have needed to change as iterator block enumerators don't throw when we might expect them to.
Diffstat (limited to 'csharp/src/ProtocolBuffers.Test')
-rw-r--r-- | csharp/src/ProtocolBuffers.Test/Collections/RepeatedFieldTest.cs | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/csharp/src/ProtocolBuffers.Test/Collections/RepeatedFieldTest.cs b/csharp/src/ProtocolBuffers.Test/Collections/RepeatedFieldTest.cs index 6eff8683..25be7731 100644 --- a/csharp/src/ProtocolBuffers.Test/Collections/RepeatedFieldTest.cs +++ b/csharp/src/ProtocolBuffers.Test/Collections/RepeatedFieldTest.cs @@ -241,18 +241,12 @@ namespace Google.Protobuf.Collections var list = new RepeatedField<string> { "first", "second" }; using (var enumerator = list.GetEnumerator()) { - Assert.Throws<InvalidOperationException>(() => enumerator.Current.GetHashCode()); Assert.IsTrue(enumerator.MoveNext()); Assert.AreEqual("first", enumerator.Current); Assert.IsTrue(enumerator.MoveNext()); Assert.AreEqual("second", enumerator.Current); Assert.IsFalse(enumerator.MoveNext()); - Assert.Throws<InvalidOperationException>(() => enumerator.Current.GetHashCode()); Assert.IsFalse(enumerator.MoveNext()); - enumerator.Reset(); - Assert.Throws<InvalidOperationException>(() => enumerator.Current.GetHashCode()); - Assert.IsTrue(enumerator.MoveNext()); - Assert.AreEqual("first", enumerator.Current); } } |