From 2c328dfedc103b8964ee1d603ce04f515fe59661 Mon Sep 17 00:00:00 2001 From: Julian Storer Date: Fri, 19 Aug 2011 21:34:34 +0100 Subject: [PATCH] Changed some return types from being const objects to non-const objects (for better c++11 compliance in future). This may mean that you need to remove the 'const' from your own code if you override the virtual methods that have changed. --- .../Source/Utility/jucer_StoredSettings.h | 2 +- .../src/model/components/jucer_ButtonHandler.h | 4 ++-- .../src/model/components/jucer_ComboBoxHandler.h | 6 +++--- .../model/components/jucer_ComponentNameProperty.h | 6 +++--- .../components/jucer_ComponentTypeHandler.cpp | 4 ++-- .../components/jucer_GenericComponentHandler.h | 4 ++-- .../model/components/jucer_GroupComponentHandler.h | 2 +- .../components/jucer_HyperlinkButtonHandler.h | 2 +- .../model/components/jucer_ImageButtonHandler.h | 2 +- .../model/components/jucer_JucerComponentHandler.h | 4 ++-- .../src/model/components/jucer_LabelHandler.h | 10 ++++------ .../src/model/components/jucer_SliderHandler.h | 6 +++--- .../components/jucer_TabbedComponentHandler.h | 14 +++++++------- .../src/model/components/jucer_TextEditorHandler.h | 2 +- .../src/model/components/jucer_ViewportHandler.h | 4 ++-- .../model/paintelements/jucer_ColouredElement.cpp | 2 +- .../paintelements/jucer_PaintElementEllipse.h | 2 +- .../model/paintelements/jucer_PaintElementGroup.h | 2 +- .../model/paintelements/jucer_PaintElementImage.h | 2 +- .../model/paintelements/jucer_PaintElementPath.cpp | 2 +- .../paintelements/jucer_PaintElementRectangle.h | 2 +- .../jucer_PaintElementRoundedRectangle.h | 2 +- .../model/paintelements/jucer_PaintElementText.h | 8 ++++---- .../src/properties/jucer_ColourPropertyComponent.h | 4 ++-- .../src/properties/jucer_ComponentColourProperty.h | 2 +- .../src/properties/jucer_FontPropertyComponent.h | 2 +- .../src/properties/jucer_PositionPropertyBase.h | 2 +- .../the jucer/src/ui/jucer_JucerDocumentHolder.cpp | 12 ++++++------ .../the jucer/src/ui/jucer_PaintRoutinePanel.cpp | 2 +- .../src/utility/jucer_ColourEditorComponent.h | 4 ++-- .../gui/juce_MidiKeyboardComponent.cpp | 2 +- .../gui/juce_MidiKeyboardComponent.h | 2 +- .../contexts/juce_GraphicsContext.cpp | 2 +- .../juce_graphics/contexts/juce_GraphicsContext.h | 2 +- .../contexts/juce_LowLevelGraphicsContext.h | 2 +- .../juce_LowLevelGraphicsPostScriptRenderer.cpp | 2 +- .../juce_LowLevelGraphicsPostScriptRenderer.h | 2 +- .../juce_LowLevelGraphicsSoftwareRenderer.cpp | 2 +- .../juce_LowLevelGraphicsSoftwareRenderer.h | 2 +- modules/juce_graphics/images/juce_Image.h | 2 +- .../native/juce_android_GraphicsContext.cpp | 2 +- .../native/juce_mac_CoreGraphicsContext.h | 2 +- .../native/juce_mac_CoreGraphicsContext.mm | 2 +- .../native/juce_win32_Direct2DGraphicsContext.cpp | 2 +- .../juce_gui_basics/components/juce_Component.cpp | 2 +- .../juce_gui_basics/components/juce_Component.h | 2 +- .../juce_DirectoryContentsDisplayComponent.h | 2 +- .../filebrowser/juce_FileListComponent.cpp | 2 +- .../filebrowser/juce_FileListComponent.h | 2 +- .../filebrowser/juce_FileTreeComponent.cpp | 2 +- .../filebrowser/juce_FileTreeComponent.h | 2 +- .../keyboard/juce_TextInputTarget.h | 6 +++--- .../layout/juce_ResizableBorderComponent.cpp | 2 +- .../layout/juce_ResizableBorderComponent.h | 2 +- .../lookandfeel/juce_LookAndFeel.cpp | 2 +- .../juce_gui_basics/lookandfeel/juce_LookAndFeel.h | 2 +- .../properties/juce_ButtonPropertyComponent.h | 2 +- .../properties/juce_TextPropertyComponent.cpp | 2 +- .../properties/juce_TextPropertyComponent.h | 2 +- modules/juce_gui_basics/widgets/juce_ListBox.cpp | 2 +- modules/juce_gui_basics/widgets/juce_ListBox.h | 2 +- .../widgets/juce_TableHeaderComponent.cpp | 2 +- .../widgets/juce_TableHeaderComponent.h | 2 +- .../juce_gui_basics/widgets/juce_TextEditor.cpp | 6 +++--- modules/juce_gui_basics/widgets/juce_TextEditor.h | 6 +++--- .../code_editor/juce_CPlusPlusCodeTokeniser.cpp | 2 +- .../code_editor/juce_CPlusPlusCodeTokeniser.h | 2 +- .../code_editor/juce_CodeEditorComponent.cpp | 10 +++++----- .../code_editor/juce_CodeEditorComponent.h | 12 ++++++------ .../code_editor/juce_CodeTokeniser.h | 2 +- .../juce_gui_extra/misc/juce_ColourSelector.cpp | 4 ++-- modules/juce_gui_extra/misc/juce_ColourSelector.h | 4 ++-- .../juce_opengl/native/juce_ios_OpenGLComponent.mm | 2 +- .../native/juce_linux_OpenGLComponent.cpp | 2 +- .../juce_opengl/native/juce_mac_OpenGLComponent.mm | 2 +- .../native/juce_win32_OpenGLComponent.cpp | 2 +- .../juce_opengl/opengl/juce_OpenGLComponent.cpp | 2 +- modules/juce_opengl/opengl/juce_OpenGLComponent.h | 2 +- modules/juce_opengl/opengl/juce_OpenGLContext.h | 2 +- 79 files changed, 126 insertions(+), 128 deletions(-) diff --git a/extras/Introjucer/Source/Utility/jucer_StoredSettings.h b/extras/Introjucer/Source/Utility/jucer_StoredSettings.h index 3bc522f381..6b99043817 100644 --- a/extras/Introjucer/Source/Utility/jucer_StoredSettings.h +++ b/extras/Introjucer/Source/Utility/jucer_StoredSettings.h @@ -64,7 +64,7 @@ public: ColourSelectorWithSwatches() {} int getNumSwatches() const { return StoredSettings::getInstance()->swatchColours.size(); } - const Colour getSwatchColour (int index) const { return StoredSettings::getInstance()->swatchColours [index]; } + Colour getSwatchColour (int index) const { return StoredSettings::getInstance()->swatchColours [index]; } void setSwatchColour (int index, const Colour& newColour) const { StoredSettings::getInstance()->swatchColours.set (index, newColour); } }; diff --git a/extras/the jucer/src/model/components/jucer_ButtonHandler.h b/extras/the jucer/src/model/components/jucer_ButtonHandler.h index cb3a42dc95..80d974c9cb 100644 --- a/extras/the jucer/src/model/components/jucer_ButtonHandler.h +++ b/extras/the jucer/src/model/components/jucer_ButtonHandler.h @@ -193,7 +193,7 @@ private: "Change button text"); } - const String getText() const + String getText() const { return component->getButtonText(); } @@ -290,7 +290,7 @@ private: "Change radio group ID"); } - const String getText() const + String getText() const { return String (component->getRadioGroupId()); } diff --git a/extras/the jucer/src/model/components/jucer_ComboBoxHandler.h b/extras/the jucer/src/model/components/jucer_ComboBoxHandler.h index 4d46902f2a..844eb0dd5d 100644 --- a/extras/the jucer/src/model/components/jucer_ComboBoxHandler.h +++ b/extras/the jucer/src/model/components/jucer_ComboBoxHandler.h @@ -301,7 +301,7 @@ private: "Change combo box items"); } - const String getText() const + String getText() const { return component->getProperties() ["items"]; } @@ -354,7 +354,7 @@ private: "Change combo box text when nothing selected"); } - const String getText() const + String getText() const { return component->getTextWhenNothingSelected(); } @@ -405,7 +405,7 @@ private: "Change combo box 'no items' text"); } - const String getText() const + String getText() const { return component->getTextWhenNoChoicesAvailable(); } diff --git a/extras/the jucer/src/model/components/jucer_ComponentNameProperty.h b/extras/the jucer/src/model/components/jucer_ComponentNameProperty.h index 49a0b06259..b3e0961d7e 100644 --- a/extras/the jucer/src/model/components/jucer_ComponentNameProperty.h +++ b/extras/the jucer/src/model/components/jucer_ComponentNameProperty.h @@ -53,7 +53,7 @@ public: "Change component name"); } - const String getText() const + String getText() const { return component->getName(); } @@ -113,7 +113,7 @@ public: "Change component member name"); } - const String getText() const + String getText() const { return document.getComponentLayout()->getComponentMemberVariableName (component); } @@ -172,7 +172,7 @@ public: "Change component virtual class name"); } - const String getText() const + String getText() const { return document.getComponentLayout()->getComponentVirtualClassName (component); } diff --git a/extras/the jucer/src/model/components/jucer_ComponentTypeHandler.cpp b/extras/the jucer/src/model/components/jucer_ComponentTypeHandler.cpp index 043e1e1399..e07acffb3b 100644 --- a/extras/the jucer/src/model/components/jucer_ComponentTypeHandler.cpp +++ b/extras/the jucer/src/model/components/jucer_ComponentTypeHandler.cpp @@ -268,7 +268,7 @@ public: { } - const String getText() const + String getText() const { SettableTooltipClient* ttc = dynamic_cast (component); return ttc->getTooltip(); @@ -372,7 +372,7 @@ public: { } - const String getText() const + String getText() const { return String (component->getExplicitFocusOrder()); } diff --git a/extras/the jucer/src/model/components/jucer_GenericComponentHandler.h b/extras/the jucer/src/model/components/jucer_GenericComponentHandler.h index d7109b31b9..df3a817fde 100644 --- a/extras/the jucer/src/model/components/jucer_GenericComponentHandler.h +++ b/extras/the jucer/src/model/components/jucer_GenericComponentHandler.h @@ -156,7 +156,7 @@ private: "Change generic component class"); } - const String getText() const + String getText() const { return component->actualClassName; } @@ -206,7 +206,7 @@ private: "Change generic component class"); } - const String getText() const + String getText() const { return component->constructorParams; } diff --git a/extras/the jucer/src/model/components/jucer_GroupComponentHandler.h b/extras/the jucer/src/model/components/jucer_GroupComponentHandler.h index fd820442bc..9a93fc21ba 100644 --- a/extras/the jucer/src/model/components/jucer_GroupComponentHandler.h +++ b/extras/the jucer/src/model/components/jucer_GroupComponentHandler.h @@ -133,7 +133,7 @@ private: "Change group title"); } - const String getText() const + String getText() const { return component->getText(); } diff --git a/extras/the jucer/src/model/components/jucer_HyperlinkButtonHandler.h b/extras/the jucer/src/model/components/jucer_HyperlinkButtonHandler.h index 0761f0465d..c3f7b9dd5d 100644 --- a/extras/the jucer/src/model/components/jucer_HyperlinkButtonHandler.h +++ b/extras/the jucer/src/model/components/jucer_HyperlinkButtonHandler.h @@ -119,7 +119,7 @@ private: "Change hyperlink URL"); } - const String getText() const + String getText() const { return component->getURL().toString (false); } diff --git a/extras/the jucer/src/model/components/jucer_ImageButtonHandler.h b/extras/the jucer/src/model/components/jucer_ImageButtonHandler.h index ebc29c6df6..eca7fb2542 100644 --- a/extras/the jucer/src/model/components/jucer_ImageButtonHandler.h +++ b/extras/the jucer/src/model/components/jucer_ImageButtonHandler.h @@ -501,7 +501,7 @@ public: setImageColour (layout, owner, role, newColour, true); } - const Colour getColour() const + Colour getColour() const { return getImageColour (owner, role); } diff --git a/extras/the jucer/src/model/components/jucer_JucerComponentHandler.h b/extras/the jucer/src/model/components/jucer_JucerComponentHandler.h index 66fbb9133e..64c8e969b9 100644 --- a/extras/the jucer/src/model/components/jucer_JucerComponentHandler.h +++ b/extras/the jucer/src/model/components/jucer_JucerComponentHandler.h @@ -217,7 +217,7 @@ private: mw->openFile (component->findFile()); } - const String getButtonText() const + String getButtonText() const { return "Open file for editing"; } @@ -241,7 +241,7 @@ private: "Change Viewport content constructor params"); } - const String getText() const + String getText() const { return component->getConstructorParams(); } diff --git a/extras/the jucer/src/model/components/jucer_LabelHandler.h b/extras/the jucer/src/model/components/jucer_LabelHandler.h index 8f85677995..fc80a82f67 100644 --- a/extras/the jucer/src/model/components/jucer_LabelHandler.h +++ b/extras/the jucer/src/model/components/jucer_LabelHandler.h @@ -213,7 +213,7 @@ private: "Change Label text"); } - const String getText() const + String getText() const { return component->getText(); } @@ -314,8 +314,7 @@ private: class LabelLossOfFocusProperty : public ComponentChoiceProperty