cache textures by filename

1 files changed, 13 insertions, 18 deletions

diff --git a/src/gl.zig b/src/gl.zig
index bb8d3e5..26ce981 100644
--- a/src/gl.zig
+++ b/src/gl.zig
@@ -377,7 +377,8 @@ const UniformValue = union(UniformType) {
             .UNSIGNED_INT_SAMPLER_BUFFER,
             .UNSIGNED_INT_SAMPLER_2D_RECT,
             => |value| {
-                if (value) |texture| texture.destroy();
+                // if (value) |texture| texture.destroy();
+                _ = value;
             },
             else => {},
         }
@@ -529,21 +530,6 @@ pub const CachedUniform = struct {
         }
     }
 
-    // deinit self WITHOUT freeing contained resources
-    pub fn move(
-        self: *CachedUniform,
-        to: *CachedUniform,
-        allocator: std.mem.Allocator,
-    ) CachedUniform {
-        to.deinit(allocator);
-        to.buffer = self.buffer;
-        to.value = self.value;
-
-        self.buffer = null;
-
-        return self;
-    }
-
     pub fn tryMove(
         self: *CachedUniform,
         dest: *CachedUniform,
@@ -895,24 +881,33 @@ pub const ShaderProgram = struct {
 pub const UniformCache = struct {
     allocator: std.mem.Allocator,
     uniforms: std.StringHashMap(CachedUniform),
+    textures: std.StringHashMap(Texture),
     shader: *ShaderProgram,
 
     pub fn init(allocator: std.mem.Allocator, shader: *ShaderProgram) UniformCache {
         return UniformCache{
             .allocator = allocator,
             .uniforms = std.StringHashMap(CachedUniform).init(allocator),
+            .textures = std.StringHashMap(Texture).init(allocator),
             .shader = shader,
         };
     }
 
     pub fn deinit(self: *UniformCache) void {
-        var it = self.uniforms.iterator();
-        while (it.next()) |entry| {
+        var uit = self.uniforms.iterator();
+        while (uit.next()) |entry| {
             entry.value_ptr.deinit(self.allocator);
             self.allocator.free(entry.key_ptr.*);
         }
 
+        var tit = self.textures.iterator();
+        while (tit.next()) |entry| {
+            entry.value_ptr.destroy();
+            self.allocator.free(entry.key_ptr.*);
+        }
+
         self.uniforms.deinit();
+        self.textures.deinit();
     }
 
     pub fn get(self: *UniformCache, name: []const u8) !?*CachedUniform {