Rev 2311: Change iter_entries_by_dir to avoid extra file id safety checks. in http://bzr.arbash-meinel.com/branches/bzr/0.15-dev/unicode_id_warnings

[John Arbash Meinel]

At http://bzr.arbash-meinel.com/branches/bzr/0.15-dev/unicode_id_warnings

------------------------------------------------------------
revno: 2311
revision-id: john at arbash-meinel.com-20070302160657-acwr7ptaigpwawle
parent: john at arbash-meinel.com-20070302155652-d64161mcg1rybs1o
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: unicode_id_warnings
timestamp: Fri 2007-03-02 10:06:57 -0600
message:
  Change iter_entries_by_dir to avoid extra file id safety checks.
modified:
  bzrlib/inventory.py            inventory.py-20050309040759-6648b84ca2005b37
-------------- next part --------------
=== modified file 'bzrlib/inventory.py'
--- a/bzrlib/inventory.py	2007-03-02 15:23:21 +0000
+++ b/bzrlib/inventory.py	2007-03-02 16:06:57 +0000
@@ -981,11 +981,14 @@
             from_dir = self._byid[from_dir]
 
         if specific_file_ids is not None:
+            # TODO: jam 20070302 This could really be done as a loop rather
+            #       than a bunch of recursive calls.
             parents = set()
+            byid = self._byid
             def add_ancestors(file_id):
-                if file_id not in self:
+                if file_id not in byid:
                     return
-                parent_id = self[file_id].parent_id
+                parent_id = byid[file_id].parent_id
                 if parent_id is None:
                     return
                 if parent_id not in parents: