Extend comment on shader uniforms

1 files changed, 10 insertions, 2 deletions

diff --git a/src/rendering/src/gl.c b/src/rendering/src/gl.c
index e57e949..92599d7 100644
--- a/src/rendering/src/gl.c
+++ b/src/rendering/src/gl.c
@@ -247,8 +247,16 @@ RenderObject RenderObject_new(
 
   // It is very much a non-issue if we don't find the model view projection in
   // the shader. In fact, it is removed from a shader program if it is not used.
-  // TODO: Add more uniforms, such as mouse coords and time. Could be cool idk.
-  //if (o.mvp == -1) {
+  // TODO: Add common uniforms, should be a list of strings (uniform name) &
+  // their locations (i32), such as
+  // * mouse coords,
+  // * time,
+  // * delta time,
+  // * modelviewprojection,
+  // * window size.
+  // These should be added to the RenderObject, if found.
+
+  // if (o.mvp == -1) {
   //  WARN("Unable to find \"MVP\" input in shader program");
   //}