[PATCH] cleanup branch.find_branch_root() (was Re: About merging)

David Vanderson dave.vanderson at cox.net
Sun Jun 26 21:02:17 BST 2005


Hello,

Been lurking on the list for quite some time.  Here's my first go at a 
simple patch based on what Aaron said:

Aaron Bentley wrote:
<snip>

> One easy-but-boring thing would be factoring out the options for the
> Branch constructor.
>
> Branch.__init__ currently takes the options "init" and "find_root".
> Their functionality should be separated out into factory functions, so
> that we get init_branch() and find_root_branch().
>
> Aaron

<snip>

The attached patch (off of revision 763) removes the "find_root" option 
by making it always happen unless "init" is True.  Turns out there was 
no code that wanted find_root to be False.

Please tell me if you see any problems, especially as this is my first 
patch.

Thanks,
Dave
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cleanup_find_branch_root.patch
Type: text/x-patch
Size: 3465 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050626/41a5a84d/attachment.bin 


More information about the bazaar mailing list