diff options
author | mtklein <mtklein@google.com> | 2015-11-16 11:01:18 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-11-16 11:01:18 -0800 |
commit | 3c2d32b8e27820a6e149d9ded67cbdf2411cc5c9 (patch) | |
tree | be93bf1035eb199e1c9df4ffbe8b8961eb9ecba0 /tests/ChecksumTest.cpp | |
parent | 05b48e2a24ae78312c80ed94195f0b705fdb0547 (diff) |
Revert of Switch uses of SkChecksum::Compute to Murmur3. (patchset #2 id:20001 of https://codereview.chromium.org/1436973003/ )
Reason for revert:
gotta put back *compute = 0.
Original issue's description:
> 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
>
> Committed: https://skia.googlesource.com/skia/+/540e95483d285b555e9b1a73d18c16e7d7c0deba
TBR=bsalomon@google.com,mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1448023005
Diffstat (limited to 'tests/ChecksumTest.cpp')
-rw-r--r-- | tests/ChecksumTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/ChecksumTest.cpp b/tests/ChecksumTest.cpp index cf9d65c59e..c095a5ac11 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[] = { &murmur_noseed }; + const algorithmProc kAlgorithms[] = { &SkChecksum::Compute, &murmur_noseed }; // Put 128 random bytes into two identical buffers. Any multiple of 4 will do. const size_t kBytes = SkAlign4(128); |