Merge lp://qastaging/~hikiko/compiz/compiz.clear-skydome-background into lp://qastaging/compiz/0.9.12

Proposed by Eleni Maria Stea
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: 3940
Merged at revision: 3941
Proposed branch: lp://qastaging/~hikiko/compiz/compiz.clear-skydome-background
Merge into: lp://qastaging/compiz/0.9.12
Prerequisite: lp://qastaging/~hikiko/compiz/compiz.3d-plugin-fixes
Diff against target: 11 lines (+1/-0)
1 file modified
plugins/cube/src/cube.cpp (+1/-0)
To merge this branch: bzr merge lp://qastaging/~hikiko/compiz/compiz.clear-skydome-background
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+254872@code.qastaging.launchpad.net

Commit message

Minor change: when the skydome was working (in the c version of compiz) it was rendered on top of the background so, there was no need to clear the screen before drawing (the skydome was rendered on top of the garbage from previous renderings, hiding it). In the c++ plugin, the skydome is buggy and the background is visible (as well as some garbage from previous renderings). A temporary glClear(GL_COLOR_BUFFER_BIT) was added until we fix the skydome and the other cube bugs.

Description of the change

Minor change: when the skydome was working (in the c version of compiz) it was rendered on top of the background so, there was no need to clear the screen before drawing (the skydome was rendered on top of the garbage from previous renderings, hiding it). In the c++ plugin, the skydome is buggy and the background is visible (as well as some garbage from previous renderings). A temporary glClear(GL_COLOR_BUFFER_BIT) was added until we fix the skydome and the other cube bugs.

To post a comment you must log in.
Revision history for this message
Eleni Maria Stea (hikiko) wrote :

screenshot before the fix: https://i.imgur.com/bdlHf5y.jpg
screenshot after the fix: https://i.imgur.com/nnj4y1f.jpg

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

+1

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