Decreased spell circle scan time

Decreased spell circle scan time
Removed comment

Author: Stick404

Common/src/main/java/at/petrak/hexcasting/api/casting/circles/CircleExecutionState.java

@@ -18,11 +18,15 @@
 import net.minecraft.server.level.*;
 import net.minecraft.world.item.ItemStack;
 import net.minecraft.world.item.Items;
+import net.minecraft.world.level.ChunkPos;
 import net.minecraft.world.level.block.state.BlockState;
+import net.minecraft.world.level.chunk.ChunkAccess;
+import net.minecraft.world.level.chunk.ChunkStatus;
 import net.minecraft.world.phys.AABB;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.*;
+import java.util.concurrent.ExecutionException;
 
 /**
  * See {@link BlockEntityAbstractImpetus}, this is what's stored in it
@@ -109,29 +113,31 @@ protected CircleExecutionState(BlockPos impetusPos, Direction impetusDir,
         var negativeBlock = new BlockPos.MutableBlockPos();
         var lastBlockPos = new BlockPos.MutableBlockPos();
         BlockPos firstBlock = null;
-
+        HashMap<ChunkPos, ChunkAccess> chunkSet = new HashMap<>();
         while (!todo.isEmpty()) {
-            /*
-            Sophia/Stickia here!
-            This block of code works well enough, it gets all the ICircleComponent that it can. However,
-            this tries to do all the calculating in a single tick, so it can get *really* laggy for larger circles.
-            So, this `while` would likely need to be moved into the #tick method, so it can be spread out over time during start up.
-            Why are these comments here? Likely so Sophia can remember this for after she gets sleep, or if anyone
-            else wants to take up the challenge. As for her system will work, but not be clean lmao
-
-            As brought up by Chloe, the slates *could* change during start up.
-            Meaning someone could just place/remove slate at the same rate of discovery, and have World Ambit for almost no slate
-            cost and generally cheating the system. What would be best would be (somehow) getting the block slate
-            without loading the chunk its self, meaning servers wont have to handle 1000s of chunks being loaded at once
-            and players not cheating the system.
-            But this is easier said than done.
-            */
 
             var pair = todo.pop();
             var enterDir = pair.getFirst();
             var herePos = pair.getSecond();
 
-            BlockState hereBs = level.getLevel().getBlockState(herePos); //FUCK THIS LINE. THIS FUC-
+            BlockState hereBs;
+            var chunkPos = new ChunkPos(herePos);
+
+            if (!chunkSet.containsKey(chunkPos)) {
+                var z = level.getChunkSource().getChunkFuture(chunkPos.x,chunkPos.z, ChunkStatus.EMPTY,true);
+                try {
+                    if (z.get().left().isPresent()){
+                        chunkSet.put(chunkPos,z.get().left().get());
+                        hereBs = z.get().left().get().getBlockState(herePos);
+                    } else {
+                        hereBs = level.getLevel().getBlockState(herePos);
+                    }
+                } catch (InterruptedException | ExecutionException e) {
+                    hereBs = level.getLevel().getBlockState(herePos);
+                }
+            } else {
+                hereBs = chunkSet.get(chunkPos).getBlockState(herePos);
+            }
             if (!(hereBs.getBlock() instanceof ICircleComponent cmp)) {
                 continue;
             }