summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDayBr3ak <benj.groeneveld@gmail.com>2014-09-02 10:49:46 +0200
committerDayBr3ak <benj.groeneveld@gmail.com>2014-09-02 10:54:26 +0200
commitac4ec5117dbe0ae12e73968b6a6d6cbc1640933a (patch)
tree1a973b407190b4ae072a4481ba0bdb3ef191b456
parentadding config file entry #381 (diff)
downloadcuberite-ac4ec5117dbe0ae12e73968b6a6d6cbc1640933a.tar
cuberite-ac4ec5117dbe0ae12e73968b6a6d6cbc1640933a.tar.gz
cuberite-ac4ec5117dbe0ae12e73968b6a6d6cbc1640933a.tar.bz2
cuberite-ac4ec5117dbe0ae12e73968b6a6d6cbc1640933a.tar.lz
cuberite-ac4ec5117dbe0ae12e73968b6a6d6cbc1640933a.tar.xz
cuberite-ac4ec5117dbe0ae12e73968b6a6d6cbc1640933a.tar.zst
cuberite-ac4ec5117dbe0ae12e73968b6a6d6cbc1640933a.zip
-rw-r--r--src/Generating/BioGen.cpp93
-rw-r--r--src/Generating/BioGen.h28
2 files changed, 23 insertions, 98 deletions
diff --git a/src/Generating/BioGen.cpp b/src/Generating/BioGen.cpp
index c9aeffa54..4936fc251 100644
--- a/src/Generating/BioGen.cpp
+++ b/src/Generating/BioGen.cpp
@@ -212,27 +212,14 @@ void cBioGenCache::InitializeBiomeGen(cIniFile & a_IniFile)
// cBioGenMulticache:
cBioGenMulticache::cBioGenMulticache(cBiomeGen * a_BioGenToCache, int a_CacheSize, int a_CachesLength) :
-m_BioGenToCache(a_BioGenToCache),
-m_CacheSize(a_CacheSize),
m_CachesLength(a_CachesLength),
-m_CachesOrder(new int*[a_CachesLength * a_CachesLength]),
-m_CachesData(new sCacheData*[a_CachesLength * a_CachesLength]),
-m_NumHits(0),
-m_NumMisses(0),
-m_TotalChain(0)
+m_InternalCacheLength(a_CachesLength * a_CacheSize)
{
- for (int i = 0; i < m_CachesLength * m_CachesLength; i++) {
- sCacheData * cacheData = m_CachesData[i] = new sCacheData[m_CacheSize];
- int * cacheOrder = m_CachesOrder[i] = new int[m_CacheSize];
-
- for (int j = 0; j < m_CacheSize; j++)
- {
- cacheOrder[j] = j;
- cacheData[j].m_ChunkX = 0x7fffffff;
- cacheData[j].m_ChunkZ = 0x7fffffff;
- }
+ //m_Caches = new std::vector<cBiomeGen*>;
+ m_Caches.reserve(m_InternalCacheLength);
+ for (int i = 0; i < m_InternalCacheLength; i++) {
+ m_Caches.push_back(new cBioGenCache(a_BioGenToCache, a_CacheSize));
}
-
}
@@ -241,14 +228,7 @@ m_TotalChain(0)
cBioGenMulticache::~cBioGenMulticache()
{
- for (int i = 0; i < m_CachesLength * m_CachesLength; i++) {
- delete[] m_CachesData[i];
- delete[] m_CachesOrder[i];
- }
- delete[] m_CachesData;
- m_CachesData = NULL;
- delete[] m_CachesOrder;
- m_CachesOrder = NULL;
+ m_Caches.erase(m_Caches.cbegin(), m_Caches.cend());
}
@@ -257,57 +237,10 @@ cBioGenMulticache::~cBioGenMulticache()
void cBioGenMulticache::GenBiomes(int a_ChunkX, int a_ChunkZ, cChunkDef::BiomeMap & a_BiomeMap)
{
- if (((m_NumHits + m_NumMisses) % 1024) == 10)
- {
- LOGD("BioGenCache: %d hits, %d misses, saved %.2f %%", m_NumHits, m_NumMisses, 100.0 * m_NumHits / (m_NumHits + m_NumMisses));
- LOGD("BioGenCache: Avg cache chain length: %.2f", (float)m_TotalChain / m_NumHits);
- }
-
- int cacheIdx = ((unsigned int)a_ChunkX % m_CachesLength) * m_CachesLength + ((unsigned int)a_ChunkZ % m_CachesLength);
- sCacheData * cacheData = m_CachesData[cacheIdx];
- int * cacheOrder = m_CachesOrder[cacheIdx];
+ int cacheIdx = ((unsigned int)a_ChunkX % m_CachesLength) * m_CachesLength
+ + ((unsigned int)a_ChunkZ % m_CachesLength);
- for (int i = 0; i < m_CacheSize; i++)
- {
- if (
- (cacheData[cacheOrder[i]].m_ChunkX != a_ChunkX) ||
- (cacheData[cacheOrder[i]].m_ChunkZ != a_ChunkZ)
- )
- {
- continue;
- }
- // Found it in the cache
- int Idx = cacheOrder[i];
-
- // Move to front:
- for (int j = i; j > 0; j--)
- {
- cacheOrder[j] = cacheOrder[j - 1];
- }
- cacheOrder[0] = Idx;
-
- // Use the cached data:
- memcpy(a_BiomeMap, cacheData[Idx].m_BiomeMap, sizeof(a_BiomeMap));
-
- m_NumHits++;
- m_TotalChain += i;
- return;
- } // for i - cache
-
- // Not in the cache:
- m_NumMisses++;
- m_BioGenToCache->GenBiomes(a_ChunkX, a_ChunkZ, a_BiomeMap);
-
- // Insert it as the first item in the MRU order:
- int Idx = cacheOrder[m_CacheSize - 1];
- for (int i = m_CacheSize - 1; i > 0; i--)
- {
- cacheOrder[i] = cacheOrder[i - 1];
- } // for i - m_CacheOrder[]
- cacheOrder[0] = Idx;
- memcpy(cacheData[Idx].m_BiomeMap, a_BiomeMap, sizeof(a_BiomeMap));
- cacheData[Idx].m_ChunkX = a_ChunkX;
- cacheData[Idx].m_ChunkZ = a_ChunkZ;
+ m_Caches[cacheIdx]->GenBiomes(a_ChunkX, a_ChunkZ, a_BiomeMap);
}
@@ -316,8 +249,12 @@ void cBioGenMulticache::GenBiomes(int a_ChunkX, int a_ChunkZ, cChunkDef::BiomeMa
void cBioGenMulticache::InitializeBiomeGen(cIniFile & a_IniFile)
{
- super::InitializeBiomeGen(a_IniFile);
- m_BioGenToCache->InitializeBiomeGen(a_IniFile);
+ //super::InitializeBiomeGen(a_IniFile);
+ for (auto it = m_Caches.begin(); it != m_Caches.end(); it++)
+ {
+ cBiomeGen * tmp = *it;
+ tmp->InitializeBiomeGen(a_IniFile);
+ }
}
diff --git a/src/Generating/BioGen.h b/src/Generating/BioGen.h
index f9564ed24..feb449c06 100644
--- a/src/Generating/BioGen.h
+++ b/src/Generating/BioGen.h
@@ -77,6 +77,9 @@ protected:
} ;
+
+
+
class cBioGenMulticache :
public cBiomeGen
{
@@ -88,33 +91,18 @@ public:
~cBioGenMulticache();
protected:
-
- cBiomeGen * m_BioGenToCache;
-
- struct sCacheData
- {
- int m_ChunkX;
- int m_ChunkZ;
- cChunkDef::BiomeMap m_BiomeMap;
- };
-
- // To avoid moving large amounts of data for the MRU behavior, we MRU-ize indices to an array of the actual data
- int m_CacheSize;
int m_CachesLength;
-
- int ** m_CachesOrder; // MRU-ized order, indices into the multiple m_CachesData array
- sCacheData ** m_CachesData;
-
- // Cache statistics
- int m_NumHits;
- int m_NumMisses;
- int m_TotalChain; // Number of cache items walked to get to a hit (only added for hits)
+ int m_InternalCacheLength;
+ std::vector<cBiomeGen*> m_Caches;
virtual void GenBiomes(int a_ChunkX, int a_ChunkZ, cChunkDef::BiomeMap & a_BiomeMap) override;
virtual void InitializeBiomeGen(cIniFile & a_IniFile) override;
};
+
+
+
/// Base class for generators that use a list of available biomes. This class takes care of the list.
class cBiomeGenList :
public cBiomeGen