ayatana-commits team mailing list archive
-
ayatana-commits team
-
Mailing list archive
-
Message #03193
Re: [Merge] lp:~chrisccoulson/dbusmenu/more-memory-fixes into lp:dbusmenu
Review: Approve
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!
--
https://code.launchpad.net/~chrisccoulson/dbusmenu/more-memory-fixes/+merge/51152
Your team ayatana-commits is subscribed to branch lp:dbusmenu.
References