From 317510f1d39a7a9507d335b2738ac25bfecc5cdc Mon Sep 17 00:00:00 2001
From: Anonymous Maarten <anonymous.maarten@gmail.com>
Date: Sat, 1 Jul 2023 17:36:22 +0200
Subject: [PATCH] Don't delete a member variable. C++ automatically destructs
 member variables

---
 LEGO1/mxstringvariable.cpp | 15 +++++++--------
 LEGO1/mxstringvariable.h   |  2 +-
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/LEGO1/mxstringvariable.cpp b/LEGO1/mxstringvariable.cpp
index 617d2e6a..e63329c1 100644
--- a/LEGO1/mxstringvariable.cpp
+++ b/LEGO1/mxstringvariable.cpp
@@ -1,14 +1,6 @@
 #include "mxstringvariable.h"
 #include "mxstring.h"
 
-//FIXME: Figure out what exactly this class is used for. It is used in LegoGameState::LegoGameState when loading the background color, and for loading the "fsmovie" variable
-// OFFSET: LEGO1 0x1003bec0
-MxStringVariable::~MxStringVariable()
-{
-
-  delete &m_string;
-  delete &m_name;
-}
 
 // OFFSET: LEGO1 0x1003bea0
 MxString *MxStringVariable::GetString()
@@ -21,3 +13,10 @@ void MxStringVariable::SetString(const char *colorString)
 {
   m_string = colorString;
 }
+
+//FIXME: Figure out what exactly this class is used for. It is used in LegoGameState::LegoGameState when loading the background color, and for loading the "fsmovie" variable
+// OFFSET: LEGO1 0x1003bec0
+void MxStringVariable::Destroy()
+{
+  delete this;
+}
diff --git a/LEGO1/mxstringvariable.h b/LEGO1/mxstringvariable.h
index 5284fc5d..dd623dfb 100644
--- a/LEGO1/mxstringvariable.h
+++ b/LEGO1/mxstringvariable.h
@@ -10,7 +10,7 @@ class MxStringVariable
   MxStringVariable() {}
   virtual MxString *GetString();
   virtual void SetString(const char *colorString);
-  virtual ~MxStringVariable();
+  virtual void Destroy();
 
 protected:
   MxString m_name;