From 625b08c947b38b794d41c6e4a3717ef3eec2ce0e Mon Sep 17 00:00:00 2001 From: md_5 Date: Sat, 23 Mar 2013 19:09:47 +1100 Subject: [PATCH] Remove extra import. --- .../0045-Limit-Custom-Map-Rendering.patch | 22 ++++++------------- 1 file changed, 7 insertions(+), 15 deletions(-) diff --git a/CraftBukkit-Patches/0045-Limit-Custom-Map-Rendering.patch b/CraftBukkit-Patches/0045-Limit-Custom-Map-Rendering.patch index 623f8fc25a..ed8e5ccd13 100644 --- a/CraftBukkit-Patches/0045-Limit-Custom-Map-Rendering.patch +++ b/CraftBukkit-Patches/0045-Limit-Custom-Map-Rendering.patch @@ -1,27 +1,19 @@ -From fe1107743148622b05f1eb1c2a6f697f3a0261a5 Mon Sep 17 00:00:00 2001 +From e730f3308cb5b9849a4833b3eda0de48f56fcc58 Mon Sep 17 00:00:00 2001 From: md_5 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. --- - .../net/minecraft/server/WorldMapHumanTracker.java | 25 ++++++++++++++-------- + .../net/minecraft/server/WorldMapHumanTracker.java | 24 ++++++++++++++-------- .../org/bukkit/craftbukkit/map/CraftMapView.java | 2 +- - 2 files changed, 17 insertions(+), 10 deletions(-) + 2 files changed, 16 insertions(+), 10 deletions(-) diff --git a/src/main/java/net/minecraft/server/WorldMapHumanTracker.java b/src/main/java/net/minecraft/server/WorldMapHumanTracker.java -index bf3e6fe..c59ac18 100644 +index bf3e6fe..49449c9 100644 --- a/src/main/java/net/minecraft/server/WorldMapHumanTracker.java +++ b/src/main/java/net/minecraft/server/WorldMapHumanTracker.java -@@ -1,6 +1,7 @@ - package net.minecraft.server; - - import java.util.Iterator; -+import org.bukkit.map.MapCursor; - - public class WorldMapHumanTracker { - -@@ -41,23 +42,29 @@ public class WorldMapHumanTracker { +@@ -41,23 +41,29 @@ public class WorldMapHumanTracker { int i; int j; @@ -47,7 +39,7 @@ index bf3e6fe..c59ac18 100644 - abyte[i * 3 + 3] = (byte) cursor.getY(); + // Spigot start + for (Iterator iterator = ((custom) ? render.cursors.iterator() : this.worldMap.g.values().iterator()); iterator.hasNext(); ++i) { -+ org.bukkit.map.MapCursor cursor = (custom) ? (MapCursor) iterator.next() : null; ++ org.bukkit.map.MapCursor cursor = (custom) ? (org.bukkit.map.MapCursor) iterator.next() : null; + if (cursor != null && !cursor.isVisible()) continue; + WorldMapDecoration deco = (custom) ? null : (WorldMapDecoration) iterator.next(); + @@ -59,7 +51,7 @@ index bf3e6fe..c59ac18 100644 // CraftBukkit end boolean flag = !itemstack.z(); -@@ -92,7 +99,7 @@ public class WorldMapHumanTracker { +@@ -92,7 +98,7 @@ public class WorldMapHumanTracker { abyte1[2] = (byte) j; for (int i1 = 0; i1 < abyte1.length - 3; ++i1) {