Merge lp:~compiz-team/compiz/compiz.fix_1189369 into lp:compiz/0.9.10

Proposed by Sam Spilsbury
Status: Superseded
Proposed branch: lp:~compiz-team/compiz/compiz.fix_1189369
Merge into: lp:compiz/0.9.10
Diff against target: 27 lines (+16/-0)
1 file modified
plugins/opengl/src/window.cpp (+16/-0)
To merge this branch: bzr merge lp:~compiz-team/compiz/compiz.fix_1189369
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Compiz Maintainers Pending
Review via email: mp+168384@code.launchpad.net

This proposal has been superseded by a proposal from 2013-06-10.

Commit message

Immediately update the window matrices and regions if the number of
textures changed.

(LP: #1189369)

Description of the change

Immediately update the window matrices and regions if the number of
textures changed.

(LP: #1189369)

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

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/opengl/src/window.cpp'
--- plugins/opengl/src/window.cpp 2013-01-03 16:05:26 +0000
+++ plugins/opengl/src/window.cpp 2013-06-10 09:04:27 +0000
@@ -179,8 +179,24 @@
179 }179 }
180 else180 else
181 {181 {
182 bool immediatelyUpdateMatricesAndRegions =
183 priv->textures.size () != textures.size ();
184
182 priv->textures = textures;185 priv->textures = textures;
183 priv->needsRebind = false;186 priv->needsRebind = false;
187
188 /* If the number of textures changed, we should immediately
189 * update the matrices and regions so that they are at least
190 * initialized, but we'll queue another update just before
191 * glPaint too in case the window moved or changed size */
192 if (immediatelyUpdateMatricesAndRegions)
193 {
194 priv->setWindowMatrix ();
195 priv->updateWindowRegions ();
196
197 priv->updateState |= PrivateGLWindow::UpdateMatrix |
198 PrivateGLWindow::UpdateRegion;
199 }
184 }200 }
185 }201 }
186202

Subscribers

People subscribed via source and target branches

to all changes: