Merge lp:~alfonsosanchezbeato/qtvideo-node/mir-desktop-enablement into lp:qtvideo-node

Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Jim Hodapp
Approved revision: 73
Merged at revision: 72
Proposed branch: lp:~alfonsosanchezbeato/qtvideo-node/mir-desktop-enablement
Merge into: lp:qtvideo-node
Diff against target: 17 lines (+5/-1)
1 file modified
src/shadervideoshader.cpp (+5/-1)
To merge this branch: bzr merge lp:~alfonsosanchezbeato/qtvideo-node/mir-desktop-enablement
Reviewer Review Type Date Requested Status
Jim Hodapp (community) Approve
Review via email: mp+317179@code.launchpad.net

Commit message

Make shader work when GL_OES_EGL_image_external is not present.

Description of the change

Make shader work when GL_OES_EGL_image_external is not present.

To post a comment you must log in.
Revision history for this message
Jim Hodapp (jhodapp) :
review: Needs Information
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) :
Revision history for this message
Jim Hodapp (jhodapp) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/shadervideoshader.cpp'
--- src/shadervideoshader.cpp 2015-05-03 21:42:34 +0000
+++ src/shadervideoshader.cpp 2017-02-14 10:30:44 +0000
@@ -82,8 +82,12 @@
82const char *ShaderVideoShader::fragmentShader() const82const char *ShaderVideoShader::fragmentShader() const
83{83{
84 static const char *shader =84 static const char *shader =
85 "#extension GL_OES_EGL_image_external : require \n"85 "#extension GL_OES_EGL_image_external : enable \n"
86 "#ifdef GL_OES_EGL_image_external \n"
86 "uniform samplerExternalOES sTexture; \n"87 "uniform samplerExternalOES sTexture; \n"
88 "#else \n"
89 "uniform sampler2D sTexture; \n"
90 "#endif \n"
87 "uniform lowp float opacity; \n"91 "uniform lowp float opacity; \n"
88 "varying highp vec2 qt_TexCoord; \n"92 "varying highp vec2 qt_TexCoord; \n"
89 "void main() \n"93 "void main() \n"

Subscribers

People subscribed via source and target branches