/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tree.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
18
18
"""
19
19
 
20
20
import os
21
 
 
22
 
from bzrlib.lazy_import import lazy_import
23
 
lazy_import(globals(), """
24
 
import collections
25
 
 
 
21
from collections import deque
 
22
 
 
23
import bzrlib
26
24
from bzrlib import (
27
25
    conflicts as _mod_conflicts,
28
26
    debug,
29
27
    delta,
30
 
    errors,
31
28
    filters,
32
 
    inventory,
33
29
    osutils,
34
30
    revision as _mod_revision,
35
31
    rules,
36
 
    trace,
37
32
    )
38
 
""")
39
 
 
40
33
from bzrlib.decorators import needs_read_lock
 
34
from bzrlib.errors import BzrError, NoSuchId
 
35
from bzrlib import errors
 
36
from bzrlib.inventory import InventoryFile
41
37
from bzrlib.inter import InterObject
42
 
from bzrlib.symbol_versioning import (
43
 
    deprecated_in,
44
 
    deprecated_method,
45
 
    )
 
38
from bzrlib.osutils import fingerprint_file
 
39
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
 
40
from bzrlib.trace import note
46
41
 
47
42
 
48
43
class Tree(object):
54
49
 
55
50
    * `RevisionTree` is a tree as recorded at some point in the past.
56
51
 
 
52
    Trees contain an `Inventory` object, and also know how to retrieve
 
53
    file texts mentioned in the inventory, either from a working
 
54
    directory or from a store.
 
55
 
 
56
    It is possible for trees to contain files that are not described
 
57
    in their inventory or vice versa; for this use `filenames()`.
 
58
 
57
59
    Trees can be compared, etc, regardless of whether they are working
58
60
    trees or versioned trees.
59
61
    """
125
127
        raise NotImplementedError(self.has_filename)
126
128
 
127
129
    def has_id(self, file_id):
128
 
        raise NotImplementedError(self.has_id)
 
130
        return self.inventory.has_id(file_id)
129
131
 
130
132
    def __contains__(self, file_id):
131
133
        return self.has_id(file_id)
132
134
 
133
135
    def has_or_had_id(self, file_id):
134
 
        raise NotImplementedError(self.has_or_had_id)
 
136
        return self.inventory.has_id(file_id)
135
137
 
136
138
    def is_ignored(self, filename):
137
139
        """Check whether the filename is ignored by this tree.
141
143
        """
142
144
        return False
143
145
 
 
146
    def __iter__(self):
 
147
        return iter(self.inventory)
 
148
 
144
149
    def all_file_ids(self):
145
150
        """Iterate through all file ids, including ids for missing files."""
146
 
        raise NotImplementedError(self.all_file_ids)
 
151
        return set(self.inventory)
147
152
 
148
153
    def id2path(self, file_id):
149
154
        """Return the path for a file id.
150
155
 
151
156
        :raises NoSuchId:
152
157
        """
153
 
        raise NotImplementedError(self.id2path)
154
 
 
 
158
        return self.inventory.id2path(file_id)
 
159
 
 
160
    def is_control_filename(self, filename):
 
161
        """True if filename is the name of a control file in this tree.
 
162
 
 
163
        :param filename: A filename within the tree. This is a relative path
 
164
        from the root of this tree.
 
165
 
 
166
        This is true IF and ONLY IF the filename is part of the meta data
 
167
        that bzr controls in this tree. I.E. a random .bzr directory placed
 
168
        on disk will not be a control file for this tree.
 
169
        """
 
170
        return self.bzrdir.is_control_filename(filename)
 
171
 
 
172
    @needs_read_lock
155
173
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
156
174
        """Walk the tree in 'by_dir' order.
157
175
 
180
198
            down to specific_file_ids that have been requested. This has no
181
199
            impact if specific_file_ids is None.
182
200
        """
183
 
        raise NotImplementedError(self.iter_entries_by_dir)
184
 
 
185
 
    def list_files(self, include_root=False, from_dir=None, recursive=True):
186
 
        """List all files in this tree.
187
 
 
188
 
        :param include_root: Whether to include the entry for the tree root
189
 
        :param from_dir: Directory under which to list files
190
 
        :param recursive: Whether to list files recursively
191
 
        :return: iterator over tuples of (path, versioned, kind, file_id,
192
 
            inventory entry)
193
 
        """
194
 
        raise NotImplementedError(self.list_files)
 
201
        return self.inventory.iter_entries_by_dir(
 
202
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
195
203
 
196
204
    def iter_references(self):
197
205
        if self.supports_tree_reference():
248
256
    def _file_size(self, entry, stat_value):
249
257
        raise NotImplementedError(self._file_size)
250
258
 
 
259
    def _get_inventory(self):
 
260
        return self._inventory
 
261
 
251
262
    def get_file(self, file_id, path=None):
252
263
        """Return a file object for the file file_id in the tree.
253
264
 
294
305
        """
295
306
        return osutils.split_lines(self.get_file_text(file_id, path))
296
307
 
297
 
    def get_file_sha1(self, file_id, path=None):
298
 
        """Return the SHA1 file for a file.
299
 
 
300
 
        :param file_id: The handle for this file.
301
 
        :param path: The path that this file can be found at.
302
 
            These must point to the same object.
303
 
        """
304
 
        raise NotImplementedError(self.get_file_sha1)
305
 
 
306
308
    def get_file_mtime(self, file_id, path=None):
307
309
        """Return the modification time for a file.
308
310
 
322
324
        raise NotImplementedError(self.get_file_size)
323
325
 
324
326
    def get_file_by_path(self, path):
325
 
        raise NotImplementedError(self.get_file_by_path)
326
 
 
327
 
    def is_executable(self, file_id, path=None):
328
 
        """Check if a file is executable.
329
 
 
330
 
        :param file_id: The handle for this file.
331
 
        :param path: The path that this file can be found at.
332
 
            These must point to the same object.
333
 
        """
334
 
        raise NotImplementedError(self.is_executable)
 
327
        return self.get_file(self._inventory.path2id(path), path)
335
328
 
336
329
    def iter_files_bytes(self, desired_files):
337
330
        """Iterate through file contents.
359
352
            cur_file = (self.get_file_text(file_id),)
360
353
            yield identifier, cur_file
361
354
 
362
 
    def get_symlink_target(self, file_id, path=None):
 
355
    def get_symlink_target(self, file_id):
363
356
        """Get the target for a given file_id.
364
357
 
365
358
        It is assumed that the caller already knows that file_id is referencing
366
359
        a symlink.
367
360
        :param file_id: Handle for the symlink entry.
368
 
        :param path: The path of the file.
369
 
        If both file_id and path are supplied, an implementation may use
370
 
        either one.
371
361
        :return: The path the symlink points to.
372
362
        """
373
363
        raise NotImplementedError(self.get_symlink_target)
374
364
 
 
365
    def get_canonical_inventory_paths(self, paths):
 
366
        """Like get_canonical_inventory_path() but works on multiple items.
 
367
 
 
368
        :param paths: A sequence of paths relative to the root of the tree.
 
369
        :return: A list of paths, with each item the corresponding input path
 
370
        adjusted to account for existing elements that match case
 
371
        insensitively.
 
372
        """
 
373
        return list(self._yield_canonical_inventory_paths(paths))
 
374
 
 
375
    def get_canonical_inventory_path(self, path):
 
376
        """Returns the first inventory item that case-insensitively matches path.
 
377
 
 
378
        If a path matches exactly, it is returned. If no path matches exactly
 
379
        but more than one path matches case-insensitively, it is implementation
 
380
        defined which is returned.
 
381
 
 
382
        If no path matches case-insensitively, the input path is returned, but
 
383
        with as many path entries that do exist changed to their canonical
 
384
        form.
 
385
 
 
386
        If you need to resolve many names from the same tree, you should
 
387
        use get_canonical_inventory_paths() to avoid O(N) behaviour.
 
388
 
 
389
        :param path: A paths relative to the root of the tree.
 
390
        :return: The input path adjusted to account for existing elements
 
391
        that match case insensitively.
 
392
        """
 
393
        return self._yield_canonical_inventory_paths([path]).next()
 
394
 
 
395
    def _yield_canonical_inventory_paths(self, paths):
 
396
        for path in paths:
 
397
            # First, if the path as specified exists exactly, just use it.
 
398
            if self.path2id(path) is not None:
 
399
                yield path
 
400
                continue
 
401
            # go walkin...
 
402
            cur_id = self.get_root_id()
 
403
            cur_path = ''
 
404
            bit_iter = iter(path.split("/"))
 
405
            for elt in bit_iter:
 
406
                lelt = elt.lower()
 
407
                new_path = None
 
408
                for child in self.iter_children(cur_id):
 
409
                    try:
 
410
                        # XXX: it seem like if the child is known to be in the
 
411
                        # tree, we shouldn't need to go from its id back to
 
412
                        # its path -- mbp 2010-02-11
 
413
                        #
 
414
                        # XXX: it seems like we could be more efficient
 
415
                        # by just directly looking up the original name and
 
416
                        # only then searching all children; also by not
 
417
                        # chopping paths so much. -- mbp 2010-02-11
 
418
                        child_base = os.path.basename(self.id2path(child))
 
419
                        if (child_base == elt):
 
420
                            # if we found an exact match, we can stop now; if
 
421
                            # we found an approximate match we need to keep
 
422
                            # searching because there might be an exact match
 
423
                            # later.  
 
424
                            cur_id = child
 
425
                            new_path = osutils.pathjoin(cur_path, child_base)
 
426
                            break
 
427
                        elif child_base.lower() == lelt:
 
428
                            cur_id = child
 
429
                            new_path = osutils.pathjoin(cur_path, child_base)
 
430
                    except NoSuchId:
 
431
                        # before a change is committed we can see this error...
 
432
                        continue
 
433
                if new_path:
 
434
                    cur_path = new_path
 
435
                else:
 
436
                    # got to the end of this directory and no entries matched.
 
437
                    # Return what matched so far, plus the rest as specified.
 
438
                    cur_path = osutils.pathjoin(cur_path, elt, *list(bit_iter))
 
439
                    break
 
440
            yield cur_path
 
441
        # all done.
 
442
 
375
443
    def get_root_id(self):
376
444
        """Return the file_id for the root of this tree."""
377
445
        raise NotImplementedError(self.get_root_id)
435
503
            except errors.NoSuchRevisionInTree:
436
504
                yield self.repository.revision_tree(revision_id)
437
505
 
 
506
    @staticmethod
 
507
    def _file_revision(revision_tree, file_id):
 
508
        """Determine the revision associated with a file in a given tree."""
 
509
        revision_tree.lock_read()
 
510
        try:
 
511
            return revision_tree.inventory[file_id].revision
 
512
        finally:
 
513
            revision_tree.unlock()
 
514
 
438
515
    def _get_file_revision(self, file_id, vf, tree_revision):
439
516
        """Ensure that file_id, tree_revision is in vf to plan the merge."""
440
517
 
441
518
        if getattr(self, '_repository', None) is None:
442
519
            last_revision = tree_revision
443
 
            parent_keys = [(file_id, t.get_file_revision(file_id)) for t in
 
520
            parent_keys = [(file_id, self._file_revision(t, file_id)) for t in
444
521
                self._iter_parent_trees()]
445
522
            vf.add_lines((file_id, last_revision), parent_keys,
446
 
                         self.get_file_lines(file_id))
 
523
                         self.get_file(file_id).readlines())
447
524
            repo = self.branch.repository
448
525
            base_vf = repo.texts
449
526
        else:
450
 
            last_revision = self.get_file_revision(file_id)
 
527
            last_revision = self._file_revision(self, file_id)
451
528
            base_vf = self._repository.texts
452
529
        if base_vf not in vf.fallback_versionedfiles:
453
530
            vf.fallback_versionedfiles.append(base_vf)
454
531
        return last_revision
455
532
 
 
533
    inventory = property(_get_inventory,
 
534
                         doc="Inventory of this Tree")
 
535
 
456
536
    def _check_retrieved(self, ie, f):
457
537
        if not __debug__:
458
538
            return
459
 
        fp = osutils.fingerprint_file(f)
 
539
        fp = fingerprint_file(f)
460
540
        f.seek(0)
461
541
 
462
542
        if ie.text_size is not None:
463
543
            if ie.text_size != fp['size']:
464
 
                raise errors.BzrError(
465
 
                        "mismatched size for file %r in %r" %
466
 
                        (ie.file_id, self._store),
 
544
                raise BzrError("mismatched size for file %r in %r" % (ie.file_id, self._store),
467
545
                        ["inventory expects %d bytes" % ie.text_size,
468
546
                         "file is actually %d bytes" % fp['size'],
469
547
                         "store is probably damaged/corrupt"])
470
548
 
471
549
        if ie.text_sha1 != fp['sha1']:
472
 
            raise errors.BzrError("wrong SHA-1 for file %r in %r" %
473
 
                    (ie.file_id, self._store),
 
550
            raise BzrError("wrong SHA-1 for file %r in %r" % (ie.file_id, self._store),
474
551
                    ["inventory expects %s" % ie.text_sha1,
475
552
                     "file is actually %s" % fp['sha1'],
476
553
                     "store is probably damaged/corrupt"])
477
554
 
 
555
    @needs_read_lock
478
556
    def path2id(self, path):
479
557
        """Return the id for path in this tree."""
480
 
        raise NotImplementedError(self.path2id)
 
558
        return self._inventory.path2id(path)
481
559
 
482
560
    def paths2ids(self, paths, trees=[], require_versioned=True):
483
561
        """Return all the ids that can be reached by walking from paths.
504
582
            yield child.file_id
505
583
 
506
584
    def lock_read(self):
507
 
        """Lock this tree for multiple read only operations.
508
 
        
509
 
        :return: A bzrlib.lock.LogicalLockResult.
510
 
        """
511
585
        pass
512
586
 
513
587
    def revision_tree(self, revision_id):
540
614
 
541
615
        :return: set of paths.
542
616
        """
543
 
        raise NotImplementedError(self.filter_unversioned_files)
 
617
        # NB: we specifically *don't* call self.has_filename, because for
 
618
        # WorkingTrees that can indicate files that exist on disk but that
 
619
        # are not versioned.
 
620
        pred = self.inventory.has_filename
 
621
        return set((p for p in paths if not pred(p)))
544
622
 
545
623
    def walkdirs(self, prefix=""):
546
624
        """Walk the contents of this tree from path down.
598
676
        prefs = self.iter_search_rules([path], filter_pref_names).next()
599
677
        stk = filters._get_filter_stack_for(prefs)
600
678
        if 'filters' in debug.debug_flags:
601
 
            trace.note("*** %s content-filter: %s => %r" % (path,prefs,stk))
 
679
            note("*** %s content-filter: %s => %r" % (path,prefs,stk))
602
680
        return stk
603
681
 
604
682
    def _content_filter_stack_provider(self):
643
721
        return searcher
644
722
 
645
723
 
646
 
class InventoryTree(Tree):
647
 
    """A tree that relies on an inventory for its metadata.
648
 
 
649
 
    Trees contain an `Inventory` object, and also know how to retrieve
650
 
    file texts mentioned in the inventory, either from a working
651
 
    directory or from a store.
652
 
 
653
 
    It is possible for trees to contain files that are not described
654
 
    in their inventory or vice versa; for this use `filenames()`.
655
 
 
656
 
    Subclasses should set the _inventory attribute, which is considered
657
 
    private to external API users.
658
 
    """
659
 
 
660
 
    def get_canonical_inventory_paths(self, paths):
661
 
        """Like get_canonical_inventory_path() but works on multiple items.
662
 
 
663
 
        :param paths: A sequence of paths relative to the root of the tree.
664
 
        :return: A list of paths, with each item the corresponding input path
665
 
        adjusted to account for existing elements that match case
666
 
        insensitively.
667
 
        """
668
 
        return list(self._yield_canonical_inventory_paths(paths))
669
 
 
670
 
    def get_canonical_inventory_path(self, path):
671
 
        """Returns the first inventory item that case-insensitively matches path.
672
 
 
673
 
        If a path matches exactly, it is returned. If no path matches exactly
674
 
        but more than one path matches case-insensitively, it is implementation
675
 
        defined which is returned.
676
 
 
677
 
        If no path matches case-insensitively, the input path is returned, but
678
 
        with as many path entries that do exist changed to their canonical
679
 
        form.
680
 
 
681
 
        If you need to resolve many names from the same tree, you should
682
 
        use get_canonical_inventory_paths() to avoid O(N) behaviour.
683
 
 
684
 
        :param path: A paths relative to the root of the tree.
685
 
        :return: The input path adjusted to account for existing elements
686
 
        that match case insensitively.
687
 
        """
688
 
        return self._yield_canonical_inventory_paths([path]).next()
689
 
 
690
 
    def _yield_canonical_inventory_paths(self, paths):
691
 
        for path in paths:
692
 
            # First, if the path as specified exists exactly, just use it.
693
 
            if self.path2id(path) is not None:
694
 
                yield path
695
 
                continue
696
 
            # go walkin...
697
 
            cur_id = self.get_root_id()
698
 
            cur_path = ''
699
 
            bit_iter = iter(path.split("/"))
700
 
            for elt in bit_iter:
701
 
                lelt = elt.lower()
702
 
                new_path = None
703
 
                for child in self.iter_children(cur_id):
704
 
                    try:
705
 
                        # XXX: it seem like if the child is known to be in the
706
 
                        # tree, we shouldn't need to go from its id back to
707
 
                        # its path -- mbp 2010-02-11
708
 
                        #
709
 
                        # XXX: it seems like we could be more efficient
710
 
                        # by just directly looking up the original name and
711
 
                        # only then searching all children; also by not
712
 
                        # chopping paths so much. -- mbp 2010-02-11
713
 
                        child_base = os.path.basename(self.id2path(child))
714
 
                        if (child_base == elt):
715
 
                            # if we found an exact match, we can stop now; if
716
 
                            # we found an approximate match we need to keep
717
 
                            # searching because there might be an exact match
718
 
                            # later.  
719
 
                            cur_id = child
720
 
                            new_path = osutils.pathjoin(cur_path, child_base)
721
 
                            break
722
 
                        elif child_base.lower() == lelt:
723
 
                            cur_id = child
724
 
                            new_path = osutils.pathjoin(cur_path, child_base)
725
 
                    except errors.NoSuchId:
726
 
                        # before a change is committed we can see this error...
727
 
                        continue
728
 
                if new_path:
729
 
                    cur_path = new_path
730
 
                else:
731
 
                    # got to the end of this directory and no entries matched.
732
 
                    # Return what matched so far, plus the rest as specified.
733
 
                    cur_path = osutils.pathjoin(cur_path, elt, *list(bit_iter))
734
 
                    break
735
 
            yield cur_path
736
 
        # all done.
737
 
 
738
 
    def _get_inventory(self):
739
 
        return self._inventory
740
 
 
741
 
    inventory = property(_get_inventory,
742
 
                         doc="Inventory of this Tree")
743
 
 
744
 
    @needs_read_lock
745
 
    def path2id(self, path):
746
 
        """Return the id for path in this tree."""
747
 
        return self._inventory.path2id(path)
748
 
 
749
 
    def id2path(self, file_id):
750
 
        """Return the path for a file id.
751
 
 
752
 
        :raises NoSuchId:
753
 
        """
754
 
        return self.inventory.id2path(file_id)
755
 
 
756
 
    def has_id(self, file_id):
757
 
        return self.inventory.has_id(file_id)
758
 
 
759
 
    def has_or_had_id(self, file_id):
760
 
        return self.inventory.has_id(file_id)
761
 
 
762
 
    def all_file_ids(self):
763
 
        return set(self.inventory)
764
 
 
765
 
    @deprecated_method(deprecated_in((2, 4, 0)))
766
 
    def __iter__(self):
767
 
        return iter(self.inventory)
768
 
 
769
 
    def filter_unversioned_files(self, paths):
770
 
        """Filter out paths that are versioned.
771
 
 
772
 
        :return: set of paths.
773
 
        """
774
 
        # NB: we specifically *don't* call self.has_filename, because for
775
 
        # WorkingTrees that can indicate files that exist on disk but that
776
 
        # are not versioned.
777
 
        pred = self.inventory.has_filename
778
 
        return set((p for p in paths if not pred(p)))
779
 
 
780
 
    @needs_read_lock
781
 
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
782
 
        """Walk the tree in 'by_dir' order.
783
 
 
784
 
        This will yield each entry in the tree as a (path, entry) tuple.
785
 
        The order that they are yielded is:
786
 
 
787
 
        See Tree.iter_entries_by_dir for details.
788
 
 
789
 
        :param yield_parents: If True, yield the parents from the root leading
790
 
            down to specific_file_ids that have been requested. This has no
791
 
            impact if specific_file_ids is None.
792
 
        """
793
 
        return self.inventory.iter_entries_by_dir(
794
 
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
795
 
 
796
 
    def get_file_by_path(self, path):
797
 
        return self.get_file(self._inventory.path2id(path), path)
798
 
 
799
 
 
800
724
######################################################################
801
725
# diff
802
726
 
850
774
    return 'wtf?'
851
775
 
852
776
 
 
777
@deprecated_function(deprecated_in((1, 9, 0)))
 
778
def find_renames(old_inv, new_inv):
 
779
    for file_id in old_inv:
 
780
        if file_id not in new_inv:
 
781
            continue
 
782
        old_name = old_inv.id2path(file_id)
 
783
        new_name = new_inv.id2path(file_id)
 
784
        if old_name != new_name:
 
785
            yield (old_name, new_name)
 
786
 
 
787
 
853
788
def find_ids_across_trees(filenames, trees, require_versioned=True):
854
789
    """Find the ids corresponding to specified filenames.
855
790
 
942
877
 
943
878
    _optimisers = []
944
879
 
945
 
    @classmethod
946
 
    def is_compatible(kls, source, target):
947
 
        # The default implementation is naive and uses the public API, so
948
 
        # it works for all trees.
949
 
        return True
950
 
 
951
880
    def _changes_from_entries(self, source_entry, target_entry,
952
881
        source_path=None, target_path=None):
953
882
        """Generate a iter_changes tuple between source_entry and target_entry.
1058
987
            # All files are unversioned, so just return an empty delta
1059
988
            # _compare_trees would think we want a complete delta
1060
989
            result = delta.TreeDelta()
1061
 
            fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
 
990
            fake_entry = InventoryFile('unused', 'unused', 'unused')
1062
991
            result.unversioned = [(path, None,
1063
992
                self.target._comparison_data(fake_entry, path)[0]) for path in
1064
993
                specific_files]
1129
1058
                                     self.target.extras()
1130
1059
                if specific_files is None or
1131
1060
                    osutils.is_inside_any(specific_files, p)])
1132
 
            all_unversioned = collections.deque(all_unversioned)
 
1061
            all_unversioned = deque(all_unversioned)
1133
1062
        else:
1134
 
            all_unversioned = collections.deque()
 
1063
            all_unversioned = deque()
1135
1064
        to_paths = {}
1136
1065
        from_entries_by_dir = list(self.source.iter_entries_by_dir(
1137
1066
            specific_file_ids=specific_file_ids))
1143
1072
        # the unversioned path lookup only occurs on real trees - where there
1144
1073
        # can be extras. So the fake_entry is solely used to look up
1145
1074
        # executable it values when execute is not supported.
1146
 
        fake_entry = inventory.InventoryFile('unused', 'unused', 'unused')
 
1075
        fake_entry = InventoryFile('unused', 'unused', 'unused')
1147
1076
        for target_path, target_entry in to_entries_by_dir:
1148
1077
            while (all_unversioned and
1149
1078
                all_unversioned[0][0] < target_path.split('/')):
1197
1126
            if file_id in to_paths:
1198
1127
                # already returned
1199
1128
                continue
1200
 
            if not self.target.has_id(file_id):
 
1129
            if file_id not in self.target.all_file_ids():
1201
1130
                # common case - paths we have not emitted are not present in
1202
1131
                # target.
1203
1132
                to_path = None
1322
1251
                    yield result
1323
1252
 
1324
1253
 
1325
 
InterTree.register_optimiser(InterTree)
1326
 
 
1327
 
 
1328
1254
class MultiWalker(object):
1329
1255
    """Walk multiple trees simultaneously, getting combined results."""
1330
1256