2020-05-06 09:48:49 +00:00
|
|
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
2019-04-06 14:25:21 +00:00
|
|
|
From: Aikar <aikar@aikar.co>
|
|
|
|
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/server/World.java b/src/main/java/net/minecraft/server/World.java
|
2020-06-25 01:10:30 +00:00
|
|
|
index 3d13efc50b06748b94d6bb4a5a2f9ade1699a726..b68fe51f0518c05c9924be8b6e3e36e5125d4b03 100644
|
2019-04-06 14:25:21 +00:00
|
|
|
--- a/src/main/java/net/minecraft/server/World.java
|
|
|
|
+++ b/src/main/java/net/minecraft/server/World.java
|
2020-05-24 02:27:37 +00:00
|
|
|
@@ -999,12 +999,13 @@ public abstract class World implements GeneratorAccess, AutoCloseable {
|
2019-04-06 14:25:21 +00:00
|
|
|
return null;
|
|
|
|
} else {
|
|
|
|
// CraftBukkit start
|
|
|
|
- if (capturedTileEntities.containsKey(blockposition)) {
|
|
|
|
- return capturedTileEntities.get(blockposition);
|
|
|
|
+ TileEntity tileentity = null; // Paper
|
|
|
|
+ if (!capturedTileEntities.isEmpty() && (tileentity = capturedTileEntities.get(blockposition)) != null) { // Paper
|
|
|
|
+ return tileentity; // Paper
|
|
|
|
}
|
|
|
|
// CraftBukkit end
|
|
|
|
|
|
|
|
- TileEntity tileentity = null;
|
|
|
|
+ //TileEntity tileentity = null; // Paper - move up
|
|
|
|
|
2019-05-05 17:19:34 +00:00
|
|
|
if (this.tickingTileEntities) {
|
2019-12-12 16:20:43 +00:00
|
|
|
tileentity = this.e(blockposition);
|