GNOME Bugzilla – Bug 693339
Fix the scroll view fade shader on GLES2
Last modified: 2013-02-08 12:06:16 UTC
See patch
Created attachment 235421 [details] [review] Don't rely on implicit type conversion in the scroll view shader Before version 1.2 of GLSL it would not implicitly convert from int to float which meant that if you compare a float variable with an integer constant it will generate a compile error. In particular this means that on GLES2 (which uses GLSL 1.0) the scroll view shader will not compile on pedantic compilers, which includes Mesa. This patch just changes it to use floating point constants.
Review of attachment 235421 [details] [review]: We used to have #version 120 in the header, but cogl started emitting that automatically. Are you serious that GLES2 doesn't have 1.20 of GLSL? Sigh.
(In reply to comment #2) > Review of attachment 235421 [details] [review]: > > We used to have #version 120 in the header, but cogl started emitting that > automatically. Are you serious that GLES2 doesn't have 1.20 of GLSL? Sigh. It is a modified GLSL 1.0. Anyway even the NVIDIA compiler on GL warns here so the patch is a nice cleanup regardless of the GLES2 issue.
Ok, thanks. I've pushed it to master: http://git.gnome.org/browse/gnome-shell/commit/?id=33f69481f7c