[MERGE] clean up option.py and branch.py imports

Michael Hudson michael.hudson at canonical.com
Tue Feb 26 23:54:58 GMT 2008


So the story of this branch is that I noticed in a traceback that 
option.py lazily imported bzrlib.log then promptly used it at module 
level, which is pointless.  So I removed that, and noticed that some 
other imports were also pointlessly lazy of unused, so I fixed them, so 
I picked another module to work on -- branch.py -- and found a bunch 
more work to do there -- changes so the bzrlib.errors import can be 
properly lazy, and some more unused imports.

As it looks like doing this for all of bzrlib looks like quite a bit of 
work, I thought I'd check here that the work would be acceptable by 
submitting what I've done so far for review.

I guess in the end this should shave a couple of percentage points off 
startup time, but certainly nothing is measurable yet.

Cheers,
mwh
PS: I'll see if I can hack up an import checker that undestands lazy 
imports one of these days...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzr-mail-LcyZMO.patch
Type: text/x-diff
Size: 9026 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20080227/cc741f8d/attachment.bin 


More information about the bazaar mailing list