Fix warnings

Author: timothyschoen

Source/Components/PropertiesPanel.cpp

@@ -550,8 +550,9 @@ void PropertiesPanel::InspectorColourComponent::mouseExit(MouseEvent const& e)
     repaint();
 }
 
-struct SwatchComponent final : public Component {
+class SwatchComponent final : public Component {
 
+public:
     explicit SwatchComponent(Value const& colour)
     {
         colourValue.referTo(colour);

Source/Components/PropertiesPanel.h

@@ -306,12 +306,13 @@ class PropertiesPanel : public Component {
     };
 
     template<typename T>
-    struct EditableComponent final : public PropertiesPanelProperty, public Value::Listener {
-        std::unique_ptr<Component> label;
+    class EditableComponent final : public PropertiesPanelProperty, public Value::Listener {
         Value property;
         String allowedCharacters = "";
         double min, max;
-
+    public:
+        std::unique_ptr<Component> label;
+        
         EditableComponent(String const& propertyName, Value& value, double minimum = 0.0, double maximum = 0.0, std::function<void(bool)> onInteractionFn = nullptr)
             : PropertiesPanelProperty(propertyName)
             , property(value)

Source/Pd/Instance.cpp

@@ -32,35 +32,16 @@ EXTERN int sys_load_lib(t_canvas* canvas, char const* classname);
 
 namespace pd {
 
-struct ConsoleMessageHandler final : public Timer {
+class ConsoleMessageHandler final : public Timer {
     Instance* instance;
 
+public:
     explicit ConsoleMessageHandler(Instance* parent)
         : instance(parent)
     {
         startTimerHz(30);
     }
 
-    void timerCallback() override
-    {
-        auto item = std::tuple<void*, SmallString, bool>();
-        int numReceived = 0;
-        bool newWarning = false;
-
-        while (pendingMessages.try_dequeue(item)) {
-            auto& [object, message, type] = item;
-            addMessage(object, message.toString(), type);
-
-            numReceived++;
-            newWarning = newWarning || type;
-        }
-
-        // Check if any item got assigned
-        if (numReceived) {
-            instance->updateConsole(numReceived, newWarning);
-        }
-    }
-
     void addMessage(void* object, String const& message, bool type)
     {
         if (consoleMessages.size()) {
@@ -139,10 +120,31 @@ struct ConsoleMessageHandler final : public Timer {
             length = 0;
         }
     }
-
+    
     std::deque<std::tuple<void*, String, int, int, int>> consoleMessages;
     std::deque<std::tuple<void*, String, int, int, int>> consoleHistory;
+    
+private:
+    void timerCallback() override
+    {
+        auto item = std::tuple<void*, SmallString, bool>();
+        int numReceived = 0;
+        bool newWarning = false;
+
+        while (pendingMessages.try_dequeue(item)) {
+            auto& [object, message, type] = item;
+            addMessage(object, message.toString(), type);
 
+            numReceived++;
+            newWarning = newWarning || type;
+        }
+
+        // Check if any item got assigned
+        if (numReceived) {
+            instance->updateConsole(numReceived, newWarning);
+        }
+    }
+    
     StackArray<char, 2048> printConcatBuffer;
 
     moodycamel::ConcurrentQueue<std::tuple<void*, SmallString, bool>> pendingMessages = moodycamel::ConcurrentQueue<std::tuple<void*, SmallString, bool>>(512);