summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorTiger Wang <ziwei.tiger@hotmail.co.uk>2014-04-07 21:57:14 +0200
committerTiger Wang <ziwei.tiger@hotmail.co.uk>2014-04-07 21:57:14 +0200
commitf13cf1a0217437113ad5372ef971c3903d55a9c6 (patch)
tree92e1c42bb475ed23eeda023f08df662283153f83 /src
parentAttempt to fix errors (diff)
downloadcuberite-f13cf1a0217437113ad5372ef971c3903d55a9c6.tar
cuberite-f13cf1a0217437113ad5372ef971c3903d55a9c6.tar.gz
cuberite-f13cf1a0217437113ad5372ef971c3903d55a9c6.tar.bz2
cuberite-f13cf1a0217437113ad5372ef971c3903d55a9c6.tar.lz
cuberite-f13cf1a0217437113ad5372ef971c3903d55a9c6.tar.xz
cuberite-f13cf1a0217437113ad5372ef971c3903d55a9c6.tar.zst
cuberite-f13cf1a0217437113ad5372ef971c3903d55a9c6.zip
Diffstat (limited to '')
-rw-r--r--src/Chunk.cpp75
-rw-r--r--src/Chunk.h2
2 files changed, 29 insertions, 48 deletions
diff --git a/src/Chunk.cpp b/src/Chunk.cpp
index ede01a6cc..7b49862f0 100644
--- a/src/Chunk.cpp
+++ b/src/Chunk.cpp
@@ -291,74 +291,55 @@ void cChunk::SetAllData(
memcpy(m_HeightMap, a_HeightMap, sizeof(m_HeightMap));
}
+ if (a_HeightMap == NULL)
{
- int IdxWhereNonEmptyStarts = 0;
- { // Blocktype compression
- bool FoundNonAir = false;
- m_BlockTypes.clear();
+ CalculateHeightmap(a_BlockTypes);
+ }
- for (int Idx = NumBlocks - 1; Idx >= 0; Idx--)
+ int IdxWhereNonEmptyStarts = 0;
+ { // Blocktype compression
+ unsigned char Highest = 0;
+ int X = 0, Z = 0;
+ m_BlockTypes.clear();
+
+ for (int x = 0; x < Width; x++)
+ {
+ for (int z = 0; z < Width; z++)
{
- if (a_BlockTypes[Idx] != E_BLOCK_AIR)
+ unsigned char Height = m_HeightMap[x + z * Width];
+ if (Height > Highest)
{
- FoundNonAir = true;
- IdxWhereNonEmptyStarts = Idx;
- break;
+ Highest = Height;
+ X = x; Z = z;
}
}
- m_BlockTypes.insert(m_BlockTypes.end(), &a_BlockTypes[0], &a_BlockTypes[IdxWhereNonEmptyStarts + 1]);
}
- { // Blockmeta compression
- m_BlockMeta.clear();
- m_BlockMeta.insert(m_BlockMeta.end(), &a_BlockMeta[0], &a_BlockMeta[(IdxWhereNonEmptyStarts + 1) / 2]);
- }
+ IdxWhereNonEmptyStarts = MakeIndexNoCheck(X, Highest + 1, Z);
+
+ m_BlockTypes.insert(m_BlockTypes.end(), &a_BlockTypes[0], &a_BlockTypes[IdxWhereNonEmptyStarts]);
+ }
+
+ { // Blockmeta compression
+ m_BlockMeta.clear();
+ m_BlockMeta.insert(m_BlockMeta.end(), &a_BlockMeta[0], &a_BlockMeta[IdxWhereNonEmptyStarts / 2]);
}
if (a_BlockLight != NULL)
{
// Compress blocklight
- bool FoundNonEmpty = false;
- int IdxWhereNonEmptyStarts = 0;
m_BlockLight.clear();
-
- for (int Idx = (NumBlocks / 2) - 1; Idx >= 0; Idx--)
- {
- if (a_BlockLight[Idx] != 0)
- {
- FoundNonEmpty = true;
- IdxWhereNonEmptyStarts = Idx;
- break;
- }
- }
- m_BlockLight.insert(m_BlockLight.end(), &a_BlockLight[0], &a_BlockLight[IdxWhereNonEmptyStarts + 1]);
+ m_BlockLight.insert(m_BlockLight.end(), &a_BlockLight[0], &a_BlockLight[IdxWhereNonEmptyStarts / 2]);
}
if (a_BlockSkyLight != NULL)
{
// Compress skylight
- bool FoundNonEmpty = false;
- int IdxWhereNonEmptyStarts = 0;
m_BlockSkyLight.clear();
-
- for (int Idx = (NumBlocks / 2) - 1; Idx >= 0; Idx--)
- {
- if (a_BlockSkyLight[Idx] != 0xff)
- {
- FoundNonEmpty = true;
- IdxWhereNonEmptyStarts = Idx;
- break;
- }
- }
- m_BlockSkyLight.insert(m_BlockSkyLight.end(), &a_BlockSkyLight[0], &a_BlockSkyLight[IdxWhereNonEmptyStarts + 1]);
+ m_BlockSkyLight.insert(m_BlockSkyLight.end(), &a_BlockSkyLight[0], &a_BlockSkyLight[IdxWhereNonEmptyStarts / 2]);
}
m_IsLightValid = (a_BlockLight != NULL) && (a_BlockSkyLight != NULL);
-
- if (a_HeightMap == NULL)
- {
- CalculateHeightmap();
- }
// Clear the block entities present - either the loader / saver has better, or we'll create empty ones:
for (cBlockEntityList::iterator itr = m_BlockEntities.begin(); itr != m_BlockEntities.end(); ++itr)
@@ -1483,7 +1464,7 @@ void cChunk::WakeUpSimulators(void)
-void cChunk::CalculateHeightmap()
+void cChunk::CalculateHeightmap(const BLOCKTYPE * a_BlockTypes)
{
for (int x = 0; x < Width; x++)
{
@@ -1492,7 +1473,7 @@ void cChunk::CalculateHeightmap()
for (int y = Height - 1; y > -1; y--)
{
int index = MakeIndex( x, y, z );
- if (GetBlock(index) != E_BLOCK_AIR)
+ if (a_BlockTypes[index] != E_BLOCK_AIR)
{
m_HeightMap[x + z * Width] = (unsigned char)y;
break;
diff --git a/src/Chunk.h b/src/Chunk.h
index 6e2933ff8..9100eec58 100644
--- a/src/Chunk.h
+++ b/src/Chunk.h
@@ -267,7 +267,7 @@ public:
void UseBlockEntity(cPlayer * a_Player, int a_X, int a_Y, int a_Z); // [x, y, z] in world block coords
void CalculateLighting(); // Recalculate right now
- void CalculateHeightmap();
+ void CalculateHeightmap(const BLOCKTYPE * a_BlockTypes);
// Broadcast various packets to all clients of this chunk:
// (Please keep these alpha-sorted)