opengl: fix possible typo
authorSteve Lhomme <robux4@videolabs.io>
Tue, 15 Nov 2016 09:53:49 +0000 (10:53 +0100)
committerThomas Guillem <thomas@gllm.fr>
Wed, 16 Nov 2016 11:09:21 +0000 (12:09 +0100)
There isn't a case where program[0] is 0 and program[1] is 0. They are always
created together. So the second part of the if() is never called.

program[0] is for YUV and XYZ sources so it is odd to use it with a single
planar texture, given that's what program[1] is for.

Signed-off-by: Thomas Guillem <thomas@gllm.fr>
modules/video_output/opengl.c

index 5cc64a6..75e4a82 100644 (file)
@@ -1689,7 +1689,7 @@ int vout_display_opengl_Display(vout_display_opengl_t *vgl,
     }
 
 #ifdef SUPPORTS_SHADERS
-    if (vgl->program[0] && (vgl->chroma->plane_count == 3 || vgl->chroma->plane_count == 1))
+    if (vgl->program[0] && (vgl->chroma->plane_count == 3 || vgl->chroma->plane_count == 2))
         DrawWithShaders(vgl, left, top, right, bottom, 0);
     else if (vgl->program[1] && vgl->chroma->plane_count == 1)
         DrawWithShaders(vgl, left, top, right, bottom, 1);