Merge pull request #85 from holmraven/main

Fix concurrent exception #80 #83
Pepperoni-Jabroni · Jan 13, 2023 · 6be58a5 · 6be58a5
2 parents 684ec3c + 248e0c1
commit 6be58a5
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/src/main/java/pepjebs/mapatlases/lifecycle/MapAtlasesServerLifecycleEvents.java b/src/main/java/pepjebs/mapatlases/lifecycle/MapAtlasesServerLifecycleEvents.java
@@ -147,12 +147,7 @@ public static void mapAtlasServerTick(MinecraftServer server) {
         // Clean up disconnected players in server tick
         // since when using Disconnect event, the tick will sometimes
         // re-add the Player after they disconnect
-        // TODO: Fix concurrent exception here?
-        for (String playerName : playerToActiveMapId.keySet()) {
-            if (!seenPlayers.contains(playerName)) {
-                playerToActiveMapId.remove(playerName);
-            }
-        }
+        playerToActiveMapId.keySet().removeIf(playerName -> !seenPlayers.contains(playerName));
     }
 
     private static void updateMapDataForPlayer(