Merge lp:~vanvugt/unity/fix-901602-trunk into lp:unity

Proposed by Daniel van Vugt
Status: Merged
Approved by: Alex Launi
Approved revision: no longer in the source branch.
Merged at revision: 1787
Proposed branch: lp:~vanvugt/unity/fix-901602-trunk
Merge into: lp:unity
Diff against target: 21 lines (+3/-1)
1 file modified
plugins/unityshell/src/PanelView.cpp (+3/-1)
To merge this branch: bzr merge lp:~vanvugt/unity/fix-901602-trunk
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Review via email: mp+84920@code.launchpad.net

Description of the change

Fix uninitialized variable read (LP: #901602)

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

+1

review: Approve
Revision history for this message
Unity Merger (unity-merger) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity/63/console reported an error when processing this lp:~vanvugt/unity/fix-901602-trunk branch.
Not merging it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/unityshell/src/PanelView.cpp'
2--- plugins/unityshell/src/PanelView.cpp 2011-12-08 05:51:01 +0000
3+++ plugins/unityshell/src/PanelView.cpp 2011-12-08 10:41:28 +0000
4@@ -54,6 +54,8 @@
5
6 PanelView::PanelView(NUX_FILE_LINE_DECL)
7 : View(NUX_FILE_LINE_PARAM),
8+ _last_width(0),
9+ _last_height(0),
10 _is_dirty(true),
11 _opacity(1.0f),
12 _opacity_maximized_toggle(false),
13@@ -362,7 +364,7 @@
14 {
15 nux::Geometry geo = GetGeometry();
16
17- if (geo.width == _last_width && geo.height == _last_height && !_is_dirty)
18+ if (!_is_dirty && geo.width == _last_width && geo.height == _last_height)
19 return;
20
21 _last_width = geo.width;