Some formatting

Author: lowercasebtw

src/main/java/org/visuals/legacy/animatium/mixins/v1/entity/flame/MixinFlameFeatureRenderer.java

@@ -43,7 +43,7 @@
 
 @Mixin(FlameFeatureRenderer.class)
 public abstract class MixinFlameFeatureRenderer {
-    @ModifyExpressionValue(method = "renderFlame", at = @At(value = "FIELD", opcode = Opcodes.GETFIELD, target = "Lnet/minecraft/client/renderer/entity/state/EntityRenderState;boundingBoxWidth:F"))
+    @ModifyExpressionValue(method = "renderFlame", at = @At(value = "FIELD", target = "Lnet/minecraft/client/renderer/entity/state/EntityRenderState;boundingBoxWidth:F", opcode = Opcodes.GETFIELD))
     private float animatium$flameWidth(float original, @Local(argsOnly = true) EntityRenderState entityRenderState) {
         if (Animatium.isEnabled() && AnimatiumConfig.instance().other.flameDimensions && entityRenderState instanceof AvatarRenderState) {
             return 0.6F;
@@ -52,7 +52,7 @@ public abstract class MixinFlameFeatureRenderer {
         }
     }
 
-    @ModifyExpressionValue(method = "renderFlame", at = @At(value = "FIELD", opcode = Opcodes.GETFIELD, target = "Lnet/minecraft/client/renderer/entity/state/EntityRenderState;boundingBoxHeight:F"))
+    @ModifyExpressionValue(method = "renderFlame", at = @At(value = "FIELD", target = "Lnet/minecraft/client/renderer/entity/state/EntityRenderState;boundingBoxHeight:F", opcode = Opcodes.GETFIELD))
     private float animatium$flameHeight(float original, @Local(argsOnly = true) EntityRenderState entityRenderState) {
         if (Animatium.isEnabled() && AnimatiumConfig.instance().other.flameDimensions && entityRenderState instanceof AvatarRenderState) {
             return 1.8F;

src/main/java/org/visuals/legacy/animatium/mixins/v1/entity/projectiles/age/MixinEyeOfEnder.java

@@ -36,7 +36,7 @@
 
 @Mixin(EyeOfEnder.class)
 public abstract class MixinEyeOfEnder {
-    @WrapOperation(method = "shouldRenderAtSqrDistance", at = @At(value = "FIELD", opcode = Opcodes.GETFIELD, target = "Lnet/minecraft/world/entity/projectile/EyeOfEnder;tickCount:I"))
+    @WrapOperation(method = "shouldRenderAtSqrDistance", at = @At(value = "FIELD", target = "Lnet/minecraft/world/entity/projectile/EyeOfEnder;tickCount:I", opcode = Opcodes.GETFIELD))
     private int animatium$projectileAgeCheck(EyeOfEnder instance, Operation<Integer> original) {
         final int originalTick = original.call(instance);
         if (Animatium.isEnabled() && !AnimatiumConfig.instance().other.projectileAgeCheck) {

src/main/java/org/visuals/legacy/animatium/mixins/v1/rendering/lighting/MixinGeometryHelper_FastSmoothLighting.java

@@ -25,6 +25,7 @@
 
 package org.visuals.legacy.animatium.mixins.v1.rendering.lighting;
 
+import com.moulberry.mixinconstraints.annotations.IfModLoaded;
 import net.fabricmc.fabric.api.renderer.v1.mesh.QuadView;
 import net.fabricmc.fabric.impl.client.indigo.renderer.helper.GeometryHelper;
 import net.minecraft.core.Direction;
@@ -36,14 +37,13 @@
 import org.visuals.legacy.animatium.config.AnimatiumConfig;
 
 // Indigo Renderer Support
+@IfModLoaded("fabric")
 @Mixin(value = GeometryHelper.class, remap = false)
 public abstract class MixinGeometryHelper_FastSmoothLighting {
-
 	@Inject(method = "isQuadCubic", at = @At("HEAD"), cancellable = true)
 	private static void animatium$oldFastSmoothLighting(Direction lightFace, QuadView quad, CallbackInfoReturnable<Boolean> cir) {
 		if (Animatium.isEnabled() && AnimatiumConfig.instance().other.oldMinimumSmoothLighting) {
 			cir.setReturnValue(true);
 		}
 	}
-
 }

src/main/java/org/visuals/legacy/animatium/mixins/v1/rendering/lighting/MixinModelBlockRenderer_FastSmoothLighting.java

@@ -36,13 +36,12 @@
 
 @Mixin(ModelBlockRenderer.AmbientOcclusionRenderStorage.class)
 public abstract class MixinModelBlockRenderer_FastSmoothLighting {
-    // TODO: Figure out why expressions doesn't work for this
-    @WrapOperation(method = "calculate", at = @At(value = "FIELD", target = "Lnet/minecraft/client/renderer/block/ModelBlockRenderer$AmbientOcclusionRenderStorage;facePartial:Z", opcode = Opcodes.GETFIELD))
-    private boolean animatium$oldFastSmoothLighting(ModelBlockRenderer.AmbientOcclusionRenderStorage instance, Operation<Boolean> original) {
-        if (Animatium.isEnabled() && AnimatiumConfig.instance().other.oldMinimumSmoothLighting) {
-            return false;
-        } else {
-            return original.call(instance);
-        }
-    }
+	@WrapOperation(method = "calculate", at = @At(value = "FIELD", target = "Lnet/minecraft/client/renderer/block/ModelBlockRenderer$AmbientOcclusionRenderStorage;facePartial:Z", opcode = Opcodes.GETFIELD))
+	private boolean animatium$oldFastSmoothLighting(ModelBlockRenderer.AmbientOcclusionRenderStorage instance, Operation<Boolean> original) {
+		if (Animatium.isEnabled() && AnimatiumConfig.instance().other.oldMinimumSmoothLighting) {
+			return false;
+		} else {
+			return original.call(instance);
+		}
+	}
 }

src/main/java/org/visuals/legacy/animatium/mixins/v1/rendering/lighting/MixinSmoothLightPipeline.java

@@ -0,0 +1,46 @@
+/**
+ * Animatium
+ * The all-you-could-want legacy animations mod for modern minecraft versions.
+ * Brings back animations from the 1.7/1.8 era and more.
+ * <p>
+ * Copyright (C) 2024-2025 lowercasebtw
+ * Copyright (C) 2024-2025 mixces
+ * Copyright (C) 2024-2025 Contributors to the project retain their copyright
+ * <p>
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ * <p>
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ * <p>
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * <p>
+ * "MINECRAFT" LINKING EXCEPTION TO THE GPL
+ */
+
+package org.visuals.legacy.animatium.mixins.v1.rendering.lighting;
+
+import com.moulberry.mixinconstraints.annotations.IfModLoaded;
+import org.spongepowered.asm.mixin.Mixin;
+
+// TODO/FIX
+// Sodium Support
+@IfModLoaded("sodium")
+@Mixin(targets = "net.caffeinemc.mods.sodium.client.model.light.smooth.SmoothLightPipeline", remap = false)
+public abstract class MixinSmoothLightPipeline {
+	/*@Definition(id = "flags", local = @Local(type = int.class, name = "flags"))
+	@Expression("(flags & 1) == 0")
+	@ModifyExpressionValue(method = "calculate", at = @At("MIXINEXTRAS:EXPRESSION"), remap = false)
+	private boolean animatium$oldFastSmoothLighting(final boolean original) {
+		if (Animatium.isEnabled() && AnimatiumConfig.instance().other.oldMinimumSmoothLighting) {
+			return true;
+		} else {
+			return original;
+		}
+	}*/
+}

src/main/resources/animatium.mixins.json

@@ -115,6 +115,7 @@
 		"v1.rendering.items.usage.MixinMinecraft",
 		"v1.rendering.lighting.MixinGeometryHelper_FastSmoothLighting",
 		"v1.rendering.lighting.MixinModelBlockRenderer_FastSmoothLighting",
+		"v1.rendering.lighting.MixinSmoothLightPipeline",
 		"v1.rendering.min_transparency.MixinSpriteContents_FixTransparencyLimit",
 		"v1.rendering.outlines.MixinCompositeRenderType_BlockOutlineLineWidth",
 		"v1.rendering.outlines.MixinGameRenderer_PersistentBlockOutline",