Basic collisions using Octree. Seems buggy over octree boundaries.

include/Engine/Core/AABB.h

@@ -11,18 +11,18 @@ public:
     AABB(const glm::vec3& minPos, const glm::vec3& maxPos);
     AABB(const glm::vec4& minPos, const glm::vec4& maxPos);
     //No checks are made. Size must consist of non-negative numbers.
-    virtual void CreateFromCenter(const glm::vec3& center, const glm::vec3& size);
+    static AABB FromOriginSize(const glm::vec3& origin, const glm::vec3& size);
     virtual ~AABB();
 
     const glm::vec3& MinCorner() const { return m_MinCorner; }
     const glm::vec3& MaxCorner() const { return m_MaxCorner; }
-    const glm::vec3& Center() const { return m_Center; }
+    const glm::vec3& Origin() const { return m_Origin; }
     const glm::vec3 Size() const { return 2.0f * m_HalfSize; }
     const glm::vec3& HalfSize() const { return m_HalfSize; }
 private:
     glm::vec3 m_MinCorner;
     glm::vec3 m_MaxCorner;
-    glm::vec3 m_Center;
+    glm::vec3 m_Origin;
     glm::vec3 m_HalfSize;
 };
 

include/Engine/Core/Octree.h

@@ -2,7 +2,7 @@
 #define Octree_h__
 
 #include "../Common.h"
-#include "Core/AABB.h"
+#include "AABB.h"
 
 class Ray;
 
@@ -14,7 +14,7 @@ public:
         float CollideDistance;
     };
 
-    Octree();
+    Octree() = delete;
     ~Octree();
     //For the root Octree, [octreeBounds] should be a box containing the entire level.
     Octree(const AABB& octreeBounds, int subDivisions);

include/Engine/Core/System.h

@@ -10,6 +10,9 @@ class System
     friend class SystemPipeline;
 
 protected:
+    System()
+        : m_EventBroker(nullptr)
+    { }
     System(EventBroker* eventBroker)
         : m_EventBroker(eventBroker)
     { }
@@ -18,14 +21,13 @@ protected:
     EventBroker* m_EventBroker;
 };
 
-class PureSystem : public System
+class PureSystem : public virtual System
 {
     friend class SystemPipeline;
 
 protected:
-    PureSystem(EventBroker* eventBroker, std::string componentType)
-        : System(eventBroker)
-        , m_ComponentType(componentType)
+    PureSystem(std::string componentType)
+        : m_ComponentType(componentType)
     { }
     virtual ~PureSystem() = default;
 
@@ -34,14 +36,12 @@ protected:
     virtual void UpdateComponent(World* world, ComponentWrapper& component, double dt) = 0;
 };
 
-class ImpureSystem : public System
+class ImpureSystem : public virtual System
 {
     friend class SystemPipeline;
 
 protected:
-    ImpureSystem(EventBroker* eventBroker)
-        : System(eventBroker)
-    { }
+    ImpureSystem() = default;
     virtual ~ImpureSystem() = default;
 
     virtual void Update(World* world, double dt) = 0;

include/Engine/Core/SystemPipeline.h

@@ -32,8 +32,8 @@ public:
         System* system = new T(m_EventBroker, args...);
         group.Systems[typeid(T).name()] = system;
 
-        if (std::is_base_of<PureSystem, T>::value) {
-            PureSystem* pureSystem = static_cast<PureSystem*>(system);
+        PureSystem* pureSystem = dynamic_cast<PureSystem*>(system);
+        if (pureSystem != nullptr) {
             if (!pureSystem->m_ComponentType.empty()) {
                 group.PureSystems[pureSystem->m_ComponentType].push_back(pureSystem);
             } else {
@@ -41,8 +41,8 @@ public:
             }
         }
 
-        if (std::is_base_of<ImpureSystem, T>::value) {
-            ImpureSystem* impureSystem = static_cast<ImpureSystem*>(system);
+        ImpureSystem* impureSystem = dynamic_cast<ImpureSystem*>(system);
+        if (impureSystem != nullptr) {
             group.ImpureSystems.push_back(impureSystem);
         }
     }
@@ -56,6 +56,9 @@ public:
             }
 
             // Update
+            for (auto& system : group.ImpureSystems) {
+                system->Update(world, dt);
+            }
             for (auto& pair : group.PureSystems) {
                 const std::string& componentName = pair.first;
                 auto& systems = pair.second;
@@ -69,9 +72,6 @@ public:
                     }
                 }
             }
-            for (auto& system : group.ImpureSystems) {
-                system->Update(world, dt);
-            }
         }
     }