2013-03-23 09:41:36 +01:00
From a7d97767e0fa796db6cbc95da373f4dc1d9f40c0 Mon Sep 17 00:00:00 2001
2013-03-23 09:08:49 +01:00
From: md_5 <md_5@live.com.au>
Date: Sat, 23 Mar 2013 19:08:41 +1100
Subject: [PATCH] Limit Custom Map Rendering
The default CraftBukkit render sequence for maps is ridiculously slow. By only using it when a custom renderer has been added (rarely in most cases), we can fallback to the Vanilla renderer for general usage. This leads to a much higher effiency overall, especially if no plugins are rendering such maps.
---
2013-03-23 09:32:46 +01:00
.../java/net/minecraft/server/ItemWorldMap.java | 1 +
2013-03-23 09:09:47 +01:00
.../net/minecraft/server/WorldMapHumanTracker.java | 24 ++++++++++++++--------
2013-03-23 09:32:46 +01:00
.../java/org/bukkit/craftbukkit/CraftWorld.java | 5 +++++
2013-03-23 09:08:49 +01:00
.../org/bukkit/craftbukkit/map/CraftMapView.java | 2 +-
2013-03-23 09:32:46 +01:00
src/main/resources/configurations/bukkit.yml | 1 +
5 files changed, 23 insertions(+), 10 deletions(-)
2013-03-23 09:08:49 +01:00
2013-03-23 09:32:46 +01:00
diff --git a/src/main/java/net/minecraft/server/ItemWorldMap.java b/src/main/java/net/minecraft/server/ItemWorldMap.java
index cf3bb1f..ebc0fc7 100644
--- a/src/main/java/net/minecraft/server/ItemWorldMap.java
+++ b/src/main/java/net/minecraft/server/ItemWorldMap.java
@@ -228,6 +228,7 @@ public class ItemWorldMap extends ItemWorldMapBase {
}
public Packet c(ItemStack itemstack, World world, EntityHuman entityhuman) {
+ if (!world.getWorld().mapsEnabled) return null; // Spigot
byte[] abyte = this.getSavedMap(itemstack, world).getUpdatePacket(itemstack, world, entityhuman);
return abyte == null ? null : new Packet131ItemData((short) Item.MAP.id, (short) itemstack.getData(), abyte);
2013-03-23 09:08:49 +01:00
diff --git a/src/main/java/net/minecraft/server/WorldMapHumanTracker.java b/src/main/java/net/minecraft/server/WorldMapHumanTracker.java
2013-03-23 09:41:36 +01:00
index bf3e6fe..448800d 100644
2013-03-23 09:08:49 +01:00
--- a/src/main/java/net/minecraft/server/WorldMapHumanTracker.java
+++ b/src/main/java/net/minecraft/server/WorldMapHumanTracker.java
2013-03-23 09:09:47 +01:00
@@ -41,23 +41,29 @@ public class WorldMapHumanTracker {
2013-03-23 09:08:49 +01:00
int i;
int j;
- org.bukkit.craftbukkit.map.RenderData render = this.worldMap.mapView.render((org.bukkit.craftbukkit.entity.CraftPlayer) trackee.getBukkitEntity()); // CraftBukkit
+ // Spigot start
2013-03-23 09:41:36 +01:00
+ boolean custom = this.worldMap.mapView.renderers.size() > 1;
2013-03-23 09:08:49 +01:00
+ org.bukkit.craftbukkit.map.RenderData render = (custom) ? this.worldMap.mapView.render((org.bukkit.craftbukkit.entity.CraftPlayer) trackee.getBukkitEntity()) : null; // CraftBukkit
if (--this.g < 0) {
this.g = 4;
- abyte = new byte[render.cursors.size() * 3 + 1]; // CraftBukkit
+ abyte = new byte[((custom) ? render.cursors.size() : this.worldMap.g.size()) * 3 + 1]; // CraftBukkit
abyte[0] = 1;
i = 0;
// CraftBukkit start
- for (i = 0; i < render.cursors.size(); ++i) {
- org.bukkit.map.MapCursor cursor = render.cursors.get(i);
- if (!cursor.isVisible()) continue;
- abyte[i * 3 + 1] = (byte) (cursor.getRawType() << 4 | cursor.getDirection() & 15);
- abyte[i * 3 + 2] = (byte) cursor.getX();
- abyte[i * 3 + 3] = (byte) cursor.getY();
+ // Spigot start
+ for (Iterator iterator = ((custom) ? render.cursors.iterator() : this.worldMap.g.values().iterator()); iterator.hasNext(); ++i) {
2013-03-23 09:09:47 +01:00
+ org.bukkit.map.MapCursor cursor = (custom) ? (org.bukkit.map.MapCursor) iterator.next() : null;
2013-03-23 09:08:49 +01:00
+ if (cursor != null && !cursor.isVisible()) continue;
+ WorldMapDecoration deco = (custom) ? null : (WorldMapDecoration) iterator.next();
+
+ abyte[i * 3 + 1] = (byte) (((custom) ? cursor.getRawType() : deco.type) << 4 | ((custom) ? cursor.getDirection() : deco.rotation) & 15);
+ abyte[i * 3 + 2] = (byte) ((custom) ? cursor.getX() : deco.locX);
+ abyte[i * 3 + 3] = (byte) ((custom) ? cursor.getY() : deco.locY);
}
+ // Spigot end
// CraftBukkit end
boolean flag = !itemstack.z();
2013-03-23 09:09:47 +01:00
@@ -92,7 +98,7 @@ public class WorldMapHumanTracker {
2013-03-23 09:08:49 +01:00
abyte1[2] = (byte) j;
for (int i1 = 0; i1 < abyte1.length - 3; ++i1) {
- abyte1[i1 + 3] = render.buffer[(i1 + j) * 128 + i]; // CraftBukkit
+ abyte1[i1 + 3] = ((custom) ? render.buffer : this.worldMap.colors)[(i1 + j) * 128 + i];
}
this.c[i] = -1;
2013-03-23 09:32:46 +01:00
diff --git a/src/main/java/org/bukkit/craftbukkit/CraftWorld.java b/src/main/java/org/bukkit/craftbukkit/CraftWorld.java
index d463e40..7737870 100644
--- a/src/main/java/org/bukkit/craftbukkit/CraftWorld.java
+++ b/src/main/java/org/bukkit/craftbukkit/CraftWorld.java
@@ -112,6 +112,8 @@ public class CraftWorld implements World {
monsterTrackingRange = configuration.getInt("world-settings.default.entity-tracking-range-monsters");
maxTrackingRange = configuration.getInt("world-settings.default.entity-tracking-range-max");
+ mapsEnabled = configuration.getBoolean("world-settings.default.maps-enabled", mapsEnabled);
+
//override defaults with world specific, if they exist
growthPerTick = configuration.getInt("world-settings." + name + ".growth-chunks-per-tick", growthPerTick);
itemMergeRadius = configuration.getDouble("world-settings." + name + ".item-merge-radius", itemMergeRadius);
@@ -147,6 +149,8 @@ public class CraftWorld implements World {
if (maxTrackingRange == 0) {
System.err.println("Error! Should not have 0 maxRange");
}
+ mapsEnabled = configuration.getBoolean("world-settings." + name + ".maps-enabled", mapsEnabled);
+
server.getLogger().info("-------------- Spigot ----------------");
server.getLogger().info("-------- World Settings For [" + name + "] --------");
server.getLogger().info("Growth Per Chunk: " + growthPerTick);
@@ -176,6 +180,7 @@ public class CraftWorld implements World {
public double expMergeRadius = 3.5;
public int viewDistance;
public boolean obfuscated = false;
+ public boolean mapsEnabled = true;
//Crop growth rates:
public int wheatGrowthModifier = 100;
public int cactusGrowthModifier = 100;
2013-03-23 09:08:49 +01:00
diff --git a/src/main/java/org/bukkit/craftbukkit/map/CraftMapView.java b/src/main/java/org/bukkit/craftbukkit/map/CraftMapView.java
index b28d6a3..e70e4e4 100644
--- a/src/main/java/org/bukkit/craftbukkit/map/CraftMapView.java
+++ b/src/main/java/org/bukkit/craftbukkit/map/CraftMapView.java
@@ -18,7 +18,7 @@ import org.bukkit.map.MapView;
public final class CraftMapView implements MapView {
private final Map<CraftPlayer, RenderData> renderCache = new HashMap<CraftPlayer, RenderData>();
- private final List<MapRenderer> renderers = new ArrayList<MapRenderer>();
+ public final List<MapRenderer> renderers = new ArrayList<MapRenderer>(); // Spigot
private final Map<MapRenderer, Map<CraftPlayer, CraftMapCanvas>> canvases = new HashMap<MapRenderer, Map<CraftPlayer, CraftMapCanvas>>();
protected final WorldMap worldMap;
2013-03-23 09:32:46 +01:00
diff --git a/src/main/resources/configurations/bukkit.yml b/src/main/resources/configurations/bukkit.yml
index 4034c6f..a1fe4fe 100644
--- a/src/main/resources/configurations/bukkit.yml
+++ b/src/main/resources/configurations/bukkit.yml
@@ -61,6 +61,7 @@ world-settings:
entity-tracking-range-monsters: 48
entity-tracking-range-misc: 32
entity-tracking-range-max: 64
+ maps-enabled: true
world:
growth-chunks-per-tick: 1000
world_nether:
2013-03-23 09:08:49 +01:00
--
1.8.1-rc2
