Merge lp://qastaging/~hikiko/compiz/compiz.3d-plugin-fixes into lp://qastaging/compiz/0.9.12

Proposed by Eleni Maria Stea
Status: Merged
Approved by: Christopher Townsend
Approved revision: 3930
Merged at revision: 3940
Proposed branch: lp://qastaging/~hikiko/compiz/compiz.3d-plugin-fixes
Merge into: lp://qastaging/compiz/0.9.12
Prerequisite: lp://qastaging/~hikiko/compiz/compiz.glvector-fixes
Diff against target: 33 lines (+12/-0)
2 files modified
plugins/cube/src/cube.cpp (+10/-0)
plugins/cube/src/privates.h (+2/-0)
To merge this branch: bzr merge lp://qastaging/~hikiko/compiz/compiz.3d-plugin-fixes
Reviewer Review Type Date Requested Status
Christopher Townsend (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Stephen M. Webb Needs Fixing
Review via email: mp+251562@code.qastaging.launchpad.net

Commit message

glDisableOutputClipping was not implemented in 3d.cpp => a parent class glDisableOutputClipping was called, causing 2d clipping artifacts (the 3d cube was clipped and you couldn't see the windows move)

Description of the change

glDisableOutputClipping was not implemented in 3d.cpp => a parent class glDisableOutputClipping was called, causing 2d clipping artifacts (the 3d cube was clipped and you couldn't see the windows move)

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Stephen M. Webb (bregma) wrote :

See inline comments.

review: Needs Fixing
3930. By Eleni Maria Stea

used "#ifndef USE_GLES" around the non-GLES code.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Stephen M. Webb (bregma) wrote :

Ignore that Jenkins failure, it's just Jenkins being Jenkins as usual.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Christopher Townsend (townsend) wrote :

Ok, seems fine now. Thanks!

review: Approve

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