Refactor RenderHelper

1 files changed, 29 insertions, 37 deletions

diff --git a/src/main/java/me/xmrvizzy/skyblocker/utils/RenderHelper.java b/src/main/java/me/xmrvizzy/skyblocker/utils/RenderHelper.java
index 3c3e30d1..2e942905 100644
--- a/src/main/java/me/xmrvizzy/skyblocker/utils/RenderHelper.java
+++ b/src/main/java/me/xmrvizzy/skyblocker/utils/RenderHelper.java
@@ -10,41 +10,33 @@ import net.minecraft.util.math.Vec3d;
 import java.awt.*;
 
 public class RenderHelper {
-	private static final Vec3d DIMENSIONS = new Vec3d(1, 1, 1);
-
-	public static void renderFilledThroughWallsWithBeaconBeam(WorldRenderContext context, BlockPos pos,
-			float[] colorComponents, float alpha) {
-		renderFilledThroughWalls(context, pos, colorComponents, alpha);
-		renderBeaconBeam(context, pos, colorComponents);
-	}
-
-	public static void renderFilledThroughWalls(WorldRenderContext context, BlockPos pos, float[] colorComponents,
-			float alpha) {
-		Renderer3d.renderThroughWalls();
-		renderFilled(context, pos, colorComponents, alpha);
-		Renderer3d.stopRenderThroughWalls();
-	}
-
-	public static void renderFilledIfVisible(WorldRenderContext context, BlockPos pos, float[] colorComponents,
-			float alpha) {
-		if (FrustumUtils.isVisible(pos.getX(), pos.getY(), pos.getZ(), pos.getX() + 1, pos.getY() + 1,
-				pos.getZ() + 1)) {
-			renderFilled(context, pos, colorComponents, alpha);
-		}
-	}
-
-	public static void renderFilled(WorldRenderContext context, BlockPos pos, float[] colorComponents, float alpha) {
-		Renderer3d.renderFilled(context.matrixStack(),
-				new Color(colorComponents[0], colorComponents[1], colorComponents[2], alpha), Vec3d.of(pos),
-				DIMENSIONS);
-	}
-
-	public static void renderBeaconBeam(WorldRenderContext context, BlockPos pos, float[] colorComponents) {
-		context.matrixStack().push();
-		context.matrixStack().translate(pos.getX() - context.camera().getPos().x,
-				pos.getY() - context.camera().getPos().y, pos.getZ() - context.camera().getPos().z);
-		BeaconBlockEntityRendererInvoker.renderBeam(context.matrixStack(), context.consumers(), context.tickDelta(),
-				context.world().getTime(), 0, BeaconBlockEntityRenderer.MAX_BEAM_HEIGHT, colorComponents);
-		context.matrixStack().pop();
-	}
+    private static final Vec3d ONE = new Vec3d(1, 1, 1);
+
+    public static void renderFilledThroughWallsWithBeaconBeam(WorldRenderContext context, BlockPos pos, float[] colorComponents, float alpha) {
+        renderFilledThroughWalls(context, pos, colorComponents, alpha);
+        renderBeaconBeam(context, pos, colorComponents);
+    }
+
+    public static void renderFilledThroughWalls(WorldRenderContext context, BlockPos pos, float[] colorComponents, float alpha) {
+        Renderer3d.renderThroughWalls();
+        renderFilled(context, pos, colorComponents, alpha);
+        Renderer3d.stopRenderThroughWalls();
+    }
+
+    public static void renderFilledIfVisible(WorldRenderContext context, BlockPos pos, float[] colorComponents, float alpha) {
+        if (FrustumUtils.isVisible(pos.getX(), pos.getY(), pos.getZ(), pos.getX() + 1, pos.getY() + 1, pos.getZ() + 1)) {
+            renderFilled(context, pos, colorComponents, alpha);
+        }
+    }
+
+    public static void renderFilled(WorldRenderContext context, BlockPos pos, float[] colorComponents, float alpha) {
+        Renderer3d.renderFilled(context.matrixStack(), new Color(colorComponents[0], colorComponents[1], colorComponents[2], alpha), Vec3d.of(pos), ONE);
+    }
+
+    public static void renderBeaconBeam(WorldRenderContext context, BlockPos pos, float[] colorComponents) {
+        context.matrixStack().push();
+        context.matrixStack().translate(pos.getX() - context.camera().getPos().x, pos.getY() - context.camera().getPos().y, pos.getZ() - context.camera().getPos().z);
+        BeaconBlockEntityRendererInvoker.renderBeam(context.matrixStack(), context.consumers(), context.tickDelta(), context.world().getTime(), 0, BeaconBlockEntityRenderer.MAX_BEAM_HEIGHT, colorComponents);
+        context.matrixStack().pop();
+    }
 }