Paper-Old/Spigot-Server-Patches/0320-World-EntityHuman-Lookup-Optimizations.patch

From 3d8f1b41f66faf392991e817b0954bc7d5841738 Mon Sep 17 00:00:00 2001
From: willies952002 <admin@domnian.com>
Date: Mon, 30 Jul 2018 02:42:49 -0400
Subject: [PATCH] World EntityHuman Lookup Optimizations


diff --git a/src/main/java/net/minecraft/server/World.java b/src/main/java/net/minecraft/server/World.java
index 255829b0b5..26007862ab 100644
--- a/src/main/java/net/minecraft/server/World.java
+++ b/src/main/java/net/minecraft/server/World.java
@@ -81,6 +81,7 @@ public abstract class World implements IEntityAccess, GeneratorAccess, IIBlockAc
     private final List<TileEntity> c = Lists.newArrayList();
     private final Set<TileEntity> tileEntityListUnload = com.google.common.collect.Sets.newHashSet(); // Paper
     public final List<EntityHuman> players = Lists.newArrayList();
+    public final Map<String, EntityHuman> playersByName = Maps.newHashMap(); // Paper - World EntityHuman Lookup Optimizations
     public final List<Entity> k = Lists.newArrayList();
     protected final IntHashMap<Entity> entitiesById = new IntHashMap();
     private final long F = 16777215L;
@@ -1082,6 +1083,8 @@ public abstract class World implements IEntityAccess, GeneratorAccess, IIBlockAc
                 EntityHuman entityhuman = (EntityHuman) entity;
 
                 this.players.add(entityhuman);
+                this.playersByName.put(entityhuman.getName(), entityhuman);
+                // Paper end
                 this.everyoneSleeping();
             }
 
@@ -1124,6 +1127,7 @@ public abstract class World implements IEntityAccess, GeneratorAccess, IIBlockAc
         entity.die();
         if (entity instanceof EntityHuman) {
             this.players.remove(entity);
+            this.playersByName.remove(entity.getName()); // Paper - World EntityHuman Lookup Optimizations
             // Spigot start
             for ( WorldPersistentData worldData : worldMaps.a.values() )
             {
@@ -1157,6 +1161,7 @@ public abstract class World implements IEntityAccess, GeneratorAccess, IIBlockAc
         entity.die();
         if (entity instanceof EntityHuman) {
             this.players.remove(entity);
+            this.playersByName.remove(entity.getName()); // Paper - World EntityHuman Lookup Optimizations
             this.everyoneSleeping();
         }
 
@@ -2706,6 +2711,8 @@ public abstract class World implements IEntityAccess, GeneratorAccess, IIBlockAc
 
     @Nullable
     public EntityHuman a(String s) {
+        // Paper start - World EntityHuman Lookup Optimizations
+        /*
         for (int i = 0; i < this.players.size(); ++i) {
             EntityHuman entityhuman = (EntityHuman) this.players.get(i);
 
@@ -2715,10 +2722,15 @@ public abstract class World implements IEntityAccess, GeneratorAccess, IIBlockAc
         }
 
         return null;
+        */
+        return this.playersByName.get(s);
+        // Paper end
     }
 
     @Nullable
     public EntityHuman b(UUID uuid) {
+        // Paper start - World EntityHuman Lookup Optimizations
+        /*
         for (int i = 0; i < this.players.size(); ++i) {
             EntityHuman entityhuman = (EntityHuman) this.players.get(i);
 
@@ -2728,6 +2740,10 @@ public abstract class World implements IEntityAccess, GeneratorAccess, IIBlockAc
         }
 
         return null;
+        */
+        Entity entity = ((WorldServer)this).entitiesByUUID.get(uuid);
+        return entity instanceof EntityHuman ? (EntityHuman) entity : null;
+        // Paper end
     }
 
     public void checkSession() throws ExceptionWorldConflict {
-- 
2.18.0