Code review comment for lp:~aacid/unity-2d/fix_memory_leaks_from_qtbamf_lists
Revision history for this message
| Albert Astals Cid (aacid) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
Right, seems if that the xid is not found we are returning the last window, probably not the desired behaviour of the function.
That was already the old behaviour before my patch, i can fix that if you want, but for me it makes more sense to do so in a different merge request to keep stuff more contained. What to you think?