Add iterator cache to UnsafeList and use it in hotspots

Adds a specialized iterator for the list and a pool of iterators to avoid
object churn. Also optimizes the clear() method to reduce object creation.

1 files changed, 7 insertions, 3 deletions

diff --git a/src/main/java/net/minecraft/server/PathfinderGoalSelector.java b/src/main/java/net/minecraft/server/PathfinderGoalSelector.java
index dc16a625..9ef8ec76 100644
--- a/src/main/java/net/minecraft/server/PathfinderGoalSelector.java
+++ b/src/main/java/net/minecraft/server/PathfinderGoalSelector.java
@@ -4,10 +4,14 @@ import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
 
+import org.bukkit.craftbukkit.util.UnsafeList; // CraftBukkit
+
 public class PathfinderGoalSelector {
 
-    private List a = new ArrayList();
-    private List b = new ArrayList();
+    // CraftBukkit start - ArrayList -> UnsafeList
+    private List a = new UnsafeList();
+    private List b = new UnsafeList();
+    // CraftBukkit end
     private final MethodProfiler c;
     private int d = 0;
     private int e = 3;
@@ -21,7 +25,7 @@ public class PathfinderGoalSelector {
     }
 
     public void a() {
-        ArrayList arraylist = new ArrayList();
+        UnsafeList arraylist = new UnsafeList(); // CraftBukkit - ArrayList -> UnsafeList
         Iterator iterator;
         PathfinderGoalSelectorItem pathfindergoalselectoritem;