From 8f133acbc689e1fff036d2e2dc48471994ee0856 Mon Sep 17 00:00:00 2001
From: Stephen Seo <seo.disparate@gmail.com>
Date: Thu, 22 Nov 2018 14:28:31 +0900
Subject: [PATCH] Fix stored functions not being called in order

Manager::callForMatchingFunctions should now call all functions in the
order they were added.
---
 src/EC/Manager.hpp  |  2 +-
 src/test/ECTest.cpp | 47 +++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+), 1 deletion(-)

diff --git a/src/EC/Manager.hpp b/src/EC/Manager.hpp
index 4191ce9..4b5e4e4 100644
--- a/src/EC/Manager.hpp
+++ b/src/EC/Manager.hpp
@@ -784,7 +784,7 @@ namespace EC
 
 
     private:
-        std::unordered_map<std::size_t, std::tuple<
+        std::map<std::size_t, std::tuple<
             BitsetType,
             void*,
             std::function<void(
diff --git a/src/test/ECTest.cpp b/src/test/ECTest.cpp
index 1e76c51..f5f63fd 100644
--- a/src/test/ECTest.cpp
+++ b/src/test/ECTest.cpp
@@ -6,6 +6,7 @@
 #include <memory>
 #include <unordered_map>
 #include <mutex>
+#include <vector>
 
 #include <EC/Meta/Meta.hpp>
 #include <EC/EC.hpp>
@@ -1027,3 +1028,49 @@ TEST(EC, context)
     EXPECT_EQ(999, manager.getEntityData<C0>(e1)->x);
     EXPECT_EQ(1999, manager.getEntityData<C0>(e1)->y);
 }
+
+TEST(EC, FunctionStorageOrder)
+{
+    EC::Manager<ListComponentsAll, ListTagsAll> manager;
+    auto e0 = manager.addEntity();
+    manager.addComponent<C0>(e0, 1, 2);
+
+    std::vector<int> v;
+
+    using C0TL = EC::Meta::TypeList<C0>;
+
+    manager.addForMatchingFunction<C0TL>(
+        [&v] (std::size_t /* id */, void* /* context */, C0* c) {
+            v.push_back(c->x);
+            v.push_back(c->y);
+        });
+    manager.addForMatchingFunction<C0TL>(
+        [] (std::size_t /* id */, void* /* context */, C0* c) {
+            c->x += 2;
+            c->y += 2;
+        });
+    manager.addForMatchingFunction<C0TL>(
+        [&v] (std::size_t /* id */, void* /* context */, C0* c) {
+            v.push_back(c->x);
+            v.push_back(c->y);
+        });
+    manager.addForMatchingFunction<C0TL>(
+        [] (std::size_t /* id */, void* /* context */, C0* c) {
+            c->x += 2;
+            c->y += 2;
+        });
+    manager.addForMatchingFunction<C0TL>(
+        [&v] (std::size_t /* id */, void* /* context */, C0* c) {
+            v.push_back(c->x);
+            v.push_back(c->y);
+        });
+
+    manager.callForMatchingFunctions();
+
+    EXPECT_EQ(1, v.at(0));
+    EXPECT_EQ(2, v.at(1));
+    EXPECT_EQ(3, v.at(2));
+    EXPECT_EQ(4, v.at(3));
+    EXPECT_EQ(5, v.at(4));
+    EXPECT_EQ(6, v.at(5));
+}