feat: implement uniform caching in ShaderProgram (#13)

Author: AhmedSobhy01

src/common/shader/shader.cpp

@@ -37,6 +37,7 @@ bool our::ShaderProgram::attach(const std::string& filename, GLenum type) const
 }
 
 bool our::ShaderProgram::link() const {
+    uniformLocations.clear();
     glLinkProgram(program);
     std::string error = checkForLinkingErrors(program);
     if (!error.empty()) {

src/common/shader/shader.hpp

@@ -6,13 +6,15 @@
 #include <glm/glm.hpp>
 #include <glm/gtc/type_ptr.hpp>
 #include <string>
+#include <unordered_map>
 
 namespace our {
 
     class ShaderProgram {
     private:
         // Shader Program Handle
         GLuint program;
+        mutable std::unordered_map<std::string, GLint> uniformLocations;
 
     public:
         ShaderProgram() {
@@ -30,8 +32,12 @@ namespace our {
             glUseProgram(program);
         }
 
-        GLuint getUniformLocation(const std::string& name) {
-            return glGetUniformLocation(program, name.c_str());
+        GLint getUniformLocation(const std::string& name) {
+            if (uniformLocations.find(name) != uniformLocations.end()) return uniformLocations[name];
+
+            GLint location = glGetUniformLocation(program, name.c_str());
+            uniformLocations[name] = location;
+            return location;
         }
 
         void set(const std::string& uniform, GLfloat value) {