libappindicator

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

  1. fix-crash-708188
  2. Merge into trunk
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
1=== modified file 'src/app-indicator.c'
2--- src/app-indicator.c 2011-02-24 21:34:16 +0000
3+++ src/app-indicator.c 2011-02-25 19:20:12 +0000
4@@ -667,6 +667,8 @@
5 priv->connection = NULL;
6 }
7
8+ g_signal_handlers_disconnect_by_func(gtk_icon_theme_get_default(), G_CALLBACK(theme_changed_cb), self);
9+
10 G_OBJECT_CLASS (app_indicator_parent_class)->dispose (object);
11 return;
12 }

Subscribers

People subscribed via source and target branches