Skin: Fix texture deallocation

Author: adazem009

src/skin.cpp

@@ -12,11 +12,13 @@ Skin::Skin()
     QOpenGLContext *context = QOpenGLContext::currentContext();
     Q_ASSERT(context);
 
-    if (context) {
-        QObject::connect(context, &QOpenGLContext::aboutToBeDestroyed, &m_signalHandler, [this]() {
+    if (!m_destroyConnected && context) {
+        QObject::connect(context, &QOpenGLContext::aboutToBeDestroyed, []() {
             // Destroy textures
             m_textures.clear();
         });
+
+        m_destroyConnected = true;
     }
 }
 

src/skin.h

@@ -26,8 +26,8 @@ class Skin
         virtual void paint(QPainter *painter) = 0;
 
     private:
-        std::vector<std::shared_ptr<QOpenGLTexture>> m_textures;
-        QObject m_signalHandler; // for disconnecting signals after destroyed
+        static inline std::vector<std::shared_ptr<QOpenGLTexture>> m_textures;
+        static inline bool m_destroyConnected = false;
 };
 
 } // namespace scratchcpprender