From 540e95483d285b555e9b1a73d18c16e7d7c0deba Mon Sep 17 00:00:00 2001 From: mtklein Date: Mon, 16 Nov 2015 09:08:21 -0800 Subject: Switch uses of SkChecksum::Compute to Murmur3. SkChecksum::Compute is a very, very poorly distributed hash function. This replaces all remaining uses with Murmur3. The only interesting stuff is in src/gpu. BUG=skia: Committed: https://skia.googlesource.com/skia/+/1d024a3c909ae5cefa5e8b339e2b52dc73ee85ac Review URL: https://codereview.chromium.org/1436973003 --- tests/ChecksumTest.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests/ChecksumTest.cpp') diff --git a/tests/ChecksumTest.cpp b/tests/ChecksumTest.cpp index c095a5ac11..cf9d65c59e 100644 --- a/tests/ChecksumTest.cpp +++ b/tests/ChecksumTest.cpp @@ -18,7 +18,7 @@ static uint32_t murmur_noseed(const uint32_t* d, size_t l) { return SkChecksum:: DEF_TEST(Checksum, r) { // Algorithms to test. They're currently all uint32_t(const uint32_t*, size_t). typedef uint32_t(*algorithmProc)(const uint32_t*, size_t); - const algorithmProc kAlgorithms[] = { &SkChecksum::Compute, &murmur_noseed }; + const algorithmProc kAlgorithms[] = { &murmur_noseed }; // Put 128 random bytes into two identical buffers. Any multiple of 4 will do. const size_t kBytes = SkAlign4(128); -- cgit v1.2.3