Fix and improve FBO handling

FBO need to be rebuild when one of the attachments directly or indirectly
changes. By direct change we mean one of the FBO attachment texture being
resized or the list of attachments changing. By indirect we mean when
texture resource is recreated internally by the engine.

Failure to handle this cases resulted in FBO referencing invalid
attachments.

Change-Id: I8dd4c08e464eed7fb0eeefd61a4158304ab4245f
Task-number: QTBUG-64757
Reviewed-by: Mike Krus <mike.krus@kdab.com>

1 files changed, 26 insertions, 0 deletions

diff --git a/src/render/backend/attachmentpack.cpp b/src/render/backend/attachmentpack.cpp
index a2ac8c30c..0cd5d8673 100644
--- a/src/render/backend/attachmentpack.cpp
+++ b/src/render/backend/attachmentpack.cpp
@@ -88,6 +88,32 @@ int AttachmentPack::getDrawBufferIndex(QRenderTargetOutput::AttachmentPoint atta
     return -1;
 }
 
+bool operator ==(const Attachment &a, const Attachment &b)
+{
+    return (a.m_name == b.m_name &&
+            a.m_mipLevel == b.m_mipLevel &&
+            a.m_layer == b.m_layer &&
+            a.m_textureUuid == b.m_textureUuid &&
+            a.m_point == b.m_point &&
+            a.m_face == b.m_face);
+}
+
+bool operator !=(const Attachment &a, const Attachment &b)
+{
+    return !(a == b);
+}
+
+bool operator ==(const AttachmentPack &packA, const AttachmentPack &packB)
+{
+    return (packA.attachments() == packB.attachments() &&
+            packA.getGlDrawBuffers() == packB.getGlDrawBuffers());
+}
+
+bool operator !=(const AttachmentPack &packA, const AttachmentPack &packB)
+{
+    return !(packA == packB);
+}
+
 } // namespace Render
 } // namespace Qt3DRender