Commit prog

src/dawn/component/display/Camera.cpp

@@ -18,6 +18,7 @@ void Camera::onInit() {
 }
 
 void Camera::onDispose() {
+  std::cout << "Camera Disposing" << std::endl;
   renderTarget = nullptr;
 }
 

src/dawn/game/Game.cpp

@@ -48,4 +48,9 @@ std::shared_ptr<Scene> Game::getCurrentScene() {
 }
 
 Game::~Game() {
+  currentScene = nullptr;
+  nextFrameScene = nullptr;
+  
+  assertTrue(SCENE_ITEMS_ACTIVE == 0, "Some scene items are still active?");
+  assertTrue(SCENE_COMPONENTS_ACTIVE == 0, "Some scene components are still active?");
 }

src/dawn/scene/Scene.cpp

@@ -67,4 +67,6 @@ void Scene::removeItem(const std::shared_ptr<SceneItem> item) {
 }
 
 Scene::~Scene() {
+  sceneItemsToRemove.clear();
+  sceneItems.clear();
 }

src/dawn/scene/SceneComponent.cpp

@@ -10,7 +10,11 @@
 
 using namespace Dawn;
 
+uint64_t SCENE_COMPONENTS_ACTIVE = 0;
+
 void SceneComponent::init(const std::shared_ptr<SceneItem> item) {
+  SCENE_COMPONENTS_ACTIVE++;
+
   assertFlagOff(
     sceneComponentState,
     SCENE_COMPONENT_STATE_INIT,
@@ -45,10 +49,11 @@ void SceneComponent::dispose() {
   this->listeners.clear();
 
   this->onDispose();
-  this->item = nullptr;
+  this->item.reset();
 }
 
 std::shared_ptr<SceneItem> SceneComponent::getItem() {
+  auto item = this->item.lock();
   assertNotNull(item, "Item cannot be null?");
   return item;
 }
@@ -78,4 +83,7 @@ SceneComponent::~SceneComponent() {
       "SceneComponent is initialized but was not properly disposed!"
     );
   }
+
+  this->item.reset();
+  SCENE_COMPONENTS_ACTIVE--;
 }

src/dawn/scene/SceneComponent.hpp

@@ -9,6 +9,8 @@
 #define SCENE_COMPONENT_STATE_INIT 0x01
 #define SCENE_COMPONENT_STATE_DISPOSED 0x02
 
+extern uint64_t SCENE_COMPONENTS_ACTIVE;
+
 namespace Dawn {
   class Game;
   class Scene;
@@ -17,7 +19,7 @@ namespace Dawn {
 
   class SceneComponent : std::enable_shared_from_this<SceneComponent> {
     private:
-      std::shared_ptr<SceneItem> item;
+      std::weak_ptr<SceneItem> item;
       uint_fast8_t sceneComponentState = 0;
 
     protected:

src/dawn/scene/SceneItem.cpp

@@ -8,11 +8,15 @@
 
 using namespace Dawn;
 
+uint64_t SCENE_ITEMS_ACTIVE = 0;
+
 SceneItem::SceneItem(const std::weak_ptr<Scene> scene) :
   scene(scene),
   SceneItemTransform(),
   SceneItemComponents()
 {
+  std::cout << "Scene Item Initializing" << std::endl;
+  SCENE_ITEMS_ACTIVE++;
 }
 
 std::shared_ptr<SceneItem> SceneItem::sceneItemComponentsSelf() {
@@ -37,12 +41,6 @@ void SceneItem::remove() {
 }
 
 SceneItem::~SceneItem() {
-  std::for_each(
-    components.begin(),
-    components.end(),
-    [](auto &component) {
-      component->dispose();
-    }
-  );
-  this->components.clear();
+  std::cout << "Scene Item Disposing" << std::endl;
+  SCENE_ITEMS_ACTIVE--;
 }

src/dawn/scene/SceneItem.hpp

@@ -7,6 +7,8 @@
 #include "scene/item/SceneItemTransform.hpp"
 #include "scene/item/SceneItemComponents.hpp"
 
+extern uint64_t SCENE_ITEMS_ACTIVE;
+
 namespace Dawn {
   class Scene;
 

src/dawn/scene/item/SceneItemComponents.cpp

@@ -21,5 +21,11 @@ void SceneItemComponents::removeComponent(
 }
 
 SceneItemComponents::~SceneItemComponents() {
-
+  auto it = components.begin();
+  while(it != components.end()) {
+    auto component = *it;
+    component->dispose();
+    ++it;
+  }
+  components.clear();
 }

src/dawn/scene/item/SceneItemTransform.cpp

@@ -205,4 +205,6 @@ SceneItemTransform::~SceneItemTransform() {
       }
     }
   );
+  this->children.clear();
+  this->parent.reset();
 }

src/dawnrpg/scenes/WorldScene.cpp

@@ -48,6 +48,6 @@ void Dawn::worldScene(Scene &s) {
   auto player = createPlayerPrefab(map.map);
   player.player->camera = camera;
 
-  auto test = createTestEntityPrefab(map.map);
-  test.entity->setPosition({ 5, 0, 0 });
+  // auto test = createTestEntityPrefab(map.map);
+  // test.entity->setPosition({ 5, 0, 0 });
 }