diff options
author | mtklein <mtklein@chromium.org> | 2014-10-21 10:55:22 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-10-21 10:55:22 -0700 |
commit | 484bbe5be63d6c516e8b190634117989de0f9a26 (patch) | |
tree | 258b0b7466f447a664085df1a06a59a30a4976f6 /src/core/SkResourceCache.h | |
parent | c7663981c9864773de455ece42335dd13fc74b88 (diff) |
small tweaks to SkResourceCache::Key
- remove unused method and obsolete note
- add a reminder that yes, we are checking counts are equal
BUG=skia:
Review URL: https://codereview.chromium.org/667993002
Diffstat (limited to 'src/core/SkResourceCache.h')
-rw-r--r-- | src/core/SkResourceCache.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/core/SkResourceCache.h b/src/core/SkResourceCache.h index 665c0be498..c50b370eaf 100644 --- a/src/core/SkResourceCache.h +++ b/src/core/SkResourceCache.h @@ -40,7 +40,7 @@ public: bool operator==(const Key& other) const { const uint32_t* a = this->as32(); const uint32_t* b = other.as32(); - for (int i = 0; i < fCount32; ++i) { + for (int i = 0; i < fCount32; ++i) { // (This checks fCount == other.fCount first.) if (a[i] != b[i]) { return false; } @@ -49,13 +49,11 @@ public: } private: - // store fCount32 first, so we don't consider it in operator< int32_t fCount32; // 2 + user contents count32 uint32_t fHash; /* uint32_t fContents32[] */ const uint32_t* as32() const { return (const uint32_t*)this; } - const uint32_t* as32SkipCount() const { return this->as32() + 1; } }; struct Rec { |