Deallocate in destructors

src/Background/BackgroundStateContext.cpp

@@ -35,6 +35,11 @@ BackgroundStateContext::BackgroundStateContext(MainWindow &mainWindow) :
   completeRequestedStateTransitionIfExists();
 }
 
+BackgroundStateContext::~BackgroundStateContext()
+{
+  qDeleteAll (m_states);
+}
+
 void BackgroundStateContext::close()
 {
   LOG4CPP_INFO_S ((*mainCat)) << "BackgroundStateContext::close";

src/Background/BackgroundStateContext.h

@@ -35,6 +35,9 @@ class BackgroundStateContext
   /// Single constructor
   BackgroundStateContext(MainWindow &mainWindow);
 
+  /// Destructor deallocates memory
+  ~BackgroundStateContext();
+
   /// Open Document is being closed so remove the background
   void close();
 

src/Checker/Checker.cpp

@@ -38,6 +38,7 @@ Checker::Checker(QGraphicsScene &scene) :
 
 Checker::~Checker()
 {
+  m_gridLines.clear ();
 }
 
 void Checker::adjustPolarAngleRanges (const DocumentModelCoords &modelCoords,

src/CoordSystem/CoordSystemContext.cpp

@@ -20,13 +20,7 @@ CoordSystemContext::~CoordSystemContext()
 {
   LOG4CPP_INFO_S ((*mainCat)) << "CoordSystemContext::~CoordSystemContext";
 
-  for (int i = 0; i < m_coordSystems.count(); i++) {
-    CoordSystem *coordSystem = m_coordSystems.at (i);
-    delete coordSystem;
-  }
-
-  m_coordSystems.clear ();
-  m_coordSystemIndex = 0;
+  qDeleteAll (m_coordSystems);
 }
 
 void CoordSystemContext::addCoordSystems(unsigned int numberCoordSystemToAdd)

src/DigitizeState/DigitizeStateContext.cpp

@@ -53,6 +53,7 @@ DigitizeStateContext::DigitizeStateContext(MainWindow &mainWindow,
 
 DigitizeStateContext::~DigitizeStateContext()
 {
+  qDeleteAll (m_states);
 }
 
 QString DigitizeStateContext::activeCurve () const

src/Dlg/DlgSettingsExportFormat.cpp

@@ -55,6 +55,8 @@ DlgSettingsExportFormat::DlgSettingsExportFormat(MainWindow &mainWindow) :
   DlgSettingsAbstractBase (tr ("Export Format"),
                            "DlgSettingsExportFormat",
                            mainWindow),
+  m_validatorFunctionsPointsEvenlySpacing (nullptr),
+  m_validatorRelationsPointsEvenlySpacing (nullptr),
   m_modelExportBefore (nullptr),
   m_modelExportAfter (nullptr),
   m_haveFunction (false),
@@ -70,6 +72,9 @@ DlgSettingsExportFormat::DlgSettingsExportFormat(MainWindow &mainWindow) :
 DlgSettingsExportFormat::~DlgSettingsExportFormat()
 {
   LOG4CPP_INFO_S ((*mainCat)) << "DlgSettingsExportFormat::~DlgSettingsExportFormat";
+
+  delete m_validatorFunctionsPointsEvenlySpacing;
+  delete m_validatorRelationsPointsEvenlySpacing;  
 }
 
 void DlgSettingsExportFormat::createCurveSelection (QGridLayout *layout, int &row)

src/Dlg/DlgSettingsGridDisplay.cpp

@@ -31,6 +31,14 @@ DlgSettingsGridDisplay::DlgSettingsGridDisplay(MainWindow &mainWindow) :
   DlgSettingsAbstractBase (tr ("Grid Display"),
                            "DlgSettingsGridDisplay",
                            mainWindow),
+  m_validatorCountX (nullptr),
+  m_validatorStartX (nullptr),
+  m_validatorStepX (nullptr),
+  m_validatorStopX (nullptr),
+  m_validatorCountY (nullptr),
+  m_validatorStartY (nullptr),
+  m_validatorStepY (nullptr),
+  m_validatorStopY (nullptr),  
   m_scenePreview (nullptr),
   m_viewPreview (nullptr),
   m_modelGridDisplayBefore (nullptr),
@@ -45,6 +53,15 @@ DlgSettingsGridDisplay::DlgSettingsGridDisplay(MainWindow &mainWindow) :
 DlgSettingsGridDisplay::~DlgSettingsGridDisplay()
 {
   LOG4CPP_INFO_S ((*mainCat)) << "DlgSettingsGridDisplay::~DlgSettingsGridDisplay";
+
+  delete m_validatorCountX;
+  delete m_validatorStartX;
+  delete m_validatorStepX;
+  delete m_validatorStopX;
+  delete m_validatorCountY;
+  delete m_validatorStartY;
+  delete m_validatorStepY;
+  delete m_validatorStopY;
 }
 
 void DlgSettingsGridDisplay::createDisplayCommon (QGridLayout *layout, int &row)

src/Dlg/DlgSettingsGridRemoval.cpp

@@ -36,6 +36,15 @@ DlgSettingsGridRemoval::DlgSettingsGridRemoval(MainWindow &mainWindow) :
   DlgSettingsAbstractBase (tr ("Grid Removal"),
                            "DlgSettingsGridRemoval",
                            mainWindow),
+  m_validatorCloseDistance (nullptr),
+  m_validatorCountX (nullptr),
+  m_validatorStartX (nullptr),
+  m_validatorStepX (nullptr),
+  m_validatorStopX (nullptr),      
+  m_validatorCountY (nullptr),
+  m_validatorStartY (nullptr),
+  m_validatorStepY (nullptr),
+  m_validatorStopY (nullptr),  
   m_scenePreview (nullptr),
   m_viewPreview (nullptr),
   m_modelGridRemovalBefore (nullptr),
@@ -50,6 +59,16 @@ DlgSettingsGridRemoval::DlgSettingsGridRemoval(MainWindow &mainWindow) :
 DlgSettingsGridRemoval::~DlgSettingsGridRemoval()
 {
   LOG4CPP_INFO_S ((*mainCat)) << "DlgSettingsGridRemoval::~DlgSettingsGridRemoval";
+
+  delete m_validatorCloseDistance;
+  delete m_validatorCountX;
+  delete m_validatorStartX;
+  delete m_validatorStepX;
+  delete m_validatorStopX;      
+  delete m_validatorCountY;
+  delete m_validatorStartY;
+  delete m_validatorStepY;
+  delete m_validatorStopY;
 }
 
 void DlgSettingsGridRemoval::createOptionalSaveDefault (QHBoxLayout * /* layout */)

src/Geometry/GeometryStrategyContext.cpp

@@ -24,6 +24,7 @@ GeometryStrategyContext::GeometryStrategyContext()
 
 GeometryStrategyContext::~GeometryStrategyContext()
 {
+  qDeleteAll (m_strategies);
 }
 
 void GeometryStrategyContext::calculateGeometry (const Points &points,

src/Geometry/GeometryWindow.cpp

@@ -47,6 +47,7 @@ GeometryWindow::GeometryWindow (MainWindow *mainWindow) :
 
 GeometryWindow::~GeometryWindow()
 {
+  delete m_model;
 }
 
 void GeometryWindow::clear ()

src/Grid/GridLines.cpp

@@ -11,6 +11,11 @@ GridLines::GridLines()
 {
 }
 
+GridLines::~GridLines()
+{
+  qDeleteAll (m_gridLinesContainer);
+}
+
 void GridLines::add (GridLine *gridLine)
 {
   m_gridLinesContainer.push_back (gridLine);

src/Grid/GridLines.h

@@ -21,6 +21,9 @@ class GridLines
   /// Single constructor
   GridLines();
 
+  /// Destructor deallocates memory
+  ~GridLines();
+
   /// Add specified grid line. Ownership of all allocated QGraphicsItems is passed to new GridLine
   void add (GridLine *gridLine);
 

src/Transformation/TransformationStateContext.cpp

@@ -27,6 +27,7 @@ TransformationStateContext::TransformationStateContext(QGraphicsScene &scene,
 
 TransformationStateContext::~TransformationStateContext()
 {
+  qDeleteAll (m_states);
 }
 
 bool TransformationStateContext::isGnuplot() const

src/Tutorial/TutorialStateAxisPoints.cpp

@@ -15,7 +15,14 @@
 #include "TutorialStateContext.h"
 
 TutorialStateAxisPoints::TutorialStateAxisPoints (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_previous (nullptr),
+  m_next (nullptr)
 {
 }
 

src/Tutorial/TutorialStateChecklistWizardAbstract.cpp

@@ -15,7 +15,14 @@
 #include "TutorialStateContext.h"
 
 TutorialStateChecklistWizardAbstract::TutorialStateChecklistWizardAbstract (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_text3 (nullptr),
+  m_previous (nullptr)
 {
 }
 

src/Tutorial/TutorialStateChecklistWizardLines.cpp

@@ -12,6 +12,7 @@
 TutorialStateChecklistWizardLines::TutorialStateChecklistWizardLines (TutorialStateContext &context) : 
   TutorialStateChecklistWizardAbstract (context)
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateChecklistWizardLines::TutorialStateChecklistWizardLines";
 }
 
 void TutorialStateChecklistWizardLines::begin ()

src/Tutorial/TutorialStateChecklistWizardPoints.cpp

@@ -12,6 +12,7 @@
 TutorialStateChecklistWizardPoints::TutorialStateChecklistWizardPoints (TutorialStateContext &context) : 
   TutorialStateChecklistWizardAbstract (context)
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateChecklistWizardPoints::TutorialStateChecklistWizardPoints";
 }
 
 void TutorialStateChecklistWizardPoints::begin ()

src/Tutorial/TutorialStateColorFilter.cpp

@@ -15,8 +15,17 @@
 #include "TutorialStateContext.h"
 
 TutorialStateColorFilter::TutorialStateColorFilter (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_text3 (nullptr),
+  m_text4 (nullptr),
+  m_back (nullptr)
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateColorFilter::TutorialStateColorFilter";
 }
 
 void TutorialStateColorFilter::begin ()

src/Tutorial/TutorialStateContext.cpp

@@ -29,6 +29,11 @@ TutorialStateContext::TutorialStateContext (TutorialDlg &tutorialDlg) :
   createTimer ();
 }
 
+TutorialStateContext::~TutorialStateContext ()
+{
+  qDeleteAll (m_states);
+}
+
 void TutorialStateContext::createStates ()
 {
   LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateContext::createStates";

src/Tutorial/TutorialStateContext.h

@@ -25,6 +25,9 @@ class TutorialStateContext : public QObject
   /// Single constructor
   TutorialStateContext(TutorialDlg &tutorialDlg);
 
+  /// Destructor deallocates memory
+  ~TutorialStateContext();
+
   /// Request a transition to the specified state from the current state. A timer is used. This assumes TutorialStateContext
   /// is NOT on the stack - probably since an external event (mouse click, ...) resulted in a callback to the current state
   void requestDelayedStateTransition (TutorialState tutorialState);

src/Tutorial/TutorialStateCurveSelection.cpp

@@ -16,8 +16,18 @@
 #include "TutorialStateContext.h"
 
 TutorialStateCurveSelection::TutorialStateCurveSelection (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_text3 (nullptr),
+  m_previous (nullptr),
+  m_colorFilter (nullptr),  
+  m_next (nullptr)  
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateCurveSelection::TutorialStateCurveSelection";
 }
 
 void TutorialStateCurveSelection::begin ()

src/Tutorial/TutorialStateCurveType.cpp

@@ -16,8 +16,17 @@
 #include "TutorialStateCurveType.h"
 
 TutorialStateCurveType::TutorialStateCurveType (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_previous (nullptr),
+  m_nextLines (nullptr),
+  m_nextPoints (nullptr)
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateCurveType::TutorialStateCurveType";
 }
 
 void TutorialStateCurveType::begin ()

src/Tutorial/TutorialStateIntroduction.cpp

@@ -15,8 +15,15 @@
 #include "TutorialStateIntroduction.h"
 
 TutorialStateIntroduction::TutorialStateIntroduction (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_next (nullptr)
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateIntroduction::TutorialStateIntroduction";
 }
 
 void TutorialStateIntroduction::begin ()

src/Tutorial/TutorialStateIntroduction.h

@@ -22,7 +22,7 @@ class TutorialStateIntroduction : public TutorialStateAbstractBase
  public:
   /// Single constructor
   TutorialStateIntroduction (TutorialStateContext &context);
-
+  
   virtual void begin ();
   virtual void end ();
 

src/Tutorial/TutorialStatePointMatch.cpp

@@ -15,8 +15,17 @@
 #include "TutorialStatePointMatch.h"
 
 TutorialStatePointMatch::TutorialStatePointMatch (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_text3 (nullptr),
+  m_previous (nullptr),
+  m_next (nullptr)
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStatePointMatch::TutorialStatePointMatch";
 }
 
 void TutorialStatePointMatch::begin ()

src/Tutorial/TutorialStateSegmentFill.cpp

@@ -15,8 +15,16 @@
 #include "TutorialStateSegmentFill.h"
 
 TutorialStateSegmentFill::TutorialStateSegmentFill (TutorialStateContext &context) : 
-  TutorialStateAbstractBase (context)
+  TutorialStateAbstractBase (context),
+  m_title (nullptr),
+  m_background (nullptr),
+  m_text0 (nullptr),
+  m_text1 (nullptr),
+  m_text2 (nullptr),
+  m_previous (nullptr),
+  m_next (nullptr)
 {
+  LOG4CPP_INFO_S ((*mainCat)) << "TutorialStateSegmentFill::TutorialStateSegmentFill";
 }
 
 void TutorialStateSegmentFill::begin ()