Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Unified Diff: third_party/protobuf/csharp/src/Google.Protobuf.Test/ByteStringTest.cs

Issue 2600753002: Reverts third_party/protobuf: Update to HEAD (f52e188fe4) (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/protobuf/csharp/src/Google.Protobuf.Test/ByteStringTest.cs
diff --git a/third_party/protobuf/csharp/src/Google.Protobuf.Test/ByteStringTest.cs b/third_party/protobuf/csharp/src/Google.Protobuf.Test/ByteStringTest.cs
index 3ba8df6471f2f56925693dac0881b64f48c78b7e..8935b7829d4a95a703362b5eb317223ef66ae0b8 100644
--- a/third_party/protobuf/csharp/src/Google.Protobuf.Test/ByteStringTest.cs
+++ b/third_party/protobuf/csharp/src/Google.Protobuf.Test/ByteStringTest.cs
@@ -167,18 +167,5 @@ namespace Google.Protobuf
// Optimization which also fixes issue 61.
Assert.AreSame(ByteString.Empty, ByteString.FromBase64(""));
}
-
- [Test]
- public void GetHashCode_Regression()
- {
- // We used to have an awful hash algorithm where only the last four
- // bytes were relevant. This is a regression test for
- // https://github.com/google/protobuf/issues/2511
-
- ByteString b1 = ByteString.CopyFrom(100, 1, 2, 3, 4);
- ByteString b2 = ByteString.CopyFrom(200, 1, 2, 3, 4);
- Assert.AreNotEqual(b1.GetHashCode(), b2.GetHashCode());
- }
-
}
}

Powered by Google App Engine
This is Rietveld 408576698