Don't attempt to clean up static members during an update.

aadnk · Feb 7, 2013 · bc1955b · bc1955b
1 parent 32282bb
commit bc1955b
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/ProtocolLib/src/main/java/com/comphenix/protocol/ProtocolLibrary.java b/ProtocolLib/src/main/java/com/comphenix/protocol/ProtocolLibrary.java
@@ -40,6 +40,7 @@
 import com.comphenix.protocol.injector.PacketFilterManager.PlayerInjectHooks;
 import com.comphenix.protocol.metrics.Statistics;
 import com.comphenix.protocol.metrics.Updater;
+import com.comphenix.protocol.metrics.Updater.UpdateResult;
 import com.comphenix.protocol.reflect.compiler.BackgroundCompiler;
 
 /**
@@ -430,8 +431,10 @@ public void onDisable() {
 		reporter = null;
 
 		// Leaky ClassLoader begone!
-		CleanupStaticMembers cleanup = new CleanupStaticMembers(getClassLoader(), reporter);
-		cleanup.resetAll();
+		if (updater == null || updater.getResult() != UpdateResult.SUCCESS) {
+			CleanupStaticMembers cleanup = new CleanupStaticMembers(getClassLoader(), reporter);
+			cleanup.resetAll();
+		}
 	}
 
 	// Get the Bukkit logger first, before we try to create our own