2016-03-01 01:23:45 +01:00
|
|
|
From 6c6bd2c67ce8837dcb7b21a2468a0e5e70b9531d Mon Sep 17 00:00:00 2001
|
2014-07-21 22:46:54 +02:00
|
|
|
From: md_5 <git@md-5.net>
|
|
|
|
Date: Mon, 10 Mar 2014 09:03:28 +1100
|
|
|
|
Subject: [PATCH] Guard Entity List
|
|
|
|
|
|
|
|
|
|
|
|
diff --git a/src/main/java/net/minecraft/server/World.java b/src/main/java/net/minecraft/server/World.java
|
2016-03-01 01:23:45 +01:00
|
|
|
index a6bd744..2cc87d0 100644
|
2014-07-21 22:46:54 +02:00
|
|
|
--- a/src/main/java/net/minecraft/server/World.java
|
|
|
|
+++ b/src/main/java/net/minecraft/server/World.java
|
2016-02-29 22:33:06 +01:00
|
|
|
@@ -33,7 +33,32 @@ public abstract class World implements IBlockAccess {
|
2014-07-21 22:46:54 +02:00
|
|
|
|
2015-02-28 12:36:22 +01:00
|
|
|
private int a = 63;
|
2016-02-29 22:33:06 +01:00
|
|
|
protected boolean d;
|
2015-02-28 12:36:22 +01:00
|
|
|
- public final List<Entity> entityList = Lists.newArrayList();
|
2014-07-21 22:46:54 +02:00
|
|
|
+ // Spigot start - guard entity list from removals
|
2015-02-28 12:36:22 +01:00
|
|
|
+ public final List<Entity> entityList = new java.util.ArrayList<Entity>()
|
2014-07-21 22:46:54 +02:00
|
|
|
+ {
|
|
|
|
+ @Override
|
2015-02-28 12:36:22 +01:00
|
|
|
+ public Entity remove(int index)
|
2014-07-21 22:46:54 +02:00
|
|
|
+ {
|
|
|
|
+ guard();
|
|
|
|
+ return super.remove( index );
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+ @Override
|
|
|
|
+ public boolean remove(Object o)
|
|
|
|
+ {
|
|
|
|
+ guard();
|
|
|
|
+ return super.remove( o );
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+ private void guard()
|
|
|
|
+ {
|
|
|
|
+ if ( guardEntityList )
|
|
|
|
+ {
|
|
|
|
+ throw new java.util.ConcurrentModificationException();
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
+ };
|
|
|
|
+ // Spigot end
|
2016-02-29 22:33:06 +01:00
|
|
|
protected final List<Entity> f = Lists.newArrayList();
|
2015-02-28 12:36:22 +01:00
|
|
|
public final List<TileEntity> tileEntityList = Lists.newArrayList();
|
2016-02-29 22:33:06 +01:00
|
|
|
public final List<TileEntity> tileEntityListTick = Lists.newArrayList();
|
|
|
|
@@ -101,6 +126,7 @@ public abstract class World implements IBlockAccess {
|
|
|
|
public final org.spigotmc.SpigotWorldConfig spigotConfig; // Spigot
|
2014-07-21 22:46:54 +02:00
|
|
|
|
2016-02-29 22:33:06 +01:00
|
|
|
public final SpigotTimings.WorldTimingsHandler timings; // Spigot
|
|
|
|
+ private boolean guardEntityList; // Spigot
|
|
|
|
|
|
|
|
public CraftWorld getWorld() {
|
|
|
|
return this.world;
|
2016-03-01 01:23:45 +01:00
|
|
|
@@ -1312,6 +1338,7 @@ public abstract class World implements IBlockAccess {
|
2014-07-21 22:46:54 +02:00
|
|
|
|
|
|
|
org.spigotmc.ActivationRange.activateEntities(this); // Spigot
|
|
|
|
timings.entityTick.startTiming(); // Spigot
|
|
|
|
+ guardEntityList = true; // Spigot
|
|
|
|
// CraftBukkit start - Use field for loop variable
|
|
|
|
for (this.tickPosition = 0; this.tickPosition < this.entityList.size(); ++this.tickPosition) {
|
|
|
|
entity = (Entity) this.entityList.get(this.tickPosition);
|
2016-03-01 01:23:45 +01:00
|
|
|
@@ -1350,12 +1377,15 @@ public abstract class World implements IBlockAccess {
|
2016-02-29 22:33:06 +01:00
|
|
|
this.getChunkAt(j, l).b(entity);
|
2014-07-21 22:46:54 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
+ guardEntityList = false; // Spigot
|
|
|
|
this.entityList.remove(this.tickPosition--); // CraftBukkit - Use field for loop variable
|
|
|
|
+ guardEntityList = true; // Spigot
|
2016-02-29 22:33:06 +01:00
|
|
|
this.c(entity);
|
2014-07-21 22:46:54 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
this.methodProfiler.b();
|
|
|
|
}
|
|
|
|
+ guardEntityList = false; // Spigot
|
|
|
|
|
|
|
|
timings.entityTick.stopTiming(); // Spigot
|
|
|
|
this.methodProfiler.c("blockEntities");
|
|
|
|
--
|
2016-02-29 22:33:06 +01:00
|
|
|
2.5.0
|
2014-07-21 22:46:54 +02:00
|
|
|
|