PenLayer: Do not create FBO in stamp()

Author: adazem009

src/penlayer.cpp

@@ -37,6 +37,9 @@ PenLayer::~PenLayer()
         // Delete vertex array and buffer
         m_glF->glDeleteVertexArrays(1, &m_vao);
         m_glF->glDeleteBuffers(1, &m_vbo);
+
+        // Delete stamp FBO
+        m_glF->glDeleteFramebuffers(1, &m_stampFbo);
     }
 }
 
@@ -99,6 +102,9 @@ void PenLayer::setEngine(libscratchcpp::IEngine *newEngine)
 
             m_glF->glBindVertexArray(0);
             m_glF->glBindBuffer(GL_ARRAY_BUFFER, 0);
+
+            // Create stamp FBO
+            m_glF->glGenFramebuffers(1, &m_stampFbo);
         }
 
         clear();
@@ -268,14 +274,12 @@ void PenLayer::stamp(IRenderedTarget *target)
     m_painter->endFrame();
 
     // Create a FBO for the current texture
-    unsigned int fbo;
-    m_glF->glGenFramebuffers(1, &fbo);
-    m_glF->glBindFramebuffer(GL_FRAMEBUFFER, fbo);
+    m_glF->glBindFramebuffer(GL_FRAMEBUFFER, m_stampFbo);
     m_glF->glFramebufferTexture2D(GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D, texture.handle(), 0);
 
     if (m_glF->glCheckFramebufferStatus(GL_FRAMEBUFFER) != GL_FRAMEBUFFER_COMPLETE) {
         qWarning() << "error: framebuffer incomplete (stamp " + target->scratchTarget()->name() + ")";
-        m_glF->glDeleteFramebuffers(1, &fbo);
+        m_glF->glBindFramebuffer(GL_FRAMEBUFFER, 0);
         return;
     }
 
@@ -308,7 +312,6 @@ void PenLayer::stamp(IRenderedTarget *target)
     m_glF->glBindVertexArray(0);
     m_glF->glBindBuffer(GL_ARRAY_BUFFER, 0);
     m_glF->glBindFramebuffer(GL_FRAMEBUFFER, 0);
-    m_glF->glDeleteFramebuffers(1, &fbo);
 
     m_glF->glEnable(GL_SCISSOR_TEST);
 

src/penlayer.h

@@ -60,6 +60,7 @@ class PenLayer : public IPenLayer
         void updateTexture();
 
         static std::unordered_map<libscratchcpp::IEngine *, IPenLayer *> m_projectPenLayers;
+        static inline GLuint m_stampFbo = 0;
         bool m_antialiasingEnabled = true;
         libscratchcpp::IEngine *m_engine = nullptr;
         bool m_hqPen = false;