Fix nether render in 1.1 and 1.2

pom.xml

@@ -2,7 +2,7 @@
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>org.dynmap</groupId>
 	<artifactId>dynmap</artifactId>
-	<version>0.31</version>
+	<version>0.34</version>
 	<name>dynmap</name>
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
@@ -133,7 +133,7 @@
 		<dependency>
 			<groupId>org.bukkit</groupId>
 			<artifactId>bukkit</artifactId>
-			<version>[1.1-R1-SNAPSHOT,)</version>
+			<version>[1.1-R5-SNAPSHOT,1.7)</version>
 			<type>jar</type>
 			<scope>compile</scope>
 		</dependency>
@@ -144,21 +144,6 @@
 			<type>jar</type>
 			<scope>compile</scope>
 		</dependency>
-		<dependency>
-			<groupId>javax.servlet</groupId>
-			<artifactId>javax.servlet-api</artifactId>
-			<version>3.0.1</version>
-		</dependency>
-		<dependency>
-			<groupId>org.eclipse.jetty</groupId>
-			<artifactId>jetty-server</artifactId>
-			<version>8.0.1.v20110908</version>
-		</dependency>
-		<dependency>
-			<groupId>org.eclipse.jetty</groupId>
-			<artifactId>jetty-servlet</artifactId>
-			<version>8.0.1.v20110908</version>
-		</dependency>
 
 		<dependency>
 			<groupId>org.dynmap</groupId>

src/main/java/org/dynmap/bukkit/BlockLightLevel.java

@@ -1,80 +0,0 @@
-package org.dynmap.bukkit;
-
-import java.lang.reflect.InvocationTargetException;
-import java.lang.reflect.Method;
-
-import org.bukkit.block.Block;
-import org.dynmap.Log;
-
-/**
- * Wrapper for accessing raw light levels for given block
- */
-public class BlockLightLevel {
-    private Method gethandle;
-    private Method getrawlight;
-    private Object enum_sky;
-    private Object enum_block;
-    private boolean ready;
-    
-    @SuppressWarnings({ "unchecked", "rawtypes" })
-    public BlockLightLevel() {
-        /* Get CraftChunk.getChunkSnapshot(boolean,boolean,boolean) and CraftChunk.getHandle() */
-        try {
-            Class c = Class.forName("org.bukkit.craftbukkit.CraftChunk");
-            gethandle = c.getDeclaredMethod("getHandle", new Class[0]);
-            Class enumskyblock = Class.forName("net.minecraft.server.EnumSkyBlock");
-            Object[] enumvals = enumskyblock.getEnumConstants();
-            for(int i = 0; i < enumvals.length; i++) {
-                String ev = enumvals[i].toString();
-                if(ev.equals("Sky")) {
-                    enum_sky = enumvals[i]; 
-                }
-                else if(ev.equals("Block")) {
-                    enum_block = enumvals[i];
-                }
-            }
-            Class cc = Class.forName("net.minecraft.server.Chunk");
-            getrawlight = cc.getDeclaredMethod("a", new Class[] { enumskyblock, int.class, int.class, int.class });
-        } catch (ClassNotFoundException cnfx) {
-        } catch (NoSuchMethodException nsmx) {
-        }
-        if((gethandle != null) && (enum_sky != null) && (enum_block != null) && (getrawlight != null)) {
-            ready = true;
-        }
-        else {
-            Log.warning("Block raw light level API not available");
-        }
-    }
-    
-    public boolean isReady() {
-        return ready;
-    }
-
-    public int getSkyLightLevel(Block b) {
-        try {
-            Object hand = gethandle.invoke(b.getChunk());
-            if(hand != null) {
-                Integer v = (Integer)getrawlight.invoke(hand, enum_sky, b.getX() & 0xF, b.getY() & 0x7F, b.getZ() & 0xF);
-                return v;
-            }
-        } catch (InvocationTargetException itx) {
-        } catch (IllegalArgumentException e) {
-        } catch (IllegalAccessException e) {
-        }
-        return -1;
-    }
-
-    public int getBlockLightLevel(Block b) {
-        try {
-            Object hand = gethandle.invoke(b.getChunk());
-            if(hand != null) {
-                Integer v = (Integer)getrawlight.invoke(hand, enum_block, b.getX() & 0xF, b.getY() & 0x7F, b.getZ() & 0xF);
-                return v;
-            }
-        } catch (InvocationTargetException itx) {
-        } catch (IllegalArgumentException e) {
-        } catch (IllegalAccessException e) {
-        }
-        return -1;
-    }
-}

src/main/java/org/dynmap/bukkit/BukkitEventProcessor.java

@@ -1,449 +0,0 @@
-package org.dynmap.bukkit;
-
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.List;
-
-import org.bukkit.event.CustomEventListener;
-import org.bukkit.event.Event;
-import org.bukkit.event.Listener;
-import org.bukkit.event.block.BlockBreakEvent;
-import org.bukkit.event.block.BlockBurnEvent;
-import org.bukkit.event.block.BlockFadeEvent;
-import org.bukkit.event.block.BlockFormEvent;
-import org.bukkit.event.block.BlockFromToEvent;
-import org.bukkit.event.block.BlockListener;
-import org.bukkit.event.block.BlockPhysicsEvent;
-import org.bukkit.event.block.BlockPistonExtendEvent;
-import org.bukkit.event.block.BlockPistonRetractEvent;
-import org.bukkit.event.block.BlockPlaceEvent;
-import org.bukkit.event.block.BlockSpreadEvent;
-import org.bukkit.event.block.LeavesDecayEvent;
-import org.bukkit.event.block.SignChangeEvent;
-import org.bukkit.event.entity.EntityExplodeEvent;
-import org.bukkit.event.entity.EntityListener;
-import org.bukkit.event.player.PlayerBedLeaveEvent;
-import org.bukkit.event.player.PlayerChatEvent;
-import org.bukkit.event.player.PlayerJoinEvent;
-import org.bukkit.event.player.PlayerListener;
-import org.bukkit.event.player.PlayerLoginEvent;
-import org.bukkit.event.player.PlayerMoveEvent;
-import org.bukkit.event.player.PlayerQuitEvent;
-import org.bukkit.event.server.PluginDisableEvent;
-import org.bukkit.event.server.PluginEnableEvent;
-import org.bukkit.event.server.ServerListener;
-import org.bukkit.event.world.ChunkLoadEvent;
-import org.bukkit.event.world.ChunkPopulateEvent;
-import org.bukkit.event.world.SpawnChangeEvent;
-import org.bukkit.event.world.WorldListener;
-import org.bukkit.event.world.WorldLoadEvent;
-import org.bukkit.event.world.WorldUnloadEvent;
-import org.bukkit.plugin.Plugin;
-import org.bukkit.plugin.PluginManager;
-import org.dynmap.Log;
-
-public class BukkitEventProcessor {
-    private Plugin plugin;
-    private PluginManager pm;
-
-    private HashMap<Event.Type, List<Listener>> event_handlers = new HashMap<Event.Type, List<Listener>>();
-
-    public BukkitEventProcessor(Plugin plugin) {
-        this.plugin = plugin;
-        this.pm = plugin.getServer().getPluginManager();
-    }
-    
-    public void cleanup() {
-        /* Clean up all registered handlers */
-        for(Event.Type t : event_handlers.keySet()) {
-            List<Listener> ll = event_handlers.get(t);
-            ll.clear(); /* Empty list - we use presence of list to remember that we've registered with Bukkit */
-        }
-        if(plugin.getServer().getVersion().contains("(MC: 1.0") == false) {
-            event_handlers.clear();
-        }
-    }
-    
-    private BlockListener ourBlockEventHandler = new BlockListener() {
-        
-        @Override
-        public void onBlockPlace(BlockPlaceEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockPlace(event);
-                }
-            }
-        }
-
-        @Override
-        public void onBlockBreak(BlockBreakEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockBreak(event);
-                }
-            }
-        }
-
-        @Override
-        public void onLeavesDecay(LeavesDecayEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onLeavesDecay(event);
-                }
-            }
-        }
-        
-        @Override
-        public void onBlockBurn(BlockBurnEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockBurn(event);
-                }
-            }
-        }
-        
-        @Override
-        public void onBlockForm(BlockFormEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockForm(event);
-                }
-            }
-        }
-        @Override
-        public void onBlockFade(BlockFadeEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockFade(event);
-                }
-            }
-        }
-        @Override
-        public void onBlockSpread(BlockSpreadEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockSpread(event);
-                }
-            }
-        }
-        @Override
-        public void onBlockFromTo(BlockFromToEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockFromTo(event);
-                }
-            }
-        }
-        @Override
-        public void onBlockPhysics(BlockPhysicsEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockPhysics(event);
-                }
-            }
-        }
-        @Override
-        public void onBlockPistonRetract(BlockPistonRetractEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockPistonRetract(event);
-                }
-            }
-        }
-        @Override
-        public void onBlockPistonExtend(BlockPistonExtendEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onBlockPistonExtend(event);
-                }
-            }
-        }
-        @Override
-        public void onSignChange(SignChangeEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((BlockListener)l).onSignChange(event);
-                }
-            }
-        }
-    };
-    private PlayerListener ourPlayerEventHandler = new PlayerListener() {
-        @Override
-        public void onPlayerJoin(PlayerJoinEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((PlayerListener)l).onPlayerJoin(event);
-                }
-            }
-        }
-        @Override
-        public void onPlayerLogin(PlayerLoginEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((PlayerListener)l).onPlayerLogin(event);
-                }
-            }
-        }
-
-        @Override
-        public void onPlayerMove(PlayerMoveEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((PlayerListener)l).onPlayerMove(event);
-                }
-            }
-        }
-        
-        @Override
-        public void onPlayerQuit(PlayerQuitEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((PlayerListener)l).onPlayerQuit(event);
-                }
-            }
-        }
-
-        @Override
-        public void onPlayerBedLeave(PlayerBedLeaveEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((PlayerListener)l).onPlayerBedLeave(event);
-                }
-            }
-        }
-
-        @Override
-        public void onPlayerChat(PlayerChatEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((PlayerListener)l).onPlayerChat(event);
-                }
-            }
-        }
-    };
-
-    private WorldListener ourWorldEventHandler = new WorldListener() {
-        @Override
-        public void onWorldLoad(WorldLoadEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((WorldListener)l).onWorldLoad(event);
-                }
-            }
-        }
-        @Override
-        public void onWorldUnload(WorldUnloadEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((WorldListener)l).onWorldUnload(event);
-                }
-            }
-        }
-        @Override
-        public void onChunkLoad(ChunkLoadEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((WorldListener)l).onChunkLoad(event);
-                }
-            }
-        }
-        @Override
-        public void onChunkPopulate(ChunkPopulateEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((WorldListener)l).onChunkPopulate(event);
-                }
-            }
-        }
-        @Override
-        public void onSpawnChange(SpawnChangeEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((WorldListener)l).onSpawnChange(event);
-                }
-            }
-        }
-    };
-    
-    private CustomEventListener ourCustomEventHandler = new CustomEventListener() {
-        @Override
-        public void onCustomEvent(Event event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((CustomEventListener)l).onCustomEvent(event);
-                }
-            }
-        }
-    };
-    
-    private EntityListener ourEntityEventHandler = new EntityListener() {
-        @Override
-        public void onEntityExplode(EntityExplodeEvent event) {
-            if(event.isCancelled())
-                return;
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((EntityListener)l).onEntityExplode(event);
-                }
-            }
-        }
-    };
-    
-    private ServerListener ourServerEventHandler = new ServerListener() {
-    	@Override
-        public void onPluginEnable(PluginEnableEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((ServerListener)l).onPluginEnable(event);
-                }
-            }
-    	}
-    	@Override
-        public void onPluginDisable(PluginDisableEvent event) {
-            /* Call listeners */
-            List<Listener> ll = event_handlers.get(event.getType());
-            if(ll != null) {
-                for(Listener l : ll) {
-                    ((ServerListener)l).onPluginDisable(event);
-                }
-            }
-    	}
-    };
-    
-    /**
-     * Register event listener - this will be cleaned up properly on a /dynmap reload, unlike
-     * registering with Bukkit directly
-     */
-    public void registerEvent(Event.Type type, Listener listener) {
-        List<Listener> ll = event_handlers.get(type);
-        if(ll == null) {
-            switch(type) {  /* See if it is a type we're brokering */
-                case PLAYER_LOGIN:
-                case PLAYER_CHAT:
-                case PLAYER_JOIN:
-                case PLAYER_QUIT:
-                case PLAYER_MOVE:
-                case PLAYER_BED_LEAVE:
-                    pm.registerEvent(type, ourPlayerEventHandler, Event.Priority.Monitor, plugin);
-                    break;
-                case BLOCK_PLACE:
-                case BLOCK_BREAK:
-                case LEAVES_DECAY:
-                case BLOCK_BURN:
-                case BLOCK_FORM:
-                case BLOCK_FADE:
-                case BLOCK_SPREAD:
-                case BLOCK_FROMTO:
-                case BLOCK_PHYSICS:
-                case BLOCK_PISTON_EXTEND:
-                case BLOCK_PISTON_RETRACT:
-                    pm.registerEvent(type, ourBlockEventHandler, Event.Priority.Monitor, plugin);
-                    break;
-                case SIGN_CHANGE:
-                    pm.registerEvent(type, ourBlockEventHandler, Event.Priority.Low, plugin);
-                    break;
-                case WORLD_LOAD:
-                case WORLD_UNLOAD:
-                case CHUNK_LOAD:
-                case CHUNK_POPULATED:
-                case SPAWN_CHANGE:
-                    pm.registerEvent(type, ourWorldEventHandler, Event.Priority.Monitor, plugin);
-                    break;
-                case CUSTOM_EVENT:
-                    pm.registerEvent(type, ourCustomEventHandler, Event.Priority.Monitor, plugin);
-                    break;
-                case ENTITY_EXPLODE:
-                    pm.registerEvent(type, ourEntityEventHandler, Event.Priority.Monitor, plugin);
-                    break;
-                case PLUGIN_ENABLE:
-                case PLUGIN_DISABLE:
-                	pm.registerEvent(type, ourServerEventHandler, Event.Priority.Monitor, plugin);
-                	break;
-                default:
-                    Log.severe("registerEvent() in DynmapPlugin does not handle " + type);
-                    return;
-            }
-            ll = new ArrayList<Listener>();
-            event_handlers.put(type, ll);   /* Add list for this event */
-        }
-        ll.add(listener);
-    }
-}

src/main/java/org/dynmap/bukkit/BukkitWorld.java

@@ -13,10 +13,9 @@ import org.dynmap.utils.MapChunkCache;
 
 public class BukkitWorld extends DynmapWorld {
     private World world;
-    private static BlockLightLevel bll = new BlockLightLevel();
     
     public BukkitWorld(World w) {
-        super(w.getName(), w.getMaxHeight());
+        super(normalizeWorldName(w.getName()), w.getMaxHeight(), w.getSeaLevel());
         
         world = w;
     }
@@ -31,7 +30,7 @@ public class BukkitWorld extends DynmapWorld {
         DynmapLocation dloc = new DynmapLocation();
         Location sloc = world.getSpawnLocation();
         dloc.x = sloc.getBlockX(); dloc.y = sloc.getBlockY();
-        dloc.z = sloc.getBlockZ(); dloc.world = sloc.getWorld().getName();
+        dloc.z = sloc.getBlockZ(); dloc.world = normalizeWorldName(sloc.getWorld().getName());
         return dloc;
     }
     /* Get world time */
@@ -67,12 +66,12 @@ public class BukkitWorld extends DynmapWorld {
     /* Test if sky light level is requestable */
     @Override
     public boolean canGetSkyLightLevel() {
-        return bll.isReady();
+        return true;
     }
     /* Return sky light level */
     @Override
     public int getSkyLightLevel(int x, int y, int z) {
-        return bll.getSkyLightLevel(world.getBlockAt(x, y, z));
+        return world.getBlockAt(x, y, z).getLightFromSky();
     }
     /**
      * Get world environment ID (lower case - normal, the_end, nether)
@@ -94,4 +93,8 @@ public class BukkitWorld extends DynmapWorld {
     public World getWorld() {
         return world;
     }
+    
+    public static String normalizeWorldName(String n) {
+        return n.replace('/', '-');
+    }
 }

src/main/java/org/dynmap/bukkit/NewMapChunkCache.java

@@ -6,19 +6,16 @@ import java.lang.reflect.Method;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.ListIterator;
-import java.util.TreeSet;
 
 import org.bukkit.World;
 import org.bukkit.Chunk;
 import org.bukkit.block.Biome;
 import org.bukkit.entity.Entity;
 import org.bukkit.ChunkSnapshot;
-import org.dynmap.DynmapAPI;
 import org.dynmap.DynmapChunk;
 import org.dynmap.DynmapCore;
 import org.dynmap.DynmapWorld;
 import org.dynmap.Log;
-import org.dynmap.MapManager;
 import org.dynmap.common.BiomeMap;
 import org.dynmap.utils.MapChunkCache;
 import org.dynmap.utils.MapIterator;
@@ -35,16 +32,12 @@ public class NewMapChunkCache implements MapChunkCache {
     private static Method removeentities = null;
     private static Method getworldhandle = null;    
     private static Field  chunkbiome = null;
-    private static Field ticklist = null;
-    private static Method processticklist = null;
     private static boolean use_spout = false;
 
-    private static final int MAX_PROCESSTICKS = 20;
-    private static final int MAX_TICKLIST = 20000;
-
     private World w;
     private DynmapWorld dw;
     private Object craftworld;
+    private int nsect;
     private List<DynmapChunk> chunks;
     private ListIterator<DynmapChunk> iterator;
     private int x_min, x_max, z_min, z_max;
@@ -57,7 +50,9 @@ public class NewMapChunkCache implements MapChunkCache {
     private boolean do_save = false;
     private boolean isempty = true;
     private ChunkSnapshot[] snaparray; /* Index = (x-x_min) + ((z-z_min)*x_dim) */
-    private TreeSet<?> ourticklist;
+    private byte[][] swampcnt;
+    private BiomeMap[][] biomemap;
+    private boolean[][] isSectionNotEmpty; /* Indexed by snapshot index, then by section index */
     
     private int chunks_read;    /* Number of chunks actually loaded */
     private int chunks_attempted;   /* Number of chunks attempted to load */
@@ -79,12 +74,15 @@ public class NewMapChunkCache implements MapChunkCache {
         private BlockStep laststep;
         private int typeid = -1;
         private int blkdata = -1;
-        private int lastbiome_x = Integer.MIN_VALUE, lastbiome_z = Integer.MIN_VALUE;
-        private BiomeMap lastbiome;
-        private int lastcountswamp_x = Integer.MIN_VALUE, lastcountswamp_z = Integer.MIN_VALUE, lastcountswamp;
         private final int worldheight;
+        private final int x_base;
+        private final int z_base;
         
         OurMapIterator(int x0, int y0, int z0) {
+            x_base = x_min << 4;
+            z_base = z_min << 4;
+            if(biome)
+                biomePrep();
             initialize(x0, y0, z0);
             worldheight = w.getMaxHeight();
         }
@@ -121,53 +119,94 @@ public class NewMapChunkCache implements MapChunkCache {
         public final int getBlockEmittedLight() {
             return snap.getBlockEmittedLight(bx, y, bz);
         }
-        public final BiomeMap getBiome() {
-            if((x == lastbiome_x) && (z == lastbiome_z)) {
-                return lastbiome;
+        private void biomePrep() {
+            if(swampcnt != null)
+                return;
+            int x_size = x_dim << 4;
+            int z_size = (z_max - z_min + 1) << 4;
+            swampcnt = new byte[x_size][];
+            biomemap = new BiomeMap[x_size][];
+            for(int i = 0; i < x_size; i++) {
+                swampcnt[i] = new byte[z_size];
+                biomemap[i] = new BiomeMap[z_size];
+            }
+            for(int i = 0; i < x_size; i++) {
+                initialize(i + x_base, 64, z_base);
+                for(int j = 0; j < z_size; j++) {
+                    Biome bb = snap.getBiome(bx, bz);
+                    if(bb == null)
+                        biomemap[i][j] = BiomeMap.NULL;
+                    else
+                        biomemap[i][j] = biome_to_bmap[bb.ordinal()];
+                    if(biomemap[i][j] == BiomeMap.SWAMPLAND) {
+                        for(int ii = i-1; ii < i+2; ii++) {
+                            for(int jj = j-1; jj < j+2; jj++) {
+                                if(ii < 0) continue;
+                                if(jj < 0) continue;
+                                if(ii >= x_size) continue;
+                                if(jj >= z_size) continue;
+                                swampcnt[ii][jj]++;
+                            }
+                        }
+                    }
+                    stepPosition(BlockStep.Z_PLUS);
+                }
             }
-            BiomeMap bio;
-            Biome bb = snap.getBiome(bx, bz);
-            if(bb != null)
-                bio = biome_to_bmap[bb.ordinal()];
-            else
-                bio = BiomeMap.NULL;
-            lastbiome_x = x;
-            lastbiome_z = z;
-            lastbiome = bio;
-            return bio;
         }
+        
+        public final BiomeMap getBiome() {
+            try {
+                return biomemap[x - x_base][z - z_base];
+            } catch (Exception ex) {
+                return BiomeMap.NULL;
+            }
+        }
+        
         public final int  countSmoothedSwampBiomes() {
-            if((lastcountswamp_x == x) && (lastcountswamp_z == z))
-                return lastcountswamp;
-            int cnt = 0; 
-            BlockStep s_bak = laststep;
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.X_MINUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.Z_MINUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.X_PLUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.X_PLUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.Z_PLUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.Z_PLUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.X_MINUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.X_MINUS);
-            if(getBiome() == BiomeMap.SWAMPLAND) cnt++;
-            stepPosition(BlockStep.X_PLUS);
-            stepPosition(BlockStep.Z_MINUS);
-            laststep = s_bak;
-            
-            lastcountswamp_x = x;
-            lastcountswamp_z = z;
-            lastcountswamp = cnt;
-            
-            return cnt;
+            try {
+                return swampcnt[x - x_base][z - z_base];
+            } catch (Exception ex) {
+                return 0;
+            }
         }
+        
+        public final int  countSmoothedSwampBiomes(int sx, int sz, int scale) {
+            try {
+                int xx = x - x_base;
+                int zz = z - z_base;
+                sx <<= 1;
+                sz <<= 1;
+                int s0 = swampcnt[xx][zz];
+                int w;
+                int tot;
+                if(sx < scale) {
+                    w = scale - sx;
+                    tot = (w * swampcnt[xx-1][zz]) + ((scale - w) * s0);
+                }
+                else if(sx > scale) {
+                    w = sx - scale;
+                    tot = (w * swampcnt[xx+1][zz]) + ((scale - w) * s0);
+                }
+                else {
+                    tot = scale * s0;
+                }
+                if(sz < scale) {
+                    w = scale - sz;
+                    tot += (w * swampcnt[xx][zz-1]) + ((scale - w) * s0);
+                }
+                else if(sz > scale) {
+                    w = sz - scale;
+                    tot += (w * swampcnt[xx][zz+1]) + ((scale - w) * s0);
+                }
+                else {
+                    tot += scale * s0;
+                }
+                return tot;
+            } catch (Exception ex) {
+                return 0;
+            }
+        }
+        
         public final double getRawBiomeTemperature() {
             return snap.getRawBiomeTemperature(bx, bz);
         }
@@ -312,6 +351,16 @@ public class NewMapChunkCache implements MapChunkCache {
         public int getWorldHeight() {
             return worldheight;
         }
+        @Override
+        public long getBlockKey() {
+            return (((chunkindex * worldheight) + y) << 8) | (bx << 4) | bz;
+        }
+        @Override
+        public final boolean isEmptySection() {
+            if(isSectionNotEmpty[chunkindex] == null)
+                initSectionData(chunkindex);
+             return !isSectionNotEmpty[chunkindex][y >> 4];
+        }
      }
 
     private class OurEndMapIterator extends OurMapIterator {
@@ -357,6 +406,12 @@ public class NewMapChunkCache implements MapChunkCache {
         public double getRawBiomeRainfall(int x, int z) {
             return 0.0;
         }
+        public boolean isSectionEmpty(int sy) {
+            return true;
+        }
+        public int getTopNonEmptySection() {
+            return -1;
+        }
     }
 
     /**
@@ -392,6 +447,12 @@ public class NewMapChunkCache implements MapChunkCache {
         public final int getHighestBlockYAt(int x, int z) {
             return 64;
         }
+        public boolean isSectionEmpty(int sy) {
+            return (sy < 4);
+        }
+        public int getTopNonEmptySection() {
+            return 3;
+        }
     }
     
     private static class SpoutChunkSnapshot implements ChunkSnapshot {
@@ -428,6 +489,14 @@ public class NewMapChunkCache implements MapChunkCache {
         public final int getHighestBlockYAt(int x, int z) {
             return chunk.getHighestBlockYAt(x, z);
         }
+        public boolean isSectionEmpty(int sy) {
+            //TODO return chunk.isSectionEmpty(sy);
+            return false;
+        }
+        public int getTopNonEmptySection() {
+            return 0;
+            //TODO return chunk.getTopNonEmptySection();
+        }
     }
 
     private static final EmptyChunk EMPTY = new EmptyChunk();
@@ -472,22 +541,6 @@ public class NewMapChunkCache implements MapChunkCache {
             } catch (ClassNotFoundException cnfx) {
             } catch (NoSuchFieldException nsmx) {
             }
-            /* ticklist for World */
-            try {
-                Class c = Class.forName("net.minecraft.server.World");
-                try {
-                    ticklist = c.getDeclaredField("K"); /* 1.0.0 */
-                } catch (NoSuchFieldException nsfx) {
-                    ticklist = c.getDeclaredField("N"); /* 1.8.1 */
-                }
-                ticklist.setAccessible(true);
-                if(ticklist.getType().isAssignableFrom(TreeSet.class) == false)
-                    ticklist = null;
-                processticklist = c.getDeclaredMethod("a", new Class[] { boolean.class } );
-            } catch (ClassNotFoundException cnfx) {
-            } catch (NoSuchFieldException nsmx) {
-            } catch (NoSuchMethodException nsmx) {
-            }
             use_spout = DynmapPlugin.plugin.hasSpout();
             
             init = true;
@@ -496,11 +549,10 @@ public class NewMapChunkCache implements MapChunkCache {
     public void setChunks(BukkitWorld dw, List<DynmapChunk> chunks) {
         this.dw = dw;
         this.w = dw.getWorld();
+        nsect = dw.worldheight >> 4;
         if((getworldhandle != null) && (craftworld == null)) {
             try {
                 craftworld = getworldhandle.invoke(w);   /* World.getHandle() */
-                if(ticklist != null)
-                    ourticklist = (TreeSet<?>)ticklist.get(craftworld);
             } catch (Exception x) {
             }
         }
@@ -530,6 +582,7 @@ public class NewMapChunkCache implements MapChunkCache {
         }
     
         snaparray = new ChunkSnapshot[x_dim * (z_max-z_min+1)];
+        isSectionNotEmpty = new boolean[x_dim * (z_max-z_min+1)][];
     }
     
     private ChunkSnapshot checkSpoutData(Chunk c, ChunkSnapshot ss) {
@@ -549,8 +602,6 @@ public class NewMapChunkCache implements MapChunkCache {
         if(iterator == null)
             iterator = chunks.listIterator();
 
-        checkTickList();
-        
         DynmapCore.setIgnoreChunkLoads(true);
         //boolean isnormral = w.getEnvironment() == Environment.NORMAL;
         // Load the required chunks.
@@ -575,7 +626,7 @@ public class NewMapChunkCache implements MapChunkCache {
                 }
             }
             /* Check if cached chunk snapshot found */
-            ChunkSnapshot ss = DynmapPlugin.plugin.sscache.getSnapshot(w.getName(), chunk.x, chunk.z, blockdata, biome, biomeraw, highesty); 
+            ChunkSnapshot ss = DynmapPlugin.plugin.sscache.getSnapshot(dw.getName(), chunk.x, chunk.z, blockdata, biome, biomeraw, highesty); 
             if(ss != null) {
                 if(!vis) {
                     if(hidestyle == HiddenChunkStyle.FILL_STONE_PLAIN)
@@ -616,7 +667,7 @@ public class NewMapChunkCache implements MapChunkCache {
                     else
                         ss = w.getEmptyChunkSnapshot(chunk.x, chunk.z, biome, biomeraw);
                     if(ss != null) {
-                        DynmapPlugin.plugin.sscache.putSnapshot(w.getName(), chunk.x, chunk.z, ss, blockdata, biome, biomeraw, highesty);
+                        DynmapPlugin.plugin.sscache.putSnapshot(dw.getName(), chunk.x, chunk.z, ss, blockdata, biome, biomeraw, highesty);
                     }
                 }
                 snaparray[(chunk.x-x_min) + (chunk.z - z_min)*x_dim] = ss;
@@ -744,6 +795,36 @@ public class NewMapChunkCache implements MapChunkCache {
         ChunkSnapshot ss = snaparray[((x>>4) - x_min) + ((z>>4) - z_min) * x_dim];
         return ss.getRawBiomeRainfall(x & 0xF, z & 0xF);
     }
+    private void initSectionData(int idx) {
+        isSectionNotEmpty[idx] = new boolean[nsect + 1];
+        int maxy = 0;
+        if(snaparray[idx] != EMPTY) {
+            /* Get max height */
+            if(dw.isNether()) {
+                maxy = 128;
+            }
+            else {
+                for(int i = 0; i < 16; i++) {
+                    for(int j = 0; j < 16; j++) {
+                        maxy = Math.max(maxy, snaparray[idx].getHighestBlockYAt(i, j));
+                    }
+                }
+            }
+            maxy = (maxy-1) >> 4;
+            for(int i = 0; i <= nsect; i++) {
+                if(i <= maxy) { /* Below top? */
+                    isSectionNotEmpty[idx][i] = true;
+                }
+            }
+        }
+    }
+    public boolean isEmptySection(int sx, int sy, int sz) {
+        int idx = (sx - x_min) + (sz - z_min) * x_dim;
+        if(isSectionNotEmpty[idx] == null) {
+            initSectionData(idx);
+        }
+        return !isSectionNotEmpty[idx][sy];
+    }
 
     /**
      * Get cache iterator
@@ -820,7 +901,8 @@ public class NewMapChunkCache implements MapChunkCache {
     public boolean setChunkDataTypes(boolean blockdata, boolean biome, boolean highestblocky, boolean rawbiome) {
         this.biome = biome;
         this.biomeraw = rawbiome;
-        this.highesty = highestblocky;
+        //this.highesty = highestblocky;
+        this.highesty = true;
         this.blockdata = blockdata;
         return true;
     }
@@ -845,27 +927,6 @@ public class NewMapChunkCache implements MapChunkCache {
         return exceptions;
     }
     
-    private boolean checkTickList() {
-        boolean isok = true;
-        if((ourticklist != null) && (processticklist != null)) {
-            int cnt = 0;
-            int ticksize = ourticklist.size();
-            while((cnt < MAX_PROCESSTICKS) && (ticksize > MAX_TICKLIST) && (ourticklist.size() > MAX_TICKLIST)) {
-                try {
-                    processticklist.invoke(craftworld, true);
-                } catch (Exception x) {
-                }
-                ticksize -= 1000;
-                cnt++;
-                MapManager.mapman.incExtraTickList();
-            }
-            if(cnt >= MAX_PROCESSTICKS) {   /* If still behind, delay processing */
-                isok = false;
-            }
-        }
-        return isok;
-    }
-
     static {
         Biome[] b = Biome.values();
         BiomeMap[] bm = BiomeMap.values();

src/main/java/org/dynmap/herochat/HeroChatHandler.java

@@ -1,302 +0,0 @@
-package org.dynmap.herochat;
-
-import java.lang.reflect.Method;
-import java.util.Collections;
-import java.util.List;
-
-import org.bukkit.event.CustomEventListener;
-import org.bukkit.event.Event;
-import org.bukkit.event.server.PluginEnableEvent;
-import org.bukkit.event.server.ServerListener;
-import org.bukkit.plugin.Plugin;
-import org.dynmap.Client;
-import org.dynmap.ConfigurationNode;
-import org.dynmap.DynmapCore;
-import org.dynmap.Log;
-import org.dynmap.bukkit.DynmapPlugin;
-
-/**
- * Bukkit specific module - alternate chat handler for interfacing with HeroChat
- */
-public class HeroChatHandler {
-    private static final String DEF_CHANNEL = "Global";
-    private static final List<String> DEF_CHANNELS = Collections
-            .singletonList(DEF_CHANNEL);
-
-    private List<String> hcchannels;
-    private String hcwebinputchannel;
-    private DynmapCore core;
-    private HeroChatChannel hcwebinputchan;
-
-    private class OurPluginListener extends ServerListener {
-        @Override
-        public void onPluginEnable(PluginEnableEvent event) {
-            Plugin plugin = event.getPlugin();
-            String name = plugin.getDescription().getName();
-
-            if (name.equals("HeroChat")) {
-                activateHeroChat(plugin);
-            }
-        }
-    }
-
-    /* Reflection-based access wrapper for ChannelChatEvent from HeroChat */
-    private static class HeroChatChannelChatEvent {
-        @SuppressWarnings("rawtypes")
-        private static Class channelchatevent;
-        private static Method getsource;
-        private static Method getmessage;
-        private static Method issentbyplayer;
-        private static boolean isgood = false;
-        private Event evt;
-
-        @SuppressWarnings("unchecked")
-        public static boolean initialize() {
-            try {
-                channelchatevent = Class
-                        .forName("com.herocraftonline.dthielke.herochat.event.ChannelChatEvent");
-                getsource = channelchatevent.getMethod("getSource", new Class[0]);
-                getmessage = channelchatevent.getMethod("getMessage", new Class[0]);
-                issentbyplayer = channelchatevent.getMethod("isSentByPlayer", new Class[0]);
-                isgood = true;
-            } catch (ClassNotFoundException cnfx) {
-            } catch (NoSuchMethodException nsmx) {
-            }
-            return isgood;
-        }
-
-        public HeroChatChannelChatEvent(Event evt) {
-            this.evt = evt;
-        }
-
-        public static boolean isInstance(Event evt) {
-            return channelchatevent.isInstance(evt);
-        }
-
-        public String getSource() {
-            try {
-                return (String) getsource.invoke(evt);
-            } catch (Exception x) {
-                return null;
-            }
-        }
-
-        public String getMessage() {
-            try {
-                return (String) getmessage.invoke(evt);
-            } catch (Exception x) {
-                return null;
-            }
-        }
-        
-        public boolean isSentByPlayer() {
-            try {
-                return (Boolean) issentbyplayer.invoke(evt);
-            } catch (Exception x) {
-                return true;
-            }
-        }
-    }
-
-    /* Reflection-based access wrapper for ChannelEvent from HeroChat */
-    private static class HeroChatChannelEvent {
-        @SuppressWarnings("rawtypes")
-        private static Class channelevent;
-        private static Method getchannel;
-        private static Method iscancelled;
-        private static boolean isgood = false;
-        private Event evt;
-
-        @SuppressWarnings("unchecked")
-        public static boolean initialize() {
-            try {
-                channelevent = Class
-                        .forName("com.herocraftonline.dthielke.herochat.event.ChannelEvent");
-                getchannel = channelevent.getMethod("getChannel", new Class[0]);
-                iscancelled = channelevent.getMethod("isCancelled", new Class[0]);
-                isgood = true;
-            } catch (ClassNotFoundException cnfx) {
-            } catch (NoSuchMethodException nsmx) {
-            }
-            return isgood;
-        }
-
-        public HeroChatChannelEvent(Event evt) {
-            this.evt = evt;
-        }
-
-        public static boolean isInstance(Event evt) {
-            return channelevent.isInstance(evt);
-        }
-
-        public HeroChatChannel getChannel() {
-            try {
-                Object o;
-                o = getchannel.invoke(evt);
-                if (o != null) {
-                    return new HeroChatChannel(o);
-                }
-            } catch (Exception x) {
-            }
-            return null;
-        }
-
-        public boolean isCancelled() {
-            try {
-                return (Boolean) iscancelled.invoke(evt);
-            } catch (Exception x) {
-                return true;
-            }
-        }
-    }
-
-    /* Reflection-based access wrapper for Channel from HeroChat */
-    private static class HeroChatChannel {
-        @SuppressWarnings("rawtypes")
-        private static Class channel;
-        private static Method getname;
-        private static Method getnick;
-        private static Method sendmessage;
-        private static boolean isgood = false;
-        private Object chan;
-
-        @SuppressWarnings("unchecked")
-        public static boolean initialize() {
-            try {
-                channel = Class
-                        .forName("com.herocraftonline.dthielke.herochat.channels.Channel");
-                getname = channel.getMethod("getName");
-                getnick = channel.getMethod("getNick", new Class[0]);
-                sendmessage = channel.getMethod("sendMessage", new Class[] {
-                    String.class, String.class, String.class, boolean.class } );
-                isgood = true;
-            } catch (ClassNotFoundException cnfx) {
-            } catch (NoSuchMethodException nsmx) {
-                Log.severe(nsmx);
-            }
-            return isgood;
-        }
-
-        public HeroChatChannel(Object chan) {
-            this.chan = chan;
-        }
-
-        public String getName() {
-            try {
-                return (String) getname.invoke(chan);
-            } catch (Exception x) {
-                return null;
-            }
-        }
-
-        public String getNick() {
-            try {
-                return (String) getnick.invoke(chan);
-            } catch (Exception x) {
-                return null;
-            }
-        }
-
-        public void sendMessage(String source, String msg, String format, boolean sentByPlayer) {
-            try {
-                sendmessage.invoke(chan, source, msg, format, sentByPlayer);
-            } catch (Exception x) {
-            }
-        }
-    }
-
-    private class OurEventListener extends CustomEventListener {
-        /**
-         * Handle custom events
-         */
-        @Override
-        public void onCustomEvent(Event event) {
-            if (HeroChatChannelEvent.isInstance(event)) {
-                HeroChatChannelEvent ce = new HeroChatChannelEvent(event);
-                /* Snoop for our web channel - we'll need it, and we'll see it before it matters,
-                 * since anyone that joins the channel will give us an event (and reflection on
-                 * the plugin class to get the manager didn't work, due to a dependency on the IRC
-                 * plugin that may not be present....)
-                 */
-                HeroChatChannel c = ce.getChannel();
-                if (ce.isCancelled())
-                    return;
-                if((hcwebinputchannel != null) && ((hcwebinputchannel.equals(c.getName())) ||
-                        (hcwebinputchannel.equals(c.getNick())))) {
-                    hcwebinputchan = c;
-                }                
-                if (HeroChatChannelChatEvent.isInstance(event)) {
-                    HeroChatChannelChatEvent cce = new HeroChatChannelChatEvent(
-                        event);
-                    /* Match on name or nickname of channel */
-                    if (hcchannels.contains(c.getName()) ||
-                            hcchannels.contains(c.getNick())) {
-                        if(cce.isSentByPlayer()) {  /* Player message? */
-                            org.bukkit.entity.Player p = DynmapPlugin.plugin.getServer().getPlayer(cce.getSource());
-                            if((p != null) && (core.mapManager != null)) {
-                                core.mapManager.pushUpdate(new Client.ChatMessage("player", 
-                                                                                    c.getNick(),
-                                                                                    p.getDisplayName(),
-                                                                                    cce.getMessage(),
-                                                                                    p.getName()));
-                            }
-                        }
-                    }
-                }
-            }
-        }
-    }
-
-    public HeroChatHandler(ConfigurationNode cfg, DynmapCore core) {
-        /* If we're enabling hero chat support */
-        Log.verboseinfo("HeroChat support configured");
-        this.core = core;
-        /* Now, get the monitored channel list */
-        hcchannels = cfg.getStrings("herochatchannels", DEF_CHANNELS);
-        /* And get channel to send web messages */
-        hcwebinputchannel = cfg.getString("herochatwebchannel", DEF_CHANNEL);
-        Plugin hc = DynmapPlugin.plugin.getServer().getPluginManager().getPlugin("HeroChat");
-        if(hc != null) {
-            activateHeroChat(hc);
-        }
-        else {
-            /* Set up to hear when HeroChat is enabled */
-            DynmapPlugin.plugin.bep.registerEvent(Event.Type.PLUGIN_ENABLE, new OurPluginListener());
-        }
-    }
-
-    private void activateHeroChat(Plugin herochat) {
-        if (HeroChatChannelChatEvent.initialize() == false) {
-            Log.severe("Cannot load HeroChat chat event class!");
-            return;
-        }
-        if (HeroChatChannel.initialize() == false) {
-            Log.severe("Cannot load HeroChat channel class!");
-            return;
-        }
-        if (HeroChatChannelEvent.initialize() == false) {
-            Log.severe("Cannot load HeroChat channel event class!");
-            return;
-        }
-        /* Register event handler */
-        DynmapPlugin.plugin.bep.registerEvent(Event.Type.CUSTOM_EVENT, new OurEventListener());
-        Log.verboseinfo("HeroChat integration active");
-    }
-    /**
-     * Send message from web to appropriate HeroChat channel
-     * @param sender - sender ID
-     * @param message - message
-     * @return true if herochat is handling this, false if not
-     */
-    public boolean sendWebMessageToHeroChat(String sender, String message) {
-        if(hcwebinputchannel != null) { /* Are we handling them? */
-            if(hcwebinputchan != null) {    /* Have we seen it yet?  Maybe no if nobody has logged on or
-                                             * joined it, but then who would see it anyway?
-                                             */
-                hcwebinputchan.sendMessage(sender, message, "{default}", false);
-            }
-            return true;
-        }
-        return false;
-    }
-}

src/main/java/org/dynmap/herochat/HeroWebChatComponent.java

@@ -1,68 +0,0 @@
-package org.dynmap.herochat;
-
-import static org.dynmap.JSONUtils.s;
-
-import org.dynmap.ChatEvent;
-import org.dynmap.Client;
-import org.dynmap.Component;
-import org.dynmap.ConfigurationNode;
-import org.dynmap.DynmapCore;
-import org.dynmap.Event;
-import org.dynmap.common.DynmapListenerManager;
-import org.dynmap.common.DynmapListenerManager.EventType;
-import org.dynmap.common.DynmapPlayer;
-import org.json.simple.JSONObject;
-
-public class HeroWebChatComponent extends Component {
-    HeroChatHandler handler;
-    public HeroWebChatComponent(final DynmapCore plugin, ConfigurationNode configuration) {
-        super(plugin, configuration);
-        handler = new HeroChatHandler(configuration, plugin);
-        plugin.events.addListener("webchat", new Event.Listener<ChatEvent>() {
-            @Override
-            public void triggered(ChatEvent t) {
-                if(plugin.getServer().sendWebChatEvent(t.source, t.name, t.message)) {
-                    /* Let HeroChat take a look - only broadcast to players if it doesn't handle it */
-                    if (!handler.sendWebMessageToHeroChat(t.name, t.message)) {
-                        String msg;
-                        String msgfmt = plugin.configuration.getString("webmsgformat", null);
-                        if(msgfmt != null) {
-                            msgfmt = unescapeString(msgfmt);
-                            msg = msgfmt.replace("%playername%", t.name).replace("%message%", t.message);
-                        }
-                        else {
-                            msg = unescapeString(plugin.configuration.getString("webprefix", "\u00A72[WEB] ")) + t.name + ": " + unescapeString(plugin.configuration.getString("websuffix", "\u00A7f")) + t.message;
-                        }
-                        plugin.getServer().broadcastMessage(msg);
-                    }
-                }
-            }
-        });
-        
-        plugin.events.addListener("buildclientconfiguration", new Event.Listener<JSONObject>() {
-            @Override
-            public void triggered(JSONObject t) {
-                s(t, "allowchat", true);
-            }
-        });
-        
-        // Also make HeroChat announce joins and quits.
-        core.listenerManager.addListener(EventType.PLAYER_JOIN, new DynmapListenerManager.PlayerEventListener() {
-            @Override
-            public void playerEvent(DynmapPlayer p) { 
-                if((core.mapManager != null) && (core.playerList != null) && (core.playerList.isVisiblePlayer(p.getName()))) {
-                    core.mapManager.pushUpdate(new Client.PlayerJoinMessage(p.getDisplayName(), p.getName()));
-                }
-            }
-        });
-        core.listenerManager.addListener(EventType.PLAYER_QUIT, new DynmapListenerManager.PlayerEventListener() {
-            @Override
-            public void playerEvent(DynmapPlayer p) { 
-                if((core.mapManager != null) && (core.playerList != null) && (core.playerList.isVisiblePlayer(p.getName()))) {
-                    core.mapManager.pushUpdate(new Client.PlayerQuitMessage(p.getDisplayName(), p.getName()));
-                }
-            }
-        });
-        
-    }
-}

src/main/resources/configuration.txt

@@ -53,14 +53,6 @@ components:
     allowchat: true
     # If true, web UI users can supply name for chat using 'playername' URL parameter.  'trustclientname' must also be set true.
     allowurlname: false
-  #- class: org.dynmap.herochat.HeroWebChatComponent
-  #  # Control which HeroChat channel messages from web are directed to 
-  #  herochatwebchannel: Global
-  #  # Control which channels are monitored and reported to the web
-  #  herochatchannels:
-  #    - Global
-  #    #- Trade
-  #    #- Haggle
   
   # Note: this component is needed for the dmarker commands, and for the Marker API to be available to other plugins
   - class: org.dynmap.MarkersComponent
@@ -71,12 +63,13 @@ components:
     showspawn: true
     spawnicon: world
     spawnlabel: "Spawn"
-    # (optional) layer for showing offline player's positions
+    # (optional) layer for showing offline player's positions (for 'maxofflinetime' minutes after logoff)
     showofflineplayers: false
     offlinelabel: "Offline"
     offlineicon: offlineuser
     offlinehidebydefault: true
     offlineminzoom: 0
+    maxofflinetime: 30
     # (optional) layer for showing player's spawn beds
     showspawnbeds: false
     spawnbedlabel: "Spawn Beds"
@@ -97,6 +90,8 @@ components:
     messagettl: 5
     # Optional: set number of lines in scrollable message history: if set, messagettl is not used to age out messages
     #scrollback: 100
+    # Optional: send push button
+    sendbutton: false
   - class: org.dynmap.ClientComponent
     type: playermarkers
     showplayerfaces: true
@@ -186,6 +181,13 @@ enabletilehash: true
 # Optional - enabled BetterGrass style rendering of grass and snow block sides
 #better-grass: true
 
+# Optional - enable smooth lighting by default on all maps supporting it (can be set per map as lighting option)
+smooth-lighting: false
+
+# Default image format for HDMaps (png, jpg, jpg-q75, jpg-q80, jpg-q85, jpg-q90, jpg-q95, jpg-q100)
+# Has no effect on maps with explicit format settings
+image-format: png
+
 # Pre 0.29 render options - set all three to false if you want same results as 0.28 (i.e. you have existing maps and don't want to render)
 #  use-generated-textures: if true, use generated textures (same as client); false is static, pre 0.29 textures
 #  correct-water-lighting: if true, use corrected water lighting (same as client); false is legacy, pre 0.29 water (darker)
@@ -228,6 +230,9 @@ compass-mode: newnorth
 # Enable Kaevator's Superslopes block rendering support
 #superslopes-support: true
 
+# Enabled ComputerCraft block rendering support
+#computercraft-support: true
+
 render-triggers:
   #- playermove
   #- playerjoin
@@ -243,6 +248,8 @@ render-triggers:
   - explosion
   - blockfromto
   - blockphysics
+  - structuregrow
+  - blockgrow
 
 # Title for the web page - if not specified, defaults to the server's name (unless it is the default of 'Unknown Server')
 #webpage-title: "My Awesome Server Map"

src/main/resources/plugin.yml

@@ -64,7 +64,7 @@ commands:
            /<command> deleteicon id:<id> - remove icon 
            /<command> deleteicon <label> - remove icon 
            /<command> addcorner - add corner to corner list using current location
-           /<command> addcorner <x> <z> <world> - add corner with given x and z coordinate on given world to corner list
+           /<command> addcorner <x> <y> <z> <world> - add corner with given x, y and z coordinate on given world to corner list
            /<command> clearcorners - clear corner list
            /<command> addarea <label> - add new area with given label using corner list
            /<command> addarea id:<id> <label> - add new area with given ID using corner list
@@ -73,6 +73,13 @@ commands:
            /<command> listareas - list details of all areas
            /<command> updatearea <label> <arg>:<value> ... - update attributes of area with given label
            /<command> updatearea id:<id> <arg>:<value> ... - update attributes of area with given ID
+           /<command> addline <label> - add new poly-line with given label using corner list
+           /<command> addline id:<id> <label> - add new poly-line with given ID using corner list
+           /<command> deleteline <label> - delete poly-line with given label
+           /<command> deleteline id:<id> <label> - delete poly-line with given ID
+           /<command> listlines - list details of all poly-lines
+           /<command> updateline <label> <arg>:<value> ... - update attributes of poly-line with given label
+           /<command> updateline id:<id> <arg>:<value> ... - update attributes of poly-line with given ID
   dmap:
     description: List and modify dynmap configuration
     usage: |
@@ -240,6 +247,18 @@ permissions:
   dynmap.marker.deletearea:
     description: Allows /dmarker deletearea
     default: op
+  dynmap.marker.addline:
+    description: Allows /dmarker addline
+    default: op            
+  dynmap.marker.updateline:
+    description: Allows /dmarker updateline
+    default: op            
+  dynmap.marker.listlines:
+    description: Allows /dmarker listlines
+    default: op            
+  dynmap.marker.deleteline:
+    description: Allows /dmarker deleteline
+    default: op
   dynmap.dmap.worldlist:
     description: Allows /dmap worldlist
     default: op