From 9464a9b5603387d9991f4579450033fd0ef7bd79 Mon Sep 17 00:00:00 2001 From: Jamalam Date: Wed, 18 Dec 2024 17:35:07 +0000 Subject: [PATCH] fix: use NullOutputStream.NULL_OUTPUT_STREAM rather than NullOutputStream.INSTANCE Fixes an issue I was encountering where NullOutputStream.INSTANCE did not exist - unsure why it did not, but this change doesn't change anything since INSTANCE and NULL_OUTPUT_STREAM are the same, and other parts of the code base already use NULL_OUTPUT_STREAM --- .../dev/architectury/loom/forge/tool/ForgeToolExecutor.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/dev/architectury/loom/forge/tool/ForgeToolExecutor.java b/src/main/java/dev/architectury/loom/forge/tool/ForgeToolExecutor.java index ce3a58c0a..c793cd236 100644 --- a/src/main/java/dev/architectury/loom/forge/tool/ForgeToolExecutor.java +++ b/src/main/java/dev/architectury/loom/forge/tool/ForgeToolExecutor.java @@ -77,13 +77,13 @@ public static ExecResult exec(ExecOperations execOperations, Settings settings) if (settings.getShowVerboseStdout().get()) { spec.setStandardOutput(System.out); } else { - spec.setStandardOutput(NullOutputStream.INSTANCE); + spec.setStandardOutput(NullOutputStream.NULL_OUTPUT_STREAM); } if (settings.getShowVerboseStderr().get()) { spec.setErrorOutput(System.err); } else { - spec.setErrorOutput(NullOutputStream.INSTANCE); + spec.setErrorOutput(NullOutputStream.NULL_OUTPUT_STREAM); } }); }