Remove simple constructors/destructors in favor of compiler-generated ones.

Framework/API/inc/MantidAPI/AlgorithmProperty.h

@@ -70,8 +70,6 @@ public:
   inline AlgorithmProperty *clone() const override {
     return new AlgorithmProperty(*this);
   }
-  /// Destructor
-  ~AlgorithmProperty() override;
 
   /// Add the value of another property. Doesn't make sense here.
   AlgorithmProperty &operator+=(Kernel::Property const *) override {

Framework/API/inc/MantidAPI/DataProcessorAlgorithm.h

@@ -42,7 +42,6 @@ namespace API {
 class DLLExport DataProcessorAlgorithm : public Algorithm {
 public:
   DataProcessorAlgorithm();
-  ~DataProcessorAlgorithm() override;
   std::string getPropertyValue(const std::string &name) const override;
   TypedValue getProperty(const std::string &name) const override;
 

Framework/API/inc/MantidAPI/MultiPeriodGroupAlgorithm.h

@@ -39,7 +39,6 @@ National Laboratory & European Spallation Source
 class DLLExport MultiPeriodGroupAlgorithm : public Algorithm {
 public:
   MultiPeriodGroupAlgorithm();
-  ~MultiPeriodGroupAlgorithm() override;
 
 private:
   /// Overriden from Algorithm base

Framework/API/inc/MantidAPI/MultiPeriodGroupWorker.h

@@ -48,10 +48,10 @@ public:
   /// Convenience typdef for workspace names.
   typedef std::vector<WorkspaceGroup_sptr> VecWSGroupType;
   /// Constructor
-  MultiPeriodGroupWorker();
-  /// Copy constructor
+  MultiPeriodGroupWorker() = default;
+  /// Constructor
   MultiPeriodGroupWorker(const std::string &workspacePropertyName);
-  virtual ~MultiPeriodGroupWorker();
+  virtual ~MultiPeriodGroupWorker() = default;
   /// Flag to indicate use of a custom workspace property
   bool useCustomWorkspaceProperty() const;
   /// Check groups

Framework/API/inc/MantidAPI/Progress.h

@@ -44,8 +44,6 @@ public:
   Progress(Algorithm *alg, double start, double end, int numSteps);
   Progress(Algorithm *alg, double start, double end, int64_t numSteps);
   Progress(Algorithm *alg, double start, double end, size_t numSteps);
-  ~Progress() override;
-
   void doReport(const std::string &msg = "") override;
   bool hasCancellationBeenRequested() const override;
 

Framework/API/src/AlgorithmProperty.cpp

@@ -41,11 +41,6 @@ AlgorithmProperty &AlgorithmProperty::operator=(const AlgorithmProperty &rhs) {
   return *this;
 }
 
-/**
- * Destructor
- */
-AlgorithmProperty::~AlgorithmProperty() = default;
-
 /**
  * Return the algorithm as string
  * @returns The algorithm serialized as a string

Framework/API/src/DataProcessorAlgorithm.cpp

@@ -33,11 +33,6 @@ DataProcessorAlgorithm::DataProcessorAlgorithm()
   enableHistoryRecordingForChild(true);
 }
 
-//----------------------------------------------------------------------------------------------
-/** Destructor
- */
-DataProcessorAlgorithm::~DataProcessorAlgorithm() = default;
-
 //---------------------------------------------------------------------------------------------
 /** Create a Child Algorithm.  A call to this method creates a child algorithm
 *object.

Framework/API/src/MultiPeriodGroupAlgorithm.cpp

@@ -12,11 +12,6 @@ namespace API {
 MultiPeriodGroupAlgorithm::MultiPeriodGroupAlgorithm()
     : m_worker(new MultiPeriodGroupWorker) {}
 
-//----------------------------------------------------------------------------------------------
-/** Destructor
- */
-MultiPeriodGroupAlgorithm::~MultiPeriodGroupAlgorithm() = default;
-
 /** Check the input workspace properties for groups.
  *
  * Overriden from base Algorithm class.

Framework/API/src/MultiPeriodGroupWorker.cpp

@@ -10,12 +10,6 @@ using namespace Mantid::Kernel;
 namespace Mantid {
 namespace API {
 
-//----------------------------------------------------------------------------------------------
-/** Constructor
- */
-MultiPeriodGroupWorker::MultiPeriodGroupWorker()
-    : m_workspacePropertyName("") {}
-
 /**
  * Constructor
  * @param workspacePropertyName : Property name to treat as source of
@@ -25,11 +19,6 @@ MultiPeriodGroupWorker::MultiPeriodGroupWorker(
     const std::string &workspacePropertyName)
     : m_workspacePropertyName(workspacePropertyName) {}
 
-//----------------------------------------------------------------------------------------------
-/** Destructor
- */
-MultiPeriodGroupWorker::~MultiPeriodGroupWorker() = default;
-
 /**
  * Try to add the input workspace to the multiperiod input group list.
  * @param ws: candidate workspace

Framework/API/src/Progress.cpp

@@ -39,9 +39,6 @@ Progress::Progress(Algorithm *alg, double start, double end, int64_t numSteps)
 Progress::Progress(Algorithm *alg, double start, double end, size_t numSteps)
     : ProgressBase(start, end, int64_t(numSteps)), m_alg(alg) {}
 
-/** Destructor */
-Progress::~Progress() = default;
-
 /** Actually do the reporting, without changing the loop counter.
  * This is called by report(), and can be called directly in
  * order to force a report.

Framework/Crystal/inc/MantidCrystal/AddPeakHKL.h

@@ -32,9 +32,6 @@ namespace Crystal {
 */
 class DLLExport AddPeakHKL : public API::Algorithm {
 public:
-  AddPeakHKL();
-  ~AddPeakHKL() override;
-
   const std::string name() const override;
   int version() const override;
   const std::string category() const override;

Framework/Crystal/inc/MantidCrystal/CalculatePeaksHKL.h

@@ -34,8 +34,6 @@ namespace Crystal {
 */
 class DLLExport CalculatePeaksHKL : public API::Algorithm {
 public:
-  CalculatePeaksHKL();
-  ~CalculatePeaksHKL() override;
 
   const std::string name() const override;
   /// Summary of algorithms purpose

Framework/Crystal/inc/MantidCrystal/CalculateUMatrix.h

@@ -36,9 +36,6 @@ namespace Crystal {
 */
 class DLLExport CalculateUMatrix : public API::Algorithm {
 public:
-  CalculateUMatrix();
-  ~CalculateUMatrix() override;
-
   /// Algorithm's name for identification
   const std::string name() const override { return "CalculateUMatrix"; };
   /// Summary of algorithms purpose

Framework/Crystal/inc/MantidCrystal/CentroidPeaks.h

@@ -17,9 +17,6 @@ namespace Crystal {
  */
 class DLLExport CentroidPeaks : public API::Algorithm {
 public:
-  CentroidPeaks();
-  ~CentroidPeaks() override;
-
   /// Algorithm's name for identification
   const std::string name() const override { return "CentroidPeaks"; };
   /// Summary of algorithms purpose

Framework/Crystal/inc/MantidCrystal/ClearUB.h

@@ -38,8 +38,6 @@ namespace Crystal {
 */
 class DLLExport ClearUB : public API::Algorithm {
 public:
-  ClearUB();
-  ~ClearUB() override;
 
   const std::string name() const override;
   /// Summary of algorithms purpose

Framework/Crystal/inc/MantidCrystal/CombinePeaksWorkspaces.h

@@ -35,9 +35,6 @@ namespace Crystal {
 */
 class DLLExport CombinePeaksWorkspaces : public API::Algorithm {
 public:
-  CombinePeaksWorkspaces();
-  ~CombinePeaksWorkspaces() override;
-
   const std::string name() const override;
   /// Summary of algorithms purpose
   const std::string summary() const override {

Framework/Crystal/inc/MantidCrystal/DiffPeaksWorkspaces.h

@@ -33,8 +33,6 @@ namespace Crystal {
 */
 class DLLExport DiffPeaksWorkspaces : public API::Algorithm {
 public:
-  DiffPeaksWorkspaces();
-  ~DiffPeaksWorkspaces() override;
 
   const std::string name() const override;
   /// Summary of algorithms purpose

Framework/Crystal/inc/MantidCrystal/FilterPeaks.h

@@ -33,8 +33,6 @@ namespace Crystal {
 */
 class DLLExport FilterPeaks : public API::Algorithm {
 public:
-  FilterPeaks();
-  ~FilterPeaks() override;
 
   const std::string name() const override;
   /// Summary of algorithms purpose

Framework/Crystal/inc/MantidCrystal/FindClusterFaces.h

@@ -33,8 +33,6 @@ namespace Crystal {
 */
 class DLLExport FindClusterFaces : public API::Algorithm {
 public:
-  FindClusterFaces();
-  ~FindClusterFaces() override;
 
   const std::string name() const override;
   /// Summary of algorithms purpose

Framework/Crystal/inc/MantidCrystal/FindUBUsingFFT.h

@@ -35,8 +35,6 @@ namespace Crystal {
   */
 class DLLExport FindUBUsingFFT : public API::Algorithm {
 public:
-  FindUBUsingFFT();
-  ~FindUBUsingFFT() override;
 
   /// Algorithm's name for identification
   const std::string name() const override;