[vlc-commits] vout: ios: remove useless glFlush

Alexandre Janniaux git at videolan.org
Wed Jan 22 10:08:43 CET 2020


vlc | branch: master | Alexandre Janniaux <ajanni at videolabs.io> | Fri Dec  6 19:38:56 2019 +0100| [807341b2f149c713f842798809bc5bc23161c253] | committer: Thomas Guillem

vout: ios: remove useless glFlush

glFlush send batched command from the CPU-side client to the GPU and
thus creates more commands, while glFinish wait for all CPU-side client
commands to be executed, including the one needing to be flushed.

Also, glFlush doesn't block on the command queue so it doesn't matter
for the issue fixed in 1399ebae2e123cf3ba112946014b59ef34d639dd.

Signed-off-by: Thomas Guillem <thomas at gllm.fr>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=807341b2f149c713f842798809bc5bc23161c253
---

 modules/video_output/ios.m | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/modules/video_output/ios.m b/modules/video_output/ios.m
index 00d7dc5f29..695cde9f50 100644
--- a/modules/video_output/ios.m
+++ b/modules/video_output/ios.m
@@ -724,10 +724,7 @@ static void GLESSwap(vlc_gl_t *gl)
      * background.*/
     EAGLContext *previousEaglContext = [EAGLContext currentContext];
     if ([EAGLContext setCurrentContext:_eaglContext])
-    {
         glFinish();
-        glFlush();
-    }
     [EAGLContext setCurrentContext:previousEaglContext];
 }
 



More information about the vlc-commits mailing list