Merge lp://qastaging/~compiz-team/compiz/compiz.fix_1056645 into lp://qastaging/compiz/0.9.8

Proposed by Sam Spilsbury
Status: Merged
Approved by: Daniel van Vugt
Approved revision: 3396
Merged at revision: 3397
Proposed branch: lp://qastaging/~compiz-team/compiz/compiz.fix_1056645
Merge into: lp://qastaging/compiz/0.9.8
Diff against target: 53 lines (+15/-5)
1 file modified
plugins/opengl/src/screen.cpp (+15/-5)
To merge this branch: bzr merge lp://qastaging/~compiz-team/compiz/compiz.fix_1056645
Reviewer Review Type Date Requested Status
jenkins (community) continuous-integration Needs Fixing
Daniel van Vugt Approve
Review via email: mp+126399@code.qastaging.launchpad.net

Commit message

Fix build error: "'swapInterval' is not a member of 'GL'" on USE_GLES
(LP: #1056645)

Description of the change

Fix FTBFS

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

/home/dan/bzr/compiz/tmp.645/plugins/opengl/src/screen.cpp: In function ‘void GL::waitForVideoSync()’:
/home/dan/bzr/compiz/tmp.645/plugins/opengl/src/screen.cpp:1722:21: error: too few arguments to function ‘void GL::fastSwapInterval(Display*, int)’
/home/dan/bzr/compiz/tmp.645/plugins/opengl/src/screen.cpp:1694:1: note: declared here

review: Needs Fixing
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Also, please give a commit comment with some details from the bug title.

3396. By Sam Spilsbury

Missing in one other place

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

fixed.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Alright, looks good now.

review: Approve
Revision history for this message
jenkins (martin-mrazik+qa) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches