From 4a4e4927851b7e4e9ab54ea299aceecc31bc130a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Wed, 12 Jun 2013 19:56:37 +0200 Subject: [PATCH] No need to have virtual destructor in AbstractFramebuffer. --- src/AbstractFramebuffer.cpp | 2 -- src/AbstractFramebuffer.h | 3 ++- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/AbstractFramebuffer.cpp b/src/AbstractFramebuffer.cpp index d64ea4b93..c410aac91 100644 --- a/src/AbstractFramebuffer.cpp +++ b/src/AbstractFramebuffer.cpp @@ -47,8 +47,6 @@ FramebufferTarget AbstractFramebuffer::readTarget = FramebufferTarget::ReadDraw; FramebufferTarget AbstractFramebuffer::drawTarget = FramebufferTarget::ReadDraw; #endif -AbstractFramebuffer::~AbstractFramebuffer() {} - void AbstractFramebuffer::bind(FramebufferTarget target) { bindInternal(target); setViewportInternal(); diff --git a/src/AbstractFramebuffer.h b/src/AbstractFramebuffer.h index 7472d4bbd..606ae8593 100644 --- a/src/AbstractFramebuffer.h +++ b/src/AbstractFramebuffer.h @@ -190,7 +190,6 @@ class MAGNUM_EXPORT AbstractFramebuffer { } explicit AbstractFramebuffer() = default; - virtual ~AbstractFramebuffer() = 0; /** * @brief Bind framebuffer for rendering @@ -269,6 +268,8 @@ class MAGNUM_EXPORT AbstractFramebuffer { #else protected: #endif + ~AbstractFramebuffer() = default; + void MAGNUM_LOCAL bindInternal(FramebufferTarget target); FramebufferTarget MAGNUM_LOCAL bindInternal(); void MAGNUM_LOCAL setViewportInternal();