Tidy up

modules/juce_audio_basics/midi/juce_MidiFile.cpp

@@ -264,7 +264,6 @@ namespace MidiFileHelpers
 
 //==============================================================================
 MidiFile::MidiFile()  : timeFormat ((short) (unsigned short) 0xe728) {}
-MidiFile::~MidiFile() {}
 
 MidiFile::MidiFile (const MidiFile& other)  : timeFormat (other.timeFormat)
 {

modules/juce_audio_basics/midi/juce_MidiFile.h

@@ -45,9 +45,6 @@ public:
     /** Creates an empty MidiFile object. */
     MidiFile();
 
-    /** Destructor. */
-    ~MidiFile();
-
     /** Creates a copy of another MidiFile. */
     MidiFile (const MidiFile&);
 

modules/juce_audio_basics/midi/juce_MidiKeyboardState.cpp

@@ -28,10 +28,6 @@ MidiKeyboardState::MidiKeyboardState()
     zerostruct (noteStates);
 }
 
-MidiKeyboardState::~MidiKeyboardState()
-{
-}
-
 //==============================================================================
 void MidiKeyboardState::reset()
 {

modules/juce_audio_basics/midi/juce_MidiKeyboardState.h

@@ -43,7 +43,6 @@ class JUCE_API  MidiKeyboardState
 public:
     //==============================================================================
     MidiKeyboardState();
-    ~MidiKeyboardState();
 
     //==============================================================================
     /** Resets the state of the object.

modules/juce_audio_basics/midi/juce_MidiMessageSequence.cpp

@@ -25,7 +25,6 @@ namespace juce
 
 MidiMessageSequence::MidiEventHolder::MidiEventHolder (const MidiMessage& mm) : message (mm) {}
 MidiMessageSequence::MidiEventHolder::MidiEventHolder (MidiMessage&& mm) : message (std::move (mm)) {}
-MidiMessageSequence::MidiEventHolder::~MidiEventHolder() {}
 
 //==============================================================================
 MidiMessageSequence::MidiMessageSequence()
@@ -63,10 +62,6 @@ MidiMessageSequence& MidiMessageSequence::operator= (MidiMessageSequence&& other
     return *this;
 }
 
-MidiMessageSequence::~MidiMessageSequence()
-{
-}
-
 void MidiMessageSequence::swapWith (MidiMessageSequence& other) noexcept
 {
     list.swapWith (other.list);

modules/juce_audio_basics/midi/juce_MidiMessageSequence.h

@@ -53,9 +53,6 @@ public:
     /** Move assignment operator */
     MidiMessageSequence& operator= (MidiMessageSequence&&) noexcept;
 
-    /** Destructor. */
-    ~MidiMessageSequence();
-
     //==============================================================================
     /** Structure used to hold midi events in the sequence.
 
@@ -68,9 +65,6 @@ public:
     {
     public:
         //==============================================================================
-        /** Destructor. */
-        ~MidiEventHolder();
-
         /** The message itself, whose timestamp is used to specify the event's time. */
         MidiMessage message;
 

modules/juce_audio_devices/native/juce_win32_Midi.cpp

@@ -258,7 +258,7 @@ private:
 
         struct MidiHeader
         {
-            MidiHeader() {}
+            MidiHeader() = default;
 
             void prepare (HMIDIIN device)
             {

modules/juce_core/containers/juce_AbstractFifo.cpp

@@ -28,8 +28,6 @@ AbstractFifo::AbstractFifo (int capacity) noexcept : bufferSize (capacity)
     jassert (bufferSize > 0);
 }
 
-AbstractFifo::~AbstractFifo() {}
-
 int AbstractFifo::getTotalSize() const noexcept    { return bufferSize; }
 int AbstractFifo::getFreeSpace() const noexcept    { return bufferSize - getNumReady() - 1; }
 

modules/juce_core/containers/juce_AbstractFifo.h

@@ -76,9 +76,6 @@ public:
     /** Creates a FIFO to manage a buffer with the specified capacity. */
     AbstractFifo (int capacity) noexcept;
 
-    /** Destructor */
-    ~AbstractFifo();
-
     //==============================================================================
     /** Returns the total size of the buffer being managed. */
     int getTotalSize() const noexcept;

modules/juce_core/files/juce_FileSearchPath.cpp

@@ -23,9 +23,6 @@
 namespace juce
 {
 
-FileSearchPath::FileSearchPath() {}
-FileSearchPath::~FileSearchPath() {}
-
 FileSearchPath::FileSearchPath (const String& path)
 {
     init (path);

modules/juce_core/files/juce_FileSearchPath.h

@@ -36,7 +36,10 @@ class JUCE_API  FileSearchPath
 public:
     //==============================================================================
     /** Creates an empty search path. */
-    FileSearchPath();
+    FileSearchPath() = default;
+
+    /** Destructor. */
+    ~FileSearchPath() = default;
 
     /** Creates a search path from a string of pathnames.
 
@@ -53,9 +56,6 @@ public:
     /** Copies another search path. */
     FileSearchPath& operator= (const FileSearchPath&);
 
-    /** Destructor. */
-    ~FileSearchPath();
-
     /** Uses a string containing a list of pathnames to re-initialise this list.
 
         This search path is cleared and the semicolon- or comma-separated folders

modules/juce_core/maths/juce_BigInteger.cpp

@@ -130,10 +130,6 @@ BigInteger& BigInteger::operator= (BigInteger&& other) noexcept
     return *this;
 }
 
-BigInteger::~BigInteger()
-{
-}
-
 void BigInteger::swapWith (BigInteger& other) noexcept
 {
     for (int i = 0; i < numPreallocatedInts; ++i)

modules/juce_core/maths/juce_BigInteger.h

@@ -69,7 +69,7 @@ public:
     BigInteger& operator= (BigInteger&&) noexcept;
 
     /** Destructor. */
-    ~BigInteger();
+    ~BigInteger() = default;
 
     //==============================================================================
     /** Copies another BigInteger onto this one. */

modules/juce_core/text/juce_LocalisedStrings.cpp

@@ -48,10 +48,6 @@ LocalisedStrings& LocalisedStrings::operator= (const LocalisedStrings& other)
     return *this;
 }
 
-LocalisedStrings::~LocalisedStrings()
-{
-}
-
 //==============================================================================
 String LocalisedStrings::translate (const String& text) const
 {

modules/juce_core/text/juce_LocalisedStrings.h

@@ -90,7 +90,7 @@ public:
     LocalisedStrings& operator= (const LocalisedStrings&);
 
     /** Destructor. */
-    ~LocalisedStrings();
+    ~LocalisedStrings() = default;
 
     //==============================================================================
     /** Selects the current set of mappings to be used by the system.

modules/juce_core/text/juce_StringArray.cpp

@@ -89,10 +89,6 @@ StringArray& StringArray::operator= (StringArray&& other) noexcept
     return *this;
 }
 
-StringArray::~StringArray()
-{
-}
-
 bool StringArray::operator== (const StringArray& other) const noexcept
 {
     return strings == other.strings;

modules/juce_core/text/juce_StringArray.h

@@ -99,7 +99,7 @@ public:
     StringArray (const wchar_t* const* strings, int numberOfStrings);
 
     /** Destructor. */
-    ~StringArray();
+    ~StringArray() = default;
 
     /** Copies the contents of another string array into this one */
     StringArray& operator= (const StringArray&);

modules/juce_core/text/juce_StringPairArray.cpp

@@ -34,10 +34,6 @@ StringPairArray::StringPairArray (const StringPairArray& other)
 {
 }
 
-StringPairArray::~StringPairArray()
-{
-}
-
 StringPairArray& StringPairArray::operator= (const StringPairArray& other)
 {
     keys = other.keys;

modules/juce_core/text/juce_StringPairArray.h

@@ -42,7 +42,7 @@ public:
     StringPairArray (const StringPairArray& other);
 
     /** Destructor. */
-    ~StringPairArray();
+    ~StringPairArray() = default;
 
     /** Copies the contents of another string array into this one */
     StringPairArray& operator= (const StringPairArray& other);

modules/juce_core/text/juce_StringPool.cpp

@@ -28,7 +28,6 @@ static const uint32 garbageCollectionInterval = 30000;
 
 
 StringPool::StringPool() noexcept  : lastGarbageCollectionTime (0) {}
-StringPool::~StringPool() {}
 
 struct StartEndString
 {

modules/juce_core/text/juce_StringPool.h

@@ -43,9 +43,6 @@ public:
     /** Creates an empty pool. */
     StringPool() noexcept;
 
-    /** Destructor */
-    ~StringPool();
-
     //==============================================================================
     /** Returns a pointer to a shared copy of the string that is passed in.
         The pool will always return the same String object when asked for a string that matches it.

modules/juce_data_structures/app_properties/juce_ApplicationProperties.cpp

@@ -26,10 +26,6 @@
 namespace juce
 {
 
-ApplicationProperties::ApplicationProperties()
-{
-}
-
 ApplicationProperties::~ApplicationProperties()
 {
     closeFiles();

modules/juce_data_structures/app_properties/juce_ApplicationProperties.h

@@ -57,7 +57,7 @@ public:
         Before using it, you must call setStorageParameters() to give it the info
         it needs to create the property files.
     */
-    ApplicationProperties();
+    ApplicationProperties() = default;
 
     /** Destructor. */
     ~ApplicationProperties();

modules/juce_events/messages/juce_MessageManager.h

@@ -203,7 +203,7 @@ public:
             Creates a new critical section to exclusively access methods which can
             only be called when the message manager is locked.
 
-            Unlike CrititcalSection, multiple instances of this lock class provide
+            Unlike CriticalSection, multiple instances of this lock class provide
             exclusive access to a single resource - the MessageManager.
         */
         Lock();

modules/juce_gui_basics/filebrowser/juce_FileSearchPathListComponent.cpp

@@ -81,10 +81,6 @@ FileSearchPathListComponent::FileSearchPathListComponent()
     updateButtons();
 }
 
-FileSearchPathListComponent::~FileSearchPathListComponent()
-{
-}
-
 void FileSearchPathListComponent::updateButtons()
 {
     const bool anythingSelected = listBox.getNumSelectedRows() > 0;

modules/juce_gui_basics/filebrowser/juce_FileSearchPathListComponent.h

@@ -45,9 +45,6 @@ public:
     /** Creates an empty FileSearchPathListComponent. */
     FileSearchPathListComponent();
 
-    /** Destructor. */
-    ~FileSearchPathListComponent() override;
-
     //==============================================================================
     /** Returns the path as it is currently shown. */
     const FileSearchPath& getPath() const noexcept                  { return path; }

modules/juce_gui_basics/native/juce_win32_Windowing.cpp

@@ -1430,7 +1430,7 @@ private:
     //==============================================================================
     struct ComBaseModule
     {
-        ComBaseModule() {}
+        ComBaseModule() = default;
         ComBaseModule (LPCWSTR libraryName) : h (::LoadLibrary (libraryName)) {}
         ComBaseModule (ComBaseModule&& o) : h (o.h) { o.h = nullptr; }
         ~ComBaseModule() { release(); }