Merge lp:~nick-dedekind/unity/lp1055995.launcher-icon-expo-terminate-6.0 into lp:unity/6.0

Proposed by Nick Dedekind
Status: Merged
Approved by: Andrea Azzarone
Approved revision: no longer in the source branch.
Merged at revision: 2747
Proposed branch: lp:~nick-dedekind/unity/lp1055995.launcher-icon-expo-terminate-6.0
Merge into: lp:unity/6.0
Diff against target: 13 lines (+3/-0)
1 file modified
launcher/LauncherIcon.cpp (+3/-0)
To merge this branch: bzr merge lp:~nick-dedekind/unity/lp1055995.launcher-icon-expo-terminate-6.0
Reviewer Review Type Date Requested Status
Andrea Azzarone (community) Approve
Review via email: mp+127265@code.launchpad.net

Commit message

Added launcher icon destructor disconnect from expo termination signal.

Description of the change

Added launcher icon destructor disconnect from expo termination signal.

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'launcher/LauncherIcon.cpp'
2--- launcher/LauncherIcon.cpp 2012-09-18 03:18:11 +0000
3+++ launcher/LauncherIcon.cpp 2012-10-01 13:02:13 +0000
4@@ -136,6 +136,9 @@
5 if (on_order_changed_connection.connected())
6 on_order_changed_connection.disconnect();
7
8+ if (on_expo_terminated_connection.connected())
9+ on_expo_terminated_connection.disconnect();
10+
11 if (_unity_theme)
12 {
13 _unity_theme = NULL;

Subscribers

People subscribed via source and target branches