From 0517dd5c58ea8c8bc87bb4026b632c47013f3ab6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= <mosra@centrum.cz>
Date: Thu, 7 Oct 2021 14:37:37 +0200
Subject: [PATCH] GL: no need to set _viewport twice in Framebuffer
 constructor.

That line was unchanged since 2012, so I don't think it carried some
meaning that escaped my attention.
---
 src/Magnum/GL/Framebuffer.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/Magnum/GL/Framebuffer.cpp b/src/Magnum/GL/Framebuffer.cpp
index eef4bd57b..160383116 100644
--- a/src/Magnum/GL/Framebuffer.cpp
+++ b/src/Magnum/GL/Framebuffer.cpp
@@ -99,7 +99,6 @@ Int Framebuffer::maxColorAttachments() {
 
 Framebuffer::Framebuffer(const Range2Di& viewport): AbstractFramebuffer{0, viewport, ObjectFlag::DeleteOnDestruction} {
     CORRADE_INTERNAL_ASSERT(viewport != Implementation::FramebufferState::DisengagedViewport);
-    _viewport = viewport;
     (this->*Context::current().state().framebuffer.createImplementation)();
     CORRADE_INTERNAL_ASSERT(_id != Implementation::State::DisengagedBinding);
 }