move chunk update back to secondary thread

src/chunkmanager.cpp

@@ -68,7 +68,7 @@ namespace chunkmanager
 	    chunkmesher::getMeshDataQueue().push(new chunkmesher::MeshData());
 
 	should_run = true;
-	//update_thread = std::thread(update);
+	update_thread = std::thread(update);
 	gen_thread = std::thread(generate);
 	mesh_thread = std::thread(mesh);
 
@@ -108,8 +108,10 @@ namespace chunkmanager
     int nUnloaded{0};
 
     std::queue<int32_t> chunks_todelete;
-    void primary_thread_update(){
+    void update(){
-	int nExplored{0}, nMeshed{0}, nGenerated{0};
+	while(should_run) {
+
+	    int nExplored{0}, nMeshed{0}, nGenerated{0};
 	    int chunkX=static_cast<int>(theCamera.getAtomicPosX() / CHUNK_SIZE);
 	    int chunkY=static_cast<int>(theCamera.getAtomicPosY() / CHUNK_SIZE);
 	    int chunkZ=static_cast<int>(theCamera.getAtomicPosZ() / CHUNK_SIZE);
@@ -211,6 +213,7 @@ namespace chunkmanager
 	    debug::window::set_parameter("update_chunks_generated", nGenerated);
 	    debug::window::set_parameter("update_chunks_meshed", nMeshed);
 	    debug::window::set_parameter("update_chunks_explored", nExplored);
+	}
     }
 
     // uint32_t is fine, since i'm limiting the coordinate to only use up to ten bits (1023). There's actually two spare bits
@@ -224,6 +227,8 @@ namespace chunkmanager
 	should_run=false;
 
 	std::cout << "waiting for secondary threads to finish\n";
+	update_thread.join();
+	std::cout << "update thread terminated\n";
 	gen_thread.join();
 	std::cout << "generation thread terminated\n";
 	mesh_thread.join();

src/main.cpp

@@ -103,7 +103,7 @@ int main()
 	// Reset blockping timeout if 200ms have passed
 	if(glfwGetTime() - lastBlockPick > 0.1) blockpick = false;
 
-	chunkmanager::primary_thread_update();
+	//chunkmanager::primary_thread_update();
 
 	// Render pass
 	renderer::render();