From 219417e0fdd84629526d1ee2cd54d7472c6753ec Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Mon, 5 Jun 2017 21:59:34 +0200 Subject: [PATCH] Fix framebuffer attachments on ES2 w/o separate read/draw bindings. Thanks to @julee for pointing this out. --- src/Magnum/AbstractFramebuffer.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/Magnum/AbstractFramebuffer.cpp b/src/Magnum/AbstractFramebuffer.cpp index 908555c40..6fd4e5656 100644 --- a/src/Magnum/AbstractFramebuffer.cpp +++ b/src/Magnum/AbstractFramebuffer.cpp @@ -183,7 +183,11 @@ FramebufferTarget AbstractFramebuffer::bindImplementationSingle() { glBindFramebuffer(GL_FRAMEBUFFER, _id); } - return FramebufferTarget{}; + /* On ES2 w/o separate read/draw bindings the return value is used as a + first parameter to glFramebufferRenderbuffer() etc. and so it needs to + be unconditionally GL_FRAMEBUFFER. That value is not part of the public + enum, though. */ + return FramebufferTarget(GL_FRAMEBUFFER); } #endif