Application Menu Indicator

Merge lp://qastaging/~ted/indicator-appmenu/lp738561 into lp://qastaging/indicator-appmenu/0.3

  1. lp738561
  2. Merge into trunk
Proposed by Ted Gould
Status: Merged
Merged at revision: 112
Proposed branch: lp://qastaging/~ted/indicator-appmenu/lp738561
Merge into: lp://qastaging/indicator-appmenu/0.3
Diff against target: 25 lines (+8/-0)
1 file modified
src/window-menus.c (+8/-0)
To merge this branch: bzr merge lp://qastaging/~ted/indicator-appmenu/lp738561
Reviewer Review Type Date Requested Status
Mikkel Kamstrup Erlandsen (community) Approve
Review via email: mp+56251@code.qastaging.launchpad.net

Description of the change

Disconnect signals as it seems this reference is living longer than expected. Probably in callbacks.

To post a comment you must log in.
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

 review approve

Good catch!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches