summaryrefslogtreecommitdiff
path: root/libs/utils/BasicHashtable.cpp
diff options
context:
space:
mode:
author Raph Levien <raph@google.com> 2012-10-25 23:11:13 -0700
committer Raph Levien <raph@google.com> 2012-10-26 16:09:22 -0700
commit8185e47822465a5c7a9cc6e56a11f16996855d79 (patch)
treea3faed051d830bd856e0c59fef93b8187a3f2bf9 /libs/utils/BasicHashtable.cpp
parentba0b9cca697a84947c08983338ce4e7f30920fd8 (diff)
Add an LRU cache plus hashing primitives
This patch adds a hashtable-based LRU cache. This should be significantly higher performance than the GenerationCache it is intended to replace. It is a large part of the fix for bug 7271109 TextLayoutCache low-level performance issues. We added a new method to BasicHashtable to detect when rehashing is needed, because the internal linked list pointers would get invalidated by that rehashing. Also, the hash_type specialized to pointers had a small flaw. Change-Id: I950c2083f96519777b851dbe157100e0a334caec
Diffstat (limited to 'libs/utils/BasicHashtable.cpp')
-rw-r--r--libs/utils/BasicHashtable.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/utils/BasicHashtable.cpp b/libs/utils/BasicHashtable.cpp
index fb8ec9f83f..fd51b7b2e0 100644
--- a/libs/utils/BasicHashtable.cpp
+++ b/libs/utils/BasicHashtable.cpp
@@ -80,7 +80,7 @@ void BasicHashtableImpl::clear() {
SharedBuffer* sb = SharedBuffer::bufferFromData(mBuckets);
if (sb->onlyOwner()) {
destroyBuckets(mBuckets, mBucketCount);
- for (size_t i = 0; i < mSize; i++) {
+ for (size_t i = 0; i < mBucketCount; i++) {
Bucket& bucket = bucketAt(mBuckets, i);
bucket.cookie = 0;
}