aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0409-Cache-block-data-strings.patch
diff options
context:
space:
mode:
authorSpottedleaf <[email protected]>2024-07-17 10:24:53 -0700
committerSpottedleaf <[email protected]>2024-07-17 10:28:32 -0700
commit00b949f1bbbf444e2b5e7b8de7c9b14fbd2133c6 (patch)
tree82639515bc5e9ae00c1e639e72137ed51e1ac688 /patches/server/0409-Cache-block-data-strings.patch
parent967f98aa81da851740aeb429778e46159fd188df (diff)
downloadPaper-00b949f1bbbf444e2b5e7b8de7c9b14fbd2133c6.tar.gz
Paper-00b949f1bbbf444e2b5e7b8de7c9b14fbd2133c6.zip
Remove Moonrise utils to MCUtils, remove duplicated/unused utils
Diffstat (limited to 'patches/server/0409-Cache-block-data-strings.patch')
-rw-r--r--patches/server/0409-Cache-block-data-strings.patch70
1 files changed, 70 insertions, 0 deletions
diff --git a/patches/server/0409-Cache-block-data-strings.patch b/patches/server/0409-Cache-block-data-strings.patch
new file mode 100644
index 0000000000..28a1635b20
--- /dev/null
+++ b/patches/server/0409-Cache-block-data-strings.patch
@@ -0,0 +1,70 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: miclebrick <[email protected]>
+Date: Thu, 6 Dec 2018 19:52:50 -0500
+Subject: [PATCH] Cache block data strings
+
+
+diff --git a/src/main/java/net/minecraft/server/MinecraftServer.java b/src/main/java/net/minecraft/server/MinecraftServer.java
+index 6c9026cf224cf9dc75a5f61f1c051e640d1887ed..b9ba0033542bae48678e0321fbe0633a711291a5 100644
+--- a/src/main/java/net/minecraft/server/MinecraftServer.java
++++ b/src/main/java/net/minecraft/server/MinecraftServer.java
+@@ -2147,6 +2147,7 @@ public abstract class MinecraftServer extends ReentrantBlockableEventLoop<TickTa
+ this.getPlayerList().reloadResources();
+ this.functionManager.replaceLibrary(this.resources.managers.getFunctionLibrary());
+ this.structureTemplateManager.onResourceManagerReload(this.resources.resourceManager);
++ org.bukkit.craftbukkit.block.data.CraftBlockData.reloadCache(); // Paper - cache block data strings; they can be defined by datapacks so refresh it here
+ }, this);
+
+ if (this.isSameThread()) {
+diff --git a/src/main/java/org/bukkit/craftbukkit/block/data/CraftBlockData.java b/src/main/java/org/bukkit/craftbukkit/block/data/CraftBlockData.java
+index c1c5750dd2e4a9af1a115996a87eaaa1ea552c74..67ff2241aa8869b41abb0a93467b8694618264e2 100644
+--- a/src/main/java/org/bukkit/craftbukkit/block/data/CraftBlockData.java
++++ b/src/main/java/org/bukkit/craftbukkit/block/data/CraftBlockData.java
+@@ -154,7 +154,7 @@ public class CraftBlockData implements BlockData {
+ return exactMatch;
+ }
+
+- private static final Map<Class<? extends Enum<?>>, Enum<?>[]> ENUM_VALUES = new HashMap<>();
++ private static final Map<Class<? extends Enum<?>>, Enum<?>[]> ENUM_VALUES = new java.util.concurrent.ConcurrentHashMap<>(); // Paper - cache block data strings; make thread safe
+
+ /**
+ * Convert an NMS Enum (usually a BlockStateEnum) to its appropriate Bukkit
+@@ -539,7 +539,38 @@ public class CraftBlockData implements BlockData {
+ Preconditions.checkState(CraftBlockData.MAP.put(nms, bukkit) == null, "Duplicate mapping %s->%s", nms, bukkit);
+ }
+
++ // Paper start - cache block data strings
++ private static Map<String, CraftBlockData> stringDataCache = new java.util.concurrent.ConcurrentHashMap<>();
++
++ static {
++ // cache all of the default states at startup, will not cache ones with the custom states inside of the
++ // brackets in a different order, though
++ reloadCache();
++ }
++
++ public static void reloadCache() {
++ stringDataCache.clear();
++ Block.BLOCK_STATE_REGISTRY.forEach(blockData -> stringDataCache.put(blockData.toString(), blockData.createCraftBlockData()));
++ }
++ // Paper end - cache block data strings
++
+ public static CraftBlockData newData(BlockType blockType, String data) {
++
++ // Paper start - cache block data strings
++ if (blockType != null) {
++ Block block = CraftBlockType.bukkitToMinecraftNew(blockType);
++ if (block != null) {
++ net.minecraft.resources.ResourceLocation key = BuiltInRegistries.BLOCK.getKey(block);
++ data = data == null ? key.toString() : key + data;
++ }
++ }
++
++ CraftBlockData cached = stringDataCache.computeIfAbsent(data, s -> createNewData(null, s));
++ return (CraftBlockData) cached.clone();
++ }
++
++ private static CraftBlockData createNewData(BlockType blockType, String data) {
++ // Paper end - cache block data strings
+ net.minecraft.world.level.block.state.BlockState blockData;
+ Block block = blockType == null ? null : ((CraftBlockType<?>) blockType).getHandle();
+ Map<Property<?>, Comparable<?>> parsed = null;