[MERGE][#335180] LockableFiles.__del__ must die (was Re: Do you need to close a branch? (if so how))

Andrew Bennetts andrew.bennetts at canonical.com
Mon Mar 2 00:43:24 GMT 2009


Andrew Bennetts has voted tweak.
Status is now: Semi-approved
Comment:
The basic change seems good.  It's always nice to get rid of __del__ 
methods in reference cycles.  The transplanted comment in the new 
__del__ doesn't quite make sense, though.  Perhaps s/do/did/ ?

For details, see: 
http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C49A75280.1000705%40canonical.com%3E
Project: Bazaar



More information about the bazaar mailing list