ScuffedMinecraft/src/Application.cpp

@@ -380,20 +380,28 @@ int main(int argc, char *argv[])
 		projection = glm::perspective(glm::radians(camera.Zoom), windowX / windowY, 0.1f, 1000.0f);
 
 		shader.use();
-		shader.setMat4x4f( "view", view );
-		shader.setMat4x4f( "projection", projection );
+		unsigned int viewLoc = glGetUniformLocation(shader.ID, "view");
+		glUniformMatrix4fv(viewLoc, 1, GL_FALSE, glm::value_ptr(view));
+		unsigned int projectionLoc = glGetUniformLocation(shader.ID, "projection");
+		glUniformMatrix4fv(projectionLoc, 1, GL_FALSE, glm::value_ptr(projection));
 
 		waterShader.use();
-		waterShader.setMat4x4f( "view", view );
-		waterShader.setMat4x4f( "projection", projection );
+		viewLoc = glGetUniformLocation(waterShader.ID, "view");
+		glUniformMatrix4fv(viewLoc, 1, GL_FALSE, glm::value_ptr(view));
+		projectionLoc = glGetUniformLocation(waterShader.ID, "projection");
+		glUniformMatrix4fv(projectionLoc, 1, GL_FALSE, glm::value_ptr(projection));
 
 		billboardShader.use();
-		billboardShader.setMat4x4f( "view", view );
-		billboardShader.setMat4x4f( "projection", projection );
+		viewLoc = glGetUniformLocation(billboardShader.ID, "view");
+		glUniformMatrix4fv(viewLoc, 1, GL_FALSE, glm::value_ptr(view));
+		projectionLoc = glGetUniformLocation(billboardShader.ID, "projection");
+		glUniformMatrix4fv(projectionLoc, 1, GL_FALSE, glm::value_ptr(projection));
 
 		outlineShader.use();
-		outlineShader.setMat4x4f( "view", view );
-		outlineShader.setMat4x4f( "projection", projection );
+		viewLoc = glGetUniformLocation(outlineShader.ID, "view");
+		glUniformMatrix4fv(viewLoc, 1, GL_FALSE, glm::value_ptr(view));
+		projectionLoc = glGetUniformLocation(outlineShader.ID, "projection");
+		glUniformMatrix4fv(projectionLoc, 1, GL_FALSE, glm::value_ptr(projection));
 
 		Planet::planet->Update(camera.Position);
 
@@ -484,7 +492,8 @@ int main(int argc, char *argv[])
 			glEnable(GL_BLEND);
 			glEnable(GL_COLOR_LOGIC_OP);
 
-			crosshairShader.setMat4x4f( "projection", ortho );
+			unsigned int crosshairProjLoc = glGetUniformLocation(crosshairShader.ID, "projection");
+			glUniformMatrix4fv(crosshairProjLoc, 1, GL_FALSE, glm::value_ptr(ortho));
 			glBindVertexArray(crosshairVAO);
 			glDrawArrays(GL_TRIANGLES, 0, 6);
 

ScuffedMinecraft/src/Chunk.cpp

@@ -519,15 +519,18 @@ void Chunk::Render(Shader* mainShader, Shader* billboardShader)
 
 	// Render main mesh
 	mainShader->use();
-	mainShader->setMat4x4f( "model", model );
 
+	modelLoc = glGetUniformLocation(mainShader->ID, "model");
+	glUniformMatrix4fv(modelLoc, 1, GL_FALSE, glm::value_ptr(model));
 
 	glBindVertexArray(mainVAO);
 	glDrawElements(GL_TRIANGLES, numTrianglesMain, GL_UNSIGNED_INT, 0);
 
 	// Render billboard mesh
 	billboardShader->use();
-	billboardShader->setMat4x4f( "model", model );
+
+	modelLoc = glGetUniformLocation(billboardShader->ID, "model");
+	glUniformMatrix4fv(modelLoc, 1, GL_FALSE, glm::value_ptr(model));
 
 	glDisable(GL_CULL_FACE);
 	glBindVertexArray(billboardVAO);
@@ -543,11 +546,12 @@ void Chunk::RenderWater(Shader* shader)
 	//std::cout << "Rendering chunk " << chunkPos.x << ", " << chunkPos.y << ", " << chunkPos.z << '\n'
 	//	<< "Chunk VAO: " << vertexArrayObject << '\n' << "Triangles: " << numTriangles << '\n';
 
+	modelLoc = glGetUniformLocation(shader->ID, "model");
 	glBindVertexArray(waterVAO);
 
 	glm::mat4 model = glm::mat4(1.0f);
 	model = glm::translate(model, worldPos);
-	shader->setMat4x4f( "model", model );
+	glUniformMatrix4fv(modelLoc, 1, GL_FALSE, glm::value_ptr(model));
 
 	glDrawElements(GL_TRIANGLES, numTrianglesWater, GL_UNSIGNED_INT, 0);
 }

ScuffedMinecraft/src/Planet.cpp

@@ -39,15 +39,13 @@ void Planet::Update(glm::vec3 cameraPos)
 	{
 		numChunks++;
 
-		Chunk& chunk = *it->second;
-
-		if (!chunk.ready)
+		if (!(*it->second).ready)
 			chunksLoading++;
 
-		int chunkX = chunk.chunkPos.x;
-		int chunkY = chunk.chunkPos.y;
-		int chunkZ = chunk.chunkPos.z;
-		if ( chunk.ready && (abs(chunkX - camChunkX) > renderDistance ||
+		int chunkX = (*it->second).chunkPos.x;
+		int chunkY = (*it->second).chunkPos.y;
+		int chunkZ = (*it->second).chunkPos.z;
+		if ((*it->second).ready && (abs(chunkX - camChunkX) > renderDistance ||
 			abs(chunkY - camChunkY) > renderDistance ||
 			abs(chunkZ - camChunkZ) > renderDistance))
 		{
@@ -62,13 +60,13 @@ void Planet::Update(glm::vec3 cameraPos)
 			chunkDataDeleteQueue.push({ chunkX, chunkY, chunkZ - 1 });
 
 			// Delete chunk
-			delete it->second; // &chunk
+			delete it->second;
 			it = chunks.erase(it);
 		}
 		else
 		{
 			numChunksRendered++;
-			chunk.Render(solidShader, billboardShader);
+			(*it->second).Render(solidShader, billboardShader);
 			++it;
 		}
 	}
@@ -77,13 +75,11 @@ void Planet::Update(glm::vec3 cameraPos)
 	waterShader->use();
 	for (auto it = chunks.begin(); it != chunks.end(); )
 	{
-		Chunk& chunk = *it->second;
+		int chunkX = (*it->second).chunkPos.x;
+		int chunkY = (*it->second).chunkPos.y;
+		int chunkZ = (*it->second).chunkPos.z;
 
-		int chunkX = chunk.chunkPos.x;
-		int chunkY = chunk.chunkPos.y;
-		int chunkZ = chunk.chunkPos.z;
-
-		chunk.RenderWater(waterShader);
+		(*it->second).RenderWater(waterShader);
 		++it;
 	}
 
@@ -106,16 +102,11 @@ void Planet::ChunkThreadUpdate()
 				chunks.find({ pos.x, pos.y, pos.z + 1 }) == chunks.end() &&
 				chunks.find({ pos.x, pos.y, pos.z - 1 }) == chunks.end())
 			{
-				chunkMutex.lock();
-				delete it->second;
-				it = chunkData.erase( it );
-				chunkMutex.unlock();
+				delete chunkData.at(pos);
+				chunkData.erase(pos);
 			}
-			else
-			{
 			++it;
 		}
-		}
 
 		// Check if camera moved to new chunk
 		if (camChunkX != lastCamX || camChunkY != lastCamY || camChunkZ != lastCamZ)

ScuffedMinecraft/src/Shader.cpp

@@ -2,8 +2,6 @@
 
 #include <glad/glad.h>
 
-#include <glm/gtc/type_ptr.hpp>
-
 #include <fstream>
 #include <sstream>
 #include <iostream>
@@ -93,22 +91,15 @@ void Shader::use()
 	glUseProgram(ID);
 }
 
-void Shader::setBool(const std::string& name, bool value)
+void Shader::setBool(const std::string& name, bool value) const
 {
 	glUniform1i(glGetUniformLocation(ID, name.c_str()), (int)value);
 }
-
-void Shader::setInt(const std::string& name, int value)
+void Shader::setInt(const std::string& name, int value) const
 {
 	glUniform1i(glGetUniformLocation(ID, name.c_str()), value);
 }
-
-void Shader::setFloat(const std::string& name, float value)
+void Shader::setFloat(const std::string& name, float value) const
 {
 	glUniform1f(glGetUniformLocation(ID, name.c_str()), value);
 }
-
-void Shader::setMat4x4f( const std::string& name, const glm::mat4& matrix )
-{
-	glUniformMatrix4fv( glGetUniformLocation( ID, name.c_str() ), 1, GL_FALSE, glm::value_ptr( matrix ) );
-}

ScuffedMinecraft/src/Shader.h

@@ -1,7 +1,6 @@
 #pragma once
 
 #include <string>
-#include <glm/fwd.hpp>
 
 class Shader
 {
@@ -14,12 +13,7 @@ public:
 	// use/activate the shader
 	void use();
 	// utility uniform functions
-	void setBool(const std::string& name, bool value);
-	void setInt(const std::string& name, int value);
-	void setFloat(const std::string& name, float value);
-
-	void setMat4x4f( const std::string& name, const glm::mat4& matrix );
-	
-private:
-
+	void setBool(const std::string& name, bool value) const;
+	void setInt(const std::string& name, int value) const;
+	void setFloat(const std::string& name, float value) const;
 };