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

Remove redundant call to worldMaps.a(). Addresses BUKKIT-4828

Browse Source 
The WorldMapCollection object for SecondaryWorldServers(Nether, End) is
shared with the main world, so saving it again for each SecondaryWorldServer
is redundant.

This commit removes the redundant call by checking if the WorldServer is an
instanceof SecondaryWorldServer before invoking worldMaps.a().
This commit is contained in:
Nate Mortensen 2013-10-19 18:14:20 -06:00
parent 42e7fdee92
commit 6430c868c7
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/net/minecraft/server/WorldServer.java
View File

@ -779,8 +779,12 @@ public class WorldServer extends World implements org.bukkit.BlockChangeDelegate
protected void a() throws ExceptionWorldConflict { // CraftBukkit - added throws protected void a() throws ExceptionWorldConflict { // CraftBukkit - added throws
this.G(); this.G();
this.dataManager.saveWorldData(this.worldData, this.server.getPlayerList().q()); this.dataManager.saveWorldData(this.worldData, this.server.getPlayerList().q());
// CraftBukkit start - save worldMaps once, rather than once per shared world
if (!(this instanceof SecondaryWorldServer)) {
this.worldMaps.a(); this.worldMaps.a();
} }
// CraftBukkit end
}
protected void a(Entity entity) { protected void a(Entity entity) {
super.a(entity); super.a(entity);