[MERGE] Fix another LockableFiles not unlocked warning from test_smart

Andrew Bennetts andrew at canonical.com
Tue Jan 22 03:16:29 GMT 2008


This change fixes some “LockableFiles ... not explicitly unlocked” warnings from
test_smart.  They were caused by the repository not getting unlocked by the new
streaming fetch verbs when handling errors.

(A related cause is that it's not particularly obvious that a lock_read()
happens on the first iteration of the iterator returned by repo.get_data_stream,
and that the caller is required to do the matching unlock(), but this change
doesn't attempt to address that.)

-Andrew.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzr-mail-Nh341I.patch
Type: text/x-diff
Size: 4303 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20080122/72220d31/attachment.bin 


More information about the bazaar mailing list