Explain new changes in BedrockMovePlayerTranslator

connector/src/main/java/org/geysermc/connector/network/translators/bedrock/entity/player/BedrockMovePlayerTranslator.java

@@ -117,11 +117,13 @@ public class BedrockMovePlayerTranslator extends PacketTranslator<MovePlayerPack
 
                         if (notMovingUp) {
                             int floorY = position.getFloorY();
-                            boolean overworld = session.getChunkCache().isExtendedHeight();
+                            // If the client believes the world has extended height, then it also believes the void floor
-                            if (floorY <= (overworld ? BEDROCK_OVERWORLD_VOID_FLOOR_LOWER_Y : -38)
+                            // still exists, just at a lower spot
-                                    && floorY >= (overworld ? BEDROCK_OVERWORLD_VOID_FLOOR_UPPER_Y : -40)) {
+                            boolean extendedWorld = session.getChunkCache().isExtendedHeight();
-                                // Work around there being a floor at Y -40 (Y - and teleport the player below it
+                            if (floorY <= (extendedWorld ? BEDROCK_OVERWORLD_VOID_FLOOR_LOWER_Y : -38)
-                                // Moving from below Y -40 to above the void floor works fine
+                                    && floorY >= (extendedWorld ? BEDROCK_OVERWORLD_VOID_FLOOR_UPPER_Y : -40)) {
+                                // Work around there being a floor at the bottom of the world and teleport the player below it
+                                // Moving from below to above the void floor works fine
                                 entity.setPosition(entity.getPosition().sub(0, 4f, 0));
                                 MovePlayerPacket movePlayerPacket = new MovePlayerPacket();
                                 movePlayerPacket.setRuntimeEntityId(entity.getGeyserId());