X-Git-Url: http://git.localhorst.tv/?a=blobdiff_plain;f=src%2Fchunk.cpp;h=bed947e3d5fdfff7182056313a30667ad9996edb;hb=f932e8c0273794bcd954c9f5b504bad6140f7cf4;hp=04c64f26738170a98b64345ddfc9712463ce70d5;hpb=374843f5b3ae60c0d02704a8da5100ac8abe7f1a;p=blank.git

diff --git a/src/chunk.cpp b/src/chunk.cpp
index 04c64f2..bed947e 100644
--- a/src/chunk.cpp
+++ b/src/chunk.cpp
@@ -6,6 +6,12 @@
 #include <glm/gtx/transform.hpp>
 
 
+namespace {
+
+blank::Model::Buffer buf;
+
+}
+
 namespace blank {
 
 Chunk::Chunk(const BlockTypeRegistry &types)
@@ -117,7 +123,6 @@ void Chunk::CheckUpdate() {
 	if (dirty) {
 		Update();
 	}
-	model.CheckUpdate();
 }
 
 void Chunk::Update() {
@@ -127,17 +132,17 @@ void Chunk::Update() {
 		vtx_count += shape->VertexCount();
 		idx_count += shape->VertexIndexCount();
 	}
-	model.Clear();
-	model.Reserve(vtx_count, idx_count);
+	buf.Clear();
+	buf.Reserve(vtx_count, idx_count);
 
 	Model::Index vtx_counter = 0;
 	for (size_t i = 0; i < Size(); ++i) {
 		const BlockType &type = Type(blocks[i]);
-		type.FillModel(model, ToCoords(i), vtx_counter);
+		type.FillModel(buf, ToCoords(i), vtx_counter);
 		vtx_counter += type.shape->VertexCount();
 	}
 
-	model.Invalidate();
+	model.Update(buf);
 	dirty = false;
 }
 
@@ -161,9 +166,9 @@ struct ChunkLess {
 	explicit ChunkLess(const Chunk::Pos &base)
 	: base(base) { }
 
-	bool operator ()(const Chunk &a, const Chunk &b) const {
-		Chunk::Pos da(base - a.Position());
-		Chunk::Pos db(base - b.Position());
+	bool operator ()(const Chunk::Pos &a, const Chunk::Pos &b) const {
+		Chunk::Pos da(base - a);
+		Chunk::Pos db(base - b);
 		return
 			da.x * da.x + da.y * da.y + da.z * da.z <
 			db.x * db.x + db.y * db.y + db.z * db.z;
@@ -187,8 +192,7 @@ void ChunkLoader::Generate(const Chunk::Pos &from, const Chunk::Pos &to) {
 					loaded.back().Position(pos);
 					gen(loaded.back());
 				} else {
-					to_generate.emplace_back(reg);
-					to_generate.back().Position(pos);
+					to_generate.emplace_back(pos);
 				}
 			}
 		}
@@ -205,19 +209,17 @@ Chunk *ChunkLoader::Loaded(const Chunk::Pos &pos) {
 	return nullptr;
 }
 
-Chunk *ChunkLoader::Queued(const Chunk::Pos &pos) {
-	for (Chunk &chunk : to_generate) {
-		if (chunk.Position() == pos) {
-			return &chunk;
+bool ChunkLoader::Queued(const Chunk::Pos &pos) {
+	for (const Chunk::Pos &chunk : to_generate) {
+		if (chunk == pos) {
+			return true;
 		}
 	}
 	return nullptr;
 }
 
-Chunk *ChunkLoader::Known(const Chunk::Pos &pos) {
-	Chunk *chunk = Loaded(pos);
-	if (chunk) return chunk;
-
+bool ChunkLoader::Known(const Chunk::Pos &pos) {
+	if (Loaded(pos)) return true;
 	return Queued(pos);
 }
 
@@ -227,10 +229,11 @@ Chunk &ChunkLoader::ForceLoad(const Chunk::Pos &pos) {
 		return *chunk;
 	}
 
-	chunk = Queued(pos);
-	if (chunk) {
-		gen(*chunk);
-		return *chunk;
+	for (auto iter(to_generate.begin()), end(to_generate.end()); iter != end; ++iter) {
+		if (*iter == pos) {
+			to_generate.erase(iter);
+			break;
+		}
 	}
 
 	loaded.emplace_back(reg);
@@ -259,9 +262,9 @@ 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->Position().x) > unload_dist
-				|| std::abs(base.y - iter->Position().y) > unload_dist
-				|| std::abs(base.z - iter->Position().z) > unload_dist) {
+		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) {
 			iter = to_generate.erase(iter);
 		} else {
 			++iter;
@@ -273,12 +276,32 @@ void ChunkLoader::Rebase(const Chunk::Pos &new_base) {
 }
 
 void ChunkLoader::Update() {
+	bool reused = false;
 	if (!to_generate.empty()) {
-		gen(to_generate.front());
-		loaded.splice(loaded.end(), to_generate, to_generate.begin());
+		Chunk::Pos pos(to_generate.front());
+
+		for (auto iter(to_free.begin()), end(to_free.end()); iter != end; ++iter) {
+			if (iter->Position() == pos) {
+				loaded.splice(loaded.end(), to_free, iter);
+				reused = true;
+				break;
+			}
+		}
+
+		if (!reused) {
+			if (to_free.empty()) {
+				loaded.emplace_back(reg);
+			} else {
+				loaded.splice(loaded.end(), to_free, to_free.begin());
+				reused = true;
+			}
+			loaded.back().Position(pos);
+			gen(loaded.back());
+		}
+		to_generate.pop_front();
 	}
 
-	if (!to_free.empty()) {
+	if (!reused && !to_free.empty()) {
 		to_free.pop_front();
 	}
 }