world iterator: don't use abstract iterator

Iterator<BlockPosition> allocates a boxed int (BlockPosition). This removes the whole benefit of the inlined datatype, allocates useless temporary memory and is potentially a lot slower.
This commit is contained in:
Moritz Zwerger 2025-04-11 14:10:14 +02:00
parent 728ca1ce2f
commit f96047dbfe
No known key found for this signature in database
GPG Key ID: 5CAD791931B09AC4

View File

@ -22,12 +22,12 @@ import de.bixilon.minosoft.data.registries.blocks.shapes.collision.context.Entit
import de.bixilon.minosoft.data.registries.blocks.types.fluid.FluidHolder
import de.bixilon.minosoft.data.registries.blocks.types.properties.shape.collision.CollidableBlock
import de.bixilon.minosoft.data.registries.shapes.aabb.AABB
import de.bixilon.minosoft.data.registries.shapes.aabb.AABBIterator
import de.bixilon.minosoft.data.world.World
import de.bixilon.minosoft.data.world.chunk.chunk.Chunk
import de.bixilon.minosoft.data.world.positions.BlockPosition
class WorldIterator(
private val iterator: Iterator<BlockPosition>,
private val iterator: AABBIterator,
private val world: World,
private var chunk: Chunk? = null,
) : Iterator<BlockPair> {