[MERGE] Quicker initial commit

Ian Clatworthy ian.clatworthy at internode.on.net
Thu Aug 30 03:12:33 BST 2007


Aaron Bentley wrote:

> ^^^ This looks odd.
> 
> Perhaps it would be clearer as:
>[snip]
> I missed the "return False" on the first reading, which made it look
> very strange indeed.

Done. Some extra comments added as well to improve clarity here.

> I think you should add a test to make sure all Tree implementations will
> accept a path in get_file.

Done.

New bundle attached.

Ian C.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: quicker-initial-commit-2.patch
Type: text/x-patch
Size: 15724 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070830/cadfb954/attachment-0001.bin 


More information about the bazaar mailing list