Fixed fastRemoveOne when the element isn't found

For a non-empty vectors that did not contain the element to remove, the
loop would go on forever because a size_t can't become smaller than 0.

Fixed by simply iterating forwards.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/scripting/script.cpp b/src/scripting/script.cpp
index 074546ef..3e299461 100644
--- a/src/scripting/script.cpp
+++ b/src/scripting/script.cpp
@@ -141,7 +141,7 @@ void Script::loadNPC(const std::string &name,
 template<typename T>
 static void fastRemoveOne(std::vector<T> &vector, T value)
 {
-    for (size_t i = vector.size() - 1; i >= 0; --i)
+    for (size_t i = 0, size = vector.size(); i < size; ++i)
     {
         if (vector.at(i) == value)
         {