Finish removing GrTHashTable
This class is no longer used in Ganesh (in favor of SkTDynamicHash) R=jvanverth@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/421253003
This commit is contained in:
parent
4509517e03
commit
8ab8cc07cf
@ -138,7 +138,6 @@
|
||||
'<(skia_src_path)/gpu/GrTextStrike_impl.h',
|
||||
'<(skia_src_path)/gpu/GrTexture.cpp',
|
||||
'<(skia_src_path)/gpu/GrTextureAccess.cpp',
|
||||
'<(skia_src_path)/gpu/GrTHashTable.h',
|
||||
'<(skia_src_path)/gpu/GrVertexBuffer.h',
|
||||
|
||||
'<(skia_src_path)/gpu/effects/Gr1DKernelEffect.h',
|
||||
|
@ -107,7 +107,6 @@
|
||||
'../tests/GrSurfaceTest.cpp',
|
||||
'../tests/GrTBSearchTest.cpp',
|
||||
'../tests/GradientTest.cpp',
|
||||
'../tests/HashCacheTest.cpp',
|
||||
'../tests/ImageCacheTest.cpp',
|
||||
'../tests/ImageDecodingTest.cpp',
|
||||
'../tests/ImageFilterTest.cpp',
|
||||
|
@ -1,215 +0,0 @@
|
||||
|
||||
/*
|
||||
* Copyright 2010 Google Inc.
|
||||
*
|
||||
* Use of this source code is governed by a BSD-style license that can be
|
||||
* found in the LICENSE file.
|
||||
*/
|
||||
|
||||
|
||||
|
||||
#ifndef GrTHashTable_DEFINED
|
||||
#define GrTHashTable_DEFINED
|
||||
|
||||
#include "GrTypes.h"
|
||||
#include "SkTDArray.h"
|
||||
|
||||
/**
|
||||
* Key needs
|
||||
* static bool Equals(const Entry&, const Key&);
|
||||
* static bool LessThan(const Entry&, const Key&);
|
||||
* static bool Equals(const Entry&, const Entry&); for SK_DEBUG if GrTHashTable::validate() is called
|
||||
* static bool LessThan(const Entry&, const Entry&); for SK_DEBUG if GrTHashTable::validate() is called
|
||||
* uint32_t getHash() const;
|
||||
*
|
||||
* Allows duplicate key entries but on find you may get
|
||||
* any of the duplicate entries returned.
|
||||
*/
|
||||
template <typename T, typename Key, size_t kHashBits> class GrTHashTable {
|
||||
public:
|
||||
GrTHashTable() { this->clearHash(); }
|
||||
~GrTHashTable() {}
|
||||
|
||||
int count() const { return fSorted.count(); }
|
||||
|
||||
struct Any {
|
||||
// Return the first resource that matches the key.
|
||||
bool operator()(const T*) const { return true; }
|
||||
};
|
||||
|
||||
T* find(const Key& key) const { return this->find(key, Any()); }
|
||||
template <typename Filter> T* find(const Key&, Filter filter) const;
|
||||
|
||||
// return true if key was unique when inserted.
|
||||
bool insert(const Key&, T*);
|
||||
void remove(const Key&, const T*);
|
||||
|
||||
void deleteAll();
|
||||
|
||||
#ifdef SK_DEBUG
|
||||
void validate() const;
|
||||
bool contains(T*) const;
|
||||
#endif
|
||||
|
||||
// testing
|
||||
const SkTDArray<T*>& getArray() const { return fSorted; }
|
||||
SkTDArray<T*>& getArray() { return fSorted; }
|
||||
private:
|
||||
void clearHash() { sk_bzero(fHash, sizeof(fHash)); }
|
||||
|
||||
enum {
|
||||
kHashCount = 1 << kHashBits,
|
||||
kHashMask = kHashCount - 1
|
||||
};
|
||||
static unsigned hash2Index(intptr_t hash) {
|
||||
#if 0
|
||||
if (sizeof(hash) == 8) {
|
||||
hash ^= hash >> 32;
|
||||
}
|
||||
#endif
|
||||
hash ^= hash >> 16;
|
||||
if (kHashBits <= 8) {
|
||||
hash ^= hash >> 8;
|
||||
}
|
||||
return hash & kHashMask;
|
||||
}
|
||||
|
||||
mutable T* fHash[kHashCount];
|
||||
SkTDArray<T*> fSorted;
|
||||
|
||||
// search fSorted, and return the found index, or ~index of where it
|
||||
// should be inserted
|
||||
int searchArray(const Key&) const;
|
||||
};
|
||||
|
||||
///////////////////////////////////////////////////////////////////////////////
|
||||
|
||||
template <typename T, typename Key, size_t kHashBits>
|
||||
int GrTHashTable<T, Key, kHashBits>::searchArray(const Key& key) const {
|
||||
int count = fSorted.count();
|
||||
if (0 == count) {
|
||||
// we should insert it at 0
|
||||
return ~0;
|
||||
}
|
||||
|
||||
const T* const* array = fSorted.begin();
|
||||
int high = count - 1;
|
||||
int low = 0;
|
||||
while (high > low) {
|
||||
int index = (low + high) >> 1;
|
||||
if (Key::LessThan(*array[index], key)) {
|
||||
low = index + 1;
|
||||
} else {
|
||||
high = index;
|
||||
}
|
||||
}
|
||||
|
||||
// check if we found it
|
||||
if (Key::Equals(*array[high], key)) {
|
||||
// above search should have found the first occurrence if there
|
||||
// are multiple.
|
||||
SkASSERT(0 == high || Key::LessThan(*array[high - 1], key));
|
||||
return high;
|
||||
}
|
||||
|
||||
// now return the ~ of where we should insert it
|
||||
if (Key::LessThan(*array[high], key)) {
|
||||
high += 1;
|
||||
}
|
||||
return ~high;
|
||||
}
|
||||
|
||||
template <typename T, typename Key, size_t kHashBits>
|
||||
template <typename Filter>
|
||||
T* GrTHashTable<T, Key, kHashBits>::find(const Key& key, Filter filter) const {
|
||||
|
||||
int hashIndex = hash2Index(key.getHash());
|
||||
T* elem = fHash[hashIndex];
|
||||
|
||||
if (NULL != elem && Key::Equals(*elem, key) && filter(elem)) {
|
||||
return elem;
|
||||
}
|
||||
|
||||
// bsearch for the key in our sorted array
|
||||
int index = this->searchArray(key);
|
||||
if (index < 0) {
|
||||
return NULL;
|
||||
}
|
||||
|
||||
const T* const* array = fSorted.begin();
|
||||
|
||||
// above search should have found the first occurrence if there
|
||||
// are multiple.
|
||||
SkASSERT(0 == index || Key::LessThan(*array[index - 1], key));
|
||||
|
||||
for ( ; index < count() && Key::Equals(*array[index], key); ++index) {
|
||||
if (filter(fSorted[index])) {
|
||||
// update the hash
|
||||
fHash[hashIndex] = fSorted[index];
|
||||
return fSorted[index];
|
||||
}
|
||||
}
|
||||
|
||||
return NULL;
|
||||
}
|
||||
|
||||
template <typename T, typename Key, size_t kHashBits>
|
||||
bool GrTHashTable<T, Key, kHashBits>::insert(const Key& key, T* elem) {
|
||||
int index = this->searchArray(key);
|
||||
bool first = index < 0;
|
||||
if (first) {
|
||||
// turn it into the actual index
|
||||
index = ~index;
|
||||
}
|
||||
// add it to our array
|
||||
*fSorted.insert(index) = elem;
|
||||
// update our hash table (overwrites any dupe's position in the hash)
|
||||
fHash[hash2Index(key.getHash())] = elem;
|
||||
return first;
|
||||
}
|
||||
|
||||
template <typename T, typename Key, size_t kHashBits>
|
||||
void GrTHashTable<T, Key, kHashBits>::remove(const Key& key, const T* elem) {
|
||||
int index = hash2Index(key.getHash());
|
||||
if (fHash[index] == elem) {
|
||||
fHash[index] = NULL;
|
||||
}
|
||||
|
||||
// remove from our sorted array
|
||||
index = this->searchArray(key);
|
||||
SkASSERT(index >= 0);
|
||||
// if there are multiple matches searchArray will give us the first match
|
||||
// march forward until we find elem.
|
||||
while (elem != fSorted[index]) {
|
||||
++index;
|
||||
SkASSERT(index < fSorted.count());
|
||||
}
|
||||
SkASSERT(elem == fSorted[index]);
|
||||
fSorted.remove(index);
|
||||
}
|
||||
|
||||
template <typename T, typename Key, size_t kHashBits>
|
||||
void GrTHashTable<T, Key, kHashBits>::deleteAll() {
|
||||
fSorted.deleteAll();
|
||||
this->clearHash();
|
||||
}
|
||||
|
||||
#ifdef SK_DEBUG
|
||||
template <typename T, typename Key, size_t kHashBits>
|
||||
void GrTHashTable<T, Key, kHashBits>::validate() const {
|
||||
int count = fSorted.count();
|
||||
for (int i = 1; i < count; i++) {
|
||||
SkASSERT(Key::LessThan(*fSorted[i - 1], *fSorted[i]) ||
|
||||
Key::Equals(*fSorted[i - 1], *fSorted[i]));
|
||||
}
|
||||
}
|
||||
|
||||
template <typename T, typename Key, size_t kHashBits>
|
||||
bool GrTHashTable<T, Key, kHashBits>::contains(T* elem) const {
|
||||
int index = fSorted.find(elem);
|
||||
return index >= 0;
|
||||
}
|
||||
|
||||
#endif
|
||||
|
||||
#endif
|
@ -20,7 +20,6 @@
|
||||
#include "SkString.h"
|
||||
#include "SkXfermode.h"
|
||||
|
||||
class GrBinHashKeyBuilder;
|
||||
class GrGLEffect;
|
||||
class GrGLProgramEffects;
|
||||
class GrGLShaderBuilder;
|
||||
|
@ -8,7 +8,6 @@
|
||||
#ifndef GrGpuGL_DEFINED
|
||||
#define GrGpuGL_DEFINED
|
||||
|
||||
#include "GrBinHashKey.h"
|
||||
#include "GrDrawState.h"
|
||||
#include "GrGLContext.h"
|
||||
#include "GrGLIRect.h"
|
||||
|
@ -1,154 +0,0 @@
|
||||
/*
|
||||
* Copyright 2012 Google Inc.
|
||||
*
|
||||
* Use of this source code is governed by a BSD-style license that can be
|
||||
* found in the LICENSE file.
|
||||
*/
|
||||
|
||||
#include "Test.h"
|
||||
|
||||
// This is a GR test
|
||||
#if SK_SUPPORT_GPU
|
||||
#include "GrTHashTable.h"
|
||||
|
||||
struct HashElement {
|
||||
int fKey;
|
||||
int fValue;
|
||||
};
|
||||
|
||||
class GrFindPositivesFunctor {
|
||||
public:
|
||||
// only return elements with positive values
|
||||
bool operator()(const HashElement* elem) const {
|
||||
return elem->fValue > 0;
|
||||
}
|
||||
};
|
||||
|
||||
class GrFindNegativesFunctor {
|
||||
public:
|
||||
// only return elements with negative values
|
||||
bool operator()(const HashElement* elem) const {
|
||||
return elem->fValue < 0;
|
||||
}
|
||||
};
|
||||
|
||||
class HashKey {
|
||||
public:
|
||||
HashKey(int key) : fKey(key) {}
|
||||
|
||||
uint32_t getHash() const { return fKey; }
|
||||
|
||||
static bool LessThan(const HashElement& entry, const HashKey& key) {
|
||||
return entry.fKey < key.fKey;
|
||||
}
|
||||
static bool Equals(const HashElement& entry, const HashKey& key) {
|
||||
return entry.fKey == key.fKey;
|
||||
}
|
||||
|
||||
#ifdef SK_DEBUG
|
||||
static bool LessThan(const HashElement& a, const HashElement& b) {
|
||||
return a.fKey < b.fKey;
|
||||
}
|
||||
static bool Equals(const HashElement& a, const HashElement& b) {
|
||||
return a.fKey == b.fKey;
|
||||
}
|
||||
#endif
|
||||
|
||||
protected:
|
||||
int fKey;
|
||||
};
|
||||
|
||||
DEF_TEST(HashCache, reporter) {
|
||||
GrTHashTable<HashElement, HashKey, 4> cache;
|
||||
|
||||
HashElement negHashElements[10] = {
|
||||
{ 0, 0 },
|
||||
{ 1, -1 },
|
||||
{ 2, -2 },
|
||||
{ 3, -3 },
|
||||
{ 4, -4 },
|
||||
{ 5, -5 },
|
||||
{ 6, -6 },
|
||||
{ 7, -7 },
|
||||
{ 8, -8 },
|
||||
{ 9, -9 }
|
||||
};
|
||||
HashElement posHashElements[10] = {
|
||||
{ 0, 0 },
|
||||
{ 1, 1 },
|
||||
{ 2, 2 },
|
||||
{ 3, 3 },
|
||||
{ 4, 4 },
|
||||
{ 5, 5 },
|
||||
{ 6, 6 },
|
||||
{ 7, 7 },
|
||||
{ 8, 8 },
|
||||
{ 9, 9 }
|
||||
};
|
||||
|
||||
// add i: -i pairs
|
||||
for (int i = 0; i < 10; ++i) {
|
||||
cache.insert(HashKey(i), &negHashElements[i]);
|
||||
}
|
||||
|
||||
REPORTER_ASSERT(reporter, 10 == cache.count());
|
||||
|
||||
// look for all i's and assert we found the -i's
|
||||
for (int i = 0; i < 10; ++i) {
|
||||
HashElement* found = cache.find(i);
|
||||
REPORTER_ASSERT(reporter, NULL != found && -i == found->fValue);
|
||||
}
|
||||
|
||||
// look for something not in the cache
|
||||
{
|
||||
HashElement* found = cache.find(10);
|
||||
REPORTER_ASSERT(reporter, NULL == found);
|
||||
}
|
||||
|
||||
// add i:i duplicates (so each i will have a positive & negative entry)
|
||||
for (int i = 0; i < 10; ++i) {
|
||||
cache.insert(i, &posHashElements[i]);
|
||||
}
|
||||
|
||||
REPORTER_ASSERT(reporter, 20 == cache.count());
|
||||
|
||||
// test out the find functor to find all the positive values
|
||||
{
|
||||
GrFindPositivesFunctor findPos;
|
||||
|
||||
HashElement* found = cache.find(0, findPos);
|
||||
REPORTER_ASSERT(reporter, NULL == found);
|
||||
|
||||
for (int i = 1; i < 10; ++i) {
|
||||
found = cache.find(i, findPos);
|
||||
|
||||
REPORTER_ASSERT(reporter, NULL != found && found->fValue > 0);
|
||||
}
|
||||
}
|
||||
|
||||
// make sure finding the positives wasn't a fluke - find the negatives
|
||||
{
|
||||
GrFindNegativesFunctor findNeg;
|
||||
|
||||
HashElement* found = cache.find(0, findNeg);
|
||||
REPORTER_ASSERT(reporter, NULL == found);
|
||||
|
||||
for (int i = 1; i < 10; ++i) {
|
||||
found = cache.find(i, findNeg);
|
||||
|
||||
REPORTER_ASSERT(reporter, NULL != found && found->fValue < 0);
|
||||
}
|
||||
}
|
||||
|
||||
// remove the 0:0 entries
|
||||
{
|
||||
cache.remove(0, &negHashElements[0]);
|
||||
cache.remove(0, &posHashElements[0]);
|
||||
REPORTER_ASSERT(reporter, 18 == cache.count());
|
||||
|
||||
HashElement* found = cache.find(0);
|
||||
REPORTER_ASSERT(reporter, NULL == found);
|
||||
}
|
||||
}
|
||||
|
||||
#endif
|
Loading…
Reference in New Issue
Block a user