From a78732239c90185cffc9793b49b6197dcb889dcd Mon Sep 17 00:00:00 2001
From: IMS212 <ims@irisshaders.dev>
Date: Sat, 30 Mar 2024 11:12:19 -0700
Subject: [PATCH] Minor shadow fixes

---
 build.gradle.kts                                       |  3 ++-
 src/main/java/net/irisshaders/iris/Iris.java           |  4 ++--
 .../irisshaders/iris/gui/screen/ShaderPackScreen.java  |  4 ++--
 .../iris/pipeline/IrisRenderingPipeline.java           | 10 ++--------
 4 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/build.gradle.kts b/build.gradle.kts
index 616a7fc8f2..98d6a41182 100644
--- a/build.gradle.kts
+++ b/build.gradle.kts
@@ -1,4 +1,5 @@
 import org.ajoberstar.grgit.Grgit
+import org.gradle.nativeplatform.platform.internal.DefaultNativePlatform
 
 object Constants {
     // https://fabricmc.net/develop/
@@ -169,7 +170,7 @@ dependencies {
 
 tasks {
     runClient {
-        if (Constants.ACTIVATE_RENDERDOC) {
+        if (Constants.ACTIVATE_RENDERDOC && DefaultNativePlatform.getCurrentOperatingSystem().isLinux) {
             environment("LD_PRELOAD", "/usr/lib/librenderdoc.so")
         }
     }
diff --git a/src/main/java/net/irisshaders/iris/Iris.java b/src/main/java/net/irisshaders/iris/Iris.java
index c3aef6a437..d5d4291148 100644
--- a/src/main/java/net/irisshaders/iris/Iris.java
+++ b/src/main/java/net/irisshaders/iris/Iris.java
@@ -650,9 +650,9 @@ public static String getFormattedVersion() {
 		ChatFormatting color;
 		String version = getVersion();
 
-		if (version.endsWith("-development-environment")) {
+		if (FabricLoader.getInstance().isDevelopmentEnvironment()) {
 			color = ChatFormatting.GOLD;
-			version = version.replace("-development-environment", " (Development Environment)");
+			version = version + " (Development Environment)";
 		} else if (version.endsWith("-dirty") || version.contains("unknown") || version.endsWith("-nogit")) {
 			color = ChatFormatting.RED;
 		} else if (version.contains("+rev.")) {
diff --git a/src/main/java/net/irisshaders/iris/gui/screen/ShaderPackScreen.java b/src/main/java/net/irisshaders/iris/gui/screen/ShaderPackScreen.java
index 3e93bed5c1..c33fc42296 100644
--- a/src/main/java/net/irisshaders/iris/gui/screen/ShaderPackScreen.java
+++ b/src/main/java/net/irisshaders/iris/gui/screen/ShaderPackScreen.java
@@ -1,6 +1,7 @@
 package net.irisshaders.iris.gui.screen;
 
 import com.mojang.blaze3d.platform.InputConstants;
+import net.fabricmc.loader.api.FabricLoader;
 import net.irisshaders.iris.Iris;
 import net.irisshaders.iris.api.v0.IrisApi;
 import net.irisshaders.iris.gui.GuiUtil;
@@ -80,9 +81,8 @@ public ShaderPackScreen(Screen parent) {
 
 		String irisName = Iris.MODNAME + " " + Iris.getVersion();
 
-		if (irisName.contains("-development-environment")) {
+		if (FabricLoader.getInstance().isDevelopmentEnvironment()) {
 			this.developmentComponent = Component.literal("Development Environment").withStyle(ChatFormatting.GOLD);
-			irisName = irisName.replace("-development-environment", "");
 		}
 
 		this.irisTextComponent = Component.literal(irisName).withStyle(ChatFormatting.GRAY);
diff --git a/src/main/java/net/irisshaders/iris/pipeline/IrisRenderingPipeline.java b/src/main/java/net/irisshaders/iris/pipeline/IrisRenderingPipeline.java
index 8ba2463699..431a65b009 100644
--- a/src/main/java/net/irisshaders/iris/pipeline/IrisRenderingPipeline.java
+++ b/src/main/java/net/irisshaders/iris/pipeline/IrisRenderingPipeline.java
@@ -776,17 +776,11 @@ public void addGbufferOrShadowSamplers(SamplerHolder samplers, ImageHolder image
 		IrisSamplers.addCustomImages(samplerHolder, customImages);
 
 		if (IrisSamplers.hasShadowSamplers(samplerHolder)) {
-			if (!isShadowPass) {
-				shadowTargetsSupplier.get();
-			}
-
-			IrisSamplers.addShadowSamplers(samplerHolder, Objects.requireNonNull(shadowRenderTargets), null, separateHardwareSamplers);
+			IrisSamplers.addShadowSamplers(samplerHolder, shadowTargetsSupplier.get(), null, separateHardwareSamplers);
 		}
 
 		if (isShadowPass || IrisImages.hasShadowImages(images)) {
-			// Note: hasShadowSamplers currently queries for shadow images too, so the shadow render targets will be
-			// created by this point... that's sorta ugly, though.
-			IrisImages.addShadowColorImages(images, Objects.requireNonNull(shadowRenderTargets), null);
+			IrisImages.addShadowColorImages(images, shadowTargetsSupplier.get(), null);
 		}
 	}