sid
/
NazaraEngine
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Renderer/RenderTexture: Fix and inline 

Former-commit-id: 4bf8ad8acd3b978c4665df9547328333b9f9ad42
This commit is contained in:
Lynix 2015-06-21 13:18:37 +02:00
parent 54b0339025
commit 9318d269e2
3 changed files with 125 additions and 177 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
include/Nazara/Renderer/RenderTexture.hpp
View File

@ -26,8 +26,8 @@ struct NzRenderTextureImpl;
class NAZARA_RENDERER_API NzRenderTexture : public NzRenderTarget, NzNonCopyable class NAZARA_RENDERER_API NzRenderTexture : public NzRenderTarget, NzNonCopyable
{ {
public: public:
NzRenderTexture() = default; inline NzRenderTexture();
~NzRenderTexture(); inline ~NzRenderTexture();
bool AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 index, NzRenderBuffer* buffer); bool AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 index, NzRenderBuffer* buffer);
bool AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 index, nzPixelFormat format, unsigned int width, unsigned int height); bool AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 index, nzPixelFormat format, unsigned int width, unsigned int height);
@ -38,18 +38,18 @@ class NAZARA_RENDERER_API NzRenderTexture : public NzRenderTarget, NzNonCopyable
void Detach(nzAttachmentPoint attachmentPoint, nzUInt8 index); void Detach(nzAttachmentPoint attachmentPoint, nzUInt8 index);
unsigned int GetHeight() const; unsigned int GetHeight() const override;
NzRenderTargetParameters GetParameters() const; NzRenderTargetParameters GetParameters() const;
NzVector2ui GetSize() const; NzVector2ui GetSize() const;
unsigned int GetWidth() const; unsigned int GetWidth() const override;
bool IsComplete() const; bool IsComplete() const;
bool IsRenderable() const; bool IsRenderable() const;
bool IsValid() const; inline bool IsValid() const;
bool Lock() const; bool Lock() const;
void SetColorTarget(nzUInt8 target) const; inline void SetColorTarget(nzUInt8 target) const;
void SetColorTargets(const nzUInt8* targets, unsigned int targetCount) const; void SetColorTargets(const nzUInt8* targets, unsigned int targetCount) const;
void SetColorTargets(const std::initializer_list<nzUInt8>& targets) const; void SetColorTargets(const std::initializer_list<nzUInt8>& targets) const;
@ -59,7 +59,7 @@ class NAZARA_RENDERER_API NzRenderTexture : public NzRenderTarget, NzNonCopyable
unsigned int GetOpenGLID() const; unsigned int GetOpenGLID() const;
bool HasContext() const override; bool HasContext() const override;
static void Blit(NzRenderTexture* src, NzRenderTexture* dst, nzUInt32 buffers = nzRendererBuffer_Color | nzRendererBuffer_Depth | nzRendererBuffer_Stencil, bool bilinearFilter = false); static inline void Blit(NzRenderTexture* src, NzRenderTexture* dst, nzUInt32 buffers = nzRendererBuffer_Color | nzRendererBuffer_Depth | nzRendererBuffer_Stencil, bool bilinearFilter = false);
static void Blit(NzRenderTexture* src, NzRectui srcRect, NzRenderTexture* dst, NzRectui dstRect, nzUInt32 buffers = nzRendererBuffer_Color | nzRendererBuffer_Depth | nzRendererBuffer_Stencil, bool bilinearFilter = false); static void Blit(NzRenderTexture* src, NzRectui srcRect, NzRenderTexture* dst, NzRectui dstRect, nzUInt32 buffers = nzRendererBuffer_Color | nzRendererBuffer_Depth | nzRendererBuffer_Stencil, bool bilinearFilter = false);
static bool IsSupported(); static bool IsSupported();
@ -69,6 +69,9 @@ class NAZARA_RENDERER_API NzRenderTexture : public NzRenderTarget, NzNonCopyable
void EnsureTargetUpdated() const override; void EnsureTargetUpdated() const override;
private: private:
inline void InvalidateDrawBuffers() const;
inline void InvalidateSize() const;
inline void InvalidateTargets() const;
void OnContextDestroy(const NzContext* context); void OnContextDestroy(const NzContext* context);
void OnRenderBufferDestroy(const NzRenderBuffer* renderBuffer, unsigned int attachmentIndex); void OnRenderBufferDestroy(const NzRenderBuffer* renderBuffer, unsigned int attachmentIndex);
void OnTextureDestroy(const NzTexture* texture, unsigned int attachmentIndex); void OnTextureDestroy(const NzTexture* texture, unsigned int attachmentIndex);
@ -76,7 +79,13 @@ class NAZARA_RENDERER_API NzRenderTexture : public NzRenderTarget, NzNonCopyable
void UpdateSize() const; void UpdateSize() const;
void UpdateTargets() const; void UpdateTargets() const;
NzRenderTextureImpl* m_impl = nullptr; NzRenderTextureImpl* m_impl;
mutable bool m_checked ;
mutable bool m_drawBuffersUpdated;
mutable bool m_sizeUpdated;
mutable bool m_targetsUpdated;
}; };
#include <Nazara/Renderer/RenderTexture.inl>
#endif // NAZARA_RENDERTEXTURE_HPP #endif // NAZARA_RENDERTEXTURE_HPP

53
include/Nazara/Renderer/RenderTexture.inl Normal file
View File

@ -0,0 +1,53 @@
// Copyright (C) 2015 Jérôme Leclercq
// This file is part of the "Nazara Engine - Renderer module"
// For conditions of distribution and use, see copyright notice in Config.hpp
#include <Nazara/Core/Error.hpp>
#include <cstring>
#include <Nazara/Renderer/Debug.hpp>
inline NzRenderTexture::NzRenderTexture() :
m_impl(nullptr)
{
}
inline NzRenderTexture::~NzRenderTexture()
{
Destroy();
}
inline bool NzRenderTexture::IsValid() const
{
return m_impl != nullptr;
}
inline void NzRenderTexture::SetColorTarget(nzUInt8 target) const
{
SetColorTargets(&target, 1);
}
inline void NzRenderTexture::Blit(NzRenderTexture* src, NzRenderTexture* dst, nzUInt32 buffers, bool bilinearFilter)
{
Blit(src, src->GetSize(), dst, dst->GetSize(), buffers, bilinearFilter);
}
inline void NzRenderTexture::InvalidateDrawBuffers() const
{
m_drawBuffersUpdated = false;
}
inline void NzRenderTexture::InvalidateSize() const
{
m_sizeUpdated = false;
OnRenderTargetSizeChange(this);
}
inline void NzRenderTexture::InvalidateTargets() const
{
m_checked = false;
m_drawBuffersUpdated = false;
m_targetsUpdated = false;
}
#include <Nazara/Renderer/DebugOff.hpp>

224
src/Nazara/Renderer/RenderTexture.cpp
View File

@ -63,21 +63,12 @@ struct NzRenderTextureImpl
std::vector<nzUInt8> colorTargets; std::vector<nzUInt8> colorTargets;
mutable std::vector<GLenum> drawBuffers; mutable std::vector<GLenum> drawBuffers;
const NzContext* context; const NzContext* context;
bool checked = false;
bool complete = false; bool complete = false;
bool userDefinedTargets = false; bool userDefinedTargets = false;
mutable bool drawBuffersUpdated = true;
mutable bool sizeUpdated = false;
mutable bool targetsUpdated = true;
unsigned int height; unsigned int height;
unsigned int width; unsigned int width;
}; };
NzRenderTexture::~NzRenderTexture()
{
Destroy();
}
bool NzRenderTexture::AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 index, NzRenderBuffer* buffer) bool NzRenderTexture::AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 index, NzRenderBuffer* buffer)
{ {
#if NAZARA_RENDERER_SAFE #if NAZARA_RENDERER_SAFE
@ -148,6 +139,9 @@ bool NzRenderTexture::AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 in
Unlock(); Unlock();
unsigned int attachIndex = attachmentIndex[attachmentPoint] + index; unsigned int attachIndex = attachmentIndex[attachmentPoint] + index;
if (attachIndex >= m_impl->attachments.size())
m_impl->attachments.resize(attachIndex+1);
Attachment& attachment = m_impl->attachments[attachIndex]; Attachment& attachment = m_impl->attachments[attachIndex];
attachment.attachmentPoint = attachmentPoint; attachment.attachmentPoint = attachmentPoint;
attachment.buffer = buffer; attachment.buffer = buffer;
@ -157,15 +151,11 @@ bool NzRenderTexture::AttachBuffer(nzAttachmentPoint attachmentPoint, nzUInt8 in
attachment.height = buffer->GetHeight(); attachment.height = buffer->GetHeight();
attachment.width = buffer->GetWidth(); attachment.width = buffer->GetWidth();
m_impl->checked = false; InvalidateSize();
m_impl->sizeUpdated = false; InvalidateTargets();
if (attachmentPoint == nzAttachmentPoint_Color && !m_impl->userDefinedTargets) if (attachmentPoint == nzAttachmentPoint_Color && !m_impl->userDefinedTargets)
{
m_impl->colorTargets.push_back(index); m_impl->colorTargets.push_back(index);
m_impl->drawBuffersUpdated = false;
m_impl->targetsUpdated = false;
}
return true; return true;
} }
@ -286,6 +276,9 @@ bool NzRenderTexture::AttachTexture(nzAttachmentPoint attachmentPoint, nzUInt8 i
Unlock(); Unlock();
unsigned int attachIndex = attachmentIndex[attachmentPoint] + index; unsigned int attachIndex = attachmentIndex[attachmentPoint] + index;
if (attachIndex >= m_impl->attachments.size())
m_impl->attachments.resize(attachIndex+1);
Attachment& attachment = m_impl->attachments[attachIndex]; Attachment& attachment = m_impl->attachments[attachIndex];
attachment.attachmentPoint = attachmentPoint; attachment.attachmentPoint = attachmentPoint;
attachment.isBuffer = false; attachment.isBuffer = false;
@ -295,15 +288,11 @@ bool NzRenderTexture::AttachTexture(nzAttachmentPoint attachmentPoint, nzUInt8 i
attachment.textureDestroySlot.Connect(texture->OnTextureDestroy, std::bind(&NzRenderTexture::OnTextureDestroy, this, std::placeholders::_1, attachIndex)); attachment.textureDestroySlot.Connect(texture->OnTextureDestroy, std::bind(&NzRenderTexture::OnTextureDestroy, this, std::placeholders::_1, attachIndex));
attachment.width = texture->GetWidth(); attachment.width = texture->GetWidth();
m_impl->checked = false; InvalidateSize();
m_impl->sizeUpdated = false; InvalidateTargets();
if (attachmentPoint == nzAttachmentPoint_Color && !m_impl->userDefinedTargets) if (attachmentPoint == nzAttachmentPoint_Color && !m_impl->userDefinedTargets)
{
m_impl->colorTargets.push_back(index); m_impl->colorTargets.push_back(index);
m_impl->drawBuffersUpdated = false;
m_impl->targetsUpdated = false;
}
return true; return true;
} }
@ -341,6 +330,11 @@ bool NzRenderTexture::Create(bool lock)
m_impl->context = NzContext::GetCurrent(); m_impl->context = NzContext::GetCurrent();
m_impl->contextDestroySlot.Connect(m_impl->context->OnContextDestroy, this, &NzRenderTexture::OnContextDestroy); m_impl->contextDestroySlot.Connect(m_impl->context->OnContextDestroy, this, &NzRenderTexture::OnContextDestroy);
m_checked = false;
m_drawBuffersUpdated = true;
m_sizeUpdated = false;
m_targetsUpdated = true;
if (lock) if (lock)
{ {
// En cas d'exception, la ressource sera quand même libérée // En cas d'exception, la ressource sera quand même libérée
@ -430,30 +424,21 @@ void NzRenderTexture::Detach(nzAttachmentPoint attachmentPoint, nzUInt8 index)
attachement.textureDestroySlot.Disconnect(); attachement.textureDestroySlot.Disconnect();
} }
m_impl->sizeUpdated = false; InvalidateSize();
if (attachement.attachmentPoint == nzAttachmentPoint_Color) if (attachement.attachmentPoint == nzAttachmentPoint_Color)
{ InvalidateTargets();
m_impl->drawBuffersUpdated = false;
m_impl->targetsUpdated = false;
}
Unlock(); Unlock();
m_impl->checked = false; m_checked = false;
} }
unsigned int NzRenderTexture::GetHeight() const unsigned int NzRenderTexture::GetHeight() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return 0;
}
#endif
if (!m_impl->sizeUpdated) if (!m_sizeUpdated)
UpdateSize(); UpdateSize();
return m_impl->height; return m_impl->height;
@ -461,13 +446,7 @@ unsigned int NzRenderTexture::GetHeight() const
NzRenderTargetParameters NzRenderTexture::GetParameters() const NzRenderTargetParameters NzRenderTexture::GetParameters() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return NzRenderTargetParameters();
}
#endif
///TODO ///TODO
return NzRenderTargetParameters(); return NzRenderTargetParameters();
@ -475,15 +454,9 @@ NzRenderTargetParameters NzRenderTexture::GetParameters() const
NzVector2ui NzRenderTexture::GetSize() const NzVector2ui NzRenderTexture::GetSize() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return 0;
}
#endif
if (!m_impl->sizeUpdated) if (!m_sizeUpdated)
UpdateSize(); UpdateSize();
return NzVector2ui(m_impl->width, m_impl->height); return NzVector2ui(m_impl->width, m_impl->height);
@ -491,15 +464,9 @@ NzVector2ui NzRenderTexture::GetSize() const
unsigned int NzRenderTexture::GetWidth() const unsigned int NzRenderTexture::GetWidth() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return 0;
}
#endif
if (!m_impl->sizeUpdated) if (!m_sizeUpdated)
UpdateSize(); UpdateSize();
return m_impl->width; return m_impl->width;
@ -507,15 +474,9 @@ unsigned int NzRenderTexture::GetWidth() const
bool NzRenderTexture::IsComplete() const bool NzRenderTexture::IsComplete() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return false;
}
#endif
if (!m_impl->checked) if (!m_checked)
{ {
if (!Lock()) if (!Lock())
{ {
@ -566,7 +527,7 @@ bool NzRenderTexture::IsComplete() const
NazaraInternalError("Unknown error"); NazaraInternalError("Unknown error");
} }
m_impl->checked = true; m_checked = true;
} }
return m_impl->complete; return m_impl->complete;
@ -577,20 +538,11 @@ bool NzRenderTexture::IsRenderable() const
return IsComplete() && !m_impl->attachments.empty(); return IsComplete() && !m_impl->attachments.empty();
} }
bool NzRenderTexture::IsValid() const
{
return m_impl != nullptr;
}
bool NzRenderTexture::Lock() const bool NzRenderTexture::Lock() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return false;
}
#if NAZARA_RENDERER_SAFE
if (NzContext::GetCurrent() != m_impl->context) if (NzContext::GetCurrent() != m_impl->context)
{ {
NazaraError("RenderTexture cannot be used with this context"); NazaraError("RenderTexture cannot be used with this context");
@ -612,20 +564,11 @@ bool NzRenderTexture::Lock() const
return true; return true;
} }
void NzRenderTexture::SetColorTarget(nzUInt8 target) const
{
SetColorTargets(&target, 1);
}
void NzRenderTexture::SetColorTargets(const nzUInt8* targets, unsigned int targetCount) const void NzRenderTexture::SetColorTargets(const nzUInt8* targets, unsigned int targetCount) const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return;
}
#if NAZARA_RENDERER_SAFE
for (unsigned int i = 0; i < targetCount; ++i) for (unsigned int i = 0; i < targetCount; ++i)
{ {
unsigned int index = attachmentIndex[nzAttachmentPoint_Color] + targets[i]; unsigned int index = attachmentIndex[nzAttachmentPoint_Color] + targets[i];
@ -640,20 +583,15 @@ void NzRenderTexture::SetColorTargets(const nzUInt8* targets, unsigned int targe
m_impl->colorTargets.resize(targetCount); m_impl->colorTargets.resize(targetCount);
std::memcpy(&m_impl->colorTargets[0], targets, targetCount*sizeof(nzUInt8)); std::memcpy(&m_impl->colorTargets[0], targets, targetCount*sizeof(nzUInt8));
m_impl->drawBuffersUpdated = false;
m_impl->targetsUpdated = false;
m_impl->userDefinedTargets = true; m_impl->userDefinedTargets = true;
InvalidateDrawBuffers();
} }
void NzRenderTexture::SetColorTargets(const std::initializer_list<nzUInt8>& targets) const void NzRenderTexture::SetColorTargets(const std::initializer_list<nzUInt8>& targets) const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return;
}
#if NAZARA_RENDERER_SAFE
for (nzUInt8 target : targets) for (nzUInt8 target : targets)
{ {
unsigned int index = attachmentIndex[nzAttachmentPoint_Color] + target; unsigned int index = attachmentIndex[nzAttachmentPoint_Color] + target;
@ -671,20 +609,15 @@ void NzRenderTexture::SetColorTargets(const std::initializer_list<nzUInt8>& targ
for (nzUInt8 index : targets) for (nzUInt8 index : targets)
*ptr++ = index; *ptr++ = index;
m_impl->drawBuffersUpdated = false;
m_impl->targetsUpdated = false;
m_impl->userDefinedTargets = true; m_impl->userDefinedTargets = true;
InvalidateDrawBuffers();
} }
void NzRenderTexture::Unlock() const void NzRenderTexture::Unlock() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return;
}
#if NAZARA_RENDERER_SAFE
if (NzContext::GetCurrent() != m_impl->context) if (NzContext::GetCurrent() != m_impl->context)
{ {
NazaraError("RenderTexture cannot be used with this context"); NazaraError("RenderTexture cannot be used with this context");
@ -704,13 +637,9 @@ void NzRenderTexture::Unlock() const
unsigned int NzRenderTexture::GetOpenGLID() const unsigned int NzRenderTexture::GetOpenGLID() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return 0;
}
#if NAZARA_RENDERER_SAFE
if (NzContext::GetCurrent() != m_impl->context) if (NzContext::GetCurrent() != m_impl->context)
{ {
NazaraError("RenderTexture cannot be used with this context"); NazaraError("RenderTexture cannot be used with this context");
@ -731,46 +660,18 @@ bool NzRenderTexture::IsSupported()
return NzOpenGL::IsSupported(nzOpenGLExtension_FrameBufferObject); return NzOpenGL::IsSupported(nzOpenGLExtension_FrameBufferObject);
} }
void NzRenderTexture::Blit(NzRenderTexture* src, NzRenderTexture* dst, nzUInt32 buffers, bool bilinearFilter)
{
#if NAZARA_RENDERER_SAFE
if (!src)
{
NazaraError("Invalid source render texture");
return;
}
if (!dst)
{
NazaraError("Invalid source render texture");
return;
}
#endif
Blit(src, src->GetSize(), dst, dst->GetSize(), buffers, bilinearFilter);
}
void NzRenderTexture::Blit(NzRenderTexture* src, NzRectui srcRect, NzRenderTexture* dst, NzRectui dstRect, nzUInt32 buffers, bool bilinearFilter) void NzRenderTexture::Blit(NzRenderTexture* src, NzRectui srcRect, NzRenderTexture* dst, NzRectui dstRect, nzUInt32 buffers, bool bilinearFilter)
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(src && src->IsValid(), "Invalid source render texture");
if (!src || !src->IsValid()) NazaraAssert(dst && dst->IsValid(), "Invalid destination render texture");
{
NazaraError("Invalid source render texture");
return;
}
#if NAZARA_RENDERER_SAFE
if (srcRect.x+srcRect.width > src->GetWidth() || srcRect.y+srcRect.height > src->GetHeight()) if (srcRect.x+srcRect.width > src->GetWidth() || srcRect.y+srcRect.height > src->GetHeight())
{ {
NazaraError("Source rectangle dimensions are out of bounds"); NazaraError("Source rectangle dimensions are out of bounds");
return; return;
} }
if (!dst || !dst->IsValid())
{
NazaraError("Invalid source render texture");
return;
}
if (dstRect.x+dstRect.width > dst->GetWidth() || dstRect.y+dstRect.height > dst->GetHeight()) if (dstRect.x+dstRect.width > dst->GetWidth() || dstRect.y+dstRect.height > dst->GetHeight())
{ {
NazaraError("Destination rectangle dimensions are out of bounds"); NazaraError("Destination rectangle dimensions are out of bounds");
@ -811,13 +712,9 @@ void NzRenderTexture::Blit(NzRenderTexture* src, NzRectui srcRect, NzRenderTextu
bool NzRenderTexture::Activate() const bool NzRenderTexture::Activate() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return false;
}
#if NAZARA_RENDERER_SAFE
if (NzContext::GetCurrent() != m_impl->context) if (NzContext::GetCurrent() != m_impl->context)
{ {
NazaraError("RenderTexture cannot be used with this context"); NazaraError("RenderTexture cannot be used with this context");
@ -827,20 +724,16 @@ bool NzRenderTexture::Activate() const
glBindFramebuffer(GL_DRAW_FRAMEBUFFER, m_impl->fbo); glBindFramebuffer(GL_DRAW_FRAMEBUFFER, m_impl->fbo);
m_impl->drawBuffersUpdated = false; m_drawBuffersUpdated = false;
return true; return true;
} }
void NzRenderTexture::Desactivate() const void NzRenderTexture::Desactivate() const
{ {
#if NAZARA_RENDERER_SAFE NazaraAssert(m_impl, "Invalid render texture");
if (!m_impl)
{
NazaraError("Render texture not created");
return;
}
#if NAZARA_RENDERER_SAFE
if (NzContext::GetCurrent() != m_impl->context) if (NzContext::GetCurrent() != m_impl->context)
{ {
NazaraError("RenderTexture cannot be used with this context"); NazaraError("RenderTexture cannot be used with this context");
@ -855,7 +748,7 @@ void NzRenderTexture::Desactivate() const
void NzRenderTexture::EnsureTargetUpdated() const void NzRenderTexture::EnsureTargetUpdated() const
{ {
if (!m_impl->drawBuffersUpdated) if (!m_drawBuffersUpdated)
UpdateDrawBuffers(); UpdateDrawBuffers();
for (nzUInt8 index : m_impl->colorTargets) for (nzUInt8 index : m_impl->colorTargets)
@ -894,8 +787,7 @@ void NzRenderTexture::OnRenderBufferDestroy(const NzRenderBuffer* renderBuffer,
attachment.isUsed = false; attachment.isUsed = false;
attachment.renderBufferDestroySlot.Disconnect(); attachment.renderBufferDestroySlot.Disconnect();
m_impl->checked = false; InvalidateTargets();
m_impl->targetsUpdated = false;
} }
void NzRenderTexture::OnTextureDestroy(const NzTexture* texture, unsigned int attachmentIndex) void NzRenderTexture::OnTextureDestroy(const NzTexture* texture, unsigned int attachmentIndex)
@ -905,23 +797,17 @@ void NzRenderTexture::OnTextureDestroy(const NzTexture* texture, unsigned int at
NazaraAssert(!m_impl->attachments[attachmentIndex].isBuffer, "Invalid attachment state"); NazaraAssert(!m_impl->attachments[attachmentIndex].isBuffer, "Invalid attachment state");
NazaraUnused(texture); NazaraUnused(texture);
Attachment& attachment = m_impl->attachments[attachmentIndex]; InvalidateTargets();
attachment.isUsed = false;
attachment.texture = nullptr;
attachment.textureDestroySlot.Disconnect();
m_impl->checked = false;
m_impl->targetsUpdated = false;
} }
void NzRenderTexture::UpdateDrawBuffers() const void NzRenderTexture::UpdateDrawBuffers() const
{ {
if (!m_impl->targetsUpdated) if (!m_targetsUpdated)
UpdateTargets(); UpdateTargets();
glDrawBuffers(m_impl->drawBuffers.size(), &m_impl->drawBuffers[0]); glDrawBuffers(m_impl->drawBuffers.size(), &m_impl->drawBuffers[0]);
m_impl->drawBuffersUpdated = true; m_drawBuffersUpdated = true;
} }
void NzRenderTexture::UpdateSize() const void NzRenderTexture::UpdateSize() const
@ -937,7 +823,7 @@ void NzRenderTexture::UpdateSize() const
} }
} }
m_impl->sizeUpdated = true; m_sizeUpdated = true;
} }
void NzRenderTexture::UpdateTargets() const void NzRenderTexture::UpdateTargets() const
@ -955,5 +841,5 @@ void NzRenderTexture::UpdateTargets() const
*ptr++ = GL_COLOR_ATTACHMENT0 + index; *ptr++ = GL_COLOR_ATTACHMENT0 + index;
} }
m_impl->targetsUpdated = true; m_targetsUpdated = true;
} }