From 2f286e9ec9d872625f8f82bbcdcfc588913ec4a3 Mon Sep 17 00:00:00 2001
From: 3gg <3gg@shellblade.net>
Date: Wed, 9 Jul 2025 20:19:47 -0700
Subject: Track active material in the LLR

---
 src/llr/llr.c | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

(limited to 'src/llr/llr.c')

diff --git a/src/llr/llr.c b/src/llr/llr.c
index e5242e7..664c9a4 100644
--- a/src/llr/llr.c
+++ b/src/llr/llr.c
@@ -201,11 +201,20 @@ static void configure_state(LLR* renderer) {
     }
   }
 
+  if (renderer->material_changed || renderer->shader_changed) {
+    renderer->material_changed = false;
+
+    gfx_material_activate(renderer->shader, renderer->material);
+  }
+
   if (renderer->shader_changed) {
     renderer->shader_changed = false;
     gfx_activate_shader_program(renderer->shader);
   }
 
+  // TODO: At present, this results in many redundant calls to
+  //  glGetUniformLocation() and glUniformXyz(). Look at the trace.
+  //
   // Must be called after activating the program.
   gfx_apply_uniforms(renderer->shader);
 }
@@ -296,6 +305,14 @@ void gfx_llr_clear_skeleton(LLR* renderer) {
   renderer->skeleton_changed = true;
 }
 
+void gfx_llr_set_material(LLR* renderer, const Material* material) {
+  assert(renderer);
+  assert(material);
+
+  renderer->material         = material;
+  renderer->material_changed = true;
+}
+
 void gfx_llr_set_camera(LLR* renderer, const Camera* camera) {
   assert(renderer);
 
@@ -331,7 +348,7 @@ void gfx_llr_render_mesh(LLR* renderer, const Mesh* mesh) {
   assert(mesh->geometry);
   assert(mesh->material);
 
-  gfx_material_activate(renderer->shader, mesh->material);
+  gfx_llr_set_material(renderer, mesh->material);
   gfx_llr_render_geometry(renderer, mesh->geometry);
 }
 
-- 
cgit v1.2.3