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

Alfonso Sanchez-Beato alfonso.sanchez-beato at canonical.com
Mon Feb 20 08:39:39 UTC 2017



Diff comments:

> === 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 @@
>  const char *ShaderVideoShader::fragmentShader() const
>  {
>      static const char *shader =
> -        "#extension GL_OES_EGL_image_external : require      \n"
> +        "#extension GL_OES_EGL_image_external : enable       \n"
> +        "#ifdef GL_OES_EGL_image_external                    \n"

Correct, on desktop the GL driver what not happy with requiring GL_OES_EGL_image_external. Yes, I have tested this on phones (works, and I also checked that the extension is a must there).

>          "uniform samplerExternalOES sTexture;                \n"
> +        "#else                                               \n"
> +        "uniform sampler2D sTexture;                         \n"
> +        "#endif                                              \n"
>          "uniform lowp float opacity;                         \n"
>          "varying highp vec2 qt_TexCoord;                     \n"
>          "void main()                                         \n"


-- 
https://code.launchpad.net/~alfonsosanchezbeato/qtvideo-node/mir-desktop-enablement/+merge/317179
Your team Ubuntu Phablet Team is subscribed to branch lp:qtvideo-node.



More information about the Ubuntu-reviews mailing list