Update shaders if buffer size has changed

KOPRajs · Apr 11, 2024 · 0b3f523 · 0b3f523
1 parent e8ff7c7
commit 0b3f523
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 9 deletions.
diff --git a/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPBaseRenderer.cpp b/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPBaseRenderer.cpp
@@ -27,7 +27,7 @@ CRPBaseRenderer::CRPBaseRenderer(const CRenderSettings& renderSettings,
   : m_context(context),
     m_bufferPool(std::move(bufferPool)),
     m_renderSettings(renderSettings),
-    m_shadersNeedUpdate(true),
+    m_bShadersNeedUpdate(true),
     m_bUseShaderPreset(false)
 {
   m_bufferPool->RegisterRenderer(this);
@@ -106,6 +106,13 @@ void CRPBaseRenderer::SetBuffer(IRenderBuffer* buffer)
 {
   if (m_renderBuffer != buffer)
   {
+    // Check if buffer size has changed
+    if (m_renderBuffer != nullptr && buffer != nullptr) {
+      if (m_renderBuffer->GetWidth() != buffer->GetWidth() || m_renderBuffer->GetHeight() != buffer->GetHeight()) {
+        m_bShadersNeedUpdate = true;
+      }
+    }
+
     if (m_renderBuffer != nullptr)
       m_renderBuffer->Release();
 
@@ -157,7 +164,7 @@ void CRPBaseRenderer::SetShaderPreset(const std::string& presetPath)
   if (presetPath != m_renderSettings.VideoSettings().GetShaderPreset())
   {
     m_renderSettings.VideoSettings().SetShaderPreset(presetPath);
-    m_shadersNeedUpdate = true;
+    m_bShadersNeedUpdate = true;
   }
 }
 
@@ -245,7 +252,7 @@ void CRPBaseRenderer::MarkDirty()
  */
 void CRPBaseRenderer::Updateshaders()
 {
-  if (m_shadersNeedUpdate)
+  if (m_bShadersNeedUpdate)
   {
     if (m_shaderPreset)
     {
@@ -263,7 +270,7 @@ void CRPBaseRenderer::Updateshaders()
       m_bUseShaderPreset =
           m_shaderPreset->SetShaderPreset(m_renderSettings.VideoSettings().GetShaderPreset());
     }
-    m_shadersNeedUpdate = false;
+    m_bShadersNeedUpdate = false;
   }
 }
 

diff --git a/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPBaseRenderer.h b/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPBaseRenderer.h
@@ -105,7 +105,7 @@ class CRPBaseRenderer
   void Updateshaders();
   std::unique_ptr<SHADER::IShaderPreset> m_shaderPreset;
 
-  bool m_shadersNeedUpdate;
+  bool m_bShadersNeedUpdate;
   bool m_bUseShaderPreset;
 
 private:

diff --git a/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPRendererDMA.cpp b/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPRendererDMA.cpp
@@ -116,7 +116,7 @@ void CRPRendererDMA::Render(uint8_t alpha)
     SHADER::CShaderTextureGL target(*targetTexture);
     if (!m_shaderPreset->RenderUpdate(destPoints, &source, &target))
     {
-      m_shadersNeedUpdate = false;
+      m_bShadersNeedUpdate = false;
       m_bUseShaderPreset = false;
     }
   }

diff --git a/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPRendererOpenGL.cpp b/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPRendererOpenGL.cpp
@@ -335,7 +335,7 @@ void CRPRendererOpenGL::Render(uint8_t alpha)
     SHADER::CShaderTextureGL target(*targetTexture);
     if (!m_shaderPreset->RenderUpdate(destPoints, &source, &target))
     {
-      m_shadersNeedUpdate = false;
+      m_bShadersNeedUpdate = false;
       m_bUseShaderPreset = false;
     }
   }

diff --git a/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPRendererOpenGLES.cpp b/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPRendererOpenGLES.cpp
@@ -302,7 +302,7 @@ void CRPRendererOpenGLES::Render(uint8_t alpha)
     SHADER::CShaderTextureGL target(*targetTexture);
     if (!m_shaderPreset->RenderUpdate(destPoints, &source, &target))
     {
-      m_shadersNeedUpdate = false;
+      m_bShadersNeedUpdate = false;
       m_bUseShaderPreset = false;
     }
   }

diff --git a/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPWinRenderer.cpp b/xbmc/cores/RetroPlayer/rendering/VideoRenderers/RPWinRenderer.cpp
@@ -328,7 +328,7 @@ void CRPWinRenderer::Render(CD3DTexture& target)
     m_targetTexture.SetTexture(target);
     if (!m_shaderPreset->RenderUpdate(destPoints, renderBufferTarget, &m_targetTexture))
     {
-      m_shadersNeedUpdate = false;
+      m_bShadersNeedUpdate = false;
       m_bUseShaderPreset = false;
     }
   }