Removed debug output

README.md

@@ -2,7 +2,7 @@
 Libraries bundled along with binaries for Windows (MSVC14), available as a submodule in the *deps* directory of the source tree.
 
 | Project                                                       | Version      | License                                                                                                                                                                            |
-| ---------------------------------------------------------------- | ------------ | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| ------------------------------------------------------------- | ------------ | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
 | **[GLFW](http://www.glfw.org)**                               | 3.1.2        | [zlib/libpng License](http://www.glfw.org/license.html)                                                                                                                            |
 | **[GLM](http://glm.g-truc.net/0.9.5/index.html)**             | 0.9.7.1      | [MIT License](http://glm.g-truc.net/copying.txt)                                                                                                                                   |
 | **[GLEW](http://glew.sourceforge.net)**                       | 1.13.0       | [Modified BSD License](http://glew.sourceforge.net/glew.txt), [Mesa 3-D License](http://glew.sourceforge.net/mesa.txt), [Khronos License](http://glew.sourceforge.net/khronos.txt) |
@@ -10,13 +10,10 @@ Libraries bundled along with binaries for Windows (MSVC14), available as a submo
 | **[zlib](http://www.zlib.net)**                               | 1.28         | [zlib License](http://www.zlib.net/zlib_license.html)                                                                                                                              |
 | **[libpng](http://www.libpng.org/pub/png/libpng.html)**       | 1.6.19       | [libpng License](http://www.libpng.org/pub/png/src/libpng-LICENSE.txt)                                                                                                             |
 | **[Xerces-C++](https://xerces.apache.org/xerces-c)**          | 3.1.2        | [Apache License Version 2.0](https://www.apache.org/licenses/LICENSE-2.0)                                                                                                          |
-| **[ImGui](https://github.com/ocornut/imgui)**                    | 2015-12-12   | [MIT License](https://github.com/ocornut/imgui/blob/de3a154f3801de22c8e0bd2aeabf663a70c05972/LICENSE)                                                                              |
-| **[nativefiledialog](https://github.com/mlabbe/nativefiledialog)** | 2016-01-08   | [nativefiledialog Licence](https://github.com/mlabbe/nativefiledialog/blob/master/LICENSE)                                                                                                              |
-| **[OpenAL](https://www.openal.org/)**                            | 1.1          | [OpenAL License](resources/Licenses/OpenAL.txt)
 
 #### External libraries
 Libraries that are too big to be bundled with the project.
 
 | Project                                                    | Version     | License                                                                     | Root folder environment variable (Windows) |
 | ---------------------------------------------------------- | ----------- | --------------------------------------------------------------------------- | ------------------------------------------ |
-| **[Boost](http://www.boost.org)**                          | 1.60.0+     | [Boost Software License, Version 1.0](http://www.boost.org/LICENSE_1_0.txt) | BOOST_ROOT                                 |
+| **[Boost](http://www.boost.org)**                          | 1.59.0+     | [Boost Software License, Version 1.0](http://www.boost.org/LICENSE_1_0.txt) | BOOST_ROOT                                 |

include/Engine/Collision/CollidableOctreeSystem.h

@@ -1,24 +0,0 @@
-#ifndef CollidableOctreeSystem_h__
-#define CollidableOctreeSystem_h__
-
-#include "../Core/System.h"
-#include "../Core/Octree.h"
-#include "Collision.h"
-
-class CollidableOctreeSystem : public ImpureSystem, public PureSystem
-{
-public:
-    CollidableOctreeSystem(EventBroker* eventBroker, Octree<AABB>* octree)
-        : System(eventBroker)
-        , PureSystem("Collidable")
-        , m_Octree(octree)
-    { }
-
-    virtual void Update(World* world, double dt) override;
-    virtual void UpdateComponent(World* world, EntityWrapper& entity, ComponentWrapper& component, double dt) override;
-
-private:
-    Octree<AABB>* m_Octree;
-};
-
-#endif

include/Engine/Collision/Collision.h

@@ -1,67 +0,0 @@
-#ifndef Collision_h__
-#define Collision_h__
-
-//NOTE: Collision.h needs to be #included before <GLFW/glfw3.h>, 
-//because Collision #includes "RawModel.h", which has "Texture.h", which has "OpenGL.h" which must be #included first
-//or you will get "fatal error C1189: #error:  gl.h included before glew.h"
-
-#include <vector>
-#include <boost/optional.hpp>
-
-#include "../Core/Ray.h"
-#include "../Core/AABB.h"
-#include "../Rendering/RawModel.h"
-#include "../Core/Transform.h"
-#include "../Core/Entity.h"
-#include "../Core/EntityWrapper.h"
-
-class World;
-struct ComponentWrapper;
-
-namespace Collision
-{
-//Return true if the ray hits the box.
-bool RayAABBIntr(const Ray& ray, const AABB& box);
-bool RayVsAABB(const Ray& ray, const AABB& box);
-//Return true if the ray hits the box, also outputs distance from ray origin to intersection point in [outDistance].
-bool RayVsAABB(const Ray& ray, const AABB& box, float& outDistance);
-
-//Return true if the ray hits any of the triangles in the model. Stops checking when a hit is detected.
-bool RayVsModel(const Ray& ray, 
-    const std::vector<RawModel::Vertex>& modelVertices,
-    const std::vector<unsigned int>& modelIndices);
-//Return true if the ray hits any of the triangles in the model.
-//Also returns the position of the intersection point. Will loop through all the whole model indices.
-bool RayVsModel(const Ray& ray,
-    const std::vector<RawModel::Vertex>& modelVertices,
-    const std::vector<unsigned int>& modelIndices,
-    glm::vec3& outHitPosition);
-//Return true if the ray hits any of the triangles in the model.
-//Also returns the distance from the ray origin to the closest 
-//intersection point, and the barycentric u,v-coordinates. Will loop through all the whole model indices.
-bool RayVsModel(const Ray& ray,
-    const std::vector<RawModel::Vertex>& modelVertices,
-    const std::vector<unsigned int>& modelIndices,
-    float& outDistance,
-    float& outUCoord,
-    float& outVCoord);
-
-bool AABBvsTriangles(const AABB& box,
-    const std::vector<RawModel::Vertex>& modelVertices,
-    const std::vector<unsigned int>& modelIndices,
-    const glm::mat4& modelMatrix,
-    glm::vec3& outResolutionVector);
-
-//Return true if the boxes are intersecting.
-bool AABBVsAABB(const AABB& a, const AABB& b);
-//Return true if the boxes are intersecting.
-//Also outputs the minimum translation that box [a] would need in order to resolve collision.
-bool AABBVsAABB(const AABB& a, const AABB& b, glm::vec3& minimumTranslation);
-bool IsSameBoxProbably(const AABB& first, const AABB& second, const float epsilon = 0.0001f);
-
-// Calculates an absolute AABB from an entity AABB component
-boost::optional<AABB> EntityAbsoluteAABB(EntityWrapper& entity);
-
-}
-
-#endif

include/Engine/Collision/CollisionSystem.h

@@ -1,36 +0,0 @@
-#ifndef CollisionSystem_h__
-#define CollisionSystem_h__
-
-#include <GLFW/glfw3.h>
-#include <glm/common.hpp>
-
-#include "../Common.h"
-#include "../Core/System.h"
-#include "../Core/EventBroker.h"
-#include "../Core/EKeyUp.h"
-#include "../Core/Octree.h"
-
-class CollisionSystem : public PureSystem
-{
-public:
-    CollisionSystem(EventBroker* eventBroker, Octree<AABB>* octree)
-        : System(eventBroker)
-        , PureSystem("Collidable")
-        , m_Octree(octree)
-        , zPress(false)
-    {
-        //TODO: Debug stuff, remove later.
-        EVENT_SUBSCRIBE_MEMBER(m_EKeyUp, &CollisionSystem::OnKeyUp);
-    }
-
-    virtual void UpdateComponent(World* world, EntityWrapper& entity, ComponentWrapper& component, double dt) override;
-
-private:
-    Octree<AABB>* m_Octree;
-    bool zPress;
-
-    EventRelay<CollisionSystem, Events::KeyUp> m_EKeyUp;
-    bool OnKeyUp(const Events::KeyUp &event);
-};
-
-#endif

include/Engine/Collision/ETrigger.h

@@ -1,39 +0,0 @@
-#ifndef Events_TriggerEnter_h__
-#define Events_TriggerEnter_h__
-
-#include "../Core/EventBroker.h"
-#include "../Core/Entity.h"
-
-namespace Events
-{
-
-/** Thrown once, when an entity is only touching a trigger. */
-struct TriggerTouch : Event
-{
-    /** The id of the entity that touches the trigger. */
-    EntityID Entity;
-    /** The id of the trigger entity. */
-    EntityID Trigger;
-};
-
-/** Thrown once, when an entity has completely left a trigger. */
-struct TriggerLeave : Event
-{
-    /** The id of the entity that left the trigger. */
-    EntityID Entity;
-    /** The id of the trigger entity. */
-    EntityID Trigger;
-};
-
-/** Thrown once, when an entity is completely contained inside a trigger. */
-struct TriggerEnter : Event
-{
-    /** The id of the entity that entered the trigger. */
-    EntityID Entity;
-    /** The id of the trigger entity. */
-    EntityID Trigger;
-};
-
-}
-
-#endif

include/Engine/Collision/TriggerSystem.h

@@ -1,54 +0,0 @@
-#ifndef TriggerSystem_h__
-#define TriggerSystem_h__
-
-#include <glm/common.hpp>
-#include <unordered_set>
-
-#include "../Core/System.h"
-#include "../Core/EventBroker.h"
-#include "../Core/Octree.h"
-#include "ETrigger.h"
-
-class AABB;
-
-class TriggerSystem : public PureSystem
-{
-public:
-    TriggerSystem(EventBroker* eventBroker, Octree<AABB>* octree)
-        : System(eventBroker)
-        , PureSystem("Trigger")
-        , m_Octree(octree)
-    {
-        EVENT_SUBSCRIBE_MEMBER(m_ETouch, &TriggerSystem::OnTouch);
-        EVENT_SUBSCRIBE_MEMBER(m_EEnter, &TriggerSystem::OnEnter);
-        EVENT_SUBSCRIBE_MEMBER(m_ELeave, &TriggerSystem::OnLeave);
-    }
-
-    virtual void UpdateComponent(World* world, EntityWrapper& entity, ComponentWrapper& component, double dt) override;
-
-private:
-    Octree<AABB>* m_Octree;
-    std::unordered_map<EntityID, std::unordered_set<EntityID>> m_EntitiesTouchingTrigger;
-    std::unordered_map<EntityID, std::unordered_set<EntityID>> m_EntitiesCompletelyInTrigger;
-
-    //TODO: Only exists for debug purposes, remove later.
-    EventRelay<TriggerSystem, Events::TriggerEnter> m_EEnter;
-    bool OnEnter(const Events::TriggerEnter &event);
-    EventRelay<TriggerSystem, Events::TriggerTouch> m_ETouch;
-    bool OnTouch(const Events::TriggerTouch &event);
-    EventRelay<TriggerSystem, Events::TriggerLeave> m_ELeave;
-    bool OnLeave(const Events::TriggerLeave &event);
-
-    //True if leave event was thrown.
-    bool throwLeaveIfWasInTrigger(std::unordered_set<EntityID>& triggerSet, EntityID pId, EntityID tId);
-    template<typename Event>
-    void publish(EntityID pId, EntityID tId)
-    {
-        Event e;
-        e.Trigger = tId;
-        e.Entity = pId;
-        m_EventBroker->Publish(e);
-    }
-};
-
-#endif

include/Engine/Common.h

@@ -1,9 +1,7 @@
 #include <memory>
 #include <string>
-#include <sstream>
 #include <vector>
 #include <map>
 #include <unordered_map>
 
 #include "Core/Util/Logging.h"
-#include "Core/Util/IfDebug.h"

include/Engine/Core/AABB.h

@@ -1,29 +0,0 @@
-#ifndef AABB_h__
-#define AABB_h__
-
-#include "../GLM.h"
-
-class AABB
-{
-public:
-    AABB() = default;
-    //No checks are made. Values in minPos must be less than values in maxPos, i.e. min.x < max.x, etc.
-    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.
-    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& 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_Origin;
-    glm::vec3 m_HalfSize;
-};
-
-#endif

include/Engine/Core/ComponentInfo.h

@@ -9,24 +9,14 @@ struct ComponentInfo
 	{
 		std::string Annotation;
 		unsigned int Allocation = 0;
-        std::map<std::string, std::string> FieldAnnotations;
-        std::map<std::string, std::map<std::string, int>> FieldEnumDefinitions;
-	};
-
-    struct Field_t
-    {
-        std::string Name;
-        std::string Type;
-        unsigned int Offset;
-        unsigned int Stride;
-    };
-
-	std::string Name;
-    std::unordered_map<std::string, Field_t> Fields;
-    std::vector<std::string> FieldsInOrder;
         unsigned int Stride = 0;
+	};
+
+	std::string Name;
+	std::unordered_map<std::string, std::string> FieldTypes;
+	std::unordered_map<std::string, unsigned int> FieldOffsets;
+	Meta_t Meta;
     std::shared_ptr<char> Defaults = nullptr;
-	std::shared_ptr<Meta_t> Meta = nullptr;
 };
 
 template<>

include/Engine/Core/ComponentPool.h

@@ -20,7 +20,7 @@ public:
 	~ComponentPoolForwardIterator() = default;
 	ComponentPoolForwardIterator& operator=(const ComponentPoolForwardIterator& other) = default;
 	ComponentPoolForwardIterator& operator++();
-	ComponentPoolForwardIterator operator++(int);
+	ComponentPoolForwardIterator& operator++(int);
 	bool operator!=(const ComponentPoolForwardIterator& other) const;
 	bool operator==(const ComponentPoolForwardIterator& other) const;
 	ComponentWrapper operator*() const;
@@ -43,7 +43,7 @@ public:
 
     ComponentPool(const ::ComponentInfo& ci) 
         : m_ComponentInfo(ci)
-        , m_Pool(ci.Meta->Allocation, sizeof(EntityID) + ci.Stride)
+        , m_Pool(ci.Meta.Allocation, sizeof(EntityID) + ci.Meta.Stride)
     { }
 	ComponentPool(const ComponentPool& other) = delete;
 	ComponentPool(const ComponentPool&& other) = delete;
@@ -54,8 +54,6 @@ public:
     ComponentWrapper Allocate(EntityID entity);
     // Get the component belonging to a specific entity
     ComponentWrapper GetByEntity(EntityID ent);
-    // Returns true if the pool contains a component for the specified entity
-    bool KnowsEntity(EntityID ent);
     // Delete a component and free its memory
     void Delete(ComponentWrapper& wrapper);
 

include/Engine/Core/ComponentWrapper.h

@@ -18,31 +18,21 @@ struct ComponentWrapper
     const ::EntityID EntityID;
     char* Data;
 
-    int Enum(const char* fieldName, const char* enumKey)
+    template <typename T>
+    T& Property(std::string name)
     {
-        return Info.Meta->FieldEnumDefinitions.at(fieldName).at(enumKey);
+        unsigned int offset = Info.FieldOffsets.at(name);
+        return *reinterpret_cast<T*>(&Data[offset]);
     }
 
     template <typename T>
-    T& Field(std::string name)
+    void SetProperty(std::string name, const T value) { Property<T>(name) = value; }
-    {
-        const ComponentInfo::Field_t& field = Info.Fields.at(name);
-        if (sizeof(T) > field.Stride) {
-            std::stringstream message;
-            message << "Type size of \"" << typeid(T).name() << "\" doesn't match size of component field \"" << Info.Name << "." << name << "\"!";
-            throw new std::runtime_error(message.str().c_str());
-        } 
-        return *reinterpret_cast<T*>(&Data[field.Offset]);
-    }
-
-    template <typename T>
-    void SetField(std::string name, const T value) { Field<T>(name) = value; }
     //template <typename T>
-    //void SetField(std::string name, T& value) { Field<T>(name) = value; }
+    //void SetProperty(std::string name, T& value) { Property<T>(name) = value; }
 
     // Specialization for string literals
     template <std::size_t N>
-    void SetField(std::string name, const char(&value)[N]) { Field<std::string>(name) = std::string(value); }
+    void SetProperty(std::string name, const char(&value)[N]) { Property<std::string>(name) = std::string(value); }
 
     struct SubscriptProxy
     {
@@ -57,21 +47,18 @@ struct ComponentWrapper
         std::string m_PropertyName;
 
     public:
-        // Return the integer value of an enum type key for this field
+        template <typename T>
-        int Enum(const char* enumKey) { return m_Component->Enum(m_PropertyName.c_str(), enumKey); }
+        operator T&() { return m_Component->Property<T>(m_PropertyName); }
 
         template <typename T>
-        operator T&() { return m_Component->Field<T>(m_PropertyName); }
+        void operator=(const T val) { m_Component->SetProperty<T>(m_PropertyName, val); }
-
-        template <typename T>
-        void operator=(const T val) { m_Component->SetField<T>(m_PropertyName, val); }
         // TODO: Pass by reference and rvalue (universal reference?)
         //template <typename T>
-        //void operator=(T& val) { m_Component->SetField<T>(m_PropertyName, val); }
+        //void operator=(T& val) { m_Component->SetProperty<T>(m_PropertyName, val); }
 
         // Specialization for string literals
         template<std::size_t N>
-        void operator=(const char(&val)[N]) { m_Component->SetField<N>(m_PropertyName, val); }
+        void operator=(const char(&val)[N]) { m_Component->SetProperty<N>(m_PropertyName, val); }
     };
     SubscriptProxy operator[](std::string propertyName) { return SubscriptProxy(this, propertyName); }
 };
@@ -84,22 +71,21 @@ public:
     ComponentWrapperFactory(std::string componentTypeName, std::size_t allocation = 0)
     {
         m_ComponentInfo.Name = componentTypeName;
-        m_ComponentInfo.Meta->Allocation = allocation;
+        m_ComponentInfo.Meta.Allocation = allocation;
     }
 
     template <typename T>
     void AddProperty(std::string fieldName, T defaultValue)
     {
         m_DefaultValues.push_back(defaultValue);
-        m_ComponentInfo.Fields[fieldName].Name = typeid(T).name();
+        m_ComponentInfo.FieldTypes[fieldName] = typeid(T).name();
-        m_ComponentInfo.Fields[fieldName].Offset = m_ComponentInfo.Stride;
+        m_ComponentInfo.FieldOffsets[fieldName] = m_ComponentInfo.Meta.Stride;
-        m_ComponentInfo.Fields[fieldName].Stride = sizeof(T);
+        m_ComponentInfo.Meta.Stride += sizeof(T);
-        m_ComponentInfo.Stride += sizeof(T);
     }
     
     ComponentInfo& Finalize()
     {
-        m_ComponentInfo.Defaults = std::shared_ptr<char>(new char[m_ComponentInfo.Stride]);
+        m_ComponentInfo.Defaults = std::shared_ptr<char>(new char[m_ComponentInfo.Meta.Stride]);
         std::size_t offset = 0;
         for (auto& val : m_DefaultValues) {
             memcpy(m_ComponentInfo.Defaults.get() + offset, val.Data.get(), val.Size);

include/Engine/Core/EComponentAttached.h

@@ -1,20 +0,0 @@
-#ifndef EComponentAttached_h__
-#define EComponentAttached_h__
-
-#include "EventBroker.h"
-#include "World.h"
-#include "Entity.h"
-#include "ComponentWrapper.h"
-
-namespace Events
-{
-
-struct ComponentAttached : Event
-{
-    EntityWrapper Entity;
-    ComponentWrapper Component;
-};
-
-}
-
-#endif

include/Engine/Core/EFileDropped.h

@@ -1,16 +0,0 @@
-#ifndef EFileDropped_h__
-#define EFileDropped_h__
-
-#include "EventBroker.h"
-
-namespace Events
-{
-
-struct FileDropped : Event
-{
-    std::string Path;
-};
-
-}
-
-#endif

include/Engine/Core/EKeyboardChar.h

@@ -1,17 +0,0 @@
-#ifndef Events_KeyboardChar_h__
-#define Events_KeyboardChar_h__
-
-#include "EventBroker.h"
-
-namespace Events
-{
-
-struct KeyboardChar : Event
-{
-    double Timestamp = 0.f;
-	unsigned int Char = 0;
-};
-
-}
-
-#endif

include/Engine/Core/EMouseScroll.h

@@ -1,17 +0,0 @@
-#ifndef Events_MouseScroll_h__
-#define Events_MouseScroll_h__
-
-#include "EventBroker.h"
-
-namespace Events
-{
-
-struct MouseScroll : Event
-{
-    double DeltaX;
-    double DeltaY;
-};
-
-}
-
-#endif

include/Engine/Core/EPlayerDamage.h

@@ -1,20 +0,0 @@
-#ifndef EPlayerDamage_h__
-#define EPlayerDamage_h__
-
-#include "EventBroker.h"
-#include "../Core/Entity.h"
-
-namespace Events
-{
-
-struct PlayerDamage : Event
-{
-    double DamageAmount;
-    EntityID PlayerDamagedID;
-    //optional TypeOfDamage
-    std::string TypeOfDamage;
-};
-
-}
-
-#endif

include/Engine/Core/EPlayerDeath.h

@@ -1,20 +0,0 @@
-#ifndef EPlayerDeath_h__
-#define EPlayerDeath_h__
-
-#include "EventBroker.h"
-#include "../Core/Entity.h"
-
-namespace Events
-{
-
-struct PlayerDeath : Event
-{
-    //KilledBy,KilledByWhat is optional for now. It might be used later in the playerlog-system
-    EntityID KilledBy;
-    EntityID PlayerID;
-    std::string KilledByWhat;
-};
-
-}
-
-#endif

include/Engine/Core/EPlayerHealthPickup.h

@@ -1,18 +0,0 @@
-#ifndef EPlayerHealthPickup_h__
-#define EPlayerHealthPickup_h__
-
-#include "EventBroker.h"
-#include "../Core/Entity.h"
-
-namespace Events
-{
-
-struct PlayerHealthPickup : Event
-{
-    double HealthAmount;
-    EntityID PlayerHealedID;
-};
-
-}
-
-#endif

include/Engine/Core/Entity.h

@@ -2,7 +2,5 @@
 #define Entity_h__
 
 typedef unsigned int EntityID;
-const static unsigned int EntityID_Invalid = -1;
-
 
 #endif

include/Engine/Core/EntityFactory.h

@@ -285,7 +285,7 @@ private:
 
 					XSValue::Status status;
 					XSValue* val = XSValue::getActualValue(child->getNodeValue(), XSValue::dt_integer, status);
-					compInfo.Meta->Allocation += val->fData.fValue.f_int;
+					compInfo.Meta.Allocation += val->fData.fValue.f_int;
 				}
 
 				// Save documentation string
@@ -293,11 +293,11 @@ private:
 				if (documentationTags->getLength() != 0) {
 					auto child = documentationTags->item(0)->getFirstChild();
 					if (child != nullptr) {
-						compInfo.Meta->Annotation = XSTR(child->getNodeValue());
+						compInfo.Meta.Annotation = XSTR(child->getNodeValue());
 					}
 				}
 				// TODO: Parse annotation string XML
-				// compInfo.Meta->Allocation = ...
+				// compInfo.Meta.Allocation = ...
 			} else {
 				std::cout << "Warning: Component is missing an annotation!" << std::endl;
 			}
@@ -344,7 +344,7 @@ private:
 				fieldOffset += getTypeStride(type);
 			}
 
-            compInfo.Stride = fieldOffset;
+            compInfo.Meta.Stride = fieldOffset;
 			m_ComponentInfo[compInfo.Name] = compInfo;
 		}
 	}
@@ -367,14 +367,14 @@ private:
 
 			std::string componentName = XSTR(component->getLocalName());
 			auto& compInfo = m_ComponentInfo.at(componentName);
-			compInfo.Meta->Allocation += 1;
+			compInfo.Meta.Allocation += 1;
 		}
 
 		std::cout << "COMPONENT INFO" << std::endl;
 		for (auto& pair : m_ComponentInfo) {
 			ComponentInfo& ci = pair.second;
-			std::cout << "Component: " << ci.Name << " (" << ci.Meta->Annotation << ")" << std::endl;
+			std::cout << "Component: " << ci.Name << " (" << ci.Meta.Annotation << ")" << std::endl;
-			std::cout << "  Allocation: " << ci.Meta->Allocation << std::endl;
+			std::cout << "  Allocation: " << ci.Meta.Allocation << std::endl;
 			std::cout << "  Fields:" << std::endl;
 
 			// Calculate component size
@@ -393,7 +393,7 @@ private:
 			cs.ComponentName = ci.Name;
 			cs.Stride = stride;
 			cs.Info = ci;
-			cs.Data = new char[stride*ci.Meta->Allocation];
+			cs.Data = new char[stride*ci.Meta.Allocation];
 			m_ComponentStore[cs.ComponentName] = cs;
 		}
 	}

include/Engine/Core/EntityFile.h

@@ -1,164 +0,0 @@
-#ifndef EntityFile_h__
-#define EntityFile_h__
-
-#include <stack>
-#include <boost/lexical_cast.hpp>
-#include <xercesc/util/XercesDefs.hpp>
-#include <xercesc/util/PlatformUtils.hpp>
-#include <xercesc/sax2/SAX2XMLReader.hpp>
-#include <xercesc/sax2/XMLReaderFactory.hpp>
-#include <xercesc/sax2/DefaultHandler.hpp>
-#include <xercesc/sax2/Attributes.hpp>
-#include <xercesc/util/XMLString.hpp>
-#include <xercesc/util/XMLChar.hpp>
-#include <xercesc/sax2/DefaultHandler.hpp>
-#include <xercesc/framework/XMLDocumentHandler.hpp>
-#include <xercesc/framework/psvi/XSElementDeclaration.hpp>
-#include <xercesc/framework/psvi/XSComplexTypeDefinition.hpp>
-#include <xercesc/framework/psvi/XSParticle.hpp>
-#include <xercesc/framework/psvi/XSModelGroup.hpp>
-#include <xercesc/framework/psvi/XSAnnotation.hpp>
-#include <xercesc/framework/psvi/XSValue.hpp>
-#include <xercesc/framework/XMLGrammarPoolImpl.hpp>
-#include <xercesc/dom/DOM.hpp>
-#include <xercesc/dom/DOMLSInput.hpp>
-#include <xercesc/dom/DOMElement.hpp>
-
-#include "../GLM.h"
-#include "Util/XercesString.h"
-#include "ResourceManager.h"
-#include "Entity.h"
-#include "ComponentInfo.h"
-
-class EntityFileHandler
-{
-    friend class EntityFileSAXHandler;
-public:
-    // @param EntityID The entity found
-    // @param EntityID The parent of the entity
-    typedef std::function<void(EntityID, EntityID, const std::string&)> OnStartEntityCallback;
-    void SetStartEntityCallback(OnStartEntityCallback c) { m_OnStartEntityCallback = c; }
-    // @param EntityID The entity the component corresponds to
-    // @param std::string Type name of the component
-    typedef std::function<void(EntityID, const std::string&)> OnStartComponentCallback;
-    void SetStartComponentCallback(OnStartComponentCallback c) { m_OnStartComponentCallback = c; }
-    // @param EntityID Entity 
-    // @param std::string Component name 
-    // @param std::string Field name
-    // @param std::map<std::string, std::string> Field attribute names and values
-    typedef std::function<void(EntityID, const std::string&, const std::string&, const std::map<std::string, std::string>&)> OnStartFieldCallback;
-    void SetStartFieldCallback(OnStartFieldCallback c) { m_OnStartFieldCallback = c; }
-    // @param EntityID Entity 
-    // @param std::string Component name 
-    // @param std::string Field name
-    // @param char* Field data
-    typedef std::function<void(EntityID, const std::string&, const std::string&, const char*)> OnStartFieldDataCallback;
-    void SetStartFieldDataCallback(OnStartFieldDataCallback c) { m_OnStartFieldDataCallback = c; }
-
-private:
-    OnStartEntityCallback m_OnStartEntityCallback = nullptr;
-    OnStartComponentCallback m_OnStartComponentCallback = nullptr;
-    OnStartFieldCallback m_OnStartFieldCallback = nullptr;
-    OnStartFieldDataCallback m_OnStartFieldDataCallback = nullptr;
-};
-
-class EntityFileSAXHandler : public xercesc::DefaultHandler
-{
-public:
-    enum class State
-    {
-        Unknown,
-        Entity,
-        Component,
-        ComponentField
-    };
-
-    EntityFileSAXHandler(const EntityFileHandler* handler, xercesc::SAX2XMLReader* reader);
-
-    void startElement(const XMLCh* const _uri, const XMLCh* const _localName, const XMLCh* const _qname, const xercesc::Attributes& attrs) override;
-    void characters(const XMLCh* const chars, const XMLSize_t length) override;
-    void endElement(const XMLCh* const _uri, const XMLCh* const _localName, const XMLCh* const _qname) override;
-
-    void warning(const xercesc::SAXParseException& e);
-    void error(const xercesc::SAXParseException& e);
-    void fatalError(const xercesc::SAXParseException& e);
-
-private:
-    const EntityFileHandler* m_Handler;
-    xercesc::XMLGrammarPool* m_GrammarPool;
-    xercesc::SAX2XMLReader* m_Reader;
-    //State m_CurrentScope = State::Unknown;
-    std::stack<State> m_StateStack;
-    unsigned int m_NextEntityID = 0;
-    std::stack<EntityID> m_EntityStack;
-    std::string m_CurrentComponent;
-    std::string m_CurrentField;
-    std::map<std::string, std::string> m_CurrentAttributes;
-
-    void onStartEntity(const xercesc::Attributes& attrs); 
-    void onEndEntity();
-    void onStartEntityRef(const xercesc::Attributes& attrs);
-    void onStartComponent(const std::string& name);
-    void onEndComponent(const std::string& name);
-    void onStartComponentField(const std::string& field, const xercesc::Attributes& attrs);
-    void onEndComponentField(const std::string& field);
-    void onFieldData(char* data);
-};
-
-class EntityFileXMLErrorHandler : public xercesc::ErrorHandler
-{
-public:
-	void warning(const xercesc::SAXParseException& e) override
-	{
-		reportParseException("Warning", e);
-	}
-	void error(const xercesc::SAXParseException& e) override
-	{
-		reportParseException("Error", e);
-	}
-	void fatalError(const xercesc::SAXParseException& e) override
-	{
-		reportParseException("FATAL ERROR", e);
-	}
-	void resetErrors() override { }
-
-private:
-	void reportParseException(std::string type, const xercesc::SAXParseException& e)
-	{
-		char* message = xercesc::XMLString::transcode(e.getMessage());
-		char* systemID = xercesc::XMLString::transcode(e.getSystemId());
-		std::cerr << systemID << ":" << e.getLineNumber() << ":" << e.getColumnNumber() << std::endl;
-		std::cerr << type << ": " << message << std::endl;
-		xercesc::XMLString::release(&systemID);
-		xercesc::XMLString::release(&message);
-	}
-};
-
-class EntityFile : public Resource
-{
-    friend class ResourceManager;
-    friend class EntityFileSAXHandler;
-private:
-    EntityFile(boost::filesystem::path path);
-    ~EntityFile();
-
-public:
-    static std::size_t GetTypeStride(std::string typeName);
-    static void WriteAttributeData(char* outData, const ComponentInfo::Field_t& field, const std::map<std::string, std::string>& attributes);
-    static void WriteValueData(char* outData, const ComponentInfo::Field_t& field, const char* valueData);
-
-    xercesc::XMLGrammarPool* GrammarPool() const { return m_GrammarPool; }
-
-    void Parse(const EntityFileHandler* handler) const;
-
-private:
-    boost::filesystem::path m_FilePath;
-    xercesc::XMLGrammarPool* m_GrammarPool;
-    xercesc::SAX2XMLReader* m_SAX2XMLReader;
-	//std::map<std::string, ::ComponentInfo> m_ComponentInfo;
-    //std::vector<std::string> m_EntityReferences;
-
-    static void setReaderFeatures(xercesc::SAX2XMLReader* reader);
-};
-
-#endif

include/Engine/Core/EntityFileParser.h

@@ -1,29 +0,0 @@
-#ifndef EntityFileParser_h__
-#define EntityFileParser_h__
-
-#include "EntityFile.h"
-#include "World.h"
-
-class EntityFileParser
-{
-public:
-    EntityFileParser(const EntityFile* entityFile);
-
-    EntityID MergeEntities(World* world, EntityID baseParent = EntityID_Invalid);
-
-private:
-    const EntityFile* m_EntityFile;
-    EntityFileHandler m_Handler;
-    World* m_World = nullptr;
-    EntityID m_FirstEntity = EntityID_Invalid;
-    // Maps EntityIDs local to the file to real IDs in the world after they've been 
-    // created in order to resolve parent-child relationships.
-    std::map<EntityID, EntityID> m_EntityIDMapper;
-
-    void onStartEntity(EntityID entity, EntityID parent, const std::string& name);
-    void onStartComponent(EntityID entity, const std::string& component);
-    void onStartComponentField(EntityID entity, const std::string& componentType, const std::string& fieldName, const std::map<std::string, std::string>& attributes);
-    void onFieldData(EntityID entity, const std::string& componentType, const std::string& fieldName, const char* fieldData);
-};
-
-#endif

include/Engine/Core/EntityFilePreprocessor.h

@@ -1,38 +0,0 @@
-#ifndef EntityFilePreprocessor_h__
-#define EntityFilePreprocessor_h__
-
-#include <xercesc/framework/psvi/XSElementDeclaration.hpp>
-#include <xercesc/framework/psvi/XSComplexTypeDefinition.hpp>
-#include <xercesc/framework/psvi/XSParticle.hpp>
-#include <xercesc/framework/psvi/XSModelGroup.hpp>
-#include <xercesc/framework/psvi/XSModelGroupDefinition.hpp>
-#include <xercesc/framework/psvi/XSAnnotation.hpp>
-#include <xercesc/framework/psvi/XSValue.hpp>
-#include <xercesc/framework/MemBufFormatTarget.hpp>
-#include <xercesc/framework/MemBufInputSource.hpp>
-#include <xercesc/parsers/XercesDOMParser.hpp>
-
-#include "Util/XercesString.h"
-#include "ResourceManager.h"
-#include "World.h"
-#include "EntityFile.h"
-
-class EntityFilePreprocessor
-{
-public:
-    EntityFilePreprocessor(const EntityFile* entityFile);
-
-    void RegisterComponents(World* world);
-
-private:
-    const EntityFile* m_EntityFile;
-    std::map<std::string, unsigned int> m_ComponentCounts;
-	std::map<std::string, ComponentInfo> m_ComponentInfo;
-
-	void onStartComponent(EntityID entity, std::string type);
-    void parseComponentInfo();
-    void parseDefaults();
-    std::string parseAnnotationXML(const XMLCh* xml);
-};
-
-#endif

include/Engine/Core/EntityFileWriter.h

@@ -1,39 +0,0 @@
-#ifndef EntityFileWriter_h__
-#define EntityFileWriter_h__
-
-#include <boost/filesystem.hpp>
-#include <xercesc/dom/DOM.hpp>
-#include <xercesc/util/XMLString.hpp>
-#include <xercesc/util/PlatformUtils.hpp>
-#include <xercesc/framework/LocalFileFormatTarget.hpp>
-
-#include "Util/XercesString.h"
-#include "EntityFile.h"
-#include "World.h"
-
-class EntityFileWriter
-{
-public:
-    EntityFileWriter(boost::filesystem::path file)
-        : m_FilePath(file)
-    {
-        using namespace xercesc;
-        m_DOMImplementation = DOMImplementationRegistry::getDOMImplementation(XS::ToXMLCh("LS"));
-        m_DOMLSSerializer = static_cast<DOMImplementationLS*>(m_DOMImplementation)->createLSSerializer();
-        m_DOMLSSerializer->getDomConfig()->setParameter(XMLUni::fgDOMWRTDiscardDefaultContent, true);
-        m_DOMLSSerializer->getDomConfig()->setParameter(XMLUni::fgDOMWRTFormatPrettyPrint, true);
-    }
-
-    void WriteWorld(World* world);
-    void WriteEntity(World* world, EntityID entity);
-
-private:
-    boost::filesystem::path m_FilePath;
-    xercesc::DOMImplementation* m_DOMImplementation;
-    xercesc::DOMLSSerializer* m_DOMLSSerializer;
-
-    void appendEntityChildren(xercesc::DOMElement* parentElement, const World* world, EntityID entity);
-    void appentEntityComponents(xercesc::DOMElement* parentElemetn, const World* world, EntityID entity);
-};
-
-#endif

include/Engine/Core/EntityWrapper.h

@@ -1,32 +0,0 @@
-#ifndef EntityWrapper_h__
-#define EntityWrapper_h__
-
-#include <boost/optional.hpp>
-#include "ComponentWrapper.h"
-
-class World;
-struct EntityWrapper
-{
-    EntityWrapper()
-        : World(nullptr)
-        , ID(EntityID_Invalid)
-    { }
-
-    EntityWrapper(::World* world, EntityID id)
-        : World(world)
-        , ID(id)
-    { }
-
-    ::World* World;
-    EntityID ID;
-
-    static const EntityWrapper Invalid;
-
-    bool HasComponent(const std::string& componentName);
-
-    ComponentWrapper operator[](const std::string& componentName);
-    bool operator==(const EntityWrapper& e);
-    explicit operator EntityID();
-};
-
-#endif

include/Engine/Core/EntityXMLFile.h

@@ -0,0 +1,141 @@
+#ifndef EntityXMLFile_h__
+#define EntityXMLFile_h__
+
+#include <sstream>
+
+#include "../Common.h"
+#include "../GLM.h"
+#include <xercesc/dom/DOM.hpp>
+#include <xercesc/dom/DOMLSInput.hpp>
+#include <xercesc/dom/DOMElement.hpp>
+#include <xercesc/framework/Wrapper4InputSource.hpp>
+#include <xercesc/framework/LocalFileInputSource.hpp>
+#include <xercesc/framework/LocalFileFormatTarget.hpp>
+#include <xercesc/framework/MemBufFormatTarget.hpp>
+#include <xercesc/framework/MemBufInputSource.hpp>
+#include <xercesc/sax/HandlerBase.hpp>
+#include <xercesc/util/PlatformUtils.hpp>
+#include <xercesc/util/XMLFloat.hpp>
+#include <xercesc/framework/XMLGrammarPoolImpl.hpp>
+#include <xercesc/parsers/XercesDOMParser.hpp>
+#include <xercesc/framework/psvi/XSElementDeclaration.hpp>
+#include <xercesc/framework/psvi/XSComplexTypeDefinition.hpp>
+#include <xercesc/framework/psvi/XSParticle.hpp>
+#include <xercesc/framework/psvi/XSModelGroup.hpp>
+#include <xercesc/framework/psvi/XSAnnotation.hpp>
+#include <xercesc/framework/psvi/XSValue.hpp>
+#include <xercesc/framework/XMLValidator.hpp>
+
+#include "ResourceManager.h"
+#include "Entity.h"
+#include "ComponentInfo.h"
+class World;
+
+class EntityPreprocessorXMLErrorHandler : public xercesc::DOMErrorHandler
+{
+public:
+	bool handleError(const xercesc::DOMError &e) override
+	{
+		char* message = xercesc::XMLString::transcode(e.getMessage());
+		std::cerr << "Preprocessor DOMError: " << message << std::endl;
+		xercesc::XMLString::release(&message);
+		return false;
+	}
+};
+
+class EntityParserXMLErrorHandler : public xercesc::ErrorHandler
+{
+public:
+	void warning(const xercesc::SAXParseException& e) override
+	{
+		reportParseException("Warning", e);
+	}
+	void error(const xercesc::SAXParseException& e) override
+	{
+		reportParseException("Error", e);
+	}
+	void fatalError(const xercesc::SAXParseException& e) override
+	{
+		reportParseException("FATAL ERROR", e);
+	}
+	void resetErrors() override { }
+
+private:
+	void reportParseException(std::string type, const xercesc::SAXParseException& e)
+	{
+		char* message = xercesc::XMLString::transcode(e.getMessage());
+		char* systemID = xercesc::XMLString::transcode(e.getSystemId());
+		std::cerr << systemID << ":" << e.getLineNumber() << ":" << e.getColumnNumber() << std::endl;
+		std::cerr << type << ": " << message << std::endl;
+		xercesc::XMLString::release(&systemID);
+		xercesc::XMLString::release(&message);
+	}
+};
+
+class XSTR
+{
+public:
+	XSTR(const XMLCh* const xmlString) 
+	{
+		m_AsChar = xercesc::XMLString::transcode(xmlString);
+	}
+
+	XSTR(const char* normalString)
+	{
+		m_AsXMLCh = xercesc::XMLString::transcode(normalString);
+	}
+
+	~XSTR()
+	{
+		if (m_AsChar != nullptr) {
+			xercesc::XMLString::release(&m_AsChar);
+		}
+		if (m_AsXMLCh != nullptr) {
+			xercesc::XMLString::release(&m_AsXMLCh);
+		}
+	}
+
+	operator const char*() const { return m_AsChar; }
+	operator const XMLCh*() const { return m_AsXMLCh; }
+
+private:
+	char* m_AsChar = nullptr;
+	XMLCh* m_AsXMLCh = nullptr;
+};
+
+class EntityXMLFile : public Resource
+{
+	friend class ResourceManager;
+
+private:
+	EntityXMLFile(std::string path);
+
+public:
+    ~EntityXMLFile();
+
+    void PopulateWorld(World* world);
+
+private:
+    static unsigned int InstanceCount;
+
+	std::string m_EntityFile;
+	xercesc::XMLGrammarPool* m_GrammarPool = nullptr;
+	EntityParserXMLErrorHandler* m_ErrorHandler = nullptr;
+	xercesc::XercesDOMParser* m_DOMParser = nullptr;
+	xercesc::DOMDocument* m_DOMDocument = nullptr;
+	std::map<std::string, ComponentInfo> m_ComponentInfo;
+
+    // Preprocesses the entity file to insert include-by-copy child entities
+    // TODO: Make this work in memory instead of saving to file
+    void preprocess(std::string inPath, std::string outPath);
+
+    void parseComponentInfo();
+    void parseDefaults();
+    void predictComponentAllocation();
+    void parseEntityGraph(World* world, xercesc::DOMElement* parent, EntityID parentEntity);
+    std::size_t getTypeStride(std::string typeName);
+    float getFloatAttribute(const xercesc::DOMElement* element, const char* attribute) const;
+    void writeData(const xercesc::DOMElement* element, std::string typeName, char* outData);
+};
+
+#endif

include/Engine/Core/EventBroker.h

@@ -13,8 +13,6 @@
 	relay = decltype(relay)(std::bind(handler, this, std::placeholders::_1)); \
 	m_EventBroker->Subscribe(relay);
 
-typedef unsigned int EventID;
-
 class EventBroker;
 
 class BaseEventRelay
@@ -33,7 +31,6 @@ public:
     virtual bool Receive(const std::shared_ptr<Event> event) = 0;
 
 protected:
-    EventID m_EventID;
     std::string m_ContextTypeName;
     std::string m_EventTypeName;
     EventBroker* m_Broker;
@@ -43,7 +40,7 @@ template <typename ContextType, typename EventType>
 class EventRelay : public BaseEventRelay
 {
 public:
-    typedef std::function<bool(EventType&)> CallbackType;
+    typedef std::function<bool(const EventType&)> CallbackType;
 
     EventRelay()
         : m_Callback(nullptr)
@@ -65,7 +62,7 @@ template <typename ContextType, typename EventType>
 bool EventRelay<ContextType, EventType>::Receive(const std::shared_ptr<Event> event)
 {
     if (m_Callback != nullptr) {
-        return m_Callback(*static_cast<EventType*>(event.get()));
+        return m_Callback(*static_cast<const EventType*>(event.get()));
     } else {
         return false;
     }
@@ -98,7 +95,6 @@ public:
 
 private:
     bool m_IsProcessing = false;
-    EventID m_NextEventID = 0;
 
     typedef std::string ContextTypeName_t; // typeid(ContextType).name()
     typedef std::string EventTypeName_t; // typeid(EventType).name()
@@ -107,14 +103,14 @@ private:
     typedef std::unordered_map<ContextTypeName_t, EventRelays_t> ContextRelays_t;
     ContextRelays_t m_ContextRelays;
     std::vector<BaseEventRelay*> m_RelaysToSubscribe;
-    std::vector<std::tuple<EventID, ContextTypeName_t, EventTypeName_t>> m_RelaysToUnsubscribe;
+    std::vector<BaseEventRelay*> m_RelaysToUnsubscribe;
 
     typedef std::list<std::pair<EventTypeName_t, std::shared_ptr<Event>>> EventQueue_t;
     std::shared_ptr<EventQueue_t> m_EventQueueRead;
     std::shared_ptr<EventQueue_t> m_EventQueueWrite;
 
     void subscribeImmediate(BaseEventRelay& relay);
-    void unsubscribeImmediate(std::tuple<EventID, ContextTypeName_t, EventTypeName_t> identifier);
+    void unsubscribeImmediate(BaseEventRelay& relay);
 };
 
 template <typename EventType>

include/Engine/Core/InputManager.h

@@ -8,15 +8,12 @@
 #include "EventBroker.h"
 #include "EKeyDown.h"
 #include "EKeyUp.h"
-#include "EKeyboardChar.h"
 #include "EMousePress.h"
 #include "EMouseRelease.h"
 #include "EMouseMove.h"
-#include "EMouseScroll.h"
 #include "ELockMouse.h"
 #include "EGamepadAxis.h"
 #include "EGamepadButton.h"
-#include "EFileDropped.h"
 
 class InputManager
 {
@@ -36,7 +33,6 @@ public:
 
 	void Initialize();
 
-	static const short MAX_GAMEPADS = 4;
 
 	void Update(double dt);
 
@@ -53,12 +49,6 @@ private:
 	std::array<int, GLFW_KEY_LAST+1> m_LastKeyState;
 	std::array<int, GLFW_MOUSE_BUTTON_LAST+1> m_CurrentMouseState;
 	std::array<int, GLFW_MOUSE_BUTTON_LAST+1> m_LastMouseState;
-	typedef std::array<float, static_cast<int>(Gamepad::Axis::LAST) + 1> GamepadAxisState;
-	std::array<GamepadAxisState, MAX_GAMEPADS> m_CurrentGamepadAxisState;
-	std::array<GamepadAxisState, MAX_GAMEPADS> m_LastGamepadAxisState;
-	typedef std::array<bool, static_cast<int>(Gamepad::Button::LAST) + 1> GamepadButtonState;
-	std::array<GamepadButtonState, MAX_GAMEPADS> m_CurrentGamepadButtonState;
-	std::array<GamepadButtonState, MAX_GAMEPADS> m_LastGamepadButtonState;
 
 	double m_CurrentMouseX, m_CurrentMouseY;
 	double m_LastMouseX, m_LastMouseY;
@@ -68,12 +58,10 @@ private:
 	void PublishGamepadAxisIfChanged(int gamepadID, Gamepad::Axis axis);
 	void PublishGamepadButtonIfChanged(int gamepadID, Gamepad::Button button);
 
-    static std::vector<unsigned int> GLFWCharCallbackQueue;
+    static void GLFWMouseButtonCallback(GLFWwindow* window, int button, int action, int mods)
-    static void GLFWCharCallback(GLFWwindow* window, unsigned int c);
+    {
-    static std::vector<std::pair<double, double>> GLFWScrollCallbackQueue;
+        LOG_DEBUG("Click! %i %i %i", button, action, mods);
-    static void GLFWScrollCallback(GLFWwindow* window, double xoffset, double yoffset);
+    }
-    static std::vector<std::string> GLFWDropCallbackQueue;
-    static void GLFWDropCallback(GLFWwindow* window, int count, const char* paths[]);
 };
 
 #endif

include/Engine/Core/MemoryPool.h

@@ -5,14 +5,6 @@
 template <typename T>
 class MemoryPoolForwardIterator;
 
-namespace DisableMemoryPool
-{
-//if true -> Pool allocation is not used when calling Allocate/Free, just use regular dynamic allocation. 
-//if false -> Use pool allocation.
-//Should default to false, unless the DisableMemoryPool is true in the Config.ini files.
-extern bool Value;
-}
-
 //This is the class to use if you want to allocate blocks (slots) of raw memory, with a fixed maximum size (stride).
 //Additionally, if you know that every memory-block will contain one object of a specific type, (i.e. the stride for the slot 
 //will the size of the object type) you should use ObjectPool<T> instead, your life will become easier.
@@ -88,8 +80,8 @@ public:
 	//If element cannot be allocated in the pool, because the memory ran out, memory is allocated dynamically with malloc() "outside the pool".
 	char* Allocate()
 	{
-		for (; m_CurrentAllocSlot < m_NumSlots && m_SlotIsAllocated[m_CurrentAllocSlot] && !DisableMemoryPool::Value; ++m_CurrentAllocSlot);
+		for (; m_CurrentAllocSlot < m_NumSlots && m_SlotIsAllocated[m_CurrentAllocSlot]; ++m_CurrentAllocSlot);
-		if (m_CurrentAllocSlot < m_NumSlots && !DisableMemoryPool::Value) {
+		if (m_CurrentAllocSlot < m_NumSlots) {
 			if (m_LowestAllocatedSlot > m_CurrentAllocSlot)
 				m_LowestAllocatedSlot = m_CurrentAllocSlot;
 			//Mark the slot as allocated.
@@ -101,9 +93,7 @@ public:
 		else {
 			m_ExtraMemory.push_back((char*)malloc(m_Stride));
 			//We should preferably not enter here to avoid performance issues. Set more numMaxElements in constructor instead.
-            if (!DisableMemoryPool::Value) {
 			LOG_WARNING("Allocated slots exceed Pool size, extra memory allocated dynamically. Pool size: %u, dynamic size: %u.", m_NumSlots, m_ExtraMemory.size());
-            }
 			return m_ExtraMemory.back();
 		}
 	}
@@ -118,7 +108,7 @@ public:
 		//(i.e. IsAllocatedInPool may give false positives)
 		//if it was malloc():ed 
 		//so, we may enter here even if we shouldn't.
-		if (!DisableMemoryPool::Value && IsAllocatedInPool(obj)) {
+		if (IsAllocatedInPool(obj)) {
 			--m_NumAllocatedSlots;
 			const size_t freeSlot = (obj - m_StartAddress) / m_Stride;
 			m_SlotIsAllocated[freeSlot] = false;
@@ -263,7 +253,7 @@ public:
 	}
 
 	//Postfix increment i.e. iter++. Prefer pre-increment (++iter) for efficiency.
-	MemoryPoolForwardIterator operator++(int)
+	MemoryPoolForwardIterator& operator++(int)
 	{
 		MemoryPoolForwardIterator<T> copyIter(*this);
 		operator++();

include/Engine/Core/Octree.h

@@ -1,233 +0,0 @@
-#ifndef Octree_h__
-#define Octree_h__
-
-#include <type_traits>
-
-#include "../Common.h"
-#include "AABB.h"
-
-//Fwd declarations.
-class Ray;
-
-namespace OctSpace
-{
-struct Output;
-struct ContainedObject;
-struct Child;
-}
-
-//T needs to be AABB, or inherit from AABB.
-//T also needs to have a default constructor.
-template<typename T>
-class Octree
-{
-public:
-    Octree() = delete;
-    ~Octree();
-    //For the root Octree, [octreeBounds] should be a box containing the entire level.
-    Octree(const AABB& octreeBounds, int subDivisions);
-
-    //We cannot copy the Octree as of now, because of the recursive dynamic allocation.
-    //Define these if the Octree suddenly needs to be copied, think of the children Child* ptrs. 
-    Octree(const Octree& other) = delete;
-    Octree(const Octree&& other) = delete;
-    Octree& operator= (const Octree& other) = delete;
-    //Add a dynamic object (one that moves around) into the tree.
-    void AddDynamicObject(const T& object);
-    //Add a static object (that does not move) into the tree.
-    void AddStaticObject(const T& object);
-    //Get the objects that are in the same area as the input [box], the objects are put in [outObjects].
-    //The type Box must be AABB, or inherit from AABB.
-    template<typename Box>
-    void ObjectsInSameRegion(const Box& box, std::vector<T>& outObjects);
-    //Empty the tree of all objects, static and dynamic.
-    void ClearObjects();
-    //Empty the tree of all dynamic objects. Static objects remain in the tree.
-    void ClearDynamicObjects();
-
-    //Returns true if the ray collides with something in the tree. Result is written to [data].
-    bool RayCollides(const Ray& ray, OctSpace::Output& data);
-    //Returns true if the box collides with something in the tree. 
-    //On collision with a box, that box is written to [outBoxIntersected].
-    //Note: More efficient than calling ObjectsInSameRegion from outside and testing there.
-    bool BoxCollides(const AABB& boxToTest, AABB& outBoxIntersected);
-
-private:
-    OctSpace::Child* m_Root;
-    std::vector<OctSpace::ContainedObject> m_StaticObjects;
-    std::vector<OctSpace::ContainedObject> m_DynamicObjects;
-
-    void falsifyObjectChecks();
-};
-
-namespace OctSpace
-{
-
-struct Output
-{
-    float CollideDistance;
-};
-
-struct ContainedObject
-{
-    ContainedObject()
-        : Box(nullptr)
-        , Checked(false)
-    {}
-    template<typename BoxlikeObject>
-    ContainedObject(const BoxlikeObject& box)
-        : Box(new BoxlikeObject(box))
-        , Checked(false)
-    {}
-    std::unique_ptr<AABB> Box;
-    bool Checked;
-};
-
-struct Child
-{
-    ~Child();
-    Child(const AABB& octTreeBounds,
-        int subDivisions,
-        std::vector<ContainedObject>& staticObjects,
-        std::vector<ContainedObject>& dynamicObjects);
-    Child(const Child& other) = delete;
-    Child(const Child&& other) = delete;
-    Child& operator= (const Child& other) = delete;
-    void AddDynamicObject(const AABB& box);
-    void AddStaticObject(const AABB& box);
-    template<typename T, typename Box>
-    void ObjectsInSameRegion(const Box& box, std::vector<T>& outObjects) const;
-    void ClearObjects();
-    void ClearDynamicObjects();
-    bool RayCollides(const Ray& ray, Output& data) const;
-    bool BoxCollides(const AABB& boxToTest, AABB& outBoxIntersected) const;
-
-    Child* m_Children[8];
-    //Indices into the lists in Octree.
-    std::vector<int> m_StaticObjIndices;
-    std::vector<int> m_DynamicObjIndices;
-    AABB m_Box;
-    //Reference to the lists in Octree.
-    std::vector<ContainedObject>& m_StaticObjectsRef;
-    std::vector<ContainedObject>& m_DynamicObjectsRef;
-
-    inline bool hasChildren() const;
-    int childIndexContainingPoint(const glm::vec3& point) const;
-    std::vector<int> childIndicesContainingBox(const AABB& box) const;
-};
-
-}
-
-template<typename T>
-Octree<T>::Octree(const AABB& octTreeBounds, int subDivisions)
-    : m_Root(new OctSpace::Child(octTreeBounds, subDivisions, m_StaticObjects, m_DynamicObjects))
-{
-    static_assert(std::is_base_of<AABB, T>::value, "template argument type T in Octree must be a subclass of AABB.");
-}
-
-template<typename T>
-Octree<T>::~Octree()
-{
-    delete m_Root;
-}
-
-template<typename T>
-void Octree<T>::AddDynamicObject(const T& object)
-{
-    m_Root->AddDynamicObject(object);
-    m_DynamicObjects.emplace_back(object);
-}
-
-template<typename T>
-void Octree<T>::AddStaticObject(const T& object)
-{
-    m_Root->AddStaticObject(object);
-    m_StaticObjects.emplace_back(object);
-}
-
-template<typename T>
-template<typename Box>
-void Octree<T>::ObjectsInSameRegion(const Box& box, std::vector<T>& outObjects)
-{
-    static_assert(std::is_base_of<AABB, Box>::value, "template argument type Box in Octree<T>::ObjectsInSameRegion must be a subclass of AABB.");
-    falsifyObjectChecks();
-    m_Root->ObjectsInSameRegion(box, outObjects);
-}
-
-template<typename T>
-void Octree<T>::ClearObjects()
-{
-    m_StaticObjects.clear();
-    m_DynamicObjects.clear();
-    m_Root->ClearObjects();
-}
-
-template<typename T>
-void Octree<T>::ClearDynamicObjects()
-{
-    m_DynamicObjects.clear();
-    m_Root->ClearDynamicObjects();
-}
-
-template<typename T>
-bool Octree<T>::RayCollides(const Ray& ray, OctSpace::Output& data)
-{
-    falsifyObjectChecks();
-    data.CollideDistance = -1;
-    return m_Root->RayCollides(ray, data);
-}
-
-template<typename T>
-bool Octree<T>::BoxCollides(const AABB& boxToTest, AABB& outBoxIntersected)
-{
-    falsifyObjectChecks();
-    return m_Root->BoxCollides(boxToTest, outBoxIntersected);
-}
-
-template<typename T>
-void Octree<T>::falsifyObjectChecks()
-{
-    for (auto& obj : m_StaticObjects) {
-        obj.Checked = false;
-    }
-    for (auto& obj : m_DynamicObjects) {
-        obj.Checked = false;
-    }
-}
-
-template<typename T, typename Box>
-void OctSpace::Child::ObjectsInSameRegion(const Box& box, std::vector<T>& outObjects) const
-{
-    if (hasChildren()) {
-        for (auto i : childIndicesContainingBox(box)) {
-            m_Children[i]->ObjectsInSameRegion(box, outObjects);
-        }
-    } else {
-        size_t startIndex = outObjects.size();
-        int numDuplicates = 0;
-        outObjects.resize(outObjects.size() + m_StaticObjIndices.size() + m_DynamicObjIndices.size());
-        for (size_t i = 0; i < m_StaticObjIndices.size(); ++i) {
-            ContainedObject& obj = m_StaticObjectsRef[m_StaticObjIndices[i]];
-            if (obj.Checked) {
-                ++numDuplicates;
-            } else {
-                obj.Checked = true;
-                outObjects[startIndex + i - numDuplicates] = *static_cast<T*>(obj.Box.get());
-            }
-        }
-        for (size_t i = 0; i < m_DynamicObjIndices.size(); ++i) {
-            ContainedObject& obj = m_DynamicObjectsRef[m_DynamicObjIndices[i]];
-            if (obj.Checked) {
-                ++numDuplicates;
-            } else {
-                obj.Checked = true;
-                outObjects[startIndex + i - numDuplicates] = *static_cast<T*>(obj.Box.get());
-            }
-        }
-        for (size_t i = 0; i < numDuplicates; ++i) {
-            outObjects.pop_back();
-        }
-    }
-}
-
-#endif

include/Engine/Core/Ray.h

@@ -1,31 +0,0 @@
-#ifndef Ray_h__
-#define Ray_h__
-
-#include "../GLM.h"
-#include "../Common.h"
-
-class Ray
-{
-public:
-    Ray(const glm::vec3& origin, const glm::vec3& dir)
-        : m_Origin(origin)
-        , m_Direction(glm::normalize(dir))
-    {
-        DEBUG_IF(true) {
-            if (glm::any(glm::isnan(m_Direction))) {
-                LOG_WARNING("Ray Direction was set to the zero-vector, expect unknown side effects and/or crashes.");
-            }
-        }
-    }
-    const glm::vec3& Origin() const { return m_Origin; }
-    const glm::vec3& Direction() const { return m_Direction; }
-    //Sets the ray origin at parameter.
-    void SetOrigin(const glm::vec3& origin) { m_Origin = origin; }
-    //Normalizes the parameter and sets direction to it.
-    void SetDirection(const glm::vec3& direction) { m_Direction = glm::normalize(direction); }
-private:
-    glm::vec3 m_Origin;
-    glm::vec3 m_Direction;
-};
-
-#endif // Ray_h__

include/Engine/Core/ResourceManager.h

@@ -12,6 +12,7 @@
 /** Base Resource class.
 
 	Implement this class for every resource to be handled by the resource manager.
+	Implement Create() to return a new object of that type.
 */
 class Resource
 {
@@ -21,23 +22,6 @@ protected:
 	Resource() { }
 
 public:
-    //Should be thrown in a Resource's constructor if it cannot complete because another resource is still loading.
-    //Not actually an error, just a message to the ResourceManager.
-    struct StillLoadingException : public std::exception
-    {
-        virtual const char* what() const throw()
-        {
-            return "Resource is still loading.";
-        }
-    };
-    struct FailedLoadingException : public std::exception
-    {
-        virtual const char* what() const throw()
-        {
-            return "Resource is failed to load.";
-        }
-    };
-
 	// Pretend that this is a pure virtual function that you have to implement
 	// FIXME: Why did we do this again instead of just using the constructor?
 	// static Resource* Create(std::string resourceName);
@@ -49,15 +33,6 @@ public:
 	unsigned int ResourceID;
 };
 
-//Any class inheriting from this class will always be loaded on the master thread, not on a parallel worker thread.
-//This is important in case some instructions must be executed on the main thread, e.g. OpenGL commands, like glBindBuffer.
-//This resource can still be loaded asyncronously, but it will not be loaded in a thread, instead it's constructor will 
-//be called once on every ResourceManager::Load, just throw StillLoadingException in the constructor if it is not done yet.
-class ThreadUnsafeResource : public Resource
-{
-    friend class ResourceManager;
-};
-
 /** Singleton resource manager to keep track of and cache any external engine assets */
 class ResourceManager
 {
@@ -65,7 +40,6 @@ private:
 	ResourceManager();
 
 public:
-    static bool UseThreading;
 	/*static ResourceManager& Instance()
 	{
 	static ResourceManager s;
@@ -75,6 +49,15 @@ public:
 	template <typename T>
 	static void RegisterType(std::string typeName);
 
+	/** Preloads a resource and caches it for future use
+
+		@tparam T Resource type.
+		@param resourceName Fully qualified name of the resource to preload.
+	*/
+	template <typename T>
+	static void Preload(std::string resourceName);
+	static void Preload(std::string resourceType, std::string resourceName);
+
 	/** Checks if a resource is in cache
 
 		@param resourceType Resource type as string.
@@ -83,19 +66,14 @@ public:
 	// TODO: Templateify
 	static bool IsResourceLoaded(std::string resourceType, std::string resourceName);
 
-	/** Return value should always be a valid pointer, will throw an exception on error.
+	/** Hot-loads a resource and caches it for future use
-        If the resource has been loaded already, returns a pointer to it.
-
-        If Async is false: Hot-loads a resource, caches it for future use, and returns a pointer to it. 
-        If Async is true: If the resource is not loaded yet, starts loading the resource 
-        in the background and throws Resource::StillLoadingException immediately.
 
 		@tparam T Resource type.
-		@tparam async Set this to true if the resource should be loaded asyncronously.
 		@param resourceName Fully qualified name of the resource to load.
 	*/
-    template <typename T, bool async = false>
+	template <typename T>
-    static T* Load(const std::string& resourceName, Resource* parent = nullptr);
+	static T* Load(std::string resourceName, Resource* parent = nullptr);
+	static Resource* Load(std::string resourceType, std::string resourceName, Resource* parent = nullptr);
 
 	/** Reloads an already loaded resource, keeping its resource ID intact.
 
@@ -109,31 +87,19 @@ public:
 	static void Update();
 
 private:
-    //This is a haxy way to make sure that IsMainThread is run when the program starts, so the master thread id is set.
-    struct MasterThreadChecker
-    {
-        MasterThreadChecker()
-        {
-            ResourceManager::IsMainThread();
-        }
-    };
-    const static MasterThreadChecker m_Checker;
-
 	static std::unordered_map<std::string, std::string> m_CompilerTypenameToResourceType;
     static std::unordered_map<std::string, std::function<Resource*(std::string)>> m_FactoryFunctions; // type -> factory function
 	static std::unordered_map<std::pair<std::string, std::string>, Resource*> m_ResourceCache; // (type, name) -> resource
 	static std::unordered_map<std::string, Resource*> m_ResourceFromName; // name -> resource
 	static std::unordered_map<Resource*, Resource*> m_ResourceParents; // resource -> parent resource
 
-	static std::unordered_map<std::pair<std::string, std::string>, boost::thread> m_LoadingThreads; // (type, name) -> loading thread
-	static std::unordered_map<std::pair<std::string, std::string>, std::exception_ptr> m_LoadingThreadExceptions; // (type, name) -> exceptions
-    static boost::recursive_mutex m_Mutex;
-
 	// TODO: Getters for IDs
 	static unsigned int m_CurrentResourceTypeID;
 	static std::unordered_map<std::string, unsigned int> m_ResourceTypeIDs;
 	// Number of resources of a type. Doubles as local ID.
 	static std::unordered_map<unsigned int, unsigned int> m_ResourceCount;
+	// Flag to suppress hot-load warnings when a preloading resource chain loads another resource
+	static bool m_Preloading;
 
 	static FileWatcher m_FileWatcher;
 	static void fileWatcherCallback(std::string path, FileWatcher::FileEventFlags flags);
@@ -142,13 +108,22 @@ private:
 	static unsigned int GetNewResourceID(unsigned int typeID);
 
 	// Internal: Create a resource and cache it
-    static Resource* createResourceThrowing(const std::string& resourceType, const std::string& resourceName, Resource* parent);
+	static Resource* CreateResource(std::string resourceType, std::string resourceName, Resource* parent);
-	static Resource* createResource(const std::string& resourceType, const std::string& resourceName, Resource* parent, std::exception_ptr& exception);
-    static Resource* cacheResource(Resource* resource, const std::string& resourceType, const std::string& resourceName, Resource* parent);
-
-    static bool IsMainThread();
 };
 
+template <typename T>
+T* ResourceManager::Load(std::string resourceName, Resource* parent /* = nullptr */)
+{
+	auto resourceTypename = typeid(T).name();
+	auto it = m_CompilerTypenameToResourceType.find(resourceTypename);
+	if (it == m_CompilerTypenameToResourceType.end()) {
+		LOG_ERROR("Failed to load resource \"%s\" of type \"%s\": type not registered", resourceName.c_str(), resourceTypename);
+		return nullptr;
+	}
+
+	return static_cast<T*>(Load(it->second, resourceName, parent));
+}
+
 template <typename T>
 void ResourceManager::RegisterType(std::string typeName)
 {
@@ -156,85 +131,17 @@ void ResourceManager::RegisterType(std::string typeName)
 	m_FactoryFunctions[typeName] = [](std::string resourceName) { return new T(resourceName); };
 }
 
-template <typename T, bool async>
+template <typename T>
-static T* ResourceManager::Load(const std::string& resourceName, Resource* parent /* = nullptr */)
+void ResourceManager::Preload(std::string resourceName)
 {
 	auto resourceTypename = typeid(T).name();
-    auto iter = m_CompilerTypenameToResourceType.find(resourceTypename);
+	auto it = m_CompilerTypenameToResourceType.find(resourceTypename);
-    if (iter == m_CompilerTypenameToResourceType.end()) {
+	if (it == m_CompilerTypenameToResourceType.end()) {
 		LOG_ERROR("Failed to load resource \"%s\" of type \"%s\": type not registered", resourceName.c_str(), resourceTypename);
-        throw Resource::FailedLoadingException();
+		return;
 	}
 
-    std::string resourceType = iter->second;
+	Preload(it->second, resourceName);
-    constexpr bool mustNotLoadInThread = std::is_base_of<ThreadUnsafeResource, T>::value;
-    if (mustNotLoadInThread && !IsMainThread()) {
-        LOG_ERROR("Failed to Load \"%s\": ThreadUnsafeResource type \"%s\" load in the constructor of another resource that is loaded asyncronously.", resourceName.c_str(), iter->second.c_str());
-        throw Resource::FailedLoadingException();
-    }
-
-    auto cacheKey = std::make_pair(resourceType, resourceName);
-    decltype(m_ResourceCache)::iterator it;
-    //If a thread has already been launched to load this resource.
-    auto tIt = m_LoadingThreads.find(cacheKey);
-    if (UseThreading && tIt != m_LoadingThreads.end()) {
-        if (async) {
-            //Throw StillLoadingException if the thread is still working.
-            if (!tIt->second.try_join_for(boost::chrono::nanoseconds(1))) {
-                throw Resource::StillLoadingException();
-            }
-            //Else we know the thread has completed.
-        } else {
-            //Wait for the thread to finish loading.
-            tIt->second.join();
-        }
-        //When the thread is done, delete the thread.
-        m_LoadingThreads.erase(tIt);
-        //Rethrow the thread exception if it threw any.
-        auto excIt = m_LoadingThreadExceptions.find(cacheKey);
-        std::exception_ptr exception = excIt->second;
-        m_LoadingThreadExceptions.erase(excIt);
-        if (exception) {
-            std::rethrow_exception(exception);
-        }
-    }
-
-    //If resource has already been cached and completely loaded.
-    it = m_ResourceCache.find(cacheKey);
-    if (it != m_ResourceCache.end()) {
-        if (it->second != nullptr) {
-            return static_cast<T*>(it->second);
-        } else {
-            //Don't return null on failure, exception instead.
-            throw Resource::FailedLoadingException();
-        }
-    }
-
-    //If resource is not cached..
-    if (UseThreading && async) {
-        if (mustNotLoadInThread) {
-            try {
-                return static_cast<T*>(createResourceThrowing(resourceType, resourceName, parent));
-            } catch (const Resource::StillLoadingException&) {
-                throw;
-            }
-        } else {
-            //Create a thread that loads the resource into cache.
-            m_LoadingThreads[cacheKey] = boost::thread(createResource, resourceType, resourceName, parent, m_LoadingThreadExceptions[cacheKey]);
-            throw Resource::StillLoadingException();
-        }
-    } else {
-        //load and return the resource.
-        while (true) {
-            try {
-                return static_cast<T*>(createResourceThrowing(resourceType, resourceName, parent));
-            } catch (const Resource::StillLoadingException&) {
-                continue;
-            } catch (const std::exception&) {
-                throw;
-            }
-        }
-    }
 }
 
 #endif

include/Engine/Core/System.h

@@ -3,49 +3,23 @@
 
 #include "EventBroker.h"
 #include "World.h"
-#include "EntityWrapper.h"
 #include "ComponentWrapper.h"
 
 class System
 {
     friend class SystemPipeline;
 
-protected:
+public:
-    System()
+    System(EventBroker* eventBroker, std::string componentType)
-        : m_EventBroker(nullptr)
-    { }
-    System(EventBroker* eventBroker)
         : m_EventBroker(eventBroker)
+        , m_ComponentType(componentType)
     { }
-    virtual ~System() = default;
 
+    virtual void Update(World* world, ComponentWrapper& component, double dt) = 0;
+
+protected:
+    std::string m_ComponentType;
     EventBroker* m_EventBroker;
 };
 
-class PureSystem : public virtual System
-{
-    friend class SystemPipeline;
-
-protected:
-    PureSystem(std::string componentType)
-        : m_ComponentType(componentType)
-    { }
-    virtual ~PureSystem() = default;
-
-    const std::string m_ComponentType;
-
-    virtual void UpdateComponent(World* world, EntityWrapper& entity, ComponentWrapper& component, double dt) = 0;
-};
-
-class ImpureSystem : public virtual System
-{
-    friend class SystemPipeline;
-
-protected:
-    ImpureSystem() = default;
-    virtual ~ImpureSystem() = default;
-
-    virtual void Update(World* world, double dt) = 0;
-};
-
 #endif

include/Engine/Core/SystemPipeline.h

@@ -14,52 +14,28 @@ public:
     { }
     ~SystemPipeline()
     {
-        for (UnorderedSystems& group : m_OrderedSystemGroups) {
+        for (auto& pair : m_Systems) {
-            for (auto& pair : group.Systems) {
+            for (auto& system : pair.second) {
-                delete pair.second;
+                delete system;
             }
         }
     }
 
     template <typename T, typename... Arguments>
-    //All systems with orderlevel 0 will be updated first, then 1, 2, etc.
+    void AddSystem(Arguments... args)
-    void AddSystem(int updateOrderLevel, Arguments... args)
     {
-        if (updateOrderLevel + 1 > m_OrderedSystemGroups.size()) {
-            m_OrderedSystemGroups.resize(updateOrderLevel + 1);
-        }
-        UnorderedSystems& group = m_OrderedSystemGroups[updateOrderLevel];
         System* system = new T(m_EventBroker, args...);
-        group.Systems[typeid(T).name()] = system;
+        if (!system->m_ComponentType.empty()) {
-
+            m_Systems[system->m_ComponentType].push_back(system);
-        PureSystem* pureSystem = dynamic_cast<PureSystem*>(system);
-        if (pureSystem != nullptr) {
-            if (!pureSystem->m_ComponentType.empty()) {
-                group.PureSystems[pureSystem->m_ComponentType].push_back(pureSystem);
         } else {
-                LOG_ERROR("Failed to add pure system \"%s\": Missing component type!", typeid(T).name());
+            LOG_ERROR("Failed to add system \"%s\": Missing component type!", typeid(T).name());
-            }
+            delete system;
-        }
-
-        ImpureSystem* impureSystem = dynamic_cast<ImpureSystem*>(system);
-        if (impureSystem != nullptr) {
-            group.ImpureSystems.push_back(impureSystem);
         }
     }
 
     void Update(World* world, double dt)
     {
-        for (UnorderedSystems& group : m_OrderedSystemGroups) {
+        for (auto& pair : m_Systems) {
-            // Process events
-            for (auto& pair : group.Systems) {
-                m_EventBroker->Process(pair.first);
-            }
-
-            // 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;
             const ComponentPool* pool = world->GetComponents(componentName);
@@ -68,8 +44,7 @@ public:
             }
             for (auto& component : *pool) {
                 for (auto& system : systems) {
-                        system->UpdateComponent(world, EntityWrapper(world, component.EntityID), component, dt);
+                    system->Update(world, component, dt);
-                    }
                 }
             }
         }
@@ -77,13 +52,7 @@ public:
 
 private:
     EventBroker* m_EventBroker;
-    struct UnorderedSystems
+    std::unordered_map<std::string, std::vector<System*>> m_Systems;
-    {
-        std::map<std::string, System*> Systems;
-        std::map<std::string, std::vector<PureSystem*>> PureSystems;
-        std::vector<ImpureSystem*> ImpureSystems;
-    };
-    std::vector<UnorderedSystems> m_OrderedSystemGroups;
 };
 
 #endif

include/Engine/Core/Transform.h

@@ -1,17 +0,0 @@
-#ifndef Transform_h__
-#define Transform_h__
-
-#include "../GLM.h"
-#include "World.h"
-
-namespace Transform
-{
-
-glm::vec3 AbsolutePosition(World* world, EntityID entity);
-glm::quat AbsoluteOrientation(World* world, EntityID entity);
-glm::vec3 AbsoluteScale(World* world, EntityID entity);
-glm::mat4 ModelMatrix(EntityID entity, World* world);
-
-}
-
-#endif

include/Engine/Core/Util/IfDebug.h

@@ -1,12 +0,0 @@
-// Example:
-// DEBUG_IF(condition) {
-//    // This code is executed only in debug mode and if condition is true.
-// }
-// NOTE: condition statement is not executed at all in release mode.
-#ifndef DEBUG_IF
-#ifndef DEBUG
-#define DEBUG_IF(c) if(c)
-#else
-#define DEBUG_IF(c) if(false)
-#endif
-#endif

include/Engine/Core/Util/XercesString.h

@@ -1,46 +0,0 @@
-#ifndef Util_XercesString_h__
-#define Util_XercesString_h__
-
-#include <string>
-#include <xercesc/util/XercesDefs.hpp>
-#include <xercesc/util/XMLString.hpp>
-
-namespace XS
-{
-
-class ToString
-{
-public:
-    ToString(const XMLCh* const str) { m_AsChar = xercesc::XMLString::transcode(str); }
-    ~ToString()
-    {
-        if (m_AsChar != nullptr) {
-            xercesc::XMLString::release(&m_AsChar);
-        }
-    }
-
-    operator std::string() const { return std::string(m_AsChar); }
-private:
-    char* m_AsChar = nullptr;
-};
-
-class ToXMLCh
-{
-public:
-    ToXMLCh(const std::string str) { m_Transcoded = xercesc::XMLString::transcode(str.c_str()); }
-    ToXMLCh(const char* str) { m_Transcoded = xercesc::XMLString::transcode(str); }
-    ~ToXMLCh()
-    {
-        if (m_Transcoded != nullptr) {
-            xercesc::XMLString::release(&m_Transcoded);
-        }
-    }
-
-    operator const XMLCh*() const { return m_Transcoded; }
-private:
-    XMLCh* m_Transcoded = nullptr;
-};
-
-}
-
-#endif

include/Engine/Core/World.h

@@ -14,45 +14,24 @@ public:
 
     // Create empty entity
     EntityID CreateEntity(EntityID parent = 0);
-    // Delete entity and all components within
+
-    void DeleteEntity(EntityID entity);
-    // Check if an entity exists
-    bool ValidEntity(EntityID entity) const;
     // Register a component type and allocate space for it
     void RegisterComponent(ComponentInfo& ci);
     // Attach a component to an entity and fill it with default values
-    ComponentWrapper AttachComponent(EntityID entity, const std::string& componentType);
+    ComponentWrapper AttachComponent(EntityID entity, std::string componentType);
-    // Check if an entity has a component
-    bool HasComponent(EntityID entity, const std::string& componentType) const;
     // Get a component of an entity
-    ComponentWrapper GetComponent(EntityID entity, const std::string& componentType);
+    ComponentWrapper GetComponent(EntityID entity, std::string componentType);
-    // Delete a component off an entity
-    void DeleteComponent(EntityID entity, const std::string& componentType);
     // Get all components of the specified type
-    const ComponentPool* GetComponents(const std::string& componentType);
+    const ComponentPool* GetComponents(std::string componentType);
     // Get entity parent
     EntityID GetParent(EntityID entity);
-    // Change the parent of an entity
-    void SetParent(EntityID entity, EntityID parent);
-    // Get children of an entity
-    const std::pair<std::unordered_multimap<EntityID, EntityID>::const_iterator, std::unordered_multimap<EntityID, EntityID>::const_iterator> GetChildren(EntityID entity);
-    // Get all component pools
-    const std::unordered_map<std::string, ComponentPool*>& GetComponentPools() const { return m_ComponentPools; }
-    // Get the entity children map
-    const std::unordered_multimap<EntityID, EntityID>& GetEntityChildren() const { return m_EntityChildren; }
-    // Set the textual name of an entity
-    void SetName(EntityID entity, const std::string& name);
-    // Get the textual name of an entity
-    std::string GetName(EntityID entity) const;
 
 private:
-    EntityID m_CurrentEntityID = 0;
+    EntityID m_CurrentEntityID = 1;
 
     std::unordered_map<EntityID, EntityID> m_EntityParents;
-    // TODO: This should be a more effective structure
     std::unordered_multimap<EntityID, EntityID> m_EntityChildren;
     std::unordered_map<std::string, ComponentPool*> m_ComponentPools;
-    std::unordered_map<EntityID, std::string> m_EntityNames;
 
     EntityID generateEntityID();
 };

include/Engine/Editor/EditorSystem.h

@@ -1,94 +0,0 @@
-#include <imgui/imgui.h>
-#include <glm/gtx/common.hpp>
-#include <boost/filesystem/path.hpp>
-#include <nativefiledialog/nfd.h>
-#include "../Core/System.h"
-#include "../Core/EMousePress.h"
-#include "../Core/EMouseRelease.h"
-#include "../Core/EMouseMove.h"
-#include "../Core/ConfigFile.h"
-#include "../Input/EInputCommand.h"
-#include "../Rendering/IRenderer.h"
-#include "../Core/Transform.h"
-#include "../Core/EFileDropped.h"
-#include "../Core/EntityFilePreprocessor.h"
-#include "../Core/EntityFileParser.h"
-#include "../Core/EntityFileWriter.h"
-
-class EditorSystem : public ImpureSystem
-{
-public:
-    EditorSystem(EventBroker* eventBroker, IRenderer* renderer);
-
-    virtual void Update(World* world, double dt) override;
-
-private:
-    IRenderer* m_Renderer;
-    World* m_World = nullptr;
-    Camera* m_Camera = nullptr;
-
-    bool m_Enabled;
-    bool m_Visible;
-    boost::filesystem::path m_DefaultEntityDir;
-    boost::filesystem::path m_CurrentFile;
-    std::vector<glm::vec2> m_PickingQueue;
-
-    enum class WidgetMode
-    {
-        None,
-        Translate,
-        Rotate,
-        Scale
-    } m_WidgetMode = WidgetMode::None;
-
-    enum class WidgetSpace
-    {
-        Local,
-        Global
-    } m_WidgetSpace = WidgetSpace::Global;
-
-    EntityID m_Widget = EntityID_Invalid;
-    EntityID m_WidgetX = EntityID_Invalid;
-    EntityID m_WidgetPlaneX = EntityID_Invalid;
-    EntityID m_WidgetY = EntityID_Invalid;
-    EntityID m_WidgetPlaneY = EntityID_Invalid;
-    EntityID m_WidgetZ = EntityID_Invalid;
-    EntityID m_WidgetPlaneZ = EntityID_Invalid;
-    EntityID m_WidgetOrigin = EntityID_Invalid;
-    glm::vec3 m_WidgetCurrentAxis;
-    float m_WidgetPickingDepth = 0.f;
-    glm::vec3 m_WidgetPickingPosition = glm::vec3(0);
-
-    EntityID m_Selection = EntityID_Invalid;
-    EntityID m_LastSelection = EntityID_Invalid;
-    EntityID m_UIDraggingEntity = EntityID_Invalid;
-    glm::vec3 m_Position;
-    std::string m_LastDroppedFile;
-
-    static boost::filesystem::path openDialog(boost::filesystem::path defaultPath);
-    static boost::filesystem::path saveDialog(boost::filesystem::path defaultPath);
-
-    EventRelay<EditorSystem, Events::InputCommand> m_EInputCommand;
-    bool OnInputCommand(const Events::InputCommand& e);
-    EventRelay<EditorSystem, Events::MouseRelease> m_EMouseRelease;
-    bool OnMouseRelease(const Events::MouseRelease& e);
-    EventRelay<EditorSystem, Events::MousePress> m_EMousePress;
-    bool OnMousePress(const Events::MousePress& e);
-    EventRelay<EditorSystem, Events::MouseMove> m_EMouseMove;
-    bool OnMouseMove(const Events::MouseMove& e);
-    EventRelay<EditorSystem, Events::FileDropped> m_EFileDropped;
-    bool OnFileDropped(const Events::FileDropped& e);
-    
-    void Picking();
-    void createWidget();
-    void updateWidget();
-    void setWidgetMode(WidgetMode newMode);
-    void setWidgetSpace(WidgetSpace space);
-    void drawUI(World* world, double dt);
-    bool createDeleteButton(std::string componentType);
-    bool createEntityNode(World* world, EntityID entity);
-    void changeParent(EntityID entity, EntityID newParent);
-    void fileImport(World* world);
-    void fileSave(World* world);
-    void fileSaveAs(World* world);
-};

include/Engine/GUI/Button.h

@@ -40,7 +40,7 @@ public:
 		m_TexturePressed = resourceName;
 	}
 
-	void Draw(RenderScene& rq) override
+	void Draw(RenderQueueCollection& rq) override
 	{
 		if (m_Texture == nullptr && !m_TextureReleased.empty()) {
 			SetTexture(m_TextureReleased);

include/Engine/GUI/Frame.h

@@ -212,7 +212,7 @@ public:
 
 	virtual void Update(double dt) { }
 
-	void DrawLayered(RenderScene& rq)
+	void DrawLayered(RenderQueueCollection& rq)
 	{
 		if (this->Hidden())
 			return;
@@ -232,7 +232,7 @@ public:
 		}
 	}
 
-	virtual void Draw(RenderScene& rq) { }
+	virtual void Draw(RenderQueueCollection& rq) { }
 
 protected:
 	::EventBroker* m_EventBroker;

include/Engine/GUI/TextureFrame.h

@@ -16,7 +16,7 @@ public:
 	void EnableScissor() { m_ScissorEnabled = true; }
 	void DisableScissor() { m_ScissorEnabled = false; }
 
-	void Draw(RenderScene& rq) override
+	void Draw(RenderQueueCollection& rq) override
 	{
 		if (m_Texture == nullptr)
 			return;

include/Engine/Input/InputProxy.h

@@ -1,7 +1,6 @@
 #ifndef InputProxy_h__
 #define InputProxy_h__
 
-#include <boost/tokenizer.hpp>
 #include "../Common.h"
 #include "../Core/ResourceManager.h"
 #include "../Core/ConfigFile.h"

include/Engine/Input/XboxControllerInputHandler.h

@@ -0,0 +1,40 @@
+#ifndef XboxControllerInputHandler_h__
+#define XboxControllerInputHandler_h__
+
+#include "InputHandler.h"
+#include "Core/EKeyDown.h"
+#include "Core/EKeyUp.h"
+
+struct _XINPUT_STATE;
+typedef _XINPUT_STATE XINPUT_STATE;
+
+class XboxControllerInputHandler : public InputHandler
+{
+public:
+    XboxControllerInputHandler(EventBroker* eventBroker, InputProxy* inputProxy);
+
+
+    bool BindOrigin(std::string origin, std::string command, float value) override;
+
+    void Update(double dt) override;
+
+    virtual float GetCommandValue(std::string command) override;
+private:
+    std::unordered_map<std::string, int> m_OriginButtons;
+    std::unordered_map<std::string, int> m_OriginAxes;
+	std::unordered_map<int, std::tuple<std::string, float>> m_ButtonBindings; // GLFW_KEY... -> command string & value
+	std::unordered_map<int, std::tuple<std::string, float>> m_AxisBindings; // GLFW_KEY... -> command string & value
+    std::unordered_map<std::string, float> m_CommandValues;
+
+	static const short MAX_GAMEPADS = 4;
+
+    float getAxisValue(XINPUT_STATE& state, int axis);
+	//typedef std::array<float, static_cast<int>(Gamepad::Axis::LAST) + 1> GamepadAxisState;
+	//std::array<GamepadAxisState, MAX_GAMEPADS> m_CurrentGamepadAxisState;
+	//std::array<GamepadAxisState, MAX_GAMEPADS> m_LastGamepadAxisState;
+	//typedef std::array<bool, static_cast<int>(Gamepad::Button::LAST) + 1> GamepadButtonState;
+	//std::array<GamepadButtonState, MAX_GAMEPADS> m_CurrentGamepadButtonState;
+	//std::array<GamepadButtonState, MAX_GAMEPADS> m_LastGamepadButtonState;
+};
+
+#endif

include/Engine/Network/Client.h

@@ -1,83 +1,14 @@
 #ifndef Client_h__
 #define Client_h__
 
-#include <string>
+#include <boost\asio.hpp>
-#include <ctime>
 
-#include <glm/common.hpp>
+class Client
-#include <boost/asio.hpp>
-
-#include "Network/Network.h"
-#include "Network/MessageType.h"
-#include "Network/PlayerDefinition.h"
-#include "Network/SnapshotDefinitions.h"
-#include "Core/World.h"
-#include "Core/EventBroker.h"
-#include "Core/ConfigFile.h"
-#include "Input/EInputCommand.h"
-
-class Client : public Network
 {
-public:
+	Client();
-    Client(ConfigFile* config);
 	~Client();
-    void Start(World* world, EventBroker* eventBroker) override;
-    void Update() override;
-private:
-    // Assio UDP logic
-    boost::asio::ip::udp::endpoint m_ReceiverEndpoint;
-    boost::asio::io_service m_IOService;
-    boost::asio::ip::udp::socket m_Socket;
 
-    // Sending message to server logic
-    int bytesRead = -1;
-    char readBuf[INPUTSIZE] = { 0 };
-    int snapshotInterval = 33;
-    std::clock_t previousSnapshotMessage = std::clock();
 
-    // Packet loss logic
-    unsigned int m_PacketID = 0;
-    unsigned int m_PreviousPacketID = 0;
-    unsigned int m_SendPacketID = 0;
-
-    // Game logic
-    World* m_World;
-    std::string m_PlayerName;
-    int m_PlayerID = -1;
-
-    // Network logic
-    PlayerDefinition m_PlayerDefinitions[MAXCONNECTIONS];
-    SnapshotDefinitions m_NextSnapshot;
-    double m_DurationOfPingTime;
-    std::clock_t m_StartPingTime;
-    // Use to check if we should send disconnect message
-    // if game is turned of by closing window.
-    bool m_WasStarted = false;
-
-    // Private member functions
-    void readFromServer();
-    void sendSnapshotToServer();
-    int  receive(char* data, size_t length);
-    void send(Packet& packet);
-    void connect();
-    void disconnect();
-    void ping();
-    void moveMessageHead(char*& data, size_t& length, size_t stepSize);
-    void parseMessageType(Packet& packet);
-    void parseEventMessage(Packet& packet);
-    void updateFields(Packet& packet, const ComponentInfo& componentInfo, const EntityID& entityID, const std::string& componentType);
-    void parseConnect(Packet& packet);
-    void parsePing();
-    void parseServerPing();
-    void parseSnapshot(Packet& packet);
-    void identifyPacketLoss();
-    bool isConnected();
-    EntityID createPlayer();
-
-    // Events
-    EventBroker* m_EventBroker;
-    EventRelay<Client, Events::InputCommand> m_EInputCommand;
-    bool OnInputCommand(const Events::InputCommand &e);
 };
 
 #endif

include/Engine/Network/MessageType.h

@@ -1,17 +0,0 @@
-#ifndef MessageType_h__
-#define MessageType_h__
-
-// Message types used by both server and client.
-// Used to determine what type of message was sent.
-enum class MessageType
-{
-	Connect,
-	Disconnect,
-	ClientPing,
-	ServerPing,
-	Message,
-	Snapshot,
-	Event,
-};
-
-#endif

include/Engine/Network/Network.h

@@ -1,19 +0,0 @@
-#ifndef Network_h__
-#define Network_h__
-
-#include "Core/World.h"
-#include "Core/EventBroker.h"
-#include "Network/Packet.h"
-
-#define MAXCONNECTIONS 8
-#define INPUTSIZE 4097
-
-class Network
-{
-public:
-    virtual ~Network() { };
-    virtual void Start(World* m_world, EventBroker *eventBroker) = 0;
-    virtual void Update() = 0;
-};
-
-#endif

include/Engine/Network/Packet.h

@@ -1,66 +0,0 @@
-#ifndef Packet_h__
-#define Packet_h__
-
-#include <string>
-#include "Network/MessageType.h"
-#include "Core/Util/Logging.h"
-
-// Defines the 
-class Packet
-{
-public:
-    // arg1: Type of message (Connect, Disconnect...)
-    // arg2: PacketID for identifying packet loss.
-    Packet(MessageType type, unsigned int& packetID);
-    // Used to create packet from already existing data buffer.
-    Packet(char* data, const int sizeOfPacket);
-    ~Packet();
-    void Init(MessageType type, unsigned int& packetID);
-
-    // Add primitive types like int, float, char...
-    template<typename T>
-    void WritePrimitive(T val)
-    {
-        // Check if we are trying to add more than the package can fit.
-        if (m_MaxPacketSize < m_Offset + sizeof(T)) {
-            LOG_WARNING("Packet AddPrimitive(): You are trying to add more than we have allocated for! New size is %i bytes\n", m_MaxPacketSize*2);
-            resizeData();
-        }
-        memcpy(m_Data + m_Offset, &val, sizeof(T));
-        m_Offset += sizeof(T);
-    }
-    // Pops the first element as if it was a primitive.
-    template<typename T>
-    T ReadPrimitive()
-    {
-        if (m_Offset < m_ReturnDataOffset + sizeof(T)) {
-            LOG_WARNING("Packet PopFrontPrimitive(): You are trying to remove more than what exists in this packet!");
-            return -1;
-        }
-        T returnValue;
-        memcpy(&returnValue, m_Data + m_ReturnDataOffset, sizeof(T));
-        m_ReturnDataOffset += sizeof(T);
-        return returnValue;
-    }
-    // Add a string to the message
-    void WriteString(const std::string& str);
-    // Add data to the message
-    void WriteData(char* data, int sizeOfData);
-    // Pops the first element as if it was a string.
-    std::string ReadString();
-    char* ReadData(int SizeOfData);
-
-    int Size() { return m_Offset; };
-    char* Data() { return m_Data; };
-    unsigned int DataReadSize() { return m_ReturnDataOffset; }
-    unsigned int MaxSize() { return m_MaxPacketSize; }
-
-private:
-    char* m_Data;
-    unsigned int m_ReturnDataOffset = 0;
-    int m_Offset = 0;
-    unsigned int m_MaxPacketSize = 512;
-    void resizeData();
-};
-
-#endif

include/Engine/Network/PlayerDefinition.h

@@ -1,11 +0,0 @@
-#ifndef PlayerDefinition_h__
-#define PlayerDefinition_h__
-#include <string>
-
-struct PlayerDefinition {
-    int EntityID = -1;
-    std::string Name = "";
-    boost::asio::ip::udp::endpoint Endpoint;
-};
-
-#endif

include/Engine/Network/Server.h

@@ -1,80 +1,12 @@
 #ifndef Server_h__
 #define Server_h__
 
-#include <string>
+#include <boost\asio.hpp>
-#include <ctime>
 
-#include <glm/common.hpp>
+class Server
-#include <boost/asio/ip/udp.hpp>
-
-#include "Network/MessageType.h"
-#include "Network/PlayerDefinition.h"
-#include "Core/World.h"
-#include "Core/EventBroker.h"
-#include "Network/Network.h"
-
-class Server : public Network
 {
-public:
 	Server();
 	~Server();
-    void Start(World* m_world, EventBroker *eventBroker) override;
-    void Update() override;
-private:
-    // UDP logic
-    boost::asio::ip::udp::endpoint m_ReceiverEndpoint;
-    boost::asio::io_service m_IOService;
-    boost::asio::ip::udp::socket m_Socket;
-    PlayerDefinition m_PlayerDefinitions[MAXCONNECTIONS];
-
-    // Sending messages to client logic
-    char readBuffer[INPUTSIZE] = { 0 };
-    int bytesRead = 0;
-    // time for previouse message
-    std::clock_t previousePingMessage = std::clock();
-    std::clock_t previousSnapshotMessage = std::clock();
-    std::clock_t timOutTimer = std::clock();
-    // How often we send messages (milliseconds)
-    int intervalMs = 1000;
-    int snapshotInterval = 50;
-    int checkTimeOutInterval = 100;
-
-    //Timers
-    std::clock_t m_StartPingTime;
-    std::clock_t m_StopTimes[8];
-
-    // Game logic
-    World* m_World;
-    EventBroker* m_EventBroker;
-    // vec.size() = ammount of players to create, stores playerID's
-    std::vector<unsigned int> m_PlayersToCreate;
-    
-    // Packet loss logic
-    unsigned int m_PacketID;
-    unsigned int m_PreviousPacketID;
-    unsigned int m_SendPacketID;
-
-    // Private member functions
-    int  receive(char* data, size_t length);
-    void readFromClients();
-    void send(Packet& packet, int playerID);
-    void send(Packet& packet);
-    void moveMessageHead(char*& data, size_t& length, size_t stepSize);
-    void broadcast(std::string message);
-    void broadcast(Packet& packet);
-    void sendSnapshot();
-    void sendPing();
-    void checkForTimeOuts();
-    void disconnect(int i);
-    void parseMessageType(Packet& packet);
-    void parseEvent(Packet& packet);
-    void parseConnect(Packet& packet);
-    void parseDisconnect();
-    void parseClientPing();
-    void parseServerPing();
-    void parseSnapshot(Packet& packet);
-    void identifyPacketLoss();
-    EntityID createPlayer();
 };
 
 #endif

include/Engine/Network/SnapshotDefinitions.h

@@ -1,12 +0,0 @@
-#ifndef SnapshotDefinitions_h__
-#define SnapshotDefinitions_h__
-
-struct SnapshotDefinitions
-{
-	// "+Forward" is 8 characters * sizeof(char) = 8
-	std::string InputForward;
-	// "+Right" is 6 characters * sizeof(char) = 6
-    std::string InputRight;
-};
-
-#endif

include/Engine/Rendering/BaseTexture.h

@@ -3,7 +3,7 @@
 
 #include "../Core/ResourceManager.h"
 
-class BaseTexture : public ThreadUnsafeResource
+class BaseTexture : public Resource
 {
 	friend class ResourceManager;
 

include/Engine/Rendering/Camera.h

@@ -26,12 +26,13 @@ public:
 	glm::quat Orientation() const { return m_Orientation; }
 	void SetOrientation(glm::quat val);
 
+	/*float Pitch() const { return m_Pitch; }
+	void Pitch(float val);
+	float Yaw() const { return m_Yaw; }
+	void Yaw(float val);*/
+
 	glm::mat4 ProjectionMatrix() const { return m_ProjectionMatrix; }
-    void SetProjectionMatrix(glm::mat4 val);
-
 	glm::mat4 ViewMatrix() const { return m_ViewMatrix; }
-    void SetViewMatrix(glm::mat4 val);
-
 
 	float AspectRatio() const { return m_AspectRatio; }
 	void SetAspectRatio(float val);
@@ -45,10 +46,11 @@ public:
 	float FarClip() const { return m_FarClip; }
 	void SetFarClip(float val);
 
-    void UpdateViewMatrix();
+	
-    void UpdateProjectionMatrix();
 
 private:
+	void UpdateViewMatrix();
+	void UpdateProjectionMatrix();
 
 	glm::vec3 m_Position;
 	glm::quat m_Orientation;

include/Engine/Rendering/DebugCameraInputController.h

@@ -1,4 +1,3 @@
-#include <imgui/imgui.h>
 #include "../Input/FirstPersonInputController.h"
 
 template <typename EventContext>
@@ -9,28 +8,20 @@ public:
         : FirstPersonInputController(eventBroker, playerID)
     { }
 
-    void SetPosition(const glm::vec3 position) { m_Position = position; }
-    void SetOrientation(const glm::quat orientation) { m_Orientation = orientation; }
-
     const glm::vec3 Position() const { return m_Position; }
     void SetBaseSpeed(float speed) { m_BaseSpeed = speed; }
 
     virtual bool OnCommand(const Events::InputCommand& e) override
     {
-        ImGuiIO& io = ImGui::GetIO();
-
         if (e.Command == "PrimaryFire") {
             if (e.Value > 0) {
-                if (!io.WantCaptureMouse) {
                 LockMouse();
-                }
             } else {
                 UnlockMouse();
             }
             return false;
         }
 
-        if (!io.WantCaptureKeyboard) {
         if (e.Command == "Right") {
             float value = std::max(-1.f, std::min(e.Value, 1.f));
             m_Velocity.x = value;
@@ -39,14 +30,6 @@ public:
             float value = std::max(-1.f, std::min(e.Value, 1.f));
             m_Velocity.z = -value;
         }
-            if (e.Command == "Sprint") {
-                if (e.Value > 0.f) {
-                    m_Speed = m_BaseSpeed * 2.f * (e.Value);
-                } else {
-                    m_Speed = m_BaseSpeed;
-                }
-            }
-        }
 
         return FirstPersonInputController::OnCommand(e);
     }
@@ -54,7 +37,7 @@ public:
     void Update(double dt)
     {
         if (glm::length2(m_Velocity) > 0) {
-            m_Position += m_Orientation * (glm::normalize(m_Velocity) * m_Speed * (float)dt);
+            m_Position += (m_Orientation * (m_Velocity * m_BaseSpeed)) * (float)dt;
         }
     }
 
@@ -62,5 +45,4 @@ protected:
     glm::vec3 m_Position = glm::vec3(0, 0, 0);
     glm::vec3 m_Velocity = glm::vec3(0, 0, 0);
     float m_BaseSpeed = 2.0f;
-    float m_Speed = m_BaseSpeed;
 };

include/Engine/Rendering/DrawFinalPass.h

@@ -1,38 +0,0 @@
-#ifndef DrawFinalPass_h__
-#define DrawFinalPass_h__
-
-#include "IRenderer.h"
-#include "DrawFinalPassState.h"
-#include "LightCullingPass.h"
-#include "FrameBuffer.h"
-#include "ShaderProgram.h"
-#include "Util/UnorderedMapVec2.h"
-#include "Texture.h"
-
-class DrawFinalPass
-{
-public:
-    DrawFinalPass(IRenderer* renderer, LightCullingPass* lightCullingPass);
-    ~DrawFinalPass() { }
-    void InitializeTextures();
-    void InitializeFrameBuffers();
-    void InitializeShaderPrograms();
-
-    void Draw(RenderScene& scene);
-
-    //Getters
-
-
-private:
-    void GenerateTexture(GLuint* texture, GLenum wrapping, GLenum filtering, glm::vec2 dimensions, GLint internalFormat, GLint format, GLenum type) const;
-
-    Texture* m_WhiteTexture;
-
-    const IRenderer* m_Renderer;
-    const LightCullingPass* m_LightCullingPass;
-
-    ShaderProgram* m_ForwardPlusProgram;
-
-};
-
-#endif 

include/Engine/Rendering/DrawFinalPassState.h

@@ -1,15 +0,0 @@
-#ifndef DrawFinalPassState_h__
-#define DrawFinalPassState_h__
-
-#include "Rendering/RenderState.h"
-
-class DrawFinalPassState : public RenderState
-{
-public:
-    DrawFinalPassState();
-    ~DrawFinalPassState();
-private:
-
-};
-
-#endif

include/Engine/Rendering/DrawScenePass.h

@@ -1,42 +0,0 @@
-#ifndef DrawScenePass_h__
-#define DrawScenePass_h__
-
-#include "IRenderer.h"
-#include "DrawScenePassState.h"
-#include "FrameBuffer.h"
-#include "ShaderProgram.h"
-#include "Util/UnorderedMapVec2.h"
-#include "Texture.h"
-
-class DrawScenePass
-{
-public:
-    DrawScenePass(IRenderer* renderer);
-    ~DrawScenePass() { }
-    void InitializeTextures();
-    void InitializeFrameBuffers();
-    void InitializeShaderPrograms();
-
-    void Draw(RenderScene& scene);
-
-    //Getters
-
-
-private:
-    void GenerateTexture(GLuint* texture, GLenum wrapping, GLenum filtering, glm::vec2 dimensions, GLint internalFormat, GLint format, GLenum type) const;
-
-    static bool DepthSort(const std::shared_ptr<RenderJob> &i, const std::shared_ptr<RenderJob> &j)
-    {
-        return (i->Depth < j->Depth);
-    };
-
-    Texture* m_WhiteTexture;
-
-    const IRenderer* m_Renderer;
-
-    ShaderProgram* m_BasicForwardProgram;
-
-
-};
-
-#endif 

include/Engine/Rendering/DrawScenePassState.h

@@ -1,15 +0,0 @@
-#ifndef DrawScenePassState_h__
-#define DrawScenePassState_h__
-
-#include "Rendering/RenderState.h"
-
-class DrawScenePassState : public RenderState
-{
-public:
-    DrawScenePassState();
-    ~DrawScenePassState();
-private:
-
-};
-
-#endif

include/Engine/Rendering/DummyRenderer.h

@@ -9,7 +9,7 @@ class DummyRenderer : public IRenderer
 {
 public:
 	virtual void Initialize() override;
-	virtual void Draw(RenderFrame& rq) override;
+	virtual void Draw(RenderQueueCollection& rq) override;
 };
 
 #endif

include/Engine/Rendering/EPicking.h

@@ -0,0 +1,69 @@
+#ifndef Events_Picking_h__
+#define Events_Picking_h__
+
+#include "../OpenGL.h"
+#include "../GLM.h"
+
+#include "../Core/EventBroker.h"
+#include "Util/ScreenCoords.h"
+#include "FrameBuffer.h"
+#include "../Core/Entity.h"
+#include "Util/UnorderedMapVec2.h"
+
+namespace Events
+{
+
+/** Thrown Every frame, use functions to pick*/
+struct Picking : Event
+{
+public:
+    Picking(FrameBuffer* pickingBuffer, GLuint* depthBuffer, glm::mat4 projectionMatrix, glm::mat4 viewMatrix, Rectangle resolution, const std::unordered_map<glm::vec2, EntityID>* pickingColorsToEntity)
+        : PickingBuffer(pickingBuffer)
+        , DepthBuffer(depthBuffer)
+        , ProjectionMatrix(projectionMatrix)
+        , ViewMatrix(viewMatrix)
+        , Resolution(resolution)
+        , PickingColorsToEntity(pickingColorsToEntity)
+    { }
+        
+    
+
+    struct PickData
+    {
+        //Picked Entity
+        EntityID Entity;
+        //World position of the "pick"
+        glm::vec3 Position;
+    };
+
+    PickData Pick(glm::vec2 screenCoord) const
+    {
+        PickData pickData;
+
+        ScreenCoords::PixelData data = ScreenCoords::ToPixelData(screenCoord, PickingBuffer, *DepthBuffer);
+
+        auto it = PickingColorsToEntity->find(glm::vec2(data.Color[0], data.Color[1]));
+        if (it != PickingColorsToEntity->end()) {
+            pickData.Entity = it->second;
+        } else {
+            pickData.Entity = -1;
+        }
+        pickData.Position = ScreenCoords::ToWorldPos(screenCoord.x, Resolution.Width - screenCoord.y, data.Depth, Resolution, ProjectionMatrix, ViewMatrix);
+
+        return pickData;
+    }
+
+
+private:
+    FrameBuffer* PickingBuffer;
+    GLuint* DepthBuffer;
+    const glm::mat4 ProjectionMatrix;
+    const glm::mat4 ViewMatrix;
+    const Rectangle Resolution;
+    const std::unordered_map<glm::vec2, EntityID>* PickingColorsToEntity;
+    
+};
+
+}
+
+#endif

include/Engine/Rendering/ESetCamera.h

@@ -1,23 +0,0 @@
-#ifndef Events_SetCamera_h__
-#define Events_SetCamera_h__
-
-#include "../Core/EventBroker.h"
-#include "../Core/Entity.h"
-#include <string.h>
-
-namespace Events
-{
-
-struct SetCamera : Event
-{
-public:
-    SetCamera() { };
-    std::string Name;
-
-private:
-
-};
-
-}
-
-#endif

include/Engine/Rendering/IRenderer.h

@@ -9,17 +9,6 @@
 #include "Camera.h"
 #include "RenderQueue.h"
 #include "Model.h"
-#include "../Core/World.h" //So temp
-
-
-struct PickData
-{
-    EntityID Entity;
-    glm::vec3 Position; //World position
-    float Depth;
-    ::Camera* Camera;
-    const ::World* World;
-};
 
 class IRenderer
 {
@@ -40,22 +29,20 @@ public:
 			m_Camera = camera;
 		}
 	}
+
 	virtual void Initialize() = 0;
 	virtual void Update(double dt) = 0;
-	virtual void Draw(RenderFrame& rq) = 0;
+	virtual void Draw(RenderQueueCollection& rq) = 0;
-    virtual PickData Pick(glm::vec2 screenCord) = 0;
-
-    World* m_World; //Temp world, untill viktor merge.
 
 protected:
-	Rectangle m_Resolution = Rectangle::Rectangle(1280, 720);
+	Rectangle m_Resolution = Rectangle(1280, 720);
 	bool m_Fullscreen = false;
 	bool m_VSYNC = false;
 	int m_GLVersion[2];
 	std::string m_GLVendor;
-	GLFWwindow* m_Window = nullptr;
 	::Camera* m_DefaultCamera;
 	::Camera* m_Camera = nullptr;
+	GLFWwindow* m_Window = nullptr;
 };
 
 #endif // Renderer_h__

include/Engine/Rendering/ImGuiRenderPass.h

@@ -1,77 +0,0 @@
-#include <imgui/imgui.h>
-#include "../OpenGL.h"
-#include "IRenderer.h"
-#include "RenderState.h"
-#include "../Core/EventBroker.h"
-#include "../Core/EMousePress.h"
-#include "../Core/EMouseRelease.h"
-#include "../Core/EMouseMove.h"
-#include "../Core/EMouseScroll.h"
-#include "../Core/EKeyDown.h"
-#include "../Core/EKeyUp.h"
-#include "../Core/EKeyboardChar.h"
-
-class ImGuiRenderState : public RenderState
-{
-public:
-    ImGuiRenderState()
-        : RenderState()
-    {
-        BindFramebuffer(0);
-        Enable(GL_BLEND);
-        BlendEquation(GL_FUNC_ADD);
-        BlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);
-        Disable(GL_CULL_FACE);
-        Disable(GL_DEPTH_TEST);
-        Enable(GL_SCISSOR_TEST);
-    }
-};
-
-class ImGuiRenderPass
-{
-public:
-    ImGuiRenderPass(IRenderer* renderer, EventBroker* eventBroker);
-
-    void Update(double dt);
-    void Draw();
-
-private:
-    IRenderer* m_Renderer;
-    EventBroker* m_EventBroker;
-
-    GLFWwindow* g_Window;
-    double g_DeltaTime = 0.0;
-    float g_MouseWheel = 0.f;
-    GLuint g_FontTexture;
-    int g_ShaderHandle;
-    int g_VertHandle;
-    int g_FragHandle;
-    int g_AttribLocationTex;
-    int g_AttribLocationProjMtx;
-    int g_AttribLocationPosition;
-    int g_AttribLocationUV;
-    int g_AttribLocationColor;
-    GLuint g_VboHandle;
-    GLuint g_VaoHandle;
-    GLuint g_ElementsHandle;
-
-    EventRelay<ImGuiRenderPass, Events::MousePress> m_EMousePress;
-    bool OnMousePress(const Events::MousePress& e);
-    EventRelay<ImGuiRenderPass, Events::MouseRelease> m_EMouseRelease;
-    bool OnMouseRelease(const Events::MouseRelease& e);
-    EventRelay<ImGuiRenderPass, Events::MouseMove> m_EMouseMove;
-    bool OnMouseMove(const Events::MouseMove& e);
-    EventRelay<ImGuiRenderPass, Events::MouseScroll> m_EMouseScroll;
-    bool OnMouseScroll(const Events::MouseScroll& e);
-    EventRelay<ImGuiRenderPass, Events::KeyDown> m_EKeyDown;
-    bool OnKeyDown(const Events::KeyDown& e);
-    EventRelay<ImGuiRenderPass, Events::KeyUp> m_EKeyUp;
-    bool OnKeyUp(const Events::KeyUp& e);
-    EventRelay<ImGuiRenderPass, Events::KeyboardChar> m_EKeyboardChar;
-    bool OnKeyboardChar(const Events::KeyboardChar& e);
-
-    bool createDeviceObjects();
-    bool createFontsTexture();
-
-    void newFrame();
-};

include/Engine/Rendering/LightCullingPass.h

@@ -1,83 +0,0 @@
-#ifndef LightCullingPass_h__
-#define LightCullingPass_h__
-
-#define TILE_SIZE 16 
-#define MAX_LIGHTS_PER_TILE 200
-
-#include "IRenderer.h"
-#include "LightCullingPassState.h"
-#include "ShaderProgram.h"
-#include "RenderQueue.h"
-
-
-class LightCullingPass
-{
-public:
-    LightCullingPass(IRenderer* renderer);
-    ~LightCullingPass();
-
-    void GenerateNewFrustum(RenderScene& scene);
-    void OnResolutionChange();
-    void SetSSBOSizes();
-    void CullLights(RenderScene& scene);
-    void FillLightList(RenderScene& scene);
-
-    GLuint FrustumSSBO() const { return m_FrustumSSBO; }
-    GLuint LightSSBO() const { return m_LightSSBO; }
-    GLuint LightGridSSBO() const { return m_LightGridSSBO; }
-    GLuint LightOffsetSSBO() const { return m_LightOffsetSSBO; }
-    GLuint LightIndexSSBO() const { return m_LightIndexSSBO; }
-private:
-
-    void InitializeSSBOs();
-    void InitializeShaderPrograms();
-
-    const IRenderer* m_Renderer;
-
-    GLuint m_FrustumSSBO = 0;
-    GLuint m_LightSSBO = 0;
-    GLuint m_LightGridSSBO = 0;
-    GLuint m_LightOffsetSSBO = 0;
-    GLuint m_LightIndexSSBO = 0;
-
-    ShaderProgram* m_CalculateFrustumProgram;
-    ShaderProgram* m_LightCullProgram;
-
-    int m_NumberOfTiles = 0;
-
-    struct Plane {
-        glm::vec3 Normal;
-        float d;
-    };
-
-    struct Frustum {
-        Plane Planes[4];
-    };
-    Frustum* m_Frustums;
-
-    //This should be a component
-    struct PointLight {
-        glm::vec4 Position = glm::vec4(0.f);
-        glm::vec4 Color = glm::vec4(1.f);
-        float Radius = 5.f;
-        float Intensity = 0.8f;
-        float Falloff = 0.3f;
-        float Padding = 1337;
-    };
-    std::vector<PointLight> m_PointLights;
-
-    struct LightGrid {
-        float Start;
-        float Amount;
-        glm::vec2 Padding;
-    };
-
-    LightGrid* m_LightGrid;
-
-    int m_LightOffset = 0;
-
-    float* m_LightIndex;
-};
-
-
-#endif

include/Engine/Rendering/Model.h

@@ -4,7 +4,7 @@
 #include "RawModel.h"
 #include "../OpenGL.h"
 
-class Model : public ThreadUnsafeResource
+class Model : public RawModel
 {
 	friend class ResourceManager;
 
@@ -13,15 +13,11 @@ private:
 
 public:
 	~Model();
-    const std::vector<RawModel::MaterialGroup>& MaterialGroups() const { return m_RawModel->MaterialGroups; }
-    const glm::mat4& Matrix() const { return m_RawModel->m_Matrix; }
-    const std::vector<RawModel::Vertex>& Vertices() const { return m_RawModel->m_Vertices; }
 
 	GLuint VAO;
 	GLuint ElementBuffer;
 
 private:
-    RawModel* m_RawModel;
 	GLuint VertexBuffer;
 	GLuint DiffuseVertexColorBuffer;
 	GLuint SpecularVertexColorBuffer;

include/Engine/Rendering/ModelJob.h

@@ -1,55 +0,0 @@
-#ifndef ModelJob_h__
-#define ModelJob_h__
-
-#include <cstdint>
-
-#include "../Common.h"
-#include "../GLM.h"
-#include "../Core/ComponentWrapper.h"
-#include "Texture.h"
-#include "Model.h"
-#include "RenderJob.h"
-#include "../Core/ResourceManager.h"
-#include "Camera.h"
-#include "../Core/World.h"
-
-struct ModelJob : RenderJob
-{
-    ModelJob(Model* model, Camera* camera, glm::mat4 matrix, ::RawModel::MaterialGroup matGroup, ComponentWrapper modelComponent, World* world)
-        : RenderJob()
-    {
-        Model = model;
-        TextureID = (matGroup.Texture) ? matGroup.Texture->ResourceID : 0;
-        DiffuseTexture = matGroup.Texture.get();
-        NormalTexture = matGroup.NormalMap.get();
-        SpecularTexture = matGroup.SpecularMap.get();
-        StartIndex = matGroup.StartIndex;
-        EndIndex = matGroup.EndIndex;
-        Matrix = matrix;
-        Color = modelComponent["Color"];
-        Entity = modelComponent.EntityID;
-        World = world;
-    };
-
-    unsigned int TextureID;
-    unsigned int ShaderID;
-
-    EntityID Entity;
-    glm::mat4 Matrix;
-    const Texture* DiffuseTexture;
-    const Texture* NormalTexture;
-    const Texture* SpecularTexture;
-    float Shininess = 0.f;
-    glm::vec4 Color;
-    const ::Model* Model = nullptr;
-    unsigned int StartIndex = 0;
-    unsigned int EndIndex = 0;
-    World* World;
-
-    void CalculateHash() override
-    {
-        Hash = TextureID;
-    }
-};
-
-#endif

include/Engine/Rendering/PickingPass.h

@@ -1,66 +0,0 @@
-#ifndef PickingPass_h__
-#define PickingPass_h__
-
-
-
-#include "IRenderer.h"
-#include "PickingPassState.h"
-#include "FrameBuffer.h"
-#include "ShaderProgram.h"
-#include "Util/UnorderedMapiVec2.h"
-#include "../Core/EventBroker.h"
-#include "../Core/World.h"
-
-
-
-class PickingPass
-{
-public:
-    PickingPass(IRenderer* renderer, EventBroker* eb);
-    ~PickingPass();
-    void InitializeTextures();
-    void InitializeFrameBuffers();
-    void InitializeShaderPrograms();
-
-    void Draw(RenderScene& scene);
-    void ClearPicking();
-
-    //Getters
-    const ShaderProgram& PickingProgram() const { return *m_PickingProgram; }
-    //const std::unordered_map<glm::ivec2, EntityID>& PickingColorsToEntity() const { return m_PickingColorsToEntity; }
-    GLuint PickingTexture() const { return m_PickingTexture; }
-    GLuint DepthBuffer() const { return m_DepthBuffer; }
-    const FrameBuffer& PickingBuffer() const { return m_PickingBuffer; }
-
-    
-    PickData Pick(glm::vec2 screenCoord);
-
-private:
-    void GenerateTexture(GLuint* texture, GLenum wrapping, GLenum filtering, glm::vec2 dimensions, GLint internalFormat, GLint format, GLenum type) const;
-
-    EventBroker* m_EventBroker;
-
-    const IRenderer* m_Renderer;
-
-    ShaderProgram* m_PickingProgram;
-    Camera* m_Camera;
-
-    struct PickingInfo
-    {
-        EntityID Entity;
-        const ::World* World;
-        ::Camera* Camera;
-    };
-
-    std::unordered_map<glm::ivec2, PickingInfo> m_PickingColorsToEntity;
-
-    GLuint m_PickingTexture;
-    GLuint m_DepthBuffer;
-
-    FrameBuffer m_PickingBuffer;
-
-    int m_ColorCounter[2];
-    std::map<std::tuple<EntityID, const World*, Camera*>, glm::ivec2> m_EntityColors;
-};
-
-#endif 

include/Engine/Rendering/PickingPassState.h

@@ -1,15 +0,0 @@
-#ifndef PickingPassState_h__
-#define PickingPassState_h__
-
-#include "Rendering/RenderState.h"
-
-class PickingPassState : public RenderState
-{
-public:
-    PickingPassState(GLuint frameBuffer);
-    ~PickingPassState();
-
-private:
-};
-
-#endif

include/Engine/Rendering/PointLightJob.h

@@ -1,39 +0,0 @@
-#ifndef PointLightJob_h__
-#define PointLightJob_h__
-
-#include <cstdint>
-
-#include "../Common.h"
-#include "../GLM.h"
-#include "../Core/ComponentWrapper.h"
-#include "RenderJob.h"
-#include "../Core/Transform.h"
-#include "../Core/World.h"
-
-struct PointLightJob : RenderJob
-{
-    PointLightJob(ComponentWrapper transformComponent, ComponentWrapper pointLightComponent, World* m_World)
-        : RenderJob()
-    {
-        Position = glm::vec4((glm::vec3)transformComponent["Position"], 1.0f);
-        Position = glm::vec4(Transform::AbsolutePosition(m_World, transformComponent.EntityID), 1.f);
-        Color = (glm::vec4)pointLightComponent["Color"];
-        Radius = (double)pointLightComponent["Radius"];
-        Intensity = (double)pointLightComponent["Intensity"];
-        Falloff = (double)pointLightComponent["Falloff"];
-    };
-
-    glm::vec4 Position;
-    glm::vec4 Color;
-    float Radius;
-    float Intensity;
-    float Falloff;
-    float padding = 123;
-
-    void CalculateHash() override
-    {
-        Hash = 0;
-    }
-};
-
-#endif

include/Engine/Rendering/RawModel.h

@@ -46,18 +46,14 @@ public:
 	struct MaterialGroup
 	{
 		float Shininess;
-        float Transparency;
-        std::string TexturePath;
 		std::shared_ptr<::Texture> Texture;
-        std::string NormalMapPath;
 		std::shared_ptr<::Texture> NormalMap;
-        std::string SpecularMapPath;
 		std::shared_ptr<::Texture> SpecularMap;
 		unsigned int StartIndex;
 		unsigned int EndIndex;
 	};
 
-	std::vector<MaterialGroup> MaterialGroups;
+	std::vector<MaterialGroup> TextureGroups;
 
 	std::vector<Vertex> m_Vertices;
 	std::vector<unsigned int> m_Indices;

include/Engine/Rendering/RenderJob.h

@@ -1,32 +0,0 @@
-#ifndef RenderJob_h__
-#define RenderJob_h__
-
-#include <cstdint>
-
-#include "../Common.h"
-#include "../GLM.h"
-#include "../Core/ComponentWrapper.h"
-#include "RenderQueue.h"
-
-
-struct RenderJob
-{
-    friend class RenderQueue;
-
-public:
-
-    float Depth;
-
-protected:
-    uint64_t Hash;
-
-    virtual void CalculateHash() = 0;
-
-    bool operator<(const RenderJob& rhs)
-    {
-        return this->Hash < rhs.Hash;
-    }
-
-};
-
-#endif

include/Engine/Rendering/RenderQueue.h

@@ -8,13 +8,61 @@
 #include "../GLM.h"
 #include "../Core/Util/Rectangle.h"
 #include "../Core/Entity.h"
-#include "Camera.h"
-#include "RenderJob.h"
-#include "ModelJob.h"
-#include "PointLightJob.h"
 
+class Model;
+class Skeleton;
+class Texture;
+class RenderQueue;
+
+//TODO: Render: Remove obsolete RenderJobs and fix standard values on variables.
+
+struct RenderJob
+{
+	friend class RenderQueue;
+
+	float Depth;
+
+protected:
+	uint64_t Hash;
+
+	virtual void CalculateHash() = 0;
+
+	bool operator<(const RenderJob& rhs)
+	{
+		return this->Hash < rhs.Hash;
+	}
+};
+
+struct ModelJob : RenderJob
+{
+	unsigned int ShaderID = 0;
+	unsigned int TextureID = 0;
+
+    //TODO: RENDERER: Not sure if the best solution for pickingColor to entity link is this
+    EntityID Entity;
+
+	glm::mat4 ModelMatrix;
+	const Texture* DiffuseTexture;
+	const Texture* NormalTexture;
+	const Texture* SpecularTexture;
+	float Shininess = 0.f;
+	glm::vec4 Color;
+	const Model* Model = nullptr;
+	unsigned int StartIndex = 0;
+	unsigned int EndIndex = 0;
+
+	// Animation
+	Skeleton* Skeleton = nullptr;
+	bool NoRootMotion = true;
+	std::string AnimationName;
+	double AnimationTime = 0;
+
+	void CalculateHash() override
+	{
+		Hash = TextureID;
+	}
+};
 
-/*
 struct SpriteJob : RenderJob
 {
 	unsigned int ShaderID = 0;
@@ -34,65 +82,73 @@ struct SpriteJob : RenderJob
 
 struct PointLightJob : RenderJob
 {
-    glm::vec4 Position;
+	glm::vec3 Position;
-	glm::vec4 Color;
+	glm::vec3 SpecularColor = glm::vec3(1, 1, 1);
-	float Radius;
+	glm::vec3 DiffuseColor = glm::vec3(1, 1, 1);
-    float Intensity;
+	float Radius = 1.f;
-    float Falloff;
+    float Intensity = 0.8f;
-    float padding = 123;
 
 	void CalculateHash() override
 	{
 		Hash = 0;
 	}
 };
-*/
 
-struct RenderScene
+class RenderQueue
-{
-    ::Camera* Camera;
-    std::list<std::shared_ptr<RenderJob>> ForwardJobs;
-    std::list<std::shared_ptr<RenderJob>> PointLightJobs;
-    Rectangle Viewport;
-
-	void Clear()
-	{
-        ForwardJobs.clear();
-        PointLightJobs.clear();
-	}
-};
-
-struct RenderFrame
 {
 public:
-
+	template <typename T>
-    void Add(RenderScene &scene)
+	void Add(T &job)
 	{
-        RenderScenes.push_back(std::shared_ptr<RenderScene>(new RenderScene(scene)));
+		job.CalculateHash();
+		Jobs.push_back(std::shared_ptr<T>(new T(job)));
 		m_Size++;
 	}
 
+	void Sort()
+	{
+		Jobs.sort();
+	}
+
 	void Clear()
 	{
-        RenderScenes.clear();
+		Jobs.clear();
 		m_Size = 0;
 	}
 
 	int Size() const { return m_Size; }
-    std::list<std::shared_ptr<RenderScene>>::const_iterator begin()
+	std::list<std::shared_ptr<RenderJob>>::const_iterator begin()
 	{
-        return RenderScenes.begin();
+		return Jobs.begin();
 	}
 
-    std::list<std::shared_ptr<RenderScene>>::const_iterator end()
+	std::list<std::shared_ptr<RenderJob>>::const_iterator end()
 	{
-        return RenderScenes.end();
+		return Jobs.end();
 	}
 
-    std::list<std::shared_ptr<RenderScene>> RenderScenes;
+	std::list<std::shared_ptr<RenderJob>> Jobs;
 
 private:
 	int m_Size = 0;
 };
 
+struct RenderQueueCollection
+{
+	RenderQueue Forward;
+	RenderQueue Lights;
+
+	void Clear()
+	{
+		Forward.Clear();
+		Lights.Clear();
+	}
+
+	void Sort()
+	{
+		Forward.Sort();
+		Lights.Sort();
+	}
+};
+
 #endif

include/Engine/Rendering/RenderQueueFactory.h

@@ -0,0 +1,31 @@
+#ifndef RenderQueueFactory_h__
+#define RenderQueueFactory_h__
+
+#include "../Core/World.h"
+#include "RenderQueue.h"
+#include "../Core/ResourceManager.h"
+#include "Model.h"
+#include "../GLM.h"
+
+class RenderQueueFactory
+{
+public:
+    RenderQueueFactory();
+    void Update(World* world);
+
+    
+    RenderQueueCollection RenderQueues() const { return m_RenderQueues; }
+private:
+    RenderQueueCollection m_RenderQueues;
+
+    void FillModels(World* world, RenderQueue* renderQueue);
+    void FillLights(World* world, RenderQueue* renderQueue);
+
+    glm::mat4 ModelMatrix(World* world, EntityID entity);
+
+    glm::vec3 AbsolutePosition(World* world, EntityID entity);
+    glm::quat AbsoluteOrientation(World* world, EntityID entity);
+    glm::vec3 AbsoluteScale(World* world, EntityID entity);
+};
+
+#endif

include/Engine/Rendering/RenderState.h

@@ -1,27 +0,0 @@
-#ifndef RenderState_h__
-#define RenderState_h__
-
-#include <functional>
-#include "../Common.h"
-#include "../OpenGL.h"
-#include "../GLM.h"
-
-class RenderState 
-{
-public:
-    RenderState() = default;
-    ~RenderState();
-
-    bool Enable(GLenum cap);
-    bool Disable(GLenum cap);
-    bool CullFace(GLenum mode);
-    bool ClearColor(glm::vec4 color);
-    bool Clear(GLbitfield mask);
-    bool BindFramebuffer(GLint framebuffer);
-    bool BlendEquation(GLenum mode);
-    bool BlendFunc(GLenum sfactor, GLenum dfactor);
-
-private:
-    std::vector<std::function<void(void)>> m_ResetFunctions;
-};
-#endif

include/Engine/Rendering/RenderSystem.h

@@ -1,55 +0,0 @@
-#ifndef RenderSystem_h__
-#define RenderSystem_h__
-
-#include "../Core/System.h"
-#include "RenderQueue.h"
-#include "../GLM.h"
-#include "../OpenGL.h"
-#include "../Core/ResourceManager.h"
-#include "ESetCamera.h"
-#include "Model.h"
-#include "../Core/EKeyDown.h"
-#include "../Input/EInputCommand.h"
-#include "Camera.h"
-#include "ModelJob.h"
-#include "Renderer.h"
-#include "PointLightJob.h"
-#include "../Core/Transform.h"
-#include "DebugCameraInputController.h"
-
-class RenderSystem : public ImpureSystem
-{
-public:
-    RenderSystem(EventBroker* eventBrokerer, const IRenderer* renderer, RenderFrame* renderFrame);
-    ~RenderSystem();
-
-    virtual void Update(World* world, double dt) override;
-
-private:
-    World* m_World = nullptr;
-    const IRenderer* m_Renderer;
-
-    RenderFrame* m_RenderFrame;
-    bool m_SwitchCamera = false;
-    Camera* m_Camera;
-    DebugCameraInputController<RenderSystem>* m_DebugCameraInputController;
-    
-    std::list<ComponentWrapper> m_CameraComponents;
-
-    EventRelay<RenderSystem, Events::SetCamera> m_ESetCamera;
-    bool OnSetCamera(const Events::SetCamera &event);
-    EntityID m_CurrentCamera = EntityID_Invalid;
-
-    void switchCamera(EntityID entity);
-
-    void updateCamera(World* world, double dt);
-    void updateProjectionMatrix(ComponentWrapper& cameraComponent);
-    
-    void fillModels(std::list<std::shared_ptr<RenderJob>>& jobs, World* world);
-    void fillLight(std::list<std::shared_ptr<RenderJob>>& jobs, World* world);
-
-    EventRelay<RenderSystem, Events::InputCommand> m_EInputCommand;
-    bool OnInputCommand(const Events::InputCommand& e);
-};
-
-#endif

include/Engine/Rendering/Renderer.h

@@ -10,63 +10,55 @@
 #include "Util/UnorderedMapVec2.h"
 #include "FrameBuffer.h"
 #include "../Core/World.h"
-#include "PickingPass.h"
+
-#include "DrawScenePass.h"
-#include "LightCullingPass.h"
-#include "DrawFinalPass.h"
 #include "../Core/EventBroker.h"
-#include "ImGuiRenderPass.h"
+#include "EPicking.h"
-#include "Camera.h"
-#include "../Core/Transform.h"
 
 class Renderer : public IRenderer
 {
 public:
-    Renderer(EventBroker* eventBroker, World* world) 
+    Renderer(EventBroker* eventBroker) 
         : m_EventBroker(eventBroker)
-        , m_World(world)
     { }
 
 	virtual void Initialize() override;
 	virtual void Update(double dt) override;
-    virtual void Draw(RenderFrame& frame) override;
+	virtual void Draw(RenderQueueCollection& rq) override;
-
-    virtual PickData Pick(glm::vec2 screenCoord) override;
 
 private:
 	//----------------------Variables----------------------//
     EventBroker* m_EventBroker;
-    World* m_World;
 
 	Texture* m_ErrorTexture;
 	Texture* m_WhiteTexture;
+	float m_CameraMoveSpeed;
+    FrameBuffer m_PickingBuffer;
+    GLuint m_PickingTexture;
+    GLuint m_DepthBuffer;
 
     Model* m_ScreenQuad;
     Model* m_UnitQuad;
     Model* m_UnitSphere;
 
-    DrawScenePass* m_DrawScenePass;
+    std::unordered_map<glm::vec2, EntityID> m_PickingColorsToEntity;
-    PickingPass* m_PickingPass;
-    LightCullingPass* m_LightCullingPass;
-    ImGuiRenderPass* m_ImGuiRenderPass;
-    DrawFinalPass* m_DrawFinalPass;
 
 	//----------------------Functions----------------------//
 	void InitializeWindow();
 	void InitializeShaders();
     void InitializeTextures();
-    void InitializeRenderPasses();
+    void InitializeFrameBuffers();
     //TODO: Renderer: Get InputUpdate out of renderer
 	void InputUpdate(double dt);
-    //void PickingPass(RenderQueueCollection& rq);
+    void PickingPass(RenderQueueCollection& rq);
     void DrawScreenQuad(GLuint textureToDraw);
+    void DrawScene(RenderQueueCollection& rq);
+
 
-    static bool DepthSort(const std::shared_ptr<RenderJob> &i, const std::shared_ptr<RenderJob> &j) { return (i->Depth < j->Depth); }
-    void FillDepth(RenderScene& scene);
     void GenerateTexture(GLuint* texture, GLenum wrapping, GLenum filtering, glm::vec2 dimensions, GLint internalFormat, GLint format, GLenum type);
 	//--------------------ShaderPrograms-------------------//
-	ShaderProgram* m_BasicForwardProgram;
+	ShaderProgram m_BasicForwardProgram;
-    ShaderProgram* m_DrawScreenQuadProgram;
+    ShaderProgram m_PickingProgram;
+    ShaderProgram m_DrawScreenQuadProgram;
 };
 
 #endif

include/Engine/Rendering/ShaderProgram.h

@@ -1,9 +1,6 @@
-#ifndef ShaderProgram_h__
-#define ShaderProgram_h__
 
 #include "../Common.h"
 #include "../OpenGL.h"
-#include "../Core/ResourceManager.h"
 #include <fstream>
 
 class Shader
@@ -63,13 +60,11 @@ public:
 		: ShaderType(fileName) { }
 };
 
-class ShaderProgram : public Resource
+class ShaderProgram
 {
-    friend class ResourceManager;
-private:
-	ShaderProgram(std::string)
-		: m_ShaderProgramHandle(0) { }
 public:
+	ShaderProgram()
+		: m_ShaderProgramHandle(0) { }
 	~ShaderProgram();
 
 	void AddShader(std::shared_ptr<Shader> shader);
@@ -84,5 +79,3 @@ private:
 	GLuint m_ShaderProgramHandle;
 	std::vector<std::shared_ptr<Shader>> m_Shaders;
 };
-
-#endif

include/Engine/Rendering/Util/GLError.h

@@ -9,7 +9,7 @@ inline bool _GLERROR(const char* info, const char* file, const char* func, unsig
 	GLenum error = glGetError();
 	if (error != GL_NO_ERROR)
 	{
-		_LOG(LOG_LEVEL_ERROR, file, func, line, "GL Error: %s, %i, %s", info, error, gluErrorString(error));
+		_LOG(LOG_LEVEL_ERROR, file, func, line, "GL Error: %s %i %s", info, error, gluErrorString(error));
 		return true;
 	}
 

include/Engine/Rendering/Util/UnorderedMapiVec2.h

@@ -1,24 +0,0 @@
-#pragma once
-#ifndef UnorderedMapiVec2_h__
-#define UnorderedMapiVec2_h__
-
-#include <functional>
-#include <boost/functional/hash.hpp>
-#include <glm/vec2.hpp>
-
-template<>
-struct std::hash<glm::ivec2>
-{
-    inline std::size_t operator()(const glm::ivec2 &v) const
-    {
-        return boost::hash<float>()(v.x) ^ boost::hash<float>()(v.y);
-    }
-
-    inline bool operator()(const glm::ivec2& a, const glm::ivec2& b)const
-    {
-        return a.x == b.x && a.y == b.y;
-    }
-
-};
-
-#endif

include/Engine/Sound/EContinueSound.h

@@ -1,17 +0,0 @@
-#ifndef Events_ContinueSound_h__
-#define Events_ContinueSound_h__
-
-#include "Core/EventBroker.h"
-#include "Core/Entity.h"
-
-namespace Events
-{
-// Continues to play a sound from where it was paused.
-struct ContinueSound : Event
-{
-    EntityID EmitterID;
-};
-
-}
-
-#endif

include/Engine/Sound/EPauseSound.h

@@ -1,17 +0,0 @@
-#ifndef Events_PauseSound_h__
-#define Events_PauseSound_h__
-
-#include "Core/EventBroker.h"
-#include "Core/Entity.h"
-
-namespace Events
-{
-// Pauses a playing sound
-struct PauseSound : Event
-{
-    EntityID EmitterID;
-};
-
-}
-
-#endif

include/Engine/Sound/EPlayBackgroundMusic.h

@@ -1,18 +0,0 @@
-#ifndef Events_PlayBackgroundMusic_h__
-#define Events_PlayBackgroundMusic_h__
-
-#include <string>
-#include "Core/Entity.h"
-#include "Core/Event.h"
-
-namespace Events
-{
-// Play a sound that will be heared the same anywhere
-struct PlayBackgroundMusic : public Event
-{
-    std::string FilePath = "";
-};
-
-}
-
-#endif

include/Engine/Sound/EPlaySoundOnEntity.h

@@ -1,21 +0,0 @@
-#ifndef Events_PlaySoundOnEntity_h__
-#define Events_PlaySoundOnEntity_h__
-
-#include <string>
-#include "Core/Entity.h"
-#include "Core/Event.h"
-
-namespace Events
-{
-
-// Plays a sound on an entity with a SoundEmitter component attached.
-// Sound behavior is thereby specified in the SoundEmitter component.
-struct PlaySoundOnEntity : public Event
-{
-    EntityID EmitterID = 0;
-    std::string FilePath = "";
-};
-
-}
-
-#endif

include/Engine/Sound/EPlaySoundOnPosition.h

@@ -1,26 +0,0 @@
-#ifndef Events_PlaySoundOnPosition_h__
-#define Events_PlaySoundOnPosition_h__
-
-#include <string>
-#include <glm/common.hpp>
-#include "Core/Event.h"
-
-namespace Events
-{
-
-// Plays a sound on a given position. Idk if this would be useful.
-struct PlaySoundOnPosition : public Event
-{
-    glm::vec3 Position = glm::vec3(0);
-    std::string FilePath = "";
-    float Gain = 1;
-    float Pitch = 1;
-    bool Loop = false;
-    float MaxDistance = 20;
-    float RollOffFactor = 1;
-    float ReferenceDistance = 1;
-};
-
-}
-
-#endif

include/Engine/Sound/ESetBGMGain.h

@@ -1,16 +0,0 @@
-#ifndef Events_SetBGMGain_h__
-#define Events_SetBGMGain_h__
-
-#include "Core/Event.h"
-
-namespace Events
-{
-// Set the "volume" for all background sounds
-struct SetBGMGain : public Event
-{
-    float Gain;
-};
-
-}
-
-#endif

include/Engine/Sound/ESetSFXGain.h

@@ -1,16 +0,0 @@
-#ifndef Events_SetSFXGain_h__
-#define Events_SetSFXGain_h__
-
-#include "Core/Event.h"
-
-namespace Events
-{
-// Set the "volume" for all effect sounds
-struct SetSFXGain : public Event
-{
-    float Gain;
-};
-
-}
-
-#endif

include/Engine/Sound/EStopSound.h

@@ -1,17 +0,0 @@
-#ifndef Events_StopSound_h__
-#define Events_StopSound_h__
-
-#include "Core/EventBroker.h"
-#include "Core/Entity.h"
-
-namespace Events
-{
-// Stops a sound emitter, and will also delete it.
-struct StopSound : Event
-{
-    EntityID EmitterID;
-};
-
-}
-
-#endif

include/Engine/Sound/Sound.h

@@ -1,113 +0,0 @@
-#ifndef Sound_h__
-#define Sound_h__
-
-#include "Core/ResourceManager.h"
-
-class Sound : public Resource
-{
-    friend class ResourceManager;
-public:
-    Sound(std::string path) { m_Buffer = LoadFile(path); m_Path = path; }
-    ~Sound() { ClearBuffer(); }
-    ALuint Buffer() { return m_Buffer; }
-    std::string Path() { return m_Path; }
-    float Gain() { return m_Gain; }
-    void SetGain(float value) { m_Gain = value; }
-    void ClearBuffer() { alDeleteBuffers(1, &m_Buffer); m_BufferCache.clear(); };
-
-private:
-    float m_Gain = 1;
-    ALuint m_Buffer;
-    std::string m_Path;
-
-    // File info
-    char m_Type[4];
-    unsigned long m_Size, m_ChunkSize;
-    short m_FormatType, m_Channels;
-    unsigned long m_SampleRate, m_AvgBytesPerSec;
-    short m_BytesPerSample, m_BitsPerSample;
-    unsigned int m_DataSize;
-    std::map<std::string, ALuint> m_BufferCache;
-
-    ALuint LoadFile(std::string path)
-    {
-        if (m_BufferCache.find(path) != m_BufferCache.end()) {
-            return m_BufferCache[path];
-        }
-
-        // Open file
-        FILE *fp = fopen(path.c_str(), "rb");
-        if (!fp) {
-            printf("Sound: Failed to open file %s, no such file exists", path.c_str());
-            return 0;
-        }
-
-        //CHECK FOR VALID WAVE-FILE
-        fread(m_Type, sizeof(char), 4, fp);
-        if (m_Type[0] != 'R' || m_Type[1] != 'I' || m_Type[2] != 'F' || m_Type[3] != 'F') {
-            printf("ERROR: No RIFF in WAVE-file");
-            return 0;
-        }
-
-        fread(&m_Size, 4 * sizeof(char), 1, fp);
-        fread(m_Type, sizeof(char), 4, fp);
-        if (m_Type[0] != 'W' || m_Type[1] != 'A' || m_Type[2] != 'V' || m_Type[3] != 'E') {
-            printf("ERROR: Not WAVE-file");
-            return 0;
-        }
-
-        fread(m_Type, sizeof(char), 4, fp);
-        if (m_Type[0] != 'f' || m_Type[1] != 'm' || m_Type[2] != 't' || m_Type[3] != ' ') {
-            printf("ERROR: No fmt in WAVE-file");
-            return 0;
-        }
-
-        // READ THE DATA FROM WAVE-FILE
-        fread(&m_ChunkSize, 4 * sizeof(char), 1, fp);
-        fread(&m_FormatType, 2 * sizeof(char), 1, fp);
-        fread(&m_Channels, 2 * sizeof(char), 1, fp);
-        fread(&m_SampleRate, 4 * sizeof(char), 1, fp);
-        fread(&m_AvgBytesPerSec, 4 * sizeof(char), 1, fp);
-        fread(&m_BytesPerSample, 2 * sizeof(char), 1, fp);
-        fread(&m_BitsPerSample, 2 * sizeof(char), 1, fp);
-
-        fread(m_Type, sizeof(char), 4, fp);
-        if (m_Type[0] != 'd' || m_Type[1] != 'a' || m_Type[2] != 't' || m_Type[3] != 'a') {
-            printf("ERROR: WAVE-file Missing data");
-            return 0;
-        }
-
-        fread(&m_DataSize, 4 * sizeof(char), 1, fp);
-
-        unsigned char* buf = new unsigned char[m_DataSize];
-        fread(buf, sizeof(char), m_DataSize, fp);
-        fclose(fp);
-
-        // Create buffer
-        ALuint format = 0;
-        if (m_BitsPerSample == 8) {
-            if (m_Channels == 1) {
-                format = AL_FORMAT_MONO8;
-            } else if (m_Channels == 2) {
-                format = AL_FORMAT_STEREO8;
-            }
-        }
-        if (m_BitsPerSample == 16) {
-            if (m_Channels == 1) {
-                format = AL_FORMAT_MONO16;
-            } else if (m_Channels == 2) {
-                format = AL_FORMAT_STEREO16;
-            }
-        }
-
-        ALuint buffer;
-        alGenBuffers(1, &buffer);
-        alBufferData(buffer, format, buf, m_DataSize, m_SampleRate);
-        delete[] buf;
-
-        m_BufferCache[path] = buffer;
-        return buffer;
-    }
-};
-
-#endif

include/Engine/Sound/SoundSystem.h

@@ -1,101 +0,0 @@
-#ifndef SoundSystem_h__
-#define SoundSystem_h__
-
-#include <unordered_map>
-
-#include "glm/common.hpp"
-#include "glm/gtx/rotate_vector.hpp" // Calculate Up vector
-#include "OpenAL/al.h"
-#include "OpenAL/alc.h"
-
-#include "Core/World.h"
-#include "Core/EventBroker.h"
-#include "Core/Transform.h" // Absolute transform
-#include "Sound/Sound.h"
-#include "Sound/EPlaySoundOnEntity.h"
-#include "Sound/EPlaySoundOnPosition.h"
-#include "Sound/EPlayBackgroundMusic.h"
-#include "Sound/EPauseSound.h"
-#include "Sound/EContinueSound.h"
-#include "Sound/EStopSound.h"
-#include "Sound/ESetBGMGain.h"
-#include "Sound/ESetSFXGain.h"
-
-enum class SoundType {
-    SFX,
-    BGM
-};
-
-struct Source
-{
-    Source() { }
-    Sound* SoundResource = nullptr;
-    ALuint ALsource;
-    SoundType Type;
-};
-
-class SoundSystem
-{
-public:
-    SoundSystem() { }
-    SoundSystem(World* world, EventBroker* eventBroker, bool editorMode);
-    ~SoundSystem();
-    // Update emitters / listener
-    void Update(double dt); 
-private:
-    // Help functions for working with OpenaAL
-    void setListenerPos(glm::vec3 pos) { alListener3f(AL_POSITION, pos.x, pos.y, pos.z); };
-    void setListenerVel(glm::vec3 vel) { alListener3f(AL_VELOCITY, vel.x, vel.y, vel.z); };
-    void setListenerOri(glm::vec3 ori);
-    glm::vec3 listnerPos() { glm::vec3 pos; alGetListener3f(AL_POSITION, &pos.x, &pos.y, &pos.z); return pos; };
-    glm::vec3 listenerVel() { glm::vec3 vel; alGetListener3f(AL_VELOCITY, &vel.x, &vel.y, &vel.z); return vel; };
-    glm::vec3 listenerOri() { glm::vec3 ori; alGetListener3f(AL_ORIENTATION, &ori.x, &ori.y, &ori.z); return ori; };
-    void setSourcePos(ALuint source, glm::vec3 pos) { ALfloat spos[3] = { pos.x, pos.y, pos.z };  alSourcefv(source, AL_POSITION, spos); };
-    void setSourceVel(ALuint source, glm::vec3 vel) { ALfloat svel[3] = { vel.x, vel.y, vel.z };  alSourcefv(source, AL_VELOCITY, svel); };
-
-    // Logic 
-    void initOpenAL();
-    void updateEmitters(double dt);
-    void updateListener(double dt);
-    void deleteInactiveEmitters();
-    void addNewEmitters(double dt);
-    Source* createSource(std::string filePath);
-    void playSound(Source* source);
-    void stopSound(Source* source);
-    void stopEmitters();
-    ALenum getSourceState(ALuint source);
-    void setGain(Source* source, float gain);
-    void setSoundProperties(ALuint source, ComponentWrapper* soundComponent);
-
-    // OpenAL system variables
-    ALCdevice* m_ALCdevice = nullptr;
-    ALCcontext* m_ALCcontext = nullptr;
-
-    // Logic
-    World* m_World = nullptr;
-    EventBroker* m_EventBroker = nullptr;
-    std::unordered_map<EntityID, Source*> m_Sources;
-    float m_BGMVolumeChannel = 1.0f;
-    float m_SFXVolumeChannel = 1.f;
-    bool m_EditorEnabled = false;
-
-    // Events
-    EventRelay<SoundSystem, Events::PlaySoundOnEntity> m_EPlaySoundOnEntity;
-    bool OnPlaySoundOnEntity(const Events::PlaySoundOnEntity &e);
-    EventRelay<SoundSystem, Events::PlaySoundOnPosition> m_EPlaySoundOnPosition;
-    bool OnPlaySoundOnPosition(const Events::PlaySoundOnPosition &e);
-    EventRelay<SoundSystem, Events::PlayBackgroundMusic> m_EPlayBackgroundMusic;
-    bool OnPlayBackgroundMusic(const Events::PlayBackgroundMusic &e);
-    EventRelay<SoundSystem, Events::PauseSound> m_EPauseSound;
-    bool OnPauseSound(const Events::PauseSound &e);
-    EventRelay<SoundSystem, Events::StopSound> m_EStopSound;
-    bool OnStopSound(const Events::StopSound &e);
-    EventRelay<SoundSystem, Events::ContinueSound> m_EContinueSound;
-    bool OnContinueSound(const Events::ContinueSound &e);
-    EventRelay<SoundSystem, Events::SetBGMGain> m_ESetBGMGain; 
-    bool OnSetBGMGain(const Events::SetBGMGain &e); // Not tested
-    EventRelay<SoundSystem, Events::SetSFXGain> m_ESetSFXGain; 
-    bool OnSetSFXGain(const Events::SetSFXGain &e); // Not tested
-};
-
-#endif

include/Game/Events/ESpawnerSpawn.h

@@ -1,18 +0,0 @@
-#ifndef ESpawnerSpawn_h__
-#define ESpawnerSpawn_h__
-
-#include "Core/Event.h"
-#include "Core/EntityWrapper.h"
-
-namespace Events
-{
-
-struct SpawnerSpawn : Event
-{
-    EntityWrapper Spawner;
-    EntityWrapper Parent;
-};
-
-}
-
-#endif

include/Game/Game.h

@@ -8,26 +8,16 @@
 #include "Core/InputManager.h"
 #include "GUI/Frame.h"
 #include "Core/World.h"
+#include "Rendering/RenderQueueFactory.h"
 #include "Input/InputProxy.h"
 #include "Input/KeyboardInputHandler.h"
 #include "Input/MouseInputHandler.h"
+#include "Input/XboxControllerInputHandler.h"
 #include "Core/EKeyDown.h"
-#include "Core/EntityFilePreprocessor.h"
+#include "Core/EntityXMLFile.h"
 #include "Core/SystemPipeline.h"
-#include "Editor/EditorSystem.h"
+#include "RaptorCopterSystem.h"
-#include "Core/EntityFile.h"
+#include "PlayerSystem.h"
-#include "Rendering/RenderSystem.h"
-#include "Core/EntityFileParser.h"
-#include "Core/Octree.h"
-
-// Network
-#include <boost/thread.hpp>
-#include "Network/Network.h"
-#include "Network/Server.h"
-#include "Network/Client.h"
-
-// Sound
-#include "Sound/SoundSystem.h"
 
 class Game
 {
@@ -47,28 +37,14 @@ private:
     InputProxy* m_InputProxy;
 	GUI::Frame* m_FrameStack;
     World* m_World;
-    Octree<AABB>* m_OctreeCollision;
-    Octree<AABB>* m_OctreeFrustrumCulling;
     SystemPipeline* m_SystemPipeline;
-    RenderFrame* m_RenderFrame;
+    RenderQueueFactory* m_RenderQueueFactory;
-    // Network variables
-    boost::thread m_NetworkThread;
 
-    // Network methods
+    EventRelay<Game, Events::InputCommand> m_EInputCommand;
-    void networkFunction();
+    bool debugOnInputCommand(const Events::InputCommand& e);
-    Network* m_ClientOrServer;
-    bool m_IsClientOrServer = false;
-
-    // Sound
-    SoundSystem* m_SoundSystem;
-
-    //EventRelay<Game, Events::InputCommand> m_EInputCommand;
-    //bool debugOnInputCommand(const Events::InputCommand& e);
 
     void debugInitialize();
     void debugTick(double dt);
-	EventRelay<Client, Events::KeyDown> m_EKeyDown;
-
 };
 
 #endif

include/Game/PlayerSystem.h

@@ -0,0 +1,44 @@
+#ifndef PlayerSystem_h__
+#define PlayerSystem_h__
+
+#include <GLFW/glfw3.h>
+#include <glm/common.hpp>
+
+#include "Common.h"
+#include "Core/System.h"
+#include "Core/EventBroker.h"
+#include "Core/EKeyDown.h"
+#include "Core/EKeyUp.h"
+
+struct KeyInput
+{
+    bool Forward = false;
+    bool Left = false;
+    bool Back = false;
+    bool Right = false;
+};
+
+class PlayerSystem : public System
+{
+public:
+    PlayerSystem(EventBroker* eventBroker)
+        : System(eventBroker, "Player")
+    {
+        EVENT_SUBSCRIBE_MEMBER(m_EKeyDown, &PlayerSystem::OnKeyDown);
+        EVENT_SUBSCRIBE_MEMBER(m_EKeyUp, &PlayerSystem::OnKeyUp);
+    }
+
+    virtual void Update(World* world, ComponentWrapper& player, double dt) override;
+
+private:
+    float m_Speed = 5;
+    glm::vec3 m_Direction;
+    KeyInput input;
+
+    EventRelay<PlayerSystem, Events::KeyDown> m_EKeyDown;
+    bool OnKeyDown(const Events::KeyDown &event);
+    EventRelay<PlayerSystem, Events::KeyUp> m_EKeyUp;
+    bool OnKeyUp(const Events::KeyUp &event);
+};
+
+#endif

include/Game/RaptorCopterSystem.h

@@ -0,0 +1,18 @@
+#include "Common.h"
+#include "Core/System.h"
+
+class RaptorCopterSystem : public System
+{
+public:
+    RaptorCopterSystem(EventBroker* eventBroker)
+        : System(eventBroker, "RaptorCopter")
+    { }
+
+	virtual void Initialize() { }
+
+    virtual void Update(World* world, ComponentWrapper& raptorCopter, double dt) override
+    {
+        ComponentWrapper& transform = world->GetComponent(raptorCopter.EntityID, "Transform");
+        (glm::vec3&)transform["Orientation"] += (float)(double)raptorCopter["Speed"] * (float)dt * (glm::vec3)raptorCopter["Axis"];
+    }
+};

include/Game/Systems/HealthSystem.h

@@ -1,36 +0,0 @@
-#ifndef HealthSystem_h__
-#define HealthSystem_h__
-
-#include <GLFW/glfw3.h>
-#include <glm/common.hpp>
-
-#include "Common.h"
-#include "Core/System.h"
-#include "Core/EPlayerDamage.h"
-#include "Core/EPlayerHealthPickup.h"
-#include "Core/EPlayerDeath.h"
-
-#include <tuple>
-#include <vector>
-
-class HealthSystem : public PureSystem
-{
-public:
-    HealthSystem(EventBroker* eventBroker);
-
-    //updatecomponent
-    virtual void UpdateComponent(World* world, EntityWrapper& entity, ComponentWrapper& component, double dt) override;
-
-private:
-    //methods which will take care of specific events
-    EventRelay<HealthSystem, Events::PlayerDamage> m_EPlayerDamage;
-    bool HealthSystem::OnPlayerDamaged(const Events::PlayerDamage& e);
-    EventRelay<HealthSystem, Events::PlayerHealthPickup> m_EPlayerHealthPickup;
-    bool HealthSystem::OnPlayerHealthPickup(const Events::PlayerHealthPickup& e);
-
-    //vector which will keep track of health changes
-    std::vector<std::tuple<EntityID, double>> m_DeltaHealthVector;
-   
-};
-
-#endif

include/Game/Systems/PlayerMovementSystem.h

@@ -1,14 +0,0 @@
-#include "Common.h"
-#include "GLM.h"
-#include "Core/System.h"
-
-class PlayerMovementSystem : public PureSystem
-{
-public:
-    PlayerMovementSystem(EventBroker* eventBroker)
-        : System(eventBroker)
-        , PureSystem("Player")
-    { }
-
-    virtual void UpdateComponent(World* world, EntityWrapper& entity, ComponentWrapper& component, double dt);
-};

include/Game/Systems/PlayerSpawnSystem.h

@@ -1,18 +0,0 @@
-#include "Core/System.h"
-#include "Input/EInputCommand.h"
-#include "Systems/SpawnerSystem.h"
-#include "Events/ESpawnerSpawn.h"
-
-class PlayerSpawnSystem : public ImpureSystem
-{
-public:
-    PlayerSpawnSystem(EventBroker* eventBroker);
-
-    virtual void Update(World* world, double dt) override;
-
-private:
-    EventRelay<PlayerSpawnSystem, Events::InputCommand> m_OnInputCommand;
-    bool OnInputCommand(const Events::InputCommand& e);
-
-    std::vector<int> m_SpawnRequests;
-};

include/Game/Systems/RaptorCopterSystem.h

@@ -1,17 +0,0 @@
-#include "Common.h"
-#include "Core/System.h"
-
-class RaptorCopterSystem : public PureSystem
-{
-public:
-    RaptorCopterSystem(EventBroker* eventBroker)
-        : System(eventBroker)
-        , PureSystem("RaptorCopter")
-    { }
-
-    virtual void UpdateComponent(World* world, EntityWrapper& entity, ComponentWrapper& component, double dt) override
-    {
-        ComponentWrapper& transform = world->GetComponent(component.EntityID, "Transform");
-        (glm::vec3&)transform["Orientation"] += (float)(double)component["Speed"] * (float)dt * (glm::vec3)component["Axis"];
-    }
-};