Merge lp:~chrisccoulson/libdbusmenu/more-memory-fixes into lp:libdbusmenu/0.5
Proposed by
Chris Coulson
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Merged at revision: | 232 | ||||||||
Proposed branch: | lp:~chrisccoulson/libdbusmenu/more-memory-fixes | ||||||||
Merge into: | lp:libdbusmenu/0.5 | ||||||||
Diff against target: |
246 lines (+117/-8) 3 files modified
libdbusmenu-glib/menuitem.c (+100/-2) libdbusmenu-glib/menuitem.h (+4/-0) libdbusmenu-gtk/parser.c (+13/-6) |
||||||||
To merge this branch: | bzr merge lp:~chrisccoulson/libdbusmenu/more-memory-fixes | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Approve | ||
Review via email: mp+51152@code.launchpad.net |
Description of the change
More memory error fixes :)
To post a comment you must log in.
A few comments:
* Please don't comment out code. Either delete or fix, we have version control to find old code if we need it.
* Added a check to see if we're the parent to delete or else we could end up with really weird unparenting from non-parents.
* get_parent() needs a transfer annotation
* please try to keep it to one feature/fix per branch as it makes it easier to review and understand the impact of the change.
* new functions need to be added to "sections.txt" for the documentation.
I fixed these and merged the whole thing in. Thanks for the investigation and fixing these!