Merge lp:~mterry/libappindicator/fix-crash-708188 into lp:libappindicator

Proposed by Michael Terry
Status: Merged
Approved by: Ted Gould
Approved revision: 195
Merged at revision: 195
Proposed branch: lp:~mterry/libappindicator/fix-crash-708188
Merge into: lp:libappindicator
Diff against target: 12 lines (+2/-0)
1 file modified
src/app-indicator.c (+2/-0)
To merge this branch: bzr merge lp:~mterry/libappindicator/fix-crash-708188
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
Review via email: mp+51356@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

Note I have not confirmed this fix, but it just makes sense.

Revision history for this message
Ted Gould (ted) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/app-indicator.c'
--- src/app-indicator.c 2011-02-24 21:34:16 +0000
+++ src/app-indicator.c 2011-02-25 19:20:12 +0000
@@ -667,6 +667,8 @@
667 priv->connection = NULL;667 priv->connection = NULL;
668 }668 }
669669
670 g_signal_handlers_disconnect_by_func(gtk_icon_theme_get_default(), G_CALLBACK(theme_changed_cb), self);
671
670 G_OBJECT_CLASS (app_indicator_parent_class)->dispose (object);672 G_OBJECT_CLASS (app_indicator_parent_class)->dispose (object);
671 return;673 return;
672}674}

Subscribers

People subscribed via source and target branches