ground items: remove unnecessary copy of collected ground items

Asonn · Jun 15, 2018 · 3cdc2be · 3cdc2be
1 parent 5e916c5
commit 3cdc2be
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/...lite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsOverlay.java b/...lite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsOverlay.java
@@ -31,9 +31,8 @@
 import java.awt.Graphics2D;
 import java.awt.Rectangle;
 import java.util.AbstractMap.SimpleEntry;
-import java.util.ArrayList;
+import java.util.Collection;
 import java.util.HashMap;
-import java.util.List;
 import java.util.Map;
 import javax.inject.Inject;
 import net.runelite.api.Client;
@@ -108,7 +107,7 @@ public Dimension render(Graphics2D graphics)
 		offsetMap.clear();
 		final LocalPoint localLocation = player.getLocalLocation();
 		final Point mousePos = client.getMouseCanvasPosition();
-		final List<GroundItem> groundItemList = new ArrayList<>(plugin.getCollectedGroundItems().values());
+		final Collection<GroundItem> groundItemList = plugin.getCollectedGroundItems().values();
 		GroundItem topGroundItem = null;
 
 		if (plugin.isHotKeyPressed())

diff --git a/runelite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsPlugin.java b/runelite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsPlugin.java
@@ -190,7 +190,6 @@ protected void shutDown() throws Exception
 		mouseManager.unregisterMouseListener(inputListener);
 		keyManager.unregisterKeyListener(inputListener);
 		groundItems.clear();
-		collectedGroundItems.clear();
 		highlightedItems.invalidateAll();
 		highlightedItems = null;
 		hiddenItems.invalidateAll();