Mahtan/wagic
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Jeck - Cleaned up logic a little in Cache::Get()

Browse Source
This commit is contained in:
wagic.jeck
2009-09-19 04:11:07 +00:00
parent 3ea5ab9b88
commit a5ded3e836
1 changed files with 36 additions and 45 deletions
Download Patch File Download Diff File Expand all files Collapse all files
projects/mtg/src/WResourceManager.cpp
+21 -30
View File
@@ -811,6 +811,9 @@ void WResourceManager::CacheForState(int state){
textureWCache.Resize(HUGE_CACHE_LIMIT,HUGE_CACHE_ITEMS); textureWCache.Resize(HUGE_CACHE_LIMIT,HUGE_CACHE_ITEMS);
return; return;
#else #else
#if defined DEBUG_CACHE
textureWCache.Clear();
#endif
switch(state){ switch(state){
//Default is not to change cache sizes. //Default is not to change cache sizes.
case GAME_STATE_MENU: case GAME_STATE_MENU:
@@ -1203,38 +1206,32 @@ cacheItem * WCache<cacheItem, cacheActual>::Get(string id, int style, int submod
return NULL; return NULL;
} }
//Not managed, so look in cache.
if(it == managed.end() && style != RETRIEVE_MANAGE && style != RETRIEVE_RESOURCE ){
it = cache.end();
for(it = cache.begin();it!=cache.end();it++){
if(it->first == lookup)
break;
}
//Well, we've found something...
if(it != cache.end()) {
if(!it->second && (submode & CACHE_EXISTING))
return NULL; //A miss.
else
return it->second; //A hit.
}
}
cacheItem * item = NULL; cacheItem * item = NULL;
if(style != RETRIEVE_MANAGE) if(style != RETRIEVE_MANAGE)
item = cache[lookup]; //We don't know about this one yet. item = cache[lookup]; //We don't know about this one yet.
//We only want already existing things.
if(submode & CACHE_EXISTING || style == RETRIEVE_EXISTING){
//Didn't exist in cache.
if(!item){
mError = CACHE_ERROR_NOT_CACHED;
RemoveMiss(lookup);
}
return item;
}
//Found something. //Found something.
if(item){ if(item){
//Item went bad? //Item went bad?
if(!item->isGood()){ if(!item->isGood()){
//If we're allowed, attempt to revive it. //Attempt to revive it.
if(!(submode & CACHE_EXISTING))
item->Attempt(id,submode,mError); item->Attempt(id,submode,mError);
//Still bad, so remove it and return NULL //Still bad, so remove it and return NULL
if(submode & CACHE_EXISTING || !item->isGood()){ if(!item->isGood()){
if(!item->isLocked()){ if(!item->isLocked()){
RemoveItem(item); //Delete it. RemoveItem(item); //Delete it.
mError = CACHE_ERROR_BAD; mError = CACHE_ERROR_BAD;
@@ -1253,13 +1250,6 @@ cacheItem * WCache<cacheItem, cacheActual>::Get(string id, int style, int submod
return item; return item;
} }
//Didn't exist in cache.
if(submode & CACHE_EXISTING ){
RemoveMiss(lookup);
mError = CACHE_ERROR_NOT_CACHED;
return NULL;
}
else{
//Space in cache, make new texture //Space in cache, make new texture
item = AttemptNew(id,submode); item = AttemptNew(id,submode);
@@ -1272,12 +1262,13 @@ cacheItem * WCache<cacheItem, cacheActual>::Get(string id, int style, int submod
} }
else else
SAFE_DELETE(item); SAFE_DELETE(item);
}
} }
if(style == RETRIEVE_MANAGE){ if(style == RETRIEVE_MANAGE){
if(item){ if(item){
managed[lookup] = item; //Record a hit. UnlinkCache(item); //Remove from cached section.
managed[lookup] = item; //Promote to managed.
item->deadbolt(); //Make permanent. item->deadbolt(); //Make permanent.
} }
else if(mError == CACHE_ERROR_404) else if(mError == CACHE_ERROR_404)
@@ -1285,9 +1276,9 @@ cacheItem * WCache<cacheItem, cacheActual>::Get(string id, int style, int submod
} }
else{ else{
if(!item && mError != CACHE_ERROR_404) if(!item && mError != CACHE_ERROR_404)
RemoveMiss(lookup); RemoveMiss(lookup); //If file not found, record miss.
else else
cache[lookup] = item; cache[lookup] = item; //Record item.
} }
//Succeeded in making a new item. //Succeeded in making a new item.