Fix majors memory leaks (TODO: shared_ptr for Game objects and implement ~Game())

src/base_tab/AppendGameDialog.cpp

@@ -17,8 +17,11 @@ AppendGameDialog::AppendGameDialog(wxWindow *parent, GameBase *base)
        ID_DIALOG_CANCEL_BUTTON);
   Bind(wxEVT_BUTTON, &AppendGameDialog::OnImport, this,
        ID_DIALOG_IMPORT_BUTTON);
+  Bind(wxEVT_CLOSE_WINDOW, &AppendGameDialog::OnClose, this);
 }
 
+void AppendGameDialog::OnClose(wxCloseEvent &e) { Destroy(); }
+
 void AppendGameDialog::OnCancel(wxCommandEvent &event) { this->Close(); }
 
 void AppendGameDialog::OnImport(wxCommandEvent &event) {

src/base_tab/AppendGameDialog.hpp

@@ -1,11 +1,12 @@
 #include "gamebase/GameBase.hpp"
 
 class AppendGameDialog : public DialogAppendGame {
-  GameBase *base;
+  GameBase *base; // Should not be destroy
   std::vector<TabInfos *> tinfos;
 
 public:
   AppendGameDialog(wxWindow *parent, GameBase *base);
   void OnCancel(wxCommandEvent &event);
   void OnImport(wxCommandEvent &event);
+  void OnClose(wxCloseEvent &e);
 };

src/base_tab/BaseTab.cpp

@@ -24,6 +24,10 @@ BaseTab::BaseTab(wxFrame *parent, std::string base_file)
   LoadFile();
 }
 
+BaseTab::~BaseTab() {
+  delete base;
+}
+
 void BaseTab::OnImport(wxCommandEvent &event) {
   AppendGameDialog *dia = new AppendGameDialog(this, base);
   dia->ShowModal();

src/base_tab/BaseTab.hpp

@@ -15,6 +15,7 @@ class BaseTab : public TabBase, public TabInfos {
 
 public:
   BaseTab(wxFrame *parent, std::string base_file);
+  ~BaseTab();
   void ApplyPreferences();
   void LoadFile();
   void OnDelete(wxCommandEvent &event);

src/base_tab/gamebase/PGNGameBase.cpp

@@ -6,6 +6,10 @@ PGNGameBase::PGNGameBase(std::string pgn_file) : pgn(new pgnp::PGN()) {
   pgn->FromFile(pgn_file);
 }
 
+PGNGameBase::~PGNGameBase() {
+  delete pgn; // Should never fail since pgn is never NULL
+}
+
 bool PGNGameBase::NextGame() {
   bool game_found = false;
   try {

src/base_tab/gamebase/PGNGameBase.hpp

@@ -8,6 +8,7 @@ class PGNGameBase : public GameBase {
 
 public:
   PGNGameBase(std::string pgn_file);
+  ~PGNGameBase();
   Game *GetGame(std::uint32_t id);
   bool NextGame();
   Game *GetCurrentGame();