gl/eagl: Don't make context uncurrent just to make it current again in the next line
authorSebastian Dröge <sebastian@centricular.com>
Wed, 9 Sep 2015 06:27:17 +0000 (09:27 +0300)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 9 Dec 2017 19:32:08 +0000 (19:32 +0000)
Also binding the framebuffer again is unnecessary then as it was just bound a
few lines before while the context was current.

https://bugzilla.gnome.org/show_bug.cgi?id=754757

gst-libs/gst/gl/eagl/gstglcontext_eagl.m

index dce8168..7adcf58 100644 (file)
@@ -191,11 +191,7 @@ gst_gl_context_eagl_update_layer (GstGLContext * context)
       height);
   glFramebufferRenderbuffer (GL_FRAMEBUFFER, GL_DEPTH_ATTACHMENT,
       GL_RENDERBUFFER, depth_renderbuffer);
-  [EAGLContext setCurrentContext:nil];
 
-  [EAGLContext setCurrentContext:priv->eagl_context];
-
-  glBindFramebuffer (GL_FRAMEBUFFER, framebuffer);
   /* check creation status */
   status = glCheckFramebufferStatus (GL_FRAMEBUFFER);
   if (status != GL_FRAMEBUFFER_COMPLETE) {