DBus Menu

Merge lp:~ted/libdbusmenu/accel-ref into lp:libdbusmenu/0.5

  1. accel-ref
  2. Merge into trunk
Proposed by Ted Gould
Status: Merged
Merged at revision: 178
Proposed branch: lp:~ted/libdbusmenu/accel-ref
Merge into: lp:libdbusmenu/0.5
Diff against target: 11 lines (+1/-0)
1 file modified
libdbusmenu-gtk/client.c (+1/-0)
To merge this branch: bzr merge lp:~ted/libdbusmenu/accel-ref
Reviewer Review Type Date Requested Status
DBus Menu Team Pending
Review via email: mp+36590@code.launchpad.net

Description of the change

This makes it so we ref the object that we need. But, because there's a bug in i-appmenu we don't notice it. More importantly, if this gets merged without the corresponding patch to appmenu we'll have a memory leak.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'libdbusmenu-gtk/client.c'
2--- libdbusmenu-gtk/client.c 2010-08-30 20:22:43 +0000
3+++ libdbusmenu-gtk/client.c 2010-09-24 19:41:46 +0000
4@@ -249,6 +249,7 @@
5 }
6
7 priv->agroup = agroup;
8+ g_object_ref(priv->agroup);
9
10 return;
11 }

Subscribers

People subscribed via source and target branches