From: Irtimaled <irtimaled@gmail.com>
Date: Mon, 14 Sep 2020 06:47:32 +0000 (-0700)
Subject: Tidy up constructors on simple boxes
X-Git-Url: https://git.lizzy.rs/?a=commitdiff_plain;h=1e55815755fad05f3c271c7f86ff59f206dc5f96;p=BoundingBoxOutlineReloaded.git

Tidy up constructors on simple boxes
---

diff --git a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBeacon.java b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBeacon.java
index f6e1194..cb285ae 100644
--- a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBeacon.java
+++ b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBeacon.java
@@ -8,15 +8,14 @@ public class BoundingBoxBeacon extends BoundingBoxCuboid {
     private final Coords coords;
     private final int level;
 
-    private BoundingBoxBeacon(Coords coords, Coords minCoords, Coords maxCoords, int level, BoundingBoxType beacon) {
-        super(minCoords, maxCoords, beacon);
+    private BoundingBoxBeacon(Coords coords, Coords minCoords, Coords maxCoords, int level, BoundingBoxType type) {
+        super(minCoords, maxCoords, type);
         this.coords = coords;
         this.level = level;
     }
 
     public static BoundingBoxBeacon from(Coords coords, int level) {
-        BoundingBoxType type = BoundingBoxType.Beacon;
-        return from(coords, level, type);
+        return from(coords, level, BoundingBoxType.Beacon);
     }
 
     public static BoundingBoxBeacon from(Coords coords, int level, BoundingBoxType type) {
diff --git a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBiomeBorder.java b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBiomeBorder.java
index c8f7e5e..63c92e1 100644
--- a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBiomeBorder.java
+++ b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxBiomeBorder.java
@@ -11,7 +11,7 @@ public class BoundingBoxBiomeBorder extends AbstractBoundingBox {
     private final boolean south;
     private final boolean west;
 
-    private BoundingBoxBiomeBorder(Coords coords, boolean north, boolean east, boolean south, boolean west) {
+    public BoundingBoxBiomeBorder(Coords coords, boolean north, boolean east, boolean south, boolean west) {
         super(BoundingBoxType.BiomeBorder);
         this.coords = coords;
         this.north = north;
@@ -20,10 +20,6 @@ public class BoundingBoxBiomeBorder extends AbstractBoundingBox {
         this.west = west;
     }
 
-    public static BoundingBoxBiomeBorder from(Coords coords, boolean north, boolean east, boolean south, boolean west) {
-        return new BoundingBoxBiomeBorder(coords, north, east, south, west);
-    }
-
     @Override
     public Boolean intersectsBounds(int minX, int minZ, int maxX, int maxZ) {
         return coords.getX() >= minX &&
diff --git a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxConduit.java b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxConduit.java
index ecf9d47..4df25b2 100644
--- a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxConduit.java
+++ b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxConduit.java
@@ -13,9 +13,8 @@ public class BoundingBoxConduit extends BoundingBoxSphere {
         this.level = level;
     }
 
-    public static BoundingBoxConduit from(Coords coords, int level) {
-        int radius = 16 * level;
-        return new BoundingBoxConduit(coords, level, radius);
+    public BoundingBoxConduit(Coords coords, int level) {
+        this(coords, level, 16 * level);
     }
 
     @Override
diff --git a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxSlimeChunk.java b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxSlimeChunk.java
index dce0ad4..c7249f2 100644
--- a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxSlimeChunk.java
+++ b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxSlimeChunk.java
@@ -5,11 +5,7 @@ import com.irtimaled.bbor.common.models.BoundingBoxCuboid;
 import com.irtimaled.bbor.common.models.Coords;
 
 public class BoundingBoxSlimeChunk extends BoundingBoxCuboid {
-    private BoundingBoxSlimeChunk(Coords minCoords, Coords maxCoords) {
+    public BoundingBoxSlimeChunk(Coords minCoords, Coords maxCoords) {
         super(minCoords, maxCoords, BoundingBoxType.SlimeChunks);
     }
-
-    public static BoundingBoxSlimeChunk from(Coords minCoords, Coords maxCoords) {
-        return new BoundingBoxSlimeChunk(minCoords, maxCoords);
-    }
 }
diff --git a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxWorldSpawn.java b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxWorldSpawn.java
index d66cdad..7715a42 100644
--- a/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxWorldSpawn.java
+++ b/src/main/java/com/irtimaled/bbor/client/models/BoundingBoxWorldSpawn.java
@@ -5,11 +5,7 @@ import com.irtimaled.bbor.common.models.BoundingBoxCuboid;
 import com.irtimaled.bbor.common.models.Coords;
 
 public class BoundingBoxWorldSpawn extends BoundingBoxCuboid {
-    private BoundingBoxWorldSpawn(Coords minCoords, Coords maxCoords, BoundingBoxType type) {
+    public BoundingBoxWorldSpawn(Coords minCoords, Coords maxCoords, BoundingBoxType type) {
         super(minCoords, maxCoords, type);
     }
-
-    public static BoundingBoxWorldSpawn from(Coords minCoords, Coords maxCoords, BoundingBoxType type) {
-        return new BoundingBoxWorldSpawn(minCoords, maxCoords, type);
-    }
 }
diff --git a/src/main/java/com/irtimaled/bbor/client/providers/BiomeBorderProvider.java b/src/main/java/com/irtimaled/bbor/client/providers/BiomeBorderProvider.java
index 34990ec..7776c77 100644
--- a/src/main/java/com/irtimaled/bbor/client/providers/BiomeBorderProvider.java
+++ b/src/main/java/com/irtimaled/bbor/client/providers/BiomeBorderProvider.java
@@ -93,7 +93,7 @@ public class BiomeBorderProvider implements IBoundingBoxProvider<BoundingBoxBiom
                         boolean south = biomeIds[matchX][matchZ + 1] != biomeId;
                         boolean west = biomeIds[matchX - 1][matchZ] != biomeId;
                         if (north || east || south || west) {
-                            borders.put(coords, BoundingBoxBiomeBorder.from(coords, north, east, south, west));
+                            borders.put(coords, new BoundingBoxBiomeBorder(coords, north, east, south, west));
                         }
                     }
                 }
diff --git a/src/main/java/com/irtimaled/bbor/client/providers/ConduitProvider.java b/src/main/java/com/irtimaled/bbor/client/providers/ConduitProvider.java
index f860a0f..38de01f 100644
--- a/src/main/java/com/irtimaled/bbor/client/providers/ConduitProvider.java
+++ b/src/main/java/com/irtimaled/bbor/client/providers/ConduitProvider.java
@@ -29,7 +29,8 @@ public class ConduitProvider implements IBoundingBoxProvider<BoundingBoxConduit>
             if (blocks == null) return null;
 
             Coords coords = new Coords(conduit.getPos());
-            return BoundingBoxConduit.from(coords, conduit.isActive() ? blocks.size() / 7 : 0);
+            int level = conduit.isActive() ? blocks.size() / 7 : 0;
+            return new BoundingBoxConduit(coords, level);
         });
     }
 }
diff --git a/src/main/java/com/irtimaled/bbor/client/providers/SlimeChunkProvider.java b/src/main/java/com/irtimaled/bbor/client/providers/SlimeChunkProvider.java
index 2b9c600..7899b21 100644
--- a/src/main/java/com/irtimaled/bbor/client/providers/SlimeChunkProvider.java
+++ b/src/main/java/com/irtimaled/bbor/client/providers/SlimeChunkProvider.java
@@ -53,7 +53,7 @@ public class SlimeChunkProvider implements IBoundingBoxProvider<BoundingBoxSlime
                     int chunkZStart = chunkZ << 4;
                     Coords minCoords = new Coords(chunkXStart, 1, chunkZStart);
                     Coords maxCoords = new Coords(chunkXStart + 15, 38, chunkZStart + 15);
-                    slimeChunks.add(BoundingBoxSlimeChunk.from(minCoords, maxCoords));
+                    slimeChunks.add(new BoundingBoxSlimeChunk(minCoords, maxCoords));
                 }
             }
         }
diff --git a/src/main/java/com/irtimaled/bbor/client/providers/WorldSpawnProvider.java b/src/main/java/com/irtimaled/bbor/client/providers/WorldSpawnProvider.java
index b3884c9..bdccbf5 100644
--- a/src/main/java/com/irtimaled/bbor/client/providers/WorldSpawnProvider.java
+++ b/src/main/java/com/irtimaled/bbor/client/providers/WorldSpawnProvider.java
@@ -31,7 +31,7 @@ public class WorldSpawnProvider implements IBoundingBoxProvider<BoundingBoxWorld
         Coords minCoords = new Coords(spawnX - 10, 0, spawnZ - 10);
         Coords maxCoords = new Coords(spawnX + 10, 0, spawnZ + 10);
 
-        return BoundingBoxWorldSpawn.from(minCoords, maxCoords, BoundingBoxType.WorldSpawn);
+        return new BoundingBoxWorldSpawn(minCoords, maxCoords, BoundingBoxType.WorldSpawn);
     }
 
     private static BoundingBoxWorldSpawn buildSpawnChunksBoundingBox(int spawnX, int spawnZ, int size, BoundingBoxType type) {
@@ -46,7 +46,7 @@ public class WorldSpawnProvider implements IBoundingBoxProvider<BoundingBoxWorld
             midZ -= CHUNK_SIZE;
         }
         Coords minCoords = new Coords(midX - midOffset, 0, midZ - midOffset);
-        return BoundingBoxWorldSpawn.from(minCoords, maxCoords, type);
+        return new BoundingBoxWorldSpawn(minCoords, maxCoords, type);
     }
 
     @Override