Correctly remove faction warps in unclaimed land

src/main/java/com/massivecraft/factions/Faction.java

@@ -25,8 +25,6 @@ public interface Faction extends EconomyParticipator {
 
     public boolean removeWarp(String name);
 
-    public boolean removeWarp(LazyLocation loc);
-
     public void clearWarps();
 
     public void addAnnouncement(FPlayer fPlayer, String msg);

src/main/java/com/massivecraft/factions/zcore/persist/MemoryBoard.java

@@ -47,9 +47,10 @@ public abstract class MemoryBoard extends Board {
 
     public void removeAt(FLocation flocation) {
         Faction faction = getFactionAt(flocation);
-        for (LazyLocation loc : faction.getWarps().values()) {
+        Iterator<LazyLocation> it = faction.getWarps().values().iterator();
-            if (flocation.isInChunk(loc.getLocation())) {
+        while (it.hasNext()) {
-                faction.removeWarp(loc);
+            if (flocation.isInChunk(it.next().getLocation())) {
+                it.remove();
             }
         }
         clearOwnershipAt(flocation);

src/main/java/com/massivecraft/factions/zcore/persist/MemoryFaction.java

@@ -86,19 +86,7 @@ public abstract class MemoryFaction implements Faction, EconomyParticipator {
     }
 
     public boolean removeWarp(String name) {
-        if (warps.containsKey(name)) {
+        return warps.remove(name) != null;
-            warps.remove(name);
-            return true;
-        }
-        return false;
-    }
-
-    public boolean removeWarp(LazyLocation loc) {
-        if (warps.containsValue(loc)) {
-            warps.remove(loc);
-            return true;
-        }
-        return false;
     }
 
     public void clearWarps() {