Upstream/Paper
1
0
Fork 0
mirror of https://github.com/PaperMC/Paper.git synced 2024-11-23 11:06:29 +01:00
Code Issues Projects Releases Wiki Activity

Fix bounding box pool to use a dynamic cache size

Browse Source 
Also changed insane array copy routine. This should reduce server lag
spikes that occur periodically.
This commit is contained in:
Wesley Wolfe 2012-08-04 21:11:28 -05:00
parent 14a470985d
commit 9804665fec
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/main/java/net/minecraft/server/AABBPool.java
View File

@ -18,6 +18,8 @@ public class AABBPool {
}
public AxisAlignedBB a(double d0, double d1, double d2, double d3, double d4, double d5) {
if (this.f == 0) return new AxisAlignedBB(d0, d1, d2, d3, d4, d5); // CraftBukkit - don't pool objects indefinitely if thread doesn't adhere to contract
AxisAlignedBB axisalignedbb;
if (this.d >= this.c.size()) {
@ -37,16 +39,19 @@ public class AABBPool {
this.e = this.d;
}
if (this.f++ == this.a) {
int i = Math.max(this.e, this.c.size() - this.b);
while (this.c.size() > i) {
this.c.remove(i);
// CraftBukkit start - intelligent cache
if ((this.f++ & 0xff) == 0) {
int newSize = this.c.size() - (this.c.size() >> 3);
if (newSize > this.e) { // newSize will be 87.5%, but if we were not in that range, we clear some of the cache
for (int i = this.c.size() - 1; i > newSize; i--) { // Work down from size() to prevent insane array copies
this.c.remove(i);
}
}
this.e = 0;
this.f = 0;
// this.f = 0; // We do not reset to zero; it doubles for a flag
}
// CraftBukkit end
this.d = 0;
}