Yatopia/patches/server/0065-Don-t-unnecessarily-copy-the-passenger-list.patch

From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Ivan Pekov <ivan@mrivanplays.com>
Date: Wed, 23 Dec 2020 09:55:43 +0200
Subject: [PATCH] Don't unnecessarily copy the passenger list

It is only copied if it's needed, and that is in EntityTrackerEntry, when we've detected a change.

diff --git a/src/main/java/net/minecraft/server/Entity.java b/src/main/java/net/minecraft/server/Entity.java
index 2bacecb9cee2f93b142d63b9a79641d7bac8f868..8ba05d0dadf34baa046962cbff71a2fcfd1471d7 100644
--- a/src/main/java/net/minecraft/server/Entity.java
+++ b/src/main/java/net/minecraft/server/Entity.java
@@ -3284,7 +3284,12 @@ public abstract class Entity implements INamableTileEntity, ICommandListener, Ke
     }
 
     public List<Entity> getPassengers() {
+        // Yatopia start - don't copy passengers list
+        /*
         return (List) (this.passengers.isEmpty() ? Collections.emptyList() : Lists.newArrayList(this.passengers));
+         */
+        return this.passengers.isEmpty() ? Collections.emptyList() : this.passengers;
+        // Yatopia end
     }
 
     public boolean w(Entity entity) {
diff --git a/src/main/java/net/minecraft/server/EntityTrackerEntry.java b/src/main/java/net/minecraft/server/EntityTrackerEntry.java
index 228236bce14bfdf930570b453862dcfaae9e4823..0be11d1263d84b270c25ec8249832164dc75409d 100644
--- a/src/main/java/net/minecraft/server/EntityTrackerEntry.java
+++ b/src/main/java/net/minecraft/server/EntityTrackerEntry.java
@@ -78,7 +78,7 @@ public class EntityTrackerEntry {
         List<Entity> list = this.tracker.getPassengers();
 
         if (!list.equals(this.p)) {
-            this.p = list;
+            this.p = com.google.common.collect.ImmutableList.copyOf(list); // Yatopia - only copy list if something has changed
             this.broadcastIncludingSelf(new PacketPlayOutMount(this.tracker)); // CraftBukkit
         }