From 06b034c86e20a1891a1ba839dc31bc9f5e2abbf2 Mon Sep 17 00:00:00 2001
From: JitseB <Jitse@operamail.com>
Date: Mon, 16 Apr 2018 07:49:57 +0200
Subject: [PATCH] Removed stream function.

Stream functions generate new objects. Looping through the values is more efficient.
---
 src/net/jitse/npclib/api/NPC.java | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/net/jitse/npclib/api/NPC.java b/src/net/jitse/npclib/api/NPC.java
index c68cb6a..e2ca5c6 100644
--- a/src/net/jitse/npclib/api/NPC.java
+++ b/src/net/jitse/npclib/api/NPC.java
@@ -59,7 +59,15 @@ public abstract class NPC {
 
     public void destroy() {
         NPCManager.remove(this);
-        shown.stream().filter(u -> !autoHidden.contains(u)).forEach(u -> hide(Bukkit.getPlayer(u), true));
+
+        // Destroy NPC for every player that is still seeing it.
+        for (UUID uuid : shown) {
+            if (autoHidden.contains(uuid)) {
+                continue;
+            }
+
+            hide(Bukkit.getPlayer(uuid), true);
+        }
     }
 
     public Set<UUID> getShown() {