Only update PushBuffer if any uniform has actually changed

Author: VReaperV

src/engine/renderer/gl_shader.cpp

@@ -231,6 +231,12 @@ GLShaderManager::~GLShaderManager()
 = default;
 
 void GLShaderManager::FreeAll() {
+	for ( const std::unique_ptr<GLShader>& shader : _shaders ) {
+		if ( shader.get()->uniformStorage ) {
+			Z_Free( shader.get()->uniformStorage );
+		}
+	}
+
 	_shaders.clear();
 
 	deformShaderCount = 0;
@@ -249,8 +255,8 @@ void GLShaderManager::FreeAll() {
 			Z_Free( program.uniformBlockIndexes );
 		}
 
-		if ( program.uniformFirewall ) {
-			Z_Free( program.uniformFirewall );
+		if ( program.uniformStorage ) {
+			Z_Free( program.uniformStorage );
 		}
 	}
 
@@ -279,7 +285,7 @@ void GLShaderManager::UpdateShaderProgramUniformLocations( GLShader* shader, Sha
 	shaderProgram->uniformLocations = ( GLint* ) Z_Malloc( sizeof( GLint ) * numUniforms );
 
 	// create buffer for uniform firewall
-	shaderProgram->uniformFirewall = ( byte* ) Z_Malloc( uniformSize );
+	shaderProgram->uniformStorage = ( uint32_t* ) Z_Malloc( uniformSize );
 
 	// update uniforms
 	for (GLUniform *uniform : shader->_uniforms)
@@ -1279,10 +1285,16 @@ void GLShaderManager::InitShader( GLShader* shader ) {
 	for ( std::size_t i = 0; i < shader->_uniforms.size(); i++ ) {
 		GLUniform* uniform = shader->_uniforms[i];
 		uniform->SetLocationIndex( i );
-		uniform->SetFirewallIndex( shader->_uniformStorageSize );
-		shader->_uniformStorageSize += uniform->GetSize();
+		uniform->SetUniformStorageOffset( shader->_uniformStorageSize );
+
+		const uint32_t size = uniform->_components ? uniform->_std430Size * uniform->_components : uniform->_std430Size;
+		shader->_uniformStorageSize += size;
 	}
 
+	shader->_uniformStorageSize *= sizeof( uint32_t );
+
+	shader->uniformStorage = ( uint32_t* ) Z_Malloc( shader->_uniformStorageSize );
+
 	for ( std::size_t i = 0; i < shader->_uniformBlocks.size(); i++ ) {
 		GLUniformBlock* uniformBlock = shader->_uniformBlocks[i];
 		uniformBlock->SetLocationIndex( i );
@@ -2145,10 +2157,6 @@ bool GLCompileMacro_USE_BSP_SURFACE::HasConflictingMacros(size_t permutation, co
 	return false;
 }
 
-uint32_t* GLUniform::WriteToBuffer( uint32_t* buffer ) {
-	return buffer;
-}
-
 void GLShader::RegisterUniform( GLUniform* uniform ) {
 	_uniforms.push_back( uniform );
 }
@@ -2203,7 +2211,7 @@ GLuint GLShaderManager::SortUniforms( std::vector<GLUniform*>& uniforms ) {
 		}
 
 		if ( alignmentConsume & 3 ) {
-			tmpUniform->_std430Size += alignmentConsume;
+			tmpUniform->_nextUniformOffset += alignmentConsume;
 		}
 
 		if ( uniforms.size() ) {
@@ -2453,6 +2461,8 @@ void GLShader::WriteUniformsToBuffer( uint32_t* buffer, const Mode mode, const i
 			bufPtr = uniform->WriteToBuffer( bufPtr );
 		}
 	}
+
+	uniformsUpdated = false;
 }
 
 GLShader_generic::GLShader_generic() :