Merge lp://staging/~3v1n0/unity/panel-controller-signals-disconnect into lp://staging/unity

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Andrea Azzarone
Approved revision: no longer in the source branch.
Merged at revision: 4252
Proposed branch: lp://staging/~3v1n0/unity/panel-controller-signals-disconnect
Merge into: lp://staging/unity
Prerequisite: lp://staging/~3v1n0/unity/lowgfx-setting-sync
Diff against target: 13 lines (+1/-2)
1 file modified
panel/PanelController.cpp (+1/-2)
To merge this branch: bzr merge lp://staging/~3v1n0/unity/panel-controller-signals-disconnect
Reviewer Review Type Date Requested Status
Andrea Azzarone (community) Approve
Review via email: mp+327862@code.staging.launchpad.net

Commit message

PanelController: ensure we disconnect from signals on destruction

It's just about using the magic of deriving from sigc::trackable.
Since UScreen might be late in signaling screen changes it could fail

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) 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.