Merge lp:~sil2100/compiz/0.9.8_fix_ftbfs_arm into lp:compiz/0.9.8

Proposed by Łukasz Zemczak
Status: Merged
Approved by: Sam Spilsbury
Approved revision: 3432
Merged at revision: 3432
Proposed branch: lp:~sil2100/compiz/0.9.8_fix_ftbfs_arm
Merge into: lp:compiz/0.9.8
Diff against target: 19 lines (+2/-1)
1 file modified
plugins/text/src/text.cpp (+2/-1)
To merge this branch: bzr merge lp:~sil2100/compiz/0.9.8_fix_ftbfs_arm
Reviewer Review Type Date Requested Status
Sam Spilsbury Approve
Review via email: mp+142714@code.launchpad.net

Commit message

Fix FTBFS for ARM because of an unused variable.

Description of the change

- Problem:

FTBFS because of an unused variable.

- Fix:

Move the unused variable to the path where GLES is not used.

- Tests:

N/A

To post a comment you must log in.
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Resubmitted the patch to the correct branch.

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Yep, thats fine.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/text/src/text.cpp'
--- plugins/text/src/text.cpp 2012-11-27 03:51:57 +0000
+++ plugins/text/src/text.cpp 2013-01-10 15:40:27 +0000
@@ -492,7 +492,6 @@
492 float y,492 float y,
493 float alpha) const493 float alpha) const
494{494{
495 GLboolean wasBlend;
496 GLint oldBlendSrc, oldBlendDst;495 GLint oldBlendSrc, oldBlendDst;
497 GLushort colorData[4];496 GLushort colorData[4];
498 GLfloat textureData[8];497 GLfloat textureData[8];
@@ -509,6 +508,8 @@
509 glGetIntegerv (GL_BLEND_SRC_ALPHA, &oldBlendSrcAlpha);508 glGetIntegerv (GL_BLEND_SRC_ALPHA, &oldBlendSrcAlpha);
510 glGetIntegerv (GL_BLEND_DST_ALPHA, &oldBlendDstAlpha);509 glGetIntegerv (GL_BLEND_DST_ALPHA, &oldBlendDstAlpha);
511#else510#else
511 GLboolean wasBlend;
512
512 glGetIntegerv (GL_BLEND_SRC, &oldBlendSrc);513 glGetIntegerv (GL_BLEND_SRC, &oldBlendSrc);
513 glGetIntegerv (GL_BLEND_DST, &oldBlendDst);514 glGetIntegerv (GL_BLEND_DST, &oldBlendDst);
514515

Subscribers

People subscribed via source and target branches