From a90092e7580b32293891acee1f9426f4d2a9ec36 Mon Sep 17 00:00:00 2001 From: Jake Potrebic Date: Wed, 25 May 2022 12:21:51 -0700 Subject: [PATCH] Updated Upstream (CraftBukkit/Spigot) (#7848) Upstream has released updates that appear to apply and compile correctly. This update has not been tested by PaperMC and as with ANY update, please do your own testing CraftBukkit Changes: 95d233dc SPIGOT-1807: PlayerTeleportEvent is called twice Spigot Changes: b0819150 PR-116: Slightly reduce item merge radius in Y-direction ef2aa765 PR-119: Fix off-by-one and allow disabling max-tnt-per-tick option --- ...-Avoid-item-merge-if-stack-size-above-max-stack-size.patch | 4 ++-- ...Optimise-collision-checking-in-player-move-packet-ha.patch | 4 ++-- work/CraftBukkit | 2 +- work/Spigot | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/patches/server/0217-Avoid-item-merge-if-stack-size-above-max-stack-size.patch b/patches/server/0217-Avoid-item-merge-if-stack-size-above-max-stack-size.patch index c0afe840a..b0d783148 100644 --- a/patches/server/0217-Avoid-item-merge-if-stack-size-above-max-stack-size.patch +++ b/patches/server/0217-Avoid-item-merge-if-stack-size-above-max-stack-size.patch @@ -5,7 +5,7 @@ Subject: [PATCH] Avoid item merge if stack size above max stack size diff --git a/src/main/java/net/minecraft/world/entity/item/ItemEntity.java b/src/main/java/net/minecraft/world/entity/item/ItemEntity.java -index 943c2bd2a0836eda5b4da382c459ade6cbe40fea..b374d50678e92fabbf1d640729a8a276fc791c62 100644 +index 430809523d02801088c247e11bce351a99b8bc05..df5064e69f10bb366572e5f1da197514f3067faf 100644 --- a/src/main/java/net/minecraft/world/entity/item/ItemEntity.java +++ b/src/main/java/net/minecraft/world/entity/item/ItemEntity.java @@ -222,6 +222,10 @@ public class ItemEntity extends Entity { @@ -18,4 +18,4 @@ index 943c2bd2a0836eda5b4da382c459ade6cbe40fea..b374d50678e92fabbf1d640729a8a276 + // Paper end // Spigot start double radius = level.spigotConfig.itemMerge; - List list = this.level.getEntitiesOfClass(ItemEntity.class, this.getBoundingBox().inflate(radius, radius, radius), (entityitem) -> { + List list = this.level.getEntitiesOfClass(ItemEntity.class, this.getBoundingBox().inflate(radius, radius - 0.5D, radius), (entityitem) -> { diff --git a/patches/server/0808-Optimise-collision-checking-in-player-move-packet-ha.patch b/patches/server/0808-Optimise-collision-checking-in-player-move-packet-ha.patch index f9c026528..32c30b0f2 100644 --- a/patches/server/0808-Optimise-collision-checking-in-player-move-packet-ha.patch +++ b/patches/server/0808-Optimise-collision-checking-in-player-move-packet-ha.patch @@ -6,7 +6,7 @@ Subject: [PATCH] Optimise collision checking in player move packet handling Move collision logic to just the hasNewCollision call instead of getCubes + hasNewCollision diff --git a/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java b/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java -index 7be8b98a61c3d92ef7c473983fe620047a137630..761a87be91e82841632f4f46dec799ae28f05887 100644 +index cc0adbd20bc069e61fc333e57c2e3ccbf8d5c888..fb15e7ec56b91541a4bff810d45edec57adfaeab 100644 --- a/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java +++ b/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java @@ -587,7 +587,7 @@ public class ServerGamePacketListenerImpl implements ServerPlayerConnection, Ser @@ -137,7 +137,7 @@ index 7be8b98a61c3d92ef7c473983fe620047a137630..761a87be91e82841632f4f46dec799ae + } // else: no collision at all detected, why do we care? + } + if (!this.player.noPhysics && !this.player.isSleeping() && teleportBack) { // Paper end - optimise out extra getCubes - this.teleport(d3, d4, d5, f, f1); + this.internalTeleport(d3, d4, d5, f, f1, Collections.emptySet(), false); // CraftBukkit - SPIGOT-1807: Don't call teleport event, when the client thinks the player is falling, because the chunks are not loaded on the client yet. } else { // CraftBukkit start - fire PlayerMoveEvent @@ -1496,6 +1542,27 @@ public class ServerGamePacketListenerImpl implements ServerPlayerConnection, Ser diff --git a/work/CraftBukkit b/work/CraftBukkit index d48f2d1a5..95d233dc5 160000 --- a/work/CraftBukkit +++ b/work/CraftBukkit @@ -1 +1 @@ -Subproject commit d48f2d1a508937908a7fb9169d29f53f1b34ab76 +Subproject commit 95d233dc5b5ab0bf125ce761ace782bc2f1ac78e diff --git a/work/Spigot b/work/Spigot index b6d12d17c..b08191500 160000 --- a/work/Spigot +++ b/work/Spigot @@ -1 +1 @@ -Subproject commit b6d12d17cd1763d488a42cb9f3e6c424de5f12a8 +Subproject commit b0819150028ec7233960ec622a4b1ba57f102047