Merge lp:~jelmer/bzr/less-inventory-access into lp:bzr
Proposed by
Jelmer Vernooij
Status: | Merged |
---|---|
Approved by: | Vincent Ladeuil |
Approved revision: | no longer in the source branch. |
Merged at revision: | 6470 |
Proposed branch: | lp:~jelmer/bzr/less-inventory-access |
Merge into: | lp:bzr |
Diff against target: |
184 lines (+22/-39) 7 files modified
bzrlib/builtins.py (+0/-2) bzrlib/bundle/bundle_data.py (+6/-8) bzrlib/commit.py (+1/-3) bzrlib/inventory.py (+0/-16) bzrlib/merge.py (+2/-2) bzrlib/tests/test_bundle.py (+9/-6) bzrlib/workingtree.py (+4/-2) |
To merge this branch: | bzr merge lp:~jelmer/bzr/less-inventory-access |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Vincent Ladeuil | Approve | ||
Review via email: mp+93612@code.launchpad.net |
Commit message
Avoid direct access of tree inventories in a couple more places.
Description of the change
Remove access to Tree.root_inventory in a couple of places.
In a few places we weren't actually using the inventory after assigning it to a local variable.
I've removed Inventory.
To post a comment you must log in.
Looking at the annotation, directories is so old... I think we're safe to delete it, but you'll get the blame for not deprecating it if somebody was using it ;)