Merge lp:~bregma/indicator-appmenu/lp-1231218 into lp:indicator-appmenu/13.10

Proposed by Stephen M. Webb
Status: Merged
Approved by: Charles Kerr
Approved revision: 245
Merged at revision: 245
Proposed branch: lp:~bregma/indicator-appmenu/lp-1231218
Merge into: lp:indicator-appmenu/13.10
Diff against target: 23 lines (+3/-3)
1 file modified
src/indicator-appmenu.c (+3/-3)
To merge this branch: bzr merge lp:~bregma/indicator-appmenu/lp-1231218
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+187634@code.launchpad.net

Commit message

don't add a circular self-ref in IndicatorAppmenu objects (lp: #1231218)

Description of the change

Removes a circular self-reference that prevents IndicatorAppmenu objects from getting disposed. Fixes a memory leak in unity-panel-service.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

Completely right.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/indicator-appmenu.c'
2--- src/indicator-appmenu.c 2012-11-20 23:40:35 +0000
3+++ src/indicator-appmenu.c 2013-09-26 02:01:38 +0000
4@@ -1,7 +1,7 @@
5 /*
6 An implementation of indicator object showing menus from applications.
7
8-Copyright 2010 Canonical Ltd.
9+Copyright 2010-2013 Canonical Ltd.
10
11 Authors:
12 Ted Gould <ted@canonical.com>
13@@ -322,8 +322,8 @@
14 iapp->dbus_registration == 0 ? on_bus_acquired : NULL,
15 on_name_acquired,
16 on_name_lost,
17- g_object_ref(iapp),
18- g_object_unref);
19+ iapp,
20+ NULL);
21
22 return TRUE;
23 }

Subscribers

People subscribed via source and target branches