spigot/CraftBukkit-Patches/0126-Plug-WorldMap-Memory-Leak.patch
2016-09-05 17:35:26 +10:00

63 lines
2.8 KiB
Diff

From 46e7cbb86caac82bb9a65f3b765c2c3361d9219e Mon Sep 17 00:00:00 2001
From: md_5 <git@md-5.net>
Date: Fri, 8 Aug 2014 19:57:03 +1000
Subject: [PATCH] Plug WorldMap Memory Leak
diff --git a/src/main/java/net/minecraft/server/PersistentCollection.java b/src/main/java/net/minecraft/server/PersistentCollection.java
index fe59b75..936d6c6 100644
--- a/src/main/java/net/minecraft/server/PersistentCollection.java
+++ b/src/main/java/net/minecraft/server/PersistentCollection.java
@@ -19,7 +19,7 @@ public class PersistentCollection {
private final IDataManager b;
protected Map<String, PersistentBase> a = Maps.newHashMap();
- private final List<PersistentBase> c = Lists.newArrayList();
+ public final List<PersistentBase> c = Lists.newArrayList(); // Spigot
private final Map<String, Short> d = Maps.newHashMap();
public PersistentCollection(IDataManager idatamanager) {
diff --git a/src/main/java/net/minecraft/server/World.java b/src/main/java/net/minecraft/server/World.java
index ae4dd5d..41a59dc 100644
--- a/src/main/java/net/minecraft/server/World.java
+++ b/src/main/java/net/minecraft/server/World.java
@@ -1033,6 +1033,23 @@ public abstract class World implements IBlockAccess {
entity.die();
if (entity instanceof EntityHuman) {
this.players.remove(entity);
+ // Spigot start
+ for ( Object o : worldMaps.c )
+ {
+ if ( o instanceof WorldMap )
+ {
+ WorldMap map = (WorldMap) o;
+ map.j.remove( entity );
+ for ( Iterator<WorldMap.WorldMapHumanTracker> iter = (Iterator<WorldMap.WorldMapHumanTracker>) map.h.iterator(); iter.hasNext(); )
+ {
+ if ( iter.next().trackee == entity )
+ {
+ iter.remove();
+ }
+ }
+ }
+ }
+ // Spigot end
this.everyoneSleeping();
this.c(entity);
}
diff --git a/src/main/java/net/minecraft/server/WorldMap.java b/src/main/java/net/minecraft/server/WorldMap.java
index 988f5dc..3717dc7 100644
--- a/src/main/java/net/minecraft/server/WorldMap.java
+++ b/src/main/java/net/minecraft/server/WorldMap.java
@@ -24,7 +24,7 @@ public class WorldMap extends PersistentBase {
public byte scale;
public byte[] colors = new byte[16384];
public List<WorldMap.WorldMapHumanTracker> h = Lists.newArrayList();
- private final Map<EntityHuman, WorldMap.WorldMapHumanTracker> j = Maps.newHashMap();
+ public final Map<EntityHuman, WorldMap.WorldMapHumanTracker> j = Maps.newHashMap(); // Spigot private -> public
public Map<String, MapIcon> decorations = Maps.newLinkedHashMap();
// CraftBukkit start
--
2.7.4