[MERGE] Remove Branch.get_transaction
robertc at robertcollins.net
Mon Mar 26 06:38:04 BST 2007
Robert Collins has voted +1 (conditional).
Status is now: Conditionally approved
NEWS should mention your name. The _clear_revision_history_cache method
could actually be less specific usefully: it seems to me that any other
cached data should be cleared on unlock etc, so having overly precise
methods isn't that useful. (Of course the method to cache it should be
specific :). Perhaps you could have a '_clear_branch_cache()' which
calls '_clear_revision_history_cache()' for now, if you feel that
overriding the caching mechanism is important.
Please read the docstrings on your new methods. They dont all parse.
Lastly, the IdentityMap logic related to revision_history should be
nuked; its now cruft.
For details, see:
More information about the bazaar