X-Git-Url: http://git.localhorst.tv/?a=blobdiff_plain;f=src%2Fworld%2Fchunk.cpp;h=c114526ba46cdace3204acb226189cb0af671d27;hb=0580ff3941fe5f5ea25c96e737edba1541d9271f;hp=fe4f80267ab00d1f4bf2c572c7f278168cf03915;hpb=57ff8c89c8e172b0988a09490a2bc19d740d1c79;p=blank.git

diff --git a/src/world/chunk.cpp b/src/world/chunk.cpp
index fe4f802..c114526 100644
--- a/src/world/chunk.cpp
+++ b/src/world/chunk.cpp
@@ -169,15 +169,16 @@ void Chunk::SetBlock(int index, const Block &block) noexcept {
 	} else if (!new_type.block_light && old_type.block_light) {
 		// obstacle removed
 		int level = 0;
+		Pos pos(ToPos(index));
 		for (int face = 0; face < Block::FACE_COUNT; ++face) {
-			BlockLookup next_block(this, ToPos(index), Block::Face(face));
+			BlockLookup next_block(this, pos, Block::Face(face));
 			if (next_block) {
-				level = std::min(level, next_block.GetLight());
+				level = std::max(level, next_block.GetLight());
 			}
 		}
 		if (level > 1) {
 			SetLight(index, level - 1);
-			light_queue.emplace(this, ToPos(index));
+			light_queue.emplace(this, pos);
 			work_light();
 		}
 	}
@@ -449,7 +450,6 @@ bool Chunk::Intersection(
 	float &dist,
 	glm::vec3 &normal
 ) const noexcept {
-	// TODO: should be possible to heavily optimize this
 	int idx = 0;
 	blkid = -1;
 	dist = std::numeric_limits<float>::infinity();
@@ -627,9 +627,10 @@ ChunkLoader::ChunkLoader(const Config &config, const BlockTypeRegistry &reg, con
 , loaded()
 , to_generate()
 , to_free()
+, gen_timer(config.gen_limit)
 , load_dist(config.load_dist)
 , unload_dist(config.unload_dist) {
-
+	gen_timer.Start();
 }
 
 namespace {
@@ -774,6 +775,12 @@ Chunk &ChunkLoader::ForceLoad(const Chunk::Pos &pos) {
 	return Generate(pos);
 }
 
+bool ChunkLoader::OutOfRange(const Chunk::Pos &pos) const noexcept {
+	return std::abs(base.x - pos.x) > unload_dist
+			|| std::abs(base.y - pos.y) > unload_dist
+			|| std::abs(base.z - pos.z) > unload_dist;
+}
+
 void ChunkLoader::Rebase(const Chunk::Pos &new_base) {
 	if (new_base == base) {
 		return;
@@ -782,9 +789,7 @@ void ChunkLoader::Rebase(const Chunk::Pos &new_base) {
 
 	// unload far away chunks
 	for (auto iter(loaded.begin()), end(loaded.end()); iter != end;) {
-		if (std::abs(base.x - iter->Position().x) > unload_dist
-				|| std::abs(base.y - iter->Position().y) > unload_dist
-				|| std::abs(base.z - iter->Position().z) > unload_dist) {
+		if (OutOfRange(*iter)) {
 			auto saved = iter;
 			Remove(*saved);
 			++iter;
@@ -795,9 +800,7 @@ void ChunkLoader::Rebase(const Chunk::Pos &new_base) {
 	}
 	// abort far away queued chunks
 	for (auto iter(to_generate.begin()), end(to_generate.end()); iter != end;) {
-		if (std::abs(base.x - iter->x) > unload_dist
-				|| std::abs(base.y - iter->y) > unload_dist
-				|| std::abs(base.z - iter->z) > unload_dist) {
+		if (OutOfRange(*iter)) {
 			iter = to_generate.erase(iter);
 		} else {
 			++iter;
@@ -812,8 +815,9 @@ void ChunkLoader::GenerateSurrounding(const Chunk::Pos &pos) {
 	Generate(pos - offset, pos + offset);
 }
 
-void ChunkLoader::Update() {
-	if (to_generate.empty()) {
+void ChunkLoader::Update(int dt) {
+	gen_timer.Update(dt);
+	if (!gen_timer.Hit() || to_generate.empty()) {
 		return;
 	}
 
@@ -824,7 +828,6 @@ void ChunkLoader::Update() {
 		if (iter->Position() == pos) {
 			iter->Relink();
 			loaded.splice(loaded.end(), to_free, iter);
-			return;
 		}
 	}