chore: cleanup

Author: LooFifteen

src/main/java/net/crystalgames/scaffolding/schematic/impl/MCEditSchematic.java

@@ -1,10 +1,5 @@
 package net.crystalgames.scaffolding.schematic.impl;
 
-import java.io.OutputStream;
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Objects;
-import java.util.concurrent.CompletableFuture;
 import net.crystalgames.scaffolding.region.Region;
 import net.crystalgames.scaffolding.schematic.Schematic;
 import net.minestom.server.coordinate.Pos;
@@ -16,6 +11,12 @@
 import org.jglrxavpok.hephaistos.nbt.NBTCompound;
 import org.jglrxavpok.hephaistos.nbt.NBTException;
 
+import java.io.OutputStream;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Objects;
+import java.util.concurrent.CompletableFuture;
+
 // https://github.com/EngineHub/WorldEdit/blob/version/5.x/src/main/java/com/sk89q/worldedit/schematic/MCEditSchematicFormat.java
 public class MCEditSchematic implements Schematic {
 
@@ -128,12 +129,10 @@ public CompletableFuture<Region> build(Instance instance, Pos position) {
                 short stateId = regionBlock.stateId();
 
                 Block block = Block.fromStateId(stateId);
-                if (block != null) {
-                    blockBatch.setBlock(blockPosition.add(position), block);
-                }
+                if (block != null) blockBatch.setBlock(blockPosition.add(position), block);
             }
 
-            blockBatch.apply(instance, () -> future.complete(new Region(instance, position, position)));
+            blockBatch.apply(instance, () -> future.complete(new Region(instance, position, position.add(width, height, length))));
         });
         return future;
     }

src/main/java/net/crystalgames/scaffolding/schematic/impl/SpongeSchematic.java

@@ -12,12 +12,7 @@
 import org.jglrxavpok.hephaistos.nbt.NBTException;
 
 import java.io.OutputStream;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.LinkedHashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
+import java.util.*;
 import java.util.concurrent.CompletableFuture;
 
 // https://github.com/EngineHub/WorldEdit/blob/303f5a76b2df70d63480f2126c9ef4b228eb3c59/worldedit-core/src/main/java/com/sk89q/worldedit/extent/clipboard/io/SpongeSchematicReader.java#L261-L297
@@ -85,8 +80,7 @@ private void readBlockPalette(@NotNull NBTCompound nbtTag) throws NBTException {
         if (nbtPalette == null) throw new NBTException("Invalid Schematic: No Palette");
 
         Set<String> keys = nbtPalette.getKeys();
-        if (keys.size() != maxPalette)
-            throw new NBTException("Invalid Schematic: PaletteMax does not match Palette size");
+        if (keys.size() != maxPalette) throw new NBTException("Invalid Schematic: PaletteMax does not match Palette size");
 
         for (String key : keys) {
             Integer value = nbtPalette.getInt(key);
@@ -155,11 +149,9 @@ public CompletableFuture<Region> build(Instance instance, Pos position) {
                 short stateId = regionBlock.stateId();
 
                 Block block = Block.fromStateId(stateId);
-                if (block != null) {
-                    blockBatch.setBlock(blockPosition.add(position), block);
-                }
+                if (block != null) blockBatch.setBlock(blockPosition.add(position), block);
             }
-            blockBatch.apply(instance, () -> future.complete(new Region(instance, position, position)));
+            blockBatch.apply(instance, () -> future.complete(new Region(instance, position, position.add(width, height, length))));
         });
         return future;
     }