d231cef8cd
Take same approach we did for chunks, and only save player if its been X time since last save, instead of doing it all in 1 tick. This is even more helpful considering Player Saving is done sync for File IO.
65 lines
2.6 KiB
Diff
65 lines
2.6 KiB
Diff
From 049172acbe149163c86945185f962ce662331cc2 Mon Sep 17 00:00:00 2001
|
|
From: Alfie Cleveland <alfeh@me.com>
|
|
Date: Fri, 25 Nov 2016 13:22:40 +0000
|
|
Subject: [PATCH] Optimise removeQueue
|
|
|
|
|
|
diff --git a/src/main/java/net/minecraft/server/EntityPlayer.java b/src/main/java/net/minecraft/server/EntityPlayer.java
|
|
index ab2b4c404..fa09ef111 100644
|
|
--- a/src/main/java/net/minecraft/server/EntityPlayer.java
|
|
+++ b/src/main/java/net/minecraft/server/EntityPlayer.java
|
|
@@ -4,7 +4,9 @@ import com.google.common.collect.Lists;
|
|
import com.google.common.collect.Sets;
|
|
import com.mojang.authlib.GameProfile;
|
|
import io.netty.buffer.Unpooled;
|
|
+import java.util.ArrayDeque; // Paper
|
|
import java.util.Collection;
|
|
+import java.util.Deque; // Paper
|
|
import java.util.HashSet;
|
|
import java.util.Iterator;
|
|
import java.util.List;
|
|
@@ -35,7 +37,7 @@ public class EntityPlayer extends EntityHuman implements ICrafting {
|
|
public final PlayerInteractManager playerInteractManager;
|
|
public double d;
|
|
public double e;
|
|
- public final List<Integer> removeQueue = Lists.newLinkedList();
|
|
+ public final Deque<Integer> removeQueue = new ArrayDeque<>(); // Paper
|
|
private final ServerStatisticManager bU;
|
|
private float bV = Float.MIN_VALUE;
|
|
private int bW = Integer.MIN_VALUE;
|
|
@@ -247,10 +249,17 @@ public class EntityPlayer extends EntityHuman implements ICrafting {
|
|
Iterator iterator = this.removeQueue.iterator();
|
|
int j = 0;
|
|
|
|
- while (iterator.hasNext() && j < i) {
|
|
+ // Paper start
|
|
+ /* while (iterator.hasNext() && j < i) {
|
|
aint[j++] = ((Integer) iterator.next()).intValue();
|
|
iterator.remove();
|
|
+ } */
|
|
+
|
|
+ Integer integer;
|
|
+ while (j < i && (integer = this.removeQueue.poll()) != null) {
|
|
+ aint[j++] = integer.intValue();
|
|
}
|
|
+ // Paper end
|
|
|
|
this.playerConnection.sendPacket(new PacketPlayOutEntityDestroy(aint));
|
|
}
|
|
@@ -987,7 +996,12 @@ public class EntityPlayer extends EntityHuman implements ICrafting {
|
|
this.lastSentExp = -1;
|
|
this.lastHealthSent = -1.0F;
|
|
this.cc = -1;
|
|
- this.removeQueue.addAll(((EntityPlayer) entityhuman).removeQueue);
|
|
+ // this.removeQueue.addAll(((EntityPlayer) entityhuman).removeQueue); // Paper
|
|
+ // Paper start
|
|
+ if (this.removeQueue != ((EntityPlayer) entityhuman).removeQueue) {
|
|
+ this.removeQueue.addAll(((EntityPlayer) entityhuman).removeQueue);
|
|
+ }
|
|
+ // Paper end
|
|
}
|
|
|
|
protected void a(MobEffect mobeffect) {
|
|
--
|
|
2.11.0
|
|
|