Fix memory leaks

include/engine/box/box.h

@@ -25,6 +25,7 @@ template <typename T> struct Intersection {
     std::pair<double, double> intersectionVector = std::make_pair(
         0, 0); // Vector start from intersecting, towards intersected
     Intersection() = default;
+    virtual ~Intersection(){};
 };
 
 /**
@@ -40,7 +41,7 @@ template <typename T> class Box {
 
   public:
     Box() = default;
-    ~Box() = default;
+    virtual ~Box(){};
 
     std::pair<double, double>* getPos() { return pos; };
     std::vector<std::pair<double, double>> getVertices() {

include/engine/entity.h

@@ -20,6 +20,8 @@
 #include "engine/physics/phxbox.h"
 #include "engine/physics/point.h"
 
+#include "macros.h"
+
 namespace Engine {
 
 class InputHandler;
@@ -38,7 +40,7 @@ class Entity {
   public:
     Entity();
     Entity(std::string name);
-    ~Entity() = default;
+    virtual ~Entity();
 
     void addPoint();
     Point* getPoint();

include/engine/input/input_handler.h

@@ -34,7 +34,7 @@ struct Keybind {
 class InputHandler {
   public:
     InputHandler(/* args */) = default;
-    ~InputHandler() = default;
+    ~InputHandler(){};
     void setOwner(Entity* entity) { this->owner = entity; };
     virtual void doInput(int actionId){};
 

include/engine/physics/phxbox.h

@@ -36,6 +36,7 @@ class PhxBox {
         STATIC    ///< Cannot be pushed
     };
     PhxBox() = default;
+    ~PhxBox(){};
 
     Box<PhxBox>* getBox() { return this->box; };
 

include/engine/physics/point.h

@@ -39,6 +39,7 @@ class Point {
 
     PhxParam getParam();
     Point(/* args */) = default;
+    ~Point(){};
     Point(PhxParam phxParam) { this->phxParam = phxParam; };
 
     void setMass(double mass) { this->phxParam.mass = mass; };

include/engine/universe_master.h

@@ -48,6 +48,7 @@ typedef std::vector<InputHandler*> InpHdlTable;
 class UniverseMaster {
   public:
     UniverseMaster(/* args */) = default;
+    ~UniverseMaster();
 
     void linkWindow(sf::RenderWindow* p_window) { this->window = p_window; };
 

include/macros.h

@@ -0,0 +1,6 @@
+#ifndef MACROS_H
+#define MACROS_H
+
+#define IF_NOT_NULL_DELETE(a) (a ? delete a : void())
+
+#endif

src/engine/entity.cpp

@@ -7,6 +7,12 @@ Engine::Entity::Entity() {
     this->sprite = NULL;
 }
 
+Engine::Entity::~Entity() {
+    IF_NOT_NULL_DELETE(this->point);
+    IF_NOT_NULL_DELETE(this->sprite);
+    IF_NOT_NULL_DELETE(this->phxbox);
+}
+
 Engine::Entity::Entity(std::string name) : Entity() { this->name = name; }
 
 void Engine::Entity::addPoint() {

src/engine/universe_master.cpp

@@ -5,6 +5,15 @@
 
 using namespace Engine;
 
+Engine::UniverseMaster::~UniverseMaster() {
+    for (Entity* entity : this->allEntity) {
+        delete entity;
+    }
+    for (InputHandler* handle : this->allInpHdl) {
+        delete handle;
+    }
+}
+
 void Engine::UniverseMaster::addEntity(Entity* entity) {
     this->allEntity.push_back(entity);
     Point* point = entity->getPoint();