Merge lp:~linaro-graphics-wg/unity/fix-gles2-build into lp:unity

Proposed by Alexandros Frantzis on 2012-04-06
Status: Merged
Approved by: Unity Merger on 2012-04-13
Approved revision: 2247
Merged at revision: 2281
Proposed branch: lp:~linaro-graphics-wg/unity/fix-gles2-build
Merge into: lp:unity
Diff against target: 19 lines (+2/-0)
1 file modified
plugins/unityshell/src/OverlayRenderer.cpp (+2/-0)
To merge this branch: bzr merge lp:~linaro-graphics-wg/unity/fix-gles2-build
Reviewer Review Type Date Requested Status
Sam Spilsbury (community) 2012-04-06 Approve on 2012-04-10
Review via email: mp+101087@code.launchpad.net

Commit Message

Fix compilation with OpenGL ES 2.0.

Description of the Change

Fix compilation with OpenGL ES 2.0.

UNBLOCK

To post a comment you must log in.
Sam Spilsbury (smspillaz) :
review: Approve
Unity Merger (unity-merger) wrote :

We wanted the compiz changes without any unity changes, that was part of the deal, but now that the new compiz has been pushed, we had to distro patch with that build. I'm afraid that I have no other choice than accepting this branch in trunk.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/unityshell/src/OverlayRenderer.cpp'
2--- plugins/unityshell/src/OverlayRenderer.cpp 2012-04-05 22:01:14 +0000
3+++ plugins/unityshell/src/OverlayRenderer.cpp 2012-04-06 10:40:30 +0000
4@@ -181,6 +181,7 @@
5
6 void OverlayRendererImpl::RenderInverseMask_ASM(nux::GraphicsEngine& gfx_context, int x, int y, int width, int height, nux::ObjectPtr<nux::IOpenGLBaseTexture> device_texture, nux::TexCoordXForm &texxform, const nux::Color &color)
7 {
8+#ifndef NUX_OPENGLES_20
9 if (!inverse_texture_mask_asm_prog_.IsValid() || !inverse_texture_rect_mask_asm_prog_.IsValid())
10 {
11 InitASMInverseTextureMaskShader();
12@@ -246,6 +247,7 @@
13 CHECKGL(glDisableVertexAttribArrayARB(VertexColorLocation));
14
15 shader_program->End();
16+#endif
17 }
18
19 void OverlayRendererImpl::InitSlInverseTextureMaskShader()