Enabled some more warning flags in Xcode, and fixed the problems they caused

extras/Projucer/Builds/MacOSX/Projucer.xcodeproj/project.pbxproj

@@ -3228,7 +3228,7 @@
 				INFOPLIST_PREPROCESS = NO;
 				INSTALL_PATH = "$(HOME)/Applications";
 				MACOSX_DEPLOYMENT_TARGET = 10.11;
-				OTHER_CPLUSPLUSFLAGS = "-Wall -Wno-missing-field-initializers -Wshadow-all -Wshorten-64-to-32 -Wstrict-aliasing -Wuninitialized -Wunused-parameter -Wconversion -Wsign-compare -Wint-conversion -Wconditional-uninitialized -Woverloaded-virtual -Wreorder -Wconstant-conversion -Wsign-conversion  -Wunused-private-field -Wbool-conversion -Wextra-semi -Wno-ignored-qualifiers -Wunreachable-code -Wzero-as-null-pointer-constant";
+				OTHER_CPLUSPLUSFLAGS = "-Wall -Wno-missing-field-initializers -Wshadow-all -Wshorten-64-to-32 -Wstrict-aliasing -Wuninitialized -Wunused-parameter -Wconversion -Wsign-compare -Wint-conversion -Wconditional-uninitialized -Woverloaded-virtual -Wreorder -Wconstant-conversion -Wsign-conversion  -Wunused-private-field -Wbool-conversion -Wextra-semi -Wno-ignored-qualifiers -Wunreachable-code -Wzero-as-null-pointer-constant -Wcast-align -Winconsistent-missing-destructor-override -Wshift-sign-overflow -Wundefined-func-template -Wnullable-to-nonnull-conversion";
 				PRODUCT_BUNDLE_IDENTIFIER = com.juce.theprojucer;
 				PRODUCT_NAME = "Projucer";
 				USE_HEADERMAP = NO;
@@ -3273,7 +3273,7 @@
 				INSTALL_PATH = "$(HOME)/Applications";
 				LLVM_LTO = YES;
 				MACOSX_DEPLOYMENT_TARGET = 10.11;
-				OTHER_CPLUSPLUSFLAGS = "-Wall -Wno-missing-field-initializers -Wshadow-all -Wshorten-64-to-32 -Wstrict-aliasing -Wuninitialized -Wunused-parameter -Wconversion -Wsign-compare -Wint-conversion -Wconditional-uninitialized -Woverloaded-virtual -Wreorder -Wconstant-conversion -Wsign-conversion  -Wunused-private-field -Wbool-conversion -Wextra-semi -Wno-ignored-qualifiers -Wunreachable-code -Wzero-as-null-pointer-constant";
+				OTHER_CPLUSPLUSFLAGS = "-Wall -Wno-missing-field-initializers -Wshadow-all -Wshorten-64-to-32 -Wstrict-aliasing -Wuninitialized -Wunused-parameter -Wconversion -Wsign-compare -Wint-conversion -Wconditional-uninitialized -Woverloaded-virtual -Wreorder -Wconstant-conversion -Wsign-conversion  -Wunused-private-field -Wbool-conversion -Wextra-semi -Wno-ignored-qualifiers -Wunreachable-code -Wzero-as-null-pointer-constant -Wcast-align -Winconsistent-missing-destructor-override -Wshift-sign-overflow -Wundefined-func-template -Wnullable-to-nonnull-conversion";
 				PRODUCT_BUNDLE_IDENTIFIER = com.juce.theprojucer;
 				PRODUCT_NAME = "Projucer";
 				USE_HEADERMAP = NO;

extras/Projucer/Projucer.jucer

@@ -8,7 +8,7 @@
   <EXPORTFORMATS>
     <XCODE_MAC targetFolder="Builds/MacOSX" vstFolder="~/SDKs/vstsdk2.4" rtasFolder="~/SDKs/PT_80_SDK"
                documentExtensions=".jucer" objCExtraSuffix="zkVtji" bigIcon="rv1F4h"
-               extraLinkerFlags="" extraCompilerFlags="-Wall -Wno-missing-field-initializers -Wshadow-all -Wshorten-64-to-32 -Wstrict-aliasing -Wuninitialized -Wunused-parameter -Wconversion -Wsign-compare -Wint-conversion -Wconditional-uninitialized -Woverloaded-virtual -Wreorder -Wconstant-conversion -Wsign-conversion  -Wunused-private-field -Wbool-conversion -Wextra-semi -Wno-ignored-qualifiers -Wunreachable-code -Wzero-as-null-pointer-constant"
+               extraLinkerFlags="" extraCompilerFlags="-Wall -Wno-missing-field-initializers -Wshadow-all -Wshorten-64-to-32 -Wstrict-aliasing -Wuninitialized -Wunused-parameter -Wconversion -Wsign-compare -Wint-conversion -Wconditional-uninitialized -Woverloaded-virtual -Wreorder -Wconstant-conversion -Wsign-conversion  -Wunused-private-field -Wbool-conversion -Wextra-semi -Wno-ignored-qualifiers -Wunreachable-code -Wzero-as-null-pointer-constant -Wcast-align -Winconsistent-missing-destructor-override -Wshift-sign-overflow -Wundefined-func-template -Wnullable-to-nonnull-conversion"
                customPList="&lt;plist&gt;&#10;&lt;dict&gt;&#10;&#9;&lt;key&gt;NSAppTransportSecurity&lt;/key&gt;&#10;&#9;&lt;dict&gt;&#10;&#9;&#9;&lt;key&gt;NSAllowsArbitraryLoads&lt;/key&gt;&#10;&#9;&#9;&lt;true/&gt;&#10;&#9;&#9;&lt;key&gt;NSExceptionDomains&lt;/key&gt;&#10;&#9;&#9;&lt;dict&gt;&#10;&#9;&#9;&#9;&lt;key&gt;amazonaws.com&lt;/key&gt;&#10;&#9;&#9;&#9;&lt;dict&gt;&#10;&#9;&#9;&#9;&#9;&lt;key&gt;NSExceptionAllowsInsecureHTTPLoads&lt;/key&gt;&#10;&#9;&#9;&#9;&#9;&lt;true/&gt;&#10;&#9;&#9;&#9;&#9;&lt;key&gt;NSIncludesSubdomains&lt;/key&gt;&#10;&#9;&#9;&#9;&#9;&lt;true/&gt;&#10;&#9;&#9;&#9;&lt;/dict&gt;&#10;&#9;&#9;&lt;/dict&gt;&#10;&#9;&lt;/dict&gt;&#10;&lt;/dict&gt;&#10;&lt;/plist&gt;"
                extraFrameworks="AudioUnit; Accelerate; AVFoundation; CoreAudio; CoreAudioKit; CoreMIDI; DiscRecording; QuartzCore; AudioToolbox; OpenGL; QTKit; QuickTime">
       <CONFIGURATIONS>

extras/Projucer/Source/Application/Windows/jucer_ApplicationUsageDataWindowComponent.h

@@ -91,7 +91,7 @@ public:
         }
     }
 
-    ~ApplicationUsageDataWindowComponent()
+    ~ApplicationUsageDataWindowComponent() override
     {
         if (auto* controller = ProjucerApplication::getApp().licenseController.get())
         {

extras/Projucer/Source/Application/Windows/jucer_EditorColourSchemeWindowComponent.h

@@ -142,7 +142,7 @@ private:
                 saveSchemeState();
             }
 
-            ~EditorPanel()
+            ~EditorPanel() override
             {
                 if (hasSchemeBeenModifiedSinceSave())
                     saveScheme (true);

extras/Projucer/Source/Application/Windows/jucer_FloatingToolWindow.h

@@ -60,7 +60,7 @@ struct FloatingToolWindow  : public DialogWindow
         owner.reset (this);
     }
 
-    ~FloatingToolWindow()
+    ~FloatingToolWindow() override
     {
         if (windowPosProperty.isNotEmpty())
             getGlobalProperties().setValue (windowPosProperty, getWindowStateAsString());

extras/Projucer/Source/Application/Windows/jucer_GlobalPathsWindowComponent.h

@@ -70,7 +70,7 @@ public:
         lastUserModulePath = getAppSettings().getStoredPath (Ids::defaultUserModulePath, TargetOS::getThisOS()).get();
     }
 
-    ~GlobalPathsWindowComponent()
+    ~GlobalPathsWindowComponent() override
     {
         auto juceValue = getAppSettings().getStoredPath (Ids::defaultJuceModulePath, TargetOS::getThisOS());
         auto userValue = getAppSettings().getStoredPath (Ids::defaultUserModulePath, TargetOS::getThisOS());

extras/Projucer/Source/Application/Windows/jucer_PIPCreatorWindowComponent.h

@@ -85,7 +85,7 @@ public:
         pipTree.addListener (this);
     }
 
-    ~PIPCreatorWindowComponent()
+    ~PIPCreatorWindowComponent() override
     {
         setLookAndFeel (nullptr);
     }

extras/Projucer/Source/Application/jucer_AutoUpdater.cpp

@@ -316,7 +316,7 @@ public:
         lookAndFeelChanged();
     }
 
-    ~UpdateUserDialog()
+    ~UpdateUserDialog() override
     {
         titleLabel.reset();
         contentLabel.reset();

extras/Projucer/Source/Application/jucer_AutoUpdater.h

@@ -58,7 +58,7 @@ public:
 
     //==============================================================================
     LatestVersionChecker();
-    ~LatestVersionChecker();
+    ~LatestVersionChecker() override;
 
     static String getOSString();
 

extras/Projucer/Source/Application/jucer_MainWindow.h

@@ -42,7 +42,7 @@ class MainWindow  : public DocumentWindow,
 public:
     //==============================================================================
     MainWindow();
-    ~MainWindow();
+    ~MainWindow() override;
 
     //==============================================================================
     void closeButtonPressed() override;

extras/Projucer/Source/Application/jucer_ProjucerAnalytics.h

@@ -43,7 +43,7 @@ class ProjucerAnalyticsDestination    : public ThreadedAnalyticsDestination
 {
 public:
     ProjucerAnalyticsDestination();
-    ~ProjucerAnalyticsDestination();
+    ~ProjucerAnalyticsDestination() override;
 
     //==============================================================================
     bool logBatchedEvents (const Array<AnalyticsEvent>&) override;

extras/Projucer/Source/CodeEditor/jucer_DocumentEditorComponent.h

@@ -35,7 +35,7 @@ class DocumentEditorComponent  : public Component,
 public:
     //==============================================================================
     DocumentEditorComponent (OpenDocumentManager::Document* document);
-    ~DocumentEditorComponent();
+    ~DocumentEditorComponent() override;
 
     OpenDocumentManager::Document* getDocument() const              { return document; }
 

extras/Projucer/Source/CodeEditor/jucer_LiveBuildCodeEditor.h

@@ -40,7 +40,7 @@ public:
     {
     }
 
-    ~LiveBuildCodeEditor()
+    ~LiveBuildCodeEditor() override
     {
         for (int i = getNumChildComponents(); --i >= 0;)
             if (auto* c = dynamic_cast<DiagnosticOverlayComponent*> (getChildComponent (i)))
@@ -77,7 +77,7 @@ private:
             endPosition.setPositionMaintained (true);
         }
 
-        ~OverlayComponent()
+        ~OverlayComponent() override
         {
             setEditor (nullptr);
         }
@@ -224,7 +224,7 @@ private:
             startTimer (600);
         }
 
-        ~ComponentClassList()
+        ~ComponentClassList() override
         {
             deleteOverlays();
         }
@@ -627,7 +627,7 @@ private:
             owner.getDocument().addListener (this);
         }
 
-        ~LiteralHighlightOverlay()
+        ~LiteralHighlightOverlay() override
         {
             if (auto* p = getParentComponent())
             {

extras/Projucer/Source/CodeEditor/jucer_SourceCodeEditor.h

@@ -145,7 +145,7 @@ class SourceCodeEditor  : public DocumentEditorComponent,
 public:
     SourceCodeEditor (OpenDocumentManager::Document*, CodeDocument&);
     SourceCodeEditor (OpenDocumentManager::Document*, GenericCodeEditorComponent*);
-    ~SourceCodeEditor();
+    ~SourceCodeEditor() override;
 
     void scrollToKeepRangeOnScreen (Range<int> range);
     void highlight (Range<int> range, bool cursorAtStart);
@@ -179,7 +179,7 @@ class GenericCodeEditorComponent  : public CodeEditorComponent
 {
 public:
     GenericCodeEditorComponent (const File&, CodeDocument&, CodeTokeniser*);
-    ~GenericCodeEditorComponent();
+    ~GenericCodeEditorComponent() override;
 
     void addPopupMenuItems (PopupMenu&, const MouseEvent*) override;
     void performPopupMenuAction (int menuItemID) override;
@@ -225,7 +225,7 @@ class CppCodeEditorComponent  : public GenericCodeEditorComponent
 {
 public:
     CppCodeEditorComponent (const File&, CodeDocument&);
-    ~CppCodeEditorComponent();
+    ~CppCodeEditorComponent() override;
 
     void addPopupMenuItems (PopupMenu&, const MouseEvent*) override;
     void performPopupMenuAction (int menuItemID) override;

extras/Projucer/Source/ComponentEditor/Components/jucer_TabbedComponentHandler.h

@@ -798,7 +798,7 @@ private:
             document.addChangeListener (this);
         }
 
-        ~TabColourProperty()
+        ~TabColourProperty() override
         {
             document.removeChangeListener (this);
         }

extras/Projucer/Source/ComponentEditor/Documents/jucer_ComponentDocument.cpp

@@ -112,7 +112,7 @@ public:
             addAndMakeVisible (layout->getComponent (i));
     }
 
-    ~NormalTestComponent()
+    ~NormalTestComponent() override
     {
         for (int i = getNumChildComponents(); --i >= 0;)
             removeChildComponent (i);

extras/Projucer/Source/ComponentEditor/PaintElements/jucer_ColouredElement.h

@@ -43,7 +43,7 @@ public:
                      const bool showOutline_,
                      const bool showJointAndEnd_);
 
-    ~ColouredElement();
+    ~ColouredElement() override;
 
     //==============================================================================
     void getEditableProperties (Array<PropertyComponent*>& props, bool multipleSelected) override;

extras/Projucer/Source/ComponentEditor/PaintElements/jucer_PaintElement.h

@@ -45,7 +45,7 @@ class PaintElement  : public Component,
 public:
     //==============================================================================
     PaintElement (PaintRoutine* owner, const String& typeName);
-    virtual ~PaintElement();
+    ~PaintElement() override;
 
     //==============================================================================
     virtual void setInitialBounds (int parentWidth, int parentHeight);

extras/Projucer/Source/ComponentEditor/PaintElements/jucer_PaintElementGroup.h

@@ -34,8 +34,7 @@ class PaintElementGroup   : public PaintElement
 {
 public:
     PaintElementGroup (PaintRoutine*);
-
-    ~PaintElementGroup();
+    ~PaintElementGroup() override;
 
     void ungroup (const bool);
 

extras/Projucer/Source/ComponentEditor/PaintElements/jucer_PaintElementImage.h

@@ -36,7 +36,7 @@ class PaintElementImage   : public PaintElement
 {
 public:
     PaintElementImage (PaintRoutine*);
-    ~PaintElementImage();
+    ~PaintElementImage() override;
 
     enum StretchMode
     {

extras/Projucer/Source/ComponentEditor/PaintElements/jucer_PaintElementPath.h

@@ -64,7 +64,7 @@ class PaintElementPath   : public ColouredElement
 {
 public:
     PaintElementPath (PaintRoutine* owner);
-    ~PaintElementPath();
+    ~PaintElementPath() override;
 
     //==============================================================================
     void setInitialBounds (int parentWidth, int parentHeight) override;

extras/Projucer/Source/ComponentEditor/PaintElements/jucer_PaintElementText.h

@@ -405,7 +405,7 @@ private:
             element->getDocument()->addChangeListener (this);
         }
 
-        ~TextProperty()
+        ~TextProperty() override
         {
             element->getDocument()->removeChangeListener (this);
         }

extras/Projucer/Source/ComponentEditor/UI/jucer_ComponentLayoutEditor.h

@@ -39,8 +39,8 @@ class ComponentLayoutEditor  : public Component,
 {
 public:
     //==============================================================================
-    ComponentLayoutEditor (JucerDocument& document, ComponentLayout& layout);
-    ~ComponentLayoutEditor();
+    ComponentLayoutEditor (JucerDocument&, ComponentLayout&);
+    ~ComponentLayoutEditor() override;
 
     //==============================================================================
     void paint (Graphics&) override;

extras/Projucer/Source/ComponentEditor/UI/jucer_ComponentOverlayComponent.h

@@ -39,7 +39,7 @@ public:
     ComponentOverlayComponent (Component* const targetComponent,
                                ComponentLayout& layout);
 
-    ~ComponentOverlayComponent();
+    ~ComponentOverlayComponent() override;
 
     //==============================================================================
     virtual void showPopupMenu();

extras/Projucer/Source/ComponentEditor/UI/jucer_EditingPanelBase.h

@@ -44,7 +44,7 @@ public:
                       Component* propsPanel,
                       Component* editorComp);
 
-    ~EditingPanelBase();
+    ~EditingPanelBase() override;
 
     //==============================================================================
     void resized() override;

extras/Projucer/Source/ComponentEditor/UI/jucer_JucerDocumentEditor.cpp

@@ -54,7 +54,7 @@ public:
         document.addChangeListener (this);
     }
 
-    ~ExtraMethodsList()
+    ~ExtraMethodsList() override
     {
         document.removeChangeListener (this);
     }
@@ -165,7 +165,7 @@ public:
         doc.addChangeListener (this);
     }
 
-    ~ClassPropertiesPanel()
+    ~ClassPropertiesPanel() override
     {
         document.removeChangeListener (this);
     }

extras/Projucer/Source/ComponentEditor/UI/jucer_JucerDocumentEditor.h

@@ -39,7 +39,7 @@ class JucerDocumentEditor   : public Component,
 public:
     //==============================================================================
     JucerDocumentEditor (JucerDocument* const document);
-    ~JucerDocumentEditor();
+    ~JucerDocumentEditor() override;
 
     JucerDocument* getDocument() const noexcept         { return document.get(); }
 

extras/Projucer/Source/ComponentEditor/UI/jucer_PaintRoutinePanel.cpp

@@ -43,7 +43,7 @@ public:
         document.addChangeListener (this);
     }
 
-    ~ComponentBackgroundColourProperty()
+    ~ComponentBackgroundColourProperty() override
     {
         document.removeChangeListener (this);
     }

extras/Projucer/Source/ComponentEditor/UI/jucer_ResourceEditorPanel.h

@@ -35,7 +35,7 @@ class ResourceEditorPanel  : public Component,
 {
 public:
     ResourceEditorPanel (JucerDocument& document);
-    ~ResourceEditorPanel();
+    ~ResourceEditorPanel() override;
 
     void resized() override;
     void paint (Graphics& g) override;

extras/Projucer/Source/ComponentEditor/UI/jucer_TestComponent.h

@@ -37,7 +37,7 @@ public:
                    JucerDocument* const loadedDocument,
                    const bool alwaysFillBackground);
 
-    ~TestComponent();
+    ~TestComponent() override;
 
     //==============================================================================
     void setFilename (const String& fn);

extras/Projucer/Source/ComponentEditor/jucer_JucerDocument.h

@@ -40,7 +40,7 @@ class JucerDocument  : public ChangeBroadcaster,
 {
 public:
     JucerDocument (SourceCodeDocument* cpp);
-    ~JucerDocument();
+    ~JucerDocument() override;
 
     static bool isValidJucerCppFile (const File&);
     static XmlElement* pullMetaDataFromCppFile (const String& cpp);

extras/Projucer/Source/ComponentEditor/jucer_ObjectTypes.cpp

@@ -78,7 +78,7 @@ static const char* const elementNames[] =
     nullptr
 };
 
-const char** const elementTypeNames = (const char**) elementNames;
+const char* const* const elementTypeNames = (const char* const*) elementNames;
 const int numElementTypes = (sizeof (elementNames) / sizeof (elementNames[0])) - 1;
 
 PaintElement* createNewElement (const int index, PaintRoutine* owner)
@@ -158,7 +158,7 @@ static ComponentTypeHandler* const compTypes[] =
     nullptr
 };
 
-ComponentTypeHandler** const componentTypeHandlers = (ComponentTypeHandler**) compTypes;
+ComponentTypeHandler* const* const componentTypeHandlers = (ComponentTypeHandler* const*) compTypes;
 const int numComponentTypes = numElementsInArray (compTypes) - 1;
 
 }

extras/Projucer/Source/ComponentEditor/jucer_ObjectTypes.h

@@ -34,12 +34,12 @@ namespace ObjectTypes
 {
     //==============================================================================
     // Component types
-    extern ComponentTypeHandler** const componentTypeHandlers;
+    extern ComponentTypeHandler* const* const componentTypeHandlers;
     extern const int numComponentTypes;
 
     //==============================================================================
     // Element types
-    extern const char** const elementTypeNames;
+    extern const char* const* const elementTypeNames;
     extern const int numElementTypes;
 
     PaintElement* createNewElement (const int index, PaintRoutine* owner);

extras/Projucer/Source/Licenses/jucer_LicenseThread.h

@@ -33,7 +33,7 @@ struct NetWorkerThread   : public Thread,
 {
     NetWorkerThread()  : Thread ("License") {}
 
-    ~NetWorkerThread()
+    ~NetWorkerThread() override
     {
         JUCE_ASSERT_MESSAGE_MANAGER_IS_LOCKED
 

extras/Projucer/Source/Licenses/jucer_LicenseWebview.h

@@ -65,7 +65,7 @@ private:
         struct RedirectWebBrowserComponent : public WebBrowserComponent
         {
             RedirectWebBrowserComponent (LicenseWebviewContent& controller) : WebBrowserComponent (false), owner (controller) {}
-            virtual ~RedirectWebBrowserComponent() {}
+            ~RedirectWebBrowserComponent() override  {}
 
             bool pageAboutToLoad           (const String& url) override { return owner.pageAboutToLoad (url); }
             void pageFinishedLoading       (const String& url) override { owner.pageFinishedLoading (url); }
@@ -95,7 +95,7 @@ private:
                 }
             }
 
-            virtual ~Header()
+            ~Header() override
             {
                 if (auto* licenseController = ProjucerApplication::getApp().licenseController.get())
                     licenseController->removeLicenseStatusChangedCallback (this);

extras/Projucer/Source/LiveBuildEngine/UI/jucer_ActivityListComponent.h

@@ -45,7 +45,7 @@ public:
         activityList.addChangeListener (this);
     }
 
-    ~CurrentActivitiesComp()
+    ~CurrentActivitiesComp() override
     {
         activityList.removeChangeListener (this);
     }

extras/Projucer/Source/LiveBuildEngine/UI/jucer_BuildTabStatusComponent.h

@@ -42,7 +42,7 @@ public:
         errorList.addChangeListener (this);
     }
 
-    ~BuildStatusTabComp()
+    ~BuildStatusTabComp() override
     {
         activityList.removeChangeListener (this);
         errorList.removeChangeListener (this);

extras/Projucer/Source/LiveBuildEngine/UI/jucer_ComponentListComponent.h

@@ -48,7 +48,7 @@ public:
         owner.activityList.addListener (this);
     }
 
-    ~ComponentListComp()
+    ~ComponentListComp() override
     {
         saveOpenness();
         owner.activityList.removeListener (this);

extras/Projucer/Source/LiveBuildEngine/UI/jucer_ErrorListComponent.h

@@ -47,7 +47,7 @@ public:
         errorListChanged();
     }
 
-    ~ErrorListComp()
+    ~ErrorListComp() override
     {
         errorList.removeChangeListener (this);
     }
@@ -219,7 +219,7 @@ private:
             uniqueID << message.message << ':' << message.range.toString();
         }
 
-        ~ErrorMessageTreeItem()
+        ~ErrorMessageTreeItem() override
         {
             overlay.deleteAndZero();
         }

extras/Projucer/Source/LiveBuildEngine/jucer_CompileEngineClient.cpp

@@ -210,7 +210,7 @@ public:
         openedOk = true;
     }
 
-    ~ChildProcess()
+    ~ChildProcess() override
     {
         projectRoot.removeListener (this);
 
@@ -642,7 +642,7 @@ struct CompileEngineChildProcess::Editor  : private CodeDocument::Listener,
         document.addListener (this);
     }
 
-    ~Editor()
+    ~Editor() override
     {
         document.removeListener (this);
     }

extras/Projucer/Source/LiveBuildEngine/jucer_CompileEngineClient.h

@@ -36,7 +36,7 @@ class CompileEngineChildProcess  : public ReferenceCountedObject,
 {
 public:
     CompileEngineChildProcess (Project&);
-    ~CompileEngineChildProcess();
+    ~CompileEngineChildProcess() override;
 
     //==============================================================================
     bool openedOk() const       { return process != nullptr; }

extras/Projucer/Source/LiveBuildEngine/jucer_CompileEngineServer.cpp

@@ -67,7 +67,7 @@ struct ZombiePatrol    : private Thread,
         startTimer (1000);
     }
 
-    ~ZombiePatrol()
+    ~ZombiePatrol() override
     {
         stopThread (1000);
     }
@@ -161,7 +161,7 @@ public:
         zombieKiller.reset (new ZombiePatrol (*this));
     }
 
-    ~ServerIPC()
+    ~ServerIPC() override
     {
         zombieKiller.reset();
 

extras/Projucer/Source/Project/UI/Sidebar/jucer_FileTreeItems.h

@@ -38,7 +38,7 @@ public:
         item.state.addListener (this);
     }
 
-    ~FileTreeItemBase()
+    ~FileTreeItemBase() override
     {
         item.state.removeListener (this);
     }

extras/Projucer/Source/Project/UI/Sidebar/jucer_ModuleTreeItems.h

@@ -456,7 +456,7 @@ public:
         project.getExporterPathsModuleList().addListener (this);
     }
 
-    ~EnabledModulesItem()
+    ~EnabledModulesItem() override
     {
         ProjucerApplication::getApp().getJUCEPathModuleList().removeListener (this);
         ProjucerApplication::getApp().getUserPathsModuleList().removeListener (this);

extras/Projucer/Source/Project/UI/Sidebar/jucer_ProjectTab.h

@@ -42,7 +42,7 @@ struct ProjectSettingsComponent  : public Component,
         project.addChangeListener (this);
     }
 
-    ~ProjectSettingsComponent()
+    ~ProjectSettingsComponent() override
     {
         project.removeChangeListener (this);
     }
@@ -139,7 +139,7 @@ public:
         buildConcertina();
     }
 
-    ~ProjectTab()
+    ~ProjectTab() override
     {
         getFileTreePanel()->saveOpenness();
         getModuleTreePanel()->saveOpenness();

extras/Projucer/Source/Project/UI/Sidebar/jucer_TabComponents.h

@@ -111,7 +111,7 @@ public:
         lookAndFeelChanged();
     }
 
-    ~FindPanel()
+    ~FindPanel() override
     {
         Desktop::getInstance().removeFocusChangeListener (this);
     }
@@ -194,7 +194,7 @@ public:
         addAndMakeVisible (treeToDisplay.get());
     }
 
-    ~ConcertinaTreeComponent()
+    ~ConcertinaTreeComponent() override
     {
         treeToDisplay.reset();
         addButton.reset();

extras/Projucer/Source/Project/UI/jucer_FileGroupInformationComponent.h

@@ -50,7 +50,7 @@ public:
         addAndMakeVisible (header);
     }
 
-    ~FileGroupInformationComponent()
+    ~FileGroupInformationComponent() override
     {
         item.state.removeListener (this);
     }

extras/Projucer/Source/Project/UI/jucer_HeaderComponent.h

@@ -40,7 +40,7 @@ class HeaderComponent    : public Component,
 {
 public:
     HeaderComponent();
-    ~HeaderComponent();
+    ~HeaderComponent() override;
 
     //==========================================================================
     void resized() override;

extras/Projucer/Source/Project/UI/jucer_ProjectContentComponent.h

@@ -44,7 +44,7 @@ class ProjectContentComponent  : public Component,
 public:
     //==============================================================================
     ProjectContentComponent();
-    ~ProjectContentComponent();
+    ~ProjectContentComponent() override;
 
     Project* getProject() const noexcept    { return project; }
     virtual void setProject (Project*);

extras/Projucer/Source/Project/jucer_Project.h

@@ -42,7 +42,7 @@ class Project  : public FileBasedDocument,
 public:
     //==============================================================================
     Project (const File&);
-    ~Project();
+    ~Project() override;
 
     //==============================================================================
     // FileBasedDocument stuff..

extras/Projucer/Source/Settings/jucer_StoredSettings.h

@@ -34,7 +34,7 @@ class StoredSettings   : public ValueTree::Listener
 {
 public:
     StoredSettings();
-    ~StoredSettings();
+    ~StoredSettings() override;
 
     PropertiesFile& getGlobalProperties();
     PropertiesFile& getProjectProperties (const String& projectUID);
@@ -54,7 +54,7 @@ public:
     struct ColourSelectorWithSwatches   : public ColourSelector
     {
         ColourSelectorWithSwatches();
-        ~ColourSelectorWithSwatches();
+        ~ColourSelectorWithSwatches() override;
 
         int getNumSwatches() const override;
         Colour getSwatchColour (int index) const override;

extras/Projucer/Source/Utility/UI/PropertyComponents/jucer_LabelPropertyComponent.h

@@ -46,7 +46,7 @@ public:
         setLookAndFeel (&lf);
     }
 
-    ~LabelPropertyComponent()    { setLookAndFeel (nullptr); }
+    ~LabelPropertyComponent() override    { setLookAndFeel (nullptr); }
 
     //==============================================================================
     void refresh() override {}

extras/Projucer/Source/Utility/UI/PropertyComponents/jucer_PropertyComponentsWithEnablement.h

@@ -45,7 +45,7 @@ public:
         setEnabled (valueWithDefault.get());
     }
 
-    ~TextPropertyComponentWithEnablement()    { value.removeListener (this); }
+    ~TextPropertyComponentWithEnablement() override    { value.removeListener (this); }
 
 private:
     ValueWithDefault valueWithDefault;
@@ -72,7 +72,7 @@ public:
         setEnabled (valueWithDefault.get());
     }
 
-    ~ChoicePropertyComponentWithEnablement()    { value.removeListener (this); }
+    ~ChoicePropertyComponentWithEnablement() override    { value.removeListener (this); }
 
 private:
     ValueWithDefault valueWithDefault;

extras/Projucer/Source/Utility/UI/jucer_JucerTreeViewBase.h

@@ -35,7 +35,7 @@ class JucerTreeViewBase   : public TreeViewItem,
 {
 public:
     JucerTreeViewBase();
-    ~JucerTreeViewBase();
+    ~JucerTreeViewBase() override;
 
     int getItemWidth() const override                   { return -1; }
     int getItemHeight() const override                  { return 25; }
@@ -134,7 +134,7 @@ public:
         tree.addMouseListener (this, true);
     }
 
-    ~TreePanelBase()
+    ~TreePanelBase() override
     {
         tree.setRootItem (nullptr);
     }

extras/Projucer/Source/Utility/UI/jucer_ProjucerLookAndFeel.h

@@ -32,7 +32,7 @@ class ProjucerLookAndFeel   : public LookAndFeel_V4
 {
 public:
     ProjucerLookAndFeel();
-    ~ProjucerLookAndFeel();
+    ~ProjucerLookAndFeel() override;
 
     void drawTabButton (TabBarButton& button, Graphics&, bool isMouseOver, bool isMouseDown) override;
     int getTabButtonBestWidth (TabBarButton&, int tabDepth) override;

extras/Projucer/Source/Utility/UI/jucer_SlidingPanelComponent.h

@@ -33,7 +33,7 @@ class SlidingPanelComponent   : public Component
 {
 public:
     SlidingPanelComponent();
-    ~SlidingPanelComponent();
+    ~SlidingPanelComponent() override;
 
     /** Adds a new tab to the panel slider. */
     void addTab (const String& tabName,

modules/juce_analytics/analytics/juce_ButtonTracker.h

@@ -63,7 +63,7 @@ public:
                    int triggeredEventType = 0);
 
     /** Destructor. */
-    ~ButtonTracker();
+    ~ButtonTracker() override;
 
 private:
     /** @internal */

modules/juce_analytics/destinations/juce_ThreadedAnalyticsDestination.h

@@ -63,7 +63,7 @@ public:
     ThreadedAnalyticsDestination (const String& threadName = "Analytics thread");
 
     /** Destructor. */
-    virtual ~ThreadedAnalyticsDestination();
+    ~ThreadedAnalyticsDestination() override;
 
     //==============================================================================
     /**

modules/juce_core/containers/juce_AbstractFifo.cpp

@@ -30,8 +30,8 @@ AbstractFifo::AbstractFifo (int capacity) noexcept : bufferSize (capacity)
 
 AbstractFifo::~AbstractFifo() {}
 
-int AbstractFifo::getTotalSize() const noexcept           { return bufferSize; }
-int AbstractFifo::getFreeSpace() const noexcept           { return bufferSize - getNumReady() - 1; }
+int AbstractFifo::getTotalSize() const noexcept    { return bufferSize; }
+int AbstractFifo::getFreeSpace() const noexcept    { return bufferSize - getNumReady() - 1; }
 
 int AbstractFifo::getNumReady() const noexcept
 {
@@ -54,7 +54,8 @@ void AbstractFifo::setTotalSize (int newSize) noexcept
 }
 
 //==============================================================================
-void AbstractFifo::prepareToWrite (int numToWrite, int& startIndex1, int& blockSize1, int& startIndex2, int& blockSize2) const noexcept
+void AbstractFifo::prepareToWrite (int numToWrite, int& startIndex1, int& blockSize1,
+                                   int& startIndex2, int& blockSize2) const noexcept
 {
     auto vs = validStart.get();
     auto ve = validEnd.get();
@@ -91,7 +92,8 @@ void AbstractFifo::finishedWrite (int numWritten) noexcept
     validEnd = newEnd;
 }
 
-void AbstractFifo::prepareToRead (int numWanted, int& startIndex1, int& blockSize1, int& startIndex2, int& blockSize2) const noexcept
+void AbstractFifo::prepareToRead (int numWanted, int& startIndex1, int& blockSize1,
+                                  int& startIndex2, int& blockSize2) const noexcept
 {
     auto vs = validStart.get();
     auto ve = validEnd.get();
@@ -198,6 +200,10 @@ void AbstractFifo::ScopedReadWrite<AbstractFifo::ReadOrWrite::write>::finish (Ab
 template class AbstractFifo::ScopedReadWrite<AbstractFifo::ReadOrWrite::read>;
 template class AbstractFifo::ScopedReadWrite<AbstractFifo::ReadOrWrite::write>;
 
+AbstractFifo::ScopedRead  AbstractFifo::read  (int numToRead) noexcept     { return { *this, numToRead }; }
+AbstractFifo::ScopedWrite AbstractFifo::write (int numToWrite) noexcept    { return { *this, numToWrite }; }
+
+
 //==============================================================================
 #if JUCE_UNIT_TESTS
 

modules/juce_core/containers/juce_AbstractFifo.h

@@ -281,7 +281,7 @@ public:
         } // readHandle goes out of scope here, finishing the read operation
         @endcode
     */
-    ScopedRead read (int numToRead) noexcept      { return { *this, numToRead }; }
+    ScopedRead read (int numToRead) noexcept;
 
     /** Replaces prepareToWrite/finishedWrite with a single function.
         This function returns an object which contains the start indices and
@@ -303,7 +303,7 @@ public:
         } // writeHandle goes out of scope here, finishing the write operation
         @endcode
     */
-    ScopedWrite write (int numToWrite) noexcept    { return { *this, numToWrite }; }
+    ScopedWrite write (int numToWrite) noexcept;
 
 private:
     //==============================================================================

modules/juce_core/files/juce_FileInputStream.h

@@ -44,7 +44,7 @@ public:
     explicit FileInputStream (const File& fileToRead);
 
     /** Destructor. */
-    ~FileInputStream();
+    ~FileInputStream() override;
 
     //==============================================================================
     /** Returns the file that this stream is reading from. */

modules/juce_core/files/juce_FileOutputStream.h

@@ -67,7 +67,7 @@ public:
                       size_t bufferSizeToUse = 16384);
 
     /** Destructor. */
-    ~FileOutputStream();
+    ~FileOutputStream() override;
 
     //==============================================================================
     /** Returns the file that this stream is writing to.

modules/juce_core/files/juce_WildcardFileFilter.h

@@ -57,7 +57,7 @@ public:
                         const String& description);
 
     /** Destructor. */
-    ~WildcardFileFilter();
+    ~WildcardFileFilter() override;
 
     //==============================================================================
     /** Returns true if the filename matches one of the patterns specified. */

modules/juce_core/logging/juce_FileLogger.h

@@ -56,7 +56,7 @@ public:
                 const int64 maxInitialFileSizeBytes = 128 * 1024);
 
     /** Destructor. */
-    ~FileLogger();
+    ~FileLogger() override;
 
     //==============================================================================
     /** Returns the file that this logger is writing to. */

modules/juce_core/maths/juce_BigInteger.cpp

@@ -171,7 +171,7 @@ uint32* BigInteger::getValues() const noexcept
     jassert (heapAllocation != nullptr || allocatedSize <= numPreallocatedInts);
 
     return heapAllocation != nullptr ? heapAllocation
-                                     : (uint32*) preallocated;
+                                     : const_cast<uint32*> (preallocated);
 }
 
 uint32* BigInteger::ensureSize (const size_t numVals)

modules/juce_core/memory/juce_Memory.h

@@ -44,7 +44,7 @@ inline void deleteAndZero (Type& pointer)                           { delete poi
     a specific number of bytes,
 */
 template <typename Type, typename IntegerType>
-inline Type* addBytesToPointer (Type* basePointer, IntegerType bytes) noexcept  { return (Type*) (((char*) basePointer) + bytes); }
+inline Type* addBytesToPointer (Type* basePointer, IntegerType bytes) noexcept  { return (Type*) (const_cast<char*> (reinterpret_cast<const char*> (basePointer)) + bytes); }
 
 /** A handy function to round up a pointer to the nearest multiple of a given number of bytes.
     alignmentBytes must be a power of two. */

modules/juce_core/native/juce_mac_Network.mm

@@ -37,7 +37,7 @@ void MACAddress::findAllAddresses (Array<MACAddress>& result)
 
             if (sto.sa_family == AF_LINK)
             {
-                auto sadd = (const sockaddr_dl*) cursor->ifa_addr;
+                auto sadd = reinterpret_cast<const sockaddr_dl*> (cursor->ifa_addr);
 
                #ifndef IFT_ETHER
                 enum { IFT_ETHER = 6 };
@@ -127,7 +127,7 @@ public:
         DelegateClass::setState (delegate, this);
     }
 
-    ~URLConnectionState()
+    ~URLConnectionState() override
     {
         signalThreadShouldExit();
 
@@ -1096,38 +1096,44 @@ private:
     {
         jassert (connection == nullptr);
 
-        if (NSMutableURLRequest* req = [NSMutableURLRequest requestWithURL: [NSURL URLWithString: juceStringToNS (url.toString (! isPost))]
-                                                               cachePolicy: NSURLRequestReloadIgnoringLocalCacheData
-                                                           timeoutInterval: timeOutMs <= 0 ? 60.0 : (timeOutMs / 1000.0)])
+        if (NSURL* nsURL = [NSURL URLWithString: juceStringToNS (url.toString (! isPost))])
         {
-            [req setHTTPMethod: [NSString stringWithUTF8String: httpRequestCmd.toRawUTF8()]];
-
-            if (isPost)
+            if (NSMutableURLRequest* req = [NSMutableURLRequest requestWithURL: nsURL
+                                                                   cachePolicy: NSURLRequestReloadIgnoringLocalCacheData
+                                                               timeoutInterval: timeOutMs <= 0 ? 60.0 : (timeOutMs / 1000.0)])
             {
-                WebInputStream::createHeadersAndPostData (url, headers, postData);
+                if (NSString* httpMethod = [NSString stringWithUTF8String: httpRequestCmd.toRawUTF8()])
+                {
+                    [req setHTTPMethod: httpMethod];
 
-                if (postData.getSize() > 0)
-                    [req setHTTPBody: [NSData dataWithBytes: postData.getData()
-                                                     length: postData.getSize()]];
-            }
+                    if (isPost)
+                    {
+                        WebInputStream::createHeadersAndPostData (url, headers, postData);
 
-            StringArray headerLines;
-            headerLines.addLines (headers);
-            headerLines.removeEmptyStrings (true);
+                        if (postData.getSize() > 0)
+                            [req setHTTPBody: [NSData dataWithBytes: postData.getData()
+                                                             length: postData.getSize()]];
+                    }
 
-            for (int i = 0; i < headerLines.size(); ++i)
-            {
-                String key   = headerLines[i].upToFirstOccurrenceOf (":", false, false).trim();
-                String value = headerLines[i].fromFirstOccurrenceOf (":", false, false).trim();
+                    StringArray headerLines;
+                    headerLines.addLines (headers);
+                    headerLines.removeEmptyStrings (true);
 
-                if (key.isNotEmpty() && value.isNotEmpty())
-                    [req addValue: juceStringToNS (value) forHTTPHeaderField: juceStringToNS (key)];
-            }
+                    for (int i = 0; i < headerLines.size(); ++i)
+                    {
+                        auto key   = headerLines[i].upToFirstOccurrenceOf (":", false, false).trim();
+                        auto value = headerLines[i].fromFirstOccurrenceOf (":", false, false).trim();
+
+                        if (key.isNotEmpty() && value.isNotEmpty())
+                            [req addValue: juceStringToNS (value) forHTTPHeaderField: juceStringToNS (key)];
+                    }
 
-            // Workaround for an Apple bug. See https://github.com/AFNetworking/AFNetworking/issues/2334
-            [req HTTPBody];
+                    // Workaround for an Apple bug. See https://github.com/AFNetworking/AFNetworking/issues/2334
+                    [req HTTPBody];
 
-            connection.reset (new URLConnectionState (req, numRedirectsToFollow));
+                    connection.reset (new URLConnectionState (req, numRedirectsToFollow));
+                }
+            }
         }
     }
 

modules/juce_core/native/juce_mac_Strings.mm

@@ -92,7 +92,7 @@ String String::convertToPrecomposedUnicode() const
                                       bytesNeeded, &bytesRead,
                                       &outputBufferSize, tempOut) == noErr)
         {
-            result = String (CharPointer_UTF16 ((CharPointer_UTF16::CharType*) tempOut.get()));
+            result = String (CharPointer_UTF16 (reinterpret_cast<CharPointer_UTF16::CharType*> (tempOut.get())));
         }
 
         DisposeUnicodeToTextInfo (&conversionInfo);

modules/juce_core/native/juce_posix_IPAddress.h

@@ -71,8 +71,8 @@ namespace
         {
             if (ifa->ifa_addr->sa_family == AF_INET)
             {
-                auto interfaceAddressInfo = (sockaddr_in*) ifa->ifa_addr;
-                auto broadcastAddressInfo = (sockaddr_in*) ifa->ifa_dstaddr;
+                auto interfaceAddressInfo = reinterpret_cast<sockaddr_in*> (ifa->ifa_addr);
+                auto broadcastAddressInfo = reinterpret_cast<sockaddr_in*> (ifa->ifa_dstaddr);
 
                 if (interfaceAddressInfo->sin_addr.s_addr != INADDR_NONE)
                 {
@@ -83,8 +83,8 @@ namespace
             }
             else if (ifa->ifa_addr->sa_family == AF_INET6)
             {
-                interfaceInfo.interfaceAddress = makeAddress ((sockaddr_in6*) ifa->ifa_addr);
-                interfaceInfo.broadcastAddress = makeAddress ((sockaddr_in6*) ifa->ifa_dstaddr);
+                interfaceInfo.interfaceAddress = makeAddress (reinterpret_cast<sockaddr_in6*> (ifa->ifa_addr));
+                interfaceInfo.broadcastAddress = makeAddress (reinterpret_cast<sockaddr_in6*> (ifa->ifa_dstaddr));
                 return true;
             }
         }

modules/juce_core/network/juce_URL.cpp

@@ -46,7 +46,7 @@ struct FallbackDownloadTask  : public URL::DownloadTask,
         startThread();
     }
 
-    ~FallbackDownloadTask()
+    ~FallbackDownloadTask() override
     {
         signalThreadShouldExit();
         stream->cancel();

modules/juce_core/network/juce_WebInputStream.h

@@ -50,7 +50,7 @@ class JUCE_API WebInputStream : public InputStream
     */
     WebInputStream (const URL& url, const bool usePost);
 
-    ~WebInputStream();
+    ~WebInputStream() override;
 
 
     /** Add extra headers to http request

modules/juce_core/streams/juce_BufferedInputStream.h

@@ -61,7 +61,7 @@ public:
         This may also delete the source stream, if that option was chosen when the
         buffered stream was created.
     */
-    ~BufferedInputStream();
+    ~BufferedInputStream() override;
 
 
     //==============================================================================

modules/juce_core/streams/juce_FileInputSource.h

@@ -43,7 +43,7 @@ public:
     FileInputSource (const File& file, bool useFileTimeInHashGeneration = false);
 
     /** Destructor. */
-    ~FileInputSource();
+    ~FileInputSource() override;
 
     InputStream* createInputStream() override;
     InputStream* createInputStreamFor (const String& relatedItemPath) override;

modules/juce_core/streams/juce_MemoryInputStream.h

@@ -63,7 +63,7 @@ public:
                        bool keepInternalCopyOfData);
 
     /** Destructor. */
-    ~MemoryInputStream();
+    ~MemoryInputStream() override;
 
     /** Returns a pointer to the source data block from which this stream is reading. */
     const void* getData() const noexcept        { return data; }

modules/juce_core/streams/juce_MemoryOutputStream.h

@@ -66,7 +66,7 @@ public:
     /** Destructor.
         This will free any data that was written to it.
     */
-    ~MemoryOutputStream();
+    ~MemoryOutputStream() override;
 
     //==============================================================================
     /** Returns a pointer to the data that has been written to the stream.

modules/juce_core/streams/juce_SubregionStream.h

@@ -62,7 +62,7 @@ public:
         This may also delete the source stream, if that option was chosen when the
         buffered stream was created.
     */
-    ~SubregionStream();
+    ~SubregionStream() override;
 
 
     //==============================================================================

modules/juce_core/streams/juce_URLInputSource.h

@@ -45,7 +45,7 @@ public:
     URLInputSource (URL&& url);
 
     /** Destructor. */
-    ~URLInputSource();
+    ~URLInputSource() override;
 
     InputStream* createInputStream() override;
     InputStream* createInputStreamFor (const String& relatedItemPath) override;

modules/juce_core/text/juce_String.cpp

@@ -2005,7 +2005,7 @@ String String::createStringFromData (const void* const unknownData, int size)
 
         StringCreationHelper builder ((size_t) numChars);
 
-        auto src = (const uint16*) (data + 2);
+        auto src = reinterpret_cast<const uint16*> (data + 2);
 
         if (CharPointer_UTF16::isByteOrderMarkBigEndian (data))
         {
@@ -2075,19 +2075,19 @@ struct StringEncodingConverter
 template <>
 struct StringEncodingConverter<CharPointer_UTF8, CharPointer_UTF8>
 {
-    static CharPointer_UTF8 convert (const String& source) noexcept   { return CharPointer_UTF8 ((CharPointer_UTF8::CharType*) source.getCharPointer().getAddress()); }
+    static CharPointer_UTF8 convert (const String& source) noexcept   { return CharPointer_UTF8 (reinterpret_cast<CharPointer_UTF8::CharType*> (source.getCharPointer().getAddress())); }
 };
 
 template <>
 struct StringEncodingConverter<CharPointer_UTF16, CharPointer_UTF16>
 {
-    static CharPointer_UTF16 convert (const String& source) noexcept  { return CharPointer_UTF16 ((CharPointer_UTF16::CharType*) source.getCharPointer().getAddress()); }
+    static CharPointer_UTF16 convert (const String& source) noexcept  { return CharPointer_UTF16 (reinterpret_cast<CharPointer_UTF16::CharType*> (source.getCharPointer().getAddress())); }
 };
 
 template <>
 struct StringEncodingConverter<CharPointer_UTF32, CharPointer_UTF32>
 {
-    static CharPointer_UTF32 convert (const String& source) noexcept  { return CharPointer_UTF32 ((CharPointer_UTF32::CharType*) source.getCharPointer().getAddress()); }
+    static CharPointer_UTF32 convert (const String& source) noexcept  { return CharPointer_UTF32 (reinterpret_cast<CharPointer_UTF32::CharType*> (source.getCharPointer().getAddress())); }
 };
 
 CharPointer_UTF8  String::toUTF8()  const { return StringEncodingConverter<CharPointerType, CharPointer_UTF8 >::convert (*this); }

modules/juce_core/threads/juce_TimeSliceThread.h

@@ -98,7 +98,7 @@ public:
         should always call stopThread() with a decent timeout before deleting,
         to avoid the thread being forcibly killed (which is a Bad Thing).
     */
-    ~TimeSliceThread();
+    ~TimeSliceThread() override;
 
     //==============================================================================
     /** Adds a client to the list.

modules/juce_core/zip/juce_GZIPCompressorOutputStream.h

@@ -72,7 +72,7 @@ public:
                                 int windowBits = 0);
 
     /** Destructor. */
-    ~GZIPCompressorOutputStream();
+    ~GZIPCompressorOutputStream() override;
 
     //==============================================================================
     /** Flushes and closes the stream.

modules/juce_core/zip/juce_GZIPDecompressorInputStream.h

@@ -70,7 +70,7 @@ public:
     GZIPDecompressorInputStream (InputStream& sourceStream);
 
     /** Destructor. */
-    ~GZIPDecompressorInputStream();
+    ~GZIPDecompressorInputStream() override;
 
     //==============================================================================
     int64 getPosition() override;

modules/juce_core/zip/juce_ZipFile.cpp

@@ -154,7 +154,7 @@ struct ZipFile::ZipInputStream  : public InputStream
         }
     }
 
-    ~ZipInputStream()
+    ~ZipInputStream() override
     {
        #if JUCE_DEBUG
         if (inputStream != nullptr && inputStream == file.inputStream)

modules/juce_data_structures/app_properties/juce_PropertiesFile.h

@@ -180,7 +180,7 @@ public:
     /** Destructor.
         When deleted, the file will first call saveIfNeeded() to flush any changes to disk.
     */
-    ~PropertiesFile();
+    ~PropertiesFile() override;
 
     //==============================================================================
     /** Returns true if this file was created from a valid (or non-existent) file.

modules/juce_data_structures/values/juce_Value.h

@@ -180,7 +180,7 @@ public:
     {
     public:
         ValueSource();
-        virtual ~ValueSource();
+        virtual ~ValueSource() override;
 
         /** Returns the current value of this object. */
         virtual var getValue() const = 0;

modules/juce_data_structures/values/juce_ValueTree.cpp

@@ -815,7 +815,7 @@ struct ValueTreePropertyValueSource  : public Value::ValueSource,
         tree.addListener (this);
     }
 
-    ~ValueTreePropertyValueSource()
+    ~ValueTreePropertyValueSource() override
     {
         tree.removeListener (this);
     }

modules/juce_data_structures/values/juce_ValueTreeSynchroniser.h

@@ -55,7 +55,7 @@ public:
     ValueTreeSynchroniser (const ValueTree& tree);
 
     /** Destructor. */
-    virtual ~ValueTreeSynchroniser();
+    ~ValueTreeSynchroniser() override;
 
     /** This callback happens when the ValueTree changes and the given state-change message
         needs to be applied to any other trees that need to stay in sync with it.

modules/juce_events/interprocess/juce_ConnectedChildProcess.cpp

@@ -94,7 +94,7 @@ struct ChildProcessMaster::Connection  : public InterprocessConnection,
             startThread (4);
     }
 
-    ~Connection()
+    ~Connection() override
     {
         stopThread (10000);
     }
@@ -192,7 +192,7 @@ struct ChildProcessSlave::Connection  : public InterprocessConnection,
         startThread (4);
     }
 
-    ~Connection()
+    ~Connection() override
     {
         stopThread (10000);
     }

modules/juce_events/interprocess/juce_InterprocessConnectionServer.h

@@ -45,7 +45,7 @@ public:
     InterprocessConnectionServer();
 
     /** Destructor. */
-    ~InterprocessConnectionServer();
+    ~InterprocessConnectionServer() override;
 
     //==============================================================================
     /** Starts an internal thread which listens on the given port number.

modules/juce_events/interprocess/juce_NetworkServiceDiscovery.h

@@ -53,7 +53,7 @@ struct NetworkServiceDiscovery
                     RelativeTime minTimeBetweenBroadcasts = RelativeTime::seconds (1.5));
 
         /** Destructor */
-        ~Advertiser();
+        ~Advertiser() override;
 
     private:
         XmlElement message;
@@ -101,7 +101,7 @@ struct NetworkServiceDiscovery
         AvailableServiceList (const String& serviceTypeUID, int broadcastPort);
 
         /** Destructor */
-        ~AvailableServiceList();
+        ~AvailableServiceList() override;
 
         /** A lambda that can be set to recieve a callback when the list changes */
         std::function<void()> onChange;

modules/juce_events/messages/juce_Message.h

@@ -46,7 +46,7 @@ public:
     //==============================================================================
     /** Creates an uninitialised message. */
     Message() noexcept;
-    ~Message();
+    ~Message() override;
 
     using Ptr = ReferenceCountedObjectPtr<Message>;
 

modules/juce_events/messages/juce_MessageManager.cpp

@@ -433,7 +433,7 @@ bool MessageManagerLock::attemptLock (Thread* threadToCheck, ThreadPoolJob* jobT
     return true;
 }
 
-MessageManagerLock::~MessageManagerLock() noexcept     { mmLock.exit(); }
+MessageManagerLock::~MessageManagerLock()  { mmLock.exit(); }
 
 void MessageManagerLock::exitSignalSent()
 {

modules/juce_events/messages/juce_MessageManager.h

@@ -452,7 +452,7 @@ public:
         Make sure this object is created and deleted by the same thread,
         otherwise there are no guarantees what will happen!
    */
-    ~MessageManagerLock() noexcept;
+    ~MessageManagerLock() override;
 
     //==============================================================================
     /** Returns true if the lock was successfully acquired.

modules/juce_events/native/juce_mac_MessageManager.mm

@@ -454,7 +454,7 @@ void __attribute__ ((visibility("default"))) repostCurrentNSEvent()
     struct EventReposter  : public CallbackMessage
     {
         EventReposter() : e ([[NSApp currentEvent] retain])  {}
-        ~EventReposter()  { [e release]; }
+        ~EventReposter() override  { [e release]; }
 
         void messageCallback() override
         {

modules/juce_events/timers/juce_Timer.cpp

@@ -36,7 +36,7 @@ public:
         triggerAsyncUpdate();
     }
 
-    ~TimerThread() noexcept
+    ~TimerThread() override
     {
         signalThreadShouldExit();
         callbackArrived.signal();

modules/juce_graphics/colour/juce_PixelFormats.h

@@ -315,8 +315,7 @@ public:
 
 private:
     //==============================================================================
-    PixelARGB (const uint32 internalValue) noexcept
-        : internal (internalValue)
+    PixelARGB (uint32 internalValue) noexcept : internal (internalValue)
     {
     }
 
@@ -378,9 +377,9 @@ public:
     forcedinline uint32 getNativeARGB() const noexcept
     {
        #if JUCE_ANDROID
-        return (uint32) ((0xff << 24) | r | (g << 8) | (b << 16));
+        return (uint32) ((0xffu << 24) | r | ((uint32) g << 8) | ((uint32) b << 16));
        #else
-        return (uint32) ((0xff << 24) | b | (g << 8) | (r << 16));
+        return (uint32) ((0xffu << 24) | b | ((uint32) g << 8) | ((uint32) r << 16));
        #endif
     }
 
@@ -389,7 +388,7 @@ public:
     forcedinline uint32 getInARGBMaskOrder() const noexcept
     {
        #if JUCE_ANDROID
-        return (uint32) ((0xff << 24) | (r << 16) | (g << 8) | (b << 0));
+        return (uint32) ((0xffu << 24) | b | ((uint32) g << 8) | ((uint32) r << 16));
        #else
         return getNativeARGB();
        #endif

modules/juce_graphics/contexts/juce_LowLevelGraphicsPostScriptRenderer.h

@@ -43,7 +43,7 @@ public:
                                         int totalWidth,
                                         int totalHeight);
 
-    ~LowLevelGraphicsPostScriptRenderer();
+    ~LowLevelGraphicsPostScriptRenderer() override;
 
     //==============================================================================
     bool isVectorDevice() const override;

modules/juce_graphics/effects/juce_DropShadowEffect.h

@@ -93,7 +93,7 @@ public:
     DropShadowEffect();
 
     /** Destructor. */
-    ~DropShadowEffect();
+    ~DropShadowEffect() override;
 
     //==============================================================================
     /** Sets up parameters affecting the shadow's appearance. */

modules/juce_graphics/effects/juce_GlowEffect.h

@@ -47,7 +47,7 @@ public:
     GlowEffect();
 
     /** Destructor. */
-    ~GlowEffect();
+    ~GlowEffect() override;
 
     //==============================================================================
     /** Sets the glow's radius and colour.

modules/juce_graphics/fonts/juce_CustomTypeface.h

@@ -65,7 +65,7 @@ public:
     explicit CustomTypeface (InputStream& serialisedTypefaceStream);
 
     /** Destructor. */
-    ~CustomTypeface();
+    ~CustomTypeface() override;
 
     //==============================================================================
     /** Resets this typeface, deleting all its glyphs and settings. */

modules/juce_graphics/images/juce_Image.h

@@ -528,7 +528,7 @@ class JUCE_API  SoftwareImageType   : public ImageType
 {
 public:
     SoftwareImageType();
-    ~SoftwareImageType();
+    ~SoftwareImageType() override;
 
     ImagePixelData::Ptr create (Image::PixelFormat, int width, int height, bool clearImage) const override;
     int getTypeID() const override;
@@ -546,7 +546,7 @@ class JUCE_API  NativeImageType   : public ImageType
 {
 public:
     NativeImageType();
-    ~NativeImageType();
+    ~NativeImageType() override;
 
     ImagePixelData::Ptr create (Image::PixelFormat, int width, int height, bool clearImage) const override;
     int getTypeID() const override;