[MERGE] 1.12-preview formats
Ian Clatworthy
ian.clatworthy at internode.on.net
Mon Dec 22 08:54:35 GMT 2008
Aaron Bentley wrote:
> Performance is not the reason we do it. We do it so that
> 1. New formats are not dependant on older ones. This means older
> formats can be decomissioned as needed.
>
> 2. We care more about newer formats, especially as times goes on and
> they become default formats. This keeps their code consolidated in one
> place.
>
> It's also worth bearing in mind the fact that none of our working tree
> formats were introduced since we changed our procedures to make older
> formats subclass newer ones.
All good points. At poolie's suggestion, I've introduced some base
classes in workingtree_4.py and made WorkingTree4/WorkingTreeFormat4 and
WorkingTree5/WorkingTreeFormat5 both subclass these. The updated patch
is attached.
I searched out each of the places where WorkingTreeFormat4 was used and
I think I've got all the *necessary* bits of code changed. Let me know
please if other places ought to be changed as well, e.g. references to
_matching_from_tree_format/_matching_to_tree_format for the inter-tree
tests.
Ian C.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 1.12-preview-formats-2.patch
Type: text/x-diff
Size: 25796 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20081222/75f00bdb/attachment-0001.bin
More information about the bazaar
mailing list