Fixed bug where deleted entities retain info

diff --git a/src/EC/Manager.hpp b/src/EC/Manager.hpp
index fa3fab9cff127afbd3b66e6f012d35cceec2ce57..aa44a797f6641386efa7a8f4924fb8669f88f3f5 100644 (file)
--- a/src/EC/Manager.hpp
+++ b/src/EC/Manager.hpp
@@ -80,9 +80,6 @@ namespace EC
 
             std::get<bool>(entities[currentSize]) = true;
 
-            // TODO This shouldn't be necessary, but there is a bug
-            std::get<BitsetType>(entities[currentSize]).reset();
-
             return currentSize++;
         }
 
@@ -143,6 +140,7 @@ namespace EC
                         currentSize = 0;
                         return;
                     }
+                    std::get<BitsetType>(entities[rhs]).reset();
                     --rhs;
                 }
                 if(lhs >= rhs)

diff --git a/src/test/ECTest.cpp b/src/test/ECTest.cpp
index f76651077746642119fa86f9f31b21b31ec091bc..ee72cd2640982957fe0ded9de2ef67e07b3c40cd 100644 (file)
--- a/src/test/ECTest.cpp
+++ b/src/test/ECTest.cpp
@@ -207,3 +207,38 @@ TEST(EC, MoveComponentWithUniquePtr)
     }
 }
 
+TEST(EC, DeletedEntities)
+{
+    EC::Manager<ListComponentsAll, ListTagsAll> manager;
+
+    for(unsigned int i = 0; i < 4; ++i)
+    {
+        auto eid = manager.addEntity();
+        manager.addComponent<C0>(eid);
+        if(i >= 2)
+        {
+            manager.deleteEntity(eid);
+        }
+    }
+
+    manager.cleanup();
+
+    for(unsigned int i = 0; i < 4; ++i)
+    {
+        if(i < 2)
+        {
+            EXPECT_TRUE(manager.hasComponent<C0>(i));
+        }
+        else
+        {
+            EXPECT_FALSE(manager.hasComponent<C0>(i));
+        }
+    }
+
+    for(unsigned int i = 0; i < 2; ++i)
+    {
+        auto eid = manager.addEntity();
+        EXPECT_FALSE(manager.hasComponent<C0>(eid));
+    }
+}
+