diff options
Diffstat (limited to 'patches/server/0343-Don-t-move-existing-players-to-world-spawn.patch')
-rw-r--r-- | patches/server/0343-Don-t-move-existing-players-to-world-spawn.patch | 48 |
1 files changed, 48 insertions, 0 deletions
diff --git a/patches/server/0343-Don-t-move-existing-players-to-world-spawn.patch b/patches/server/0343-Don-t-move-existing-players-to-world-spawn.patch new file mode 100644 index 0000000000..d81ed62207 --- /dev/null +++ b/patches/server/0343-Don-t-move-existing-players-to-world-spawn.patch @@ -0,0 +1,48 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Aikar <[email protected]> +Date: Thu, 9 Apr 2020 21:20:33 -0400 +Subject: [PATCH] Don't move existing players to world spawn + +This can cause a nasty server lag the spawn chunks are not kept loaded +or they aren't finished loading yet, or if the world spawn radius is +larger than the keep loaded range. + +By skipping this, we avoid potential for a large spike on server start. + +== AT == +public net.minecraft.server.level.ServerPlayer fudgeSpawnLocation(Lnet/minecraft/server/level/ServerLevel;)V + +diff --git a/src/main/java/net/minecraft/server/level/ServerPlayer.java b/src/main/java/net/minecraft/server/level/ServerPlayer.java +index 74e8aef45702eef0dcabc7841a7c7125cc066cbe..618efe10752c968142fcc0f28a533d3515ef3367 100644 +--- a/src/main/java/net/minecraft/server/level/ServerPlayer.java ++++ b/src/main/java/net/minecraft/server/level/ServerPlayer.java +@@ -352,7 +352,7 @@ public class ServerPlayer extends Player { + this.server = server; + this.stats = server.getPlayerList().getPlayerStats(this); + this.advancements = server.getPlayerList().getPlayerAdvancements(this); +- this.fudgeSpawnLocation(world); ++ // this.fudgeSpawnLocation(world); // Paper - Don't move existing players to world spawn + this.updateOptions(clientOptions); + this.object = null; + +@@ -611,7 +611,7 @@ public class ServerPlayer extends Player { + position = Vec3.atCenterOf(world.getSharedSpawnPos()); + } + this.setLevel(world); +- this.setPos(position); ++ this.setPosRaw(position.x(), position.y(), position.z()); // Paper - don't register to chunks yet + } + this.gameMode.setLevel((ServerLevel) world); + } +diff --git a/src/main/java/net/minecraft/server/players/PlayerList.java b/src/main/java/net/minecraft/server/players/PlayerList.java +index 8ce2826b1ded161a8220273d1fc5260ed3b6772f..d2f94afe667627a14cff896b463c2b9b51bf6e29 100644 +--- a/src/main/java/net/minecraft/server/players/PlayerList.java ++++ b/src/main/java/net/minecraft/server/players/PlayerList.java +@@ -226,6 +226,7 @@ public abstract class PlayerList { + // Paper start - Entity#getEntitySpawnReason + if (optional.isEmpty()) { + player.spawnReason = org.bukkit.event.entity.CreatureSpawnEvent.SpawnReason.DEFAULT; // set Player SpawnReason to DEFAULT on first login ++ player.fudgeSpawnLocation(worldserver1); // Paper - Don't move existing players to world spawn + } + // Paper end - Entity#getEntitySpawnReason + player.setServerLevel(worldserver1); |