BlurCache: revert back to unique_ptrs

Signed-off-by: Xarblu <xarblu@protonmail.com>

Author: xarblu

src/blur.h

@@ -54,8 +54,8 @@ struct BlurRenderData
 {
     /// Temporary render targets needed for the Dual Kawase algorithm, the first texture
     /// contains not blurred background behind the window, it's cached.
-    std::vector<std::shared_ptr<GLTexture>> textures;
-    std::vector<std::shared_ptr<GLFramebuffer>> framebuffers;
+    std::vector<std::unique_ptr<GLTexture>> textures;
+    std::vector<std::unique_ptr<GLFramebuffer>> framebuffers;
 
     BBDX::BlurCacheLRU cache;
 };

src/blur_cache.cpp

@@ -341,17 +341,16 @@ void BBDX::BlurCache::prepareCache(BBDX::BlurCacheLRU &cache,
     // QUERY START
 
     // textures + FBOs used in query
-    auto cachedTextureFBO = std::pair{cacheEntry->blitTexture, cacheEntry->blitFramebuffer};
-    auto newTextureFBO = std::pair{m_paintData.blitFramebuffer->colorAttachment(), m_paintData.blitFramebuffer};
-    auto &[cachedTexture, cachedFramebuffer] = cachedTextureFBO;
-    auto &[newTexture, newFramebuffer] = newTextureFBO;
+    const auto cachedTexture = cacheEntry->blitTexture.get();
+    const auto cachedFramebuffer = cacheEntry->blitFramebuffer.get();
+    const auto newTexture = m_paintData.blitFramebuffer->colorAttachment();
 
     // check if textures differ on the pixel level
     KWin::ShaderManager::instance()->pushShader(m_textureComparePass.shader.get());
 
     // Use FBO of the cached blit; the query's draw will also
     // update this with pixels from the new blit (if it differs)
-    KWin::GLFramebuffer::pushFramebuffer(cachedFramebuffer.get());
+    KWin::GLFramebuffer::pushFramebuffer(cachedFramebuffer);
 
     QMatrix4x4 projectionMatrix;
     projectionMatrix.ortho(QRectF(0.0, 0.0, newTexture->width(), newTexture->height()));

src/blur_cache.hpp

@@ -39,10 +39,8 @@ struct BlurCacheEntry {
 
     // texture of the previous "raw" pixels (blit grabbed from scene)
     // used to create this cache entry
-    // Uses shared pointers because the validation query also gets
-    // a handle to these
-    std::shared_ptr<KWin::GLTexture> blitTexture{nullptr};
-    std::shared_ptr<KWin::GLFramebuffer> blitFramebuffer{nullptr};
+    std::unique_ptr<KWin::GLTexture> blitTexture{nullptr};
+    std::unique_ptr<KWin::GLFramebuffer> blitFramebuffer{nullptr};
 
     // backgroundRect used to create this cache entry
     KWin::Rect backgroundRect{};