aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0312-Optimize-Captured-TileEntity-Lookup.patch
diff options
context:
space:
mode:
authorNassim Jahnke <[email protected]>2022-10-02 09:56:36 +0200
committerGitHub <[email protected]>2022-10-02 09:56:36 +0200
commit928bcc8d3a058221146cea1de7d42d7e178e78f2 (patch)
treefdb518a8812097f983c41d36ea693baf5c208554 /patches/server/0312-Optimize-Captured-TileEntity-Lookup.patch
parentec3cfa9b7f147cf097692144f665893748fadc3b (diff)
downloadPaper-928bcc8d3a058221146cea1de7d42d7e178e78f2.tar.gz
Paper-928bcc8d3a058221146cea1de7d42d7e178e78f2.zip
Updated Upstream (Bukkit/CraftBukkit) (#8430)
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 Bukkit Changes: 09943450 Update SnakeYAML version 5515734f SPIGOT-7162: Incorrect description for Entity#getVehicle javadoc 6f82b381 PR-788: Add getHand() to all relevant events CraftBukkit Changes: aaf484f6f SPIGOT-7163: CraftMerchantRecipe doesn't copy demand and specialPrice from BukkitMerchantRecipe 5329dd6fd PR-1107: Add getHand() to all relevant events 93061706e SPIGOT-7045: Ocelots never spawn with babies with spawn reason OCELOT_BABY
Diffstat (limited to 'patches/server/0312-Optimize-Captured-TileEntity-Lookup.patch')
-rw-r--r--patches/server/0312-Optimize-Captured-TileEntity-Lookup.patch30
1 files changed, 30 insertions, 0 deletions
diff --git a/patches/server/0312-Optimize-Captured-TileEntity-Lookup.patch b/patches/server/0312-Optimize-Captured-TileEntity-Lookup.patch
new file mode 100644
index 0000000000..2aea08c36b
--- /dev/null
+++ b/patches/server/0312-Optimize-Captured-TileEntity-Lookup.patch
@@ -0,0 +1,30 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Aikar <[email protected]>
+Date: Sat, 6 Apr 2019 10:16:48 -0400
+Subject: [PATCH] Optimize Captured TileEntity Lookup
+
+upstream was doing a containsKey/get pattern, and always doing it at that.
+that scenario is only even valid if were in the middle of a block place.
+
+Optimize to check if the captured list even has values in it, and also to
+just do a get call since the value can never be null.
+
+diff --git a/src/main/java/net/minecraft/world/level/Level.java b/src/main/java/net/minecraft/world/level/Level.java
+index 1a474fb88dc1447fb754e8ad936ab6add470359c..cb7e92f2694cefeecebc840959a01e1e1945061a 100644
+--- a/src/main/java/net/minecraft/world/level/Level.java
++++ b/src/main/java/net/minecraft/world/level/Level.java
+@@ -845,9 +845,12 @@ public abstract class Level implements LevelAccessor, AutoCloseable {
+
+ @Nullable
+ public BlockEntity getBlockEntity(BlockPos blockposition, boolean validate) {
+- if (this.capturedTileEntities.containsKey(blockposition)) {
+- return this.capturedTileEntities.get(blockposition);
++ // Paper start - Optimize capturedTileEntities lookup
++ net.minecraft.world.level.block.entity.BlockEntity blockEntity;
++ if (!this.capturedTileEntities.isEmpty() && (blockEntity = this.capturedTileEntities.get(blockposition)) != null) {
++ return blockEntity;
+ }
++ // Paper end
+ // CraftBukkit end
+ return this.isOutsideBuildHeight(blockposition) ? null : (!this.isClientSide && !io.papermc.paper.util.TickThread.isTickThread() ? null : this.getChunkAt(blockposition).getBlockEntity(blockposition, LevelChunk.EntityCreationType.IMMEDIATE)); // Paper - rewrite chunk system
+ }