Fix managed sampler uniforms breaking on reload

changelog.md

@@ -1,3 +1,12 @@
+------------------------------------------------------
+Version 1.12.1
+------------------------------------------------------
+**Changes**
+- Optimized shader loading (should now be faster)
+
+**Fixes**
+- Fixed managed sampler uniforms losing data upon resource reloading (Ported fix from v1.10.1 to MC 1.19.4)
+
 ------------------------------------------------------
 Version 1.12.0
 ------------------------------------------------------
@@ -12,6 +21,12 @@ Version 1.11.0
 ------------------------------------------------------
 Updated to MC 1.19.4
 
+------------------------------------------------------
+Version 1.10.1
+------------------------------------------------------
+**Fixes**
+- Fixed managed sampler uniforms losing data upon resource reloading
+
 ------------------------------------------------------
 Version 1.10.0
 ------------------------------------------------------

gradle.properties

@@ -10,7 +10,7 @@ loader_version=0.14.17
 fabric_version=0.76.0+1.19.4
 
 # Mod Properties
-mod_version = 1.12.0
+mod_version = 1.12.1
 owners = Ladysnake
 maven_group = io.github.ladysnake
 archives_base_name = satin

src/main/java/ladysnake/satin/impl/ManagedSamplerUniformBase.java

@@ -60,6 +60,7 @@ public boolean findUniformTargets(List<PostEffectPass> shaders) {
         }
         this.targets = targets.toArray(new SamplerAccess[0]);
         this.locations = rawTargets.toArray(new int[0]);
+        this.syncCurrentValues();
         return this.targets.length > 0;
     }
 
@@ -76,11 +77,22 @@ private boolean findUniformTarget(SamplerAccess access) {
         if (access.satin$hasSampler(this.name)) {
             this.targets = new SamplerAccess[] {access};
             this.locations = new int[] {getSamplerLoc(access)};
+            this.syncCurrentValues();
             return true;
         }
         return false;
     }
 
+    private void syncCurrentValues() {
+        Object value = this.cachedValue;
+        if (value != null) { // after the first upload
+            this.cachedValue = null;
+            this.set(value);
+        }
+    }
+
+    protected abstract void set(Object value);
+
     @Override
     public void setDirect(int activeTexture) {
         int length = this.locations.length;

src/main/java/ladysnake/satin/impl/ManagedSamplerUniformV1.java

@@ -41,7 +41,8 @@ public void set(int textureName) {
         this.set((Object)textureName);
     }
 
-    private void set(Object value) {
+    @Override
+    protected void set(Object value) {
         SamplerAccess[] targets = this.targets;
         if (targets.length > 0 && this.cachedValue != value) {
             for (SamplerAccess target : targets) {

src/main/java/ladysnake/satin/impl/ManagedSamplerUniformV2.java

@@ -44,6 +44,11 @@ public void set(int textureName) {
         set(() -> textureName);
     }
 
+    @Override
+    protected void set(Object value) {
+        this.set((IntSupplier) value);
+    }
+
     @Override
     public void set(IntSupplier value) {
         SamplerAccess[] targets = this.targets;