~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tree.py

  • Committer: Martin Pool
  • Date: 2010-07-16 15:20:17 UTC
  • mfrom: (5346.3.1 pathnotchild)
  • mto: This revision was merged to the branch mainline in revision 5351.
  • Revision ID: mbp@canonical.com-20100716152017-t4c73h9y1uoih7fb
PathNotChild should not give a traceback.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2009 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
19
19
 
20
20
import os
21
21
from collections import deque
22
 
from cStringIO import StringIO
23
22
 
24
23
import bzrlib
25
24
from bzrlib import (
30
29
    osutils,
31
30
    revision as _mod_revision,
32
31
    rules,
33
 
    symbol_versioning,
34
32
    )
35
33
from bzrlib.decorators import needs_read_lock
36
 
from bzrlib.errors import BzrError, BzrCheckError, NoSuchId
 
34
from bzrlib.errors import BzrError, NoSuchId
37
35
from bzrlib import errors
38
 
from bzrlib.inventory import Inventory, InventoryFile
 
36
from bzrlib.inventory import InventoryFile
39
37
from bzrlib.inter import InterObject
40
38
from bzrlib.osutils import fingerprint_file
41
 
import bzrlib.revision
42
39
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
43
 
from bzrlib.trace import mutter, note
 
40
from bzrlib.trace import note
44
41
 
45
42
 
46
43
class Tree(object):
100
97
    def iter_changes(self, from_tree, include_unchanged=False,
101
98
                     specific_files=None, pb=None, extra_trees=None,
102
99
                     require_versioned=True, want_unversioned=False):
 
100
        """See InterTree.iter_changes"""
103
101
        intertree = InterTree.get(from_tree, self)
104
102
        return intertree.iter_changes(include_unchanged, specific_files, pb,
105
103
            extra_trees, require_versioned, want_unversioned=want_unversioned)
135
133
        return self.has_id(file_id)
136
134
 
137
135
    def has_or_had_id(self, file_id):
138
 
        if file_id == self.inventory.root.file_id:
139
 
            return True
140
136
        return self.inventory.has_id(file_id)
141
137
 
142
138
    def is_ignored(self, filename):
174
170
        return self.bzrdir.is_control_filename(filename)
175
171
 
176
172
    @needs_read_lock
177
 
    def iter_entries_by_dir(self, specific_file_ids=None):
 
173
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
178
174
        """Walk the tree in 'by_dir' order.
179
175
 
180
176
        This will yield each entry in the tree as a (path, entry) tuple.
197
193
 
198
194
        The yield order (ignoring root) would be::
199
195
          a, f, a/b, a/d, a/b/c, a/d/e, f/g
 
196
 
 
197
        :param yield_parents: If True, yield the parents from the root leading
 
198
            down to specific_file_ids that have been requested. This has no
 
199
            impact if specific_file_ids is None.
200
200
        """
201
201
        return self.inventory.iter_entries_by_dir(
202
 
            specific_file_ids=specific_file_ids)
 
202
            specific_file_ids=specific_file_ids, yield_parents=yield_parents)
203
203
 
204
204
    def iter_references(self):
205
 
        for path, entry in self.iter_entries_by_dir():
206
 
            if entry.kind == 'tree-reference':
207
 
                yield path, entry.file_id
 
205
        if self.supports_tree_reference():
 
206
            for path, entry in self.iter_entries_by_dir():
 
207
                if entry.kind == 'tree-reference':
 
208
                    yield path, entry.file_id
208
209
 
209
210
    def kind(self, file_id):
210
211
        raise NotImplementedError("Tree subclass %s must implement kind"
221
222
    def path_content_summary(self, path):
222
223
        """Get a summary of the information about path.
223
224
 
 
225
        All the attributes returned are for the canonical form, not the
 
226
        convenient form (if content filters are in use.)
 
227
 
224
228
        :param path: A relative path within the tree.
225
229
        :return: A tuple containing kind, size, exec, sha1-or-link.
226
230
            Kind is always present (see tree.kind()).
227
 
            size is present if kind is file, None otherwise.
 
231
            size is present if kind is file and the size of the 
 
232
                canonical form can be cheaply determined, None otherwise.
228
233
            exec is None unless kind is file and the platform supports the 'x'
229
234
                bit.
230
235
            sha1-or-link is the link target if kind is symlink, or the sha1 if
262
267
        """
263
268
        raise NotImplementedError(self.get_file)
264
269
 
 
270
    def get_file_with_stat(self, file_id, path=None):
 
271
        """Get a file handle and stat object for file_id.
 
272
 
 
273
        The default implementation returns (self.get_file, None) for backwards
 
274
        compatibility.
 
275
 
 
276
        :param file_id: The file id to read.
 
277
        :param path: The path of the file, if it is known.
 
278
        :return: A tuple (file_handle, stat_value_or_None). If the tree has
 
279
            no stat facility, or need for a stat cache feedback during commit,
 
280
            it may return None for the second element of the tuple.
 
281
        """
 
282
        return (self.get_file(file_id, path), None)
 
283
 
265
284
    def get_file_text(self, file_id, path=None):
266
285
        """Return the byte content of a file.
267
286
 
385
404
            bit_iter = iter(path.split("/"))
386
405
            for elt in bit_iter:
387
406
                lelt = elt.lower()
 
407
                new_path = None
388
408
                for child in self.iter_children(cur_id):
389
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
390
418
                        child_base = os.path.basename(self.id2path(child))
391
 
                        if child_base.lower() == lelt:
 
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.  
392
424
                            cur_id = child
393
 
                            cur_path = osutils.pathjoin(cur_path, child_base)
 
425
                            new_path = osutils.pathjoin(cur_path, child_base)
394
426
                            break
 
427
                        elif child_base.lower() == lelt:
 
428
                            cur_id = child
 
429
                            new_path = osutils.pathjoin(cur_path, child_base)
395
430
                    except NoSuchId:
396
431
                        # before a change is committed we can see this error...
397
432
                        continue
 
433
                if new_path:
 
434
                    cur_path = new_path
398
435
                else:
399
436
                    # got to the end of this directory and no entries matched.
400
437
                    # Return what matched so far, plus the rest as specified.
422
459
        raise NotImplementedError(self.annotate_iter)
423
460
 
424
461
    def _get_plan_merge_data(self, file_id, other, base):
425
 
        from bzrlib import merge, versionedfile
 
462
        from bzrlib import versionedfile
426
463
        vf = versionedfile._PlanMergeVersionedFile(file_id)
427
464
        last_revision_a = self._get_file_revision(file_id, vf, 'this:')
428
465
        last_revision_b = other._get_file_revision(file_id, vf, 'other:')
483
520
            parent_keys = [(file_id, self._file_revision(t, file_id)) for t in
484
521
                self._iter_parent_trees()]
485
522
            vf.add_lines((file_id, last_revision), parent_keys,
486
 
                         self.get_file(file_id).readlines())
 
523
                         self.get_file_lines(file_id))
487
524
            repo = self.branch.repository
488
525
            base_vf = repo.texts
489
526
        else:
545
582
            yield child.file_id
546
583
 
547
584
    def lock_read(self):
 
585
        """Lock this tree for multiple read only operations.
 
586
        
 
587
        :return: A bzrlib.lock.LogicalLockResult.
 
588
        """
548
589
        pass
549
590
 
550
591
    def revision_tree(self, revision_id):
657
698
            return None
658
699
 
659
700
    def iter_search_rules(self, path_names, pref_names=None,
660
 
        _default_searcher=rules._per_user_searcher):
 
701
        _default_searcher=None):
661
702
        """Find the preferences for filenames in a tree.
662
703
 
663
704
        :param path_names: an iterable of paths to find attributes for.
667
708
        :return: an iterator of tuple sequences, one per path-name.
668
709
          See _RulesSearcher.get_items for details on the tuple sequence.
669
710
        """
 
711
        if _default_searcher is None:
 
712
            _default_searcher = rules._per_user_searcher
670
713
        searcher = self._get_rules_searcher(_default_searcher)
671
714
        if searcher is not None:
672
715
            if pref_names is not None:
676
719
                for path in path_names:
677
720
                    yield searcher.get_items(path)
678
721
 
679
 
    @needs_read_lock
680
722
    def _get_rules_searcher(self, default_searcher):
681
723
        """Get the RulesSearcher for this tree given the default one."""
682
724
        searcher = default_searcher
810
852
        new_pending = set()
811
853
        for file_id in pending:
812
854
            for tree in trees:
813
 
                if not tree.has_id(file_id):
 
855
                if not tree.has_or_had_id(file_id):
814
856
                    continue
815
857
                for child_id in tree.iter_children(file_id):
816
858
                    if child_id not in interesting_ids:
831
873
    will pass through to InterTree as appropriate.
832
874
    """
833
875
 
 
876
    # Formats that will be used to test this InterTree. If both are
 
877
    # None, this InterTree will not be tested (e.g. because a complex
 
878
    # setup is required)
 
879
    _matching_from_tree_format = None
 
880
    _matching_to_tree_format = None
 
881
 
834
882
    _optimisers = []
835
883
 
 
884
    def _changes_from_entries(self, source_entry, target_entry,
 
885
        source_path=None, target_path=None):
 
886
        """Generate a iter_changes tuple between source_entry and target_entry.
 
887
 
 
888
        :param source_entry: An inventory entry from self.source, or None.
 
889
        :param target_entry: An inventory entry from self.target, or None.
 
890
        :param source_path: The path of source_entry, if known. If not known
 
891
            it will be looked up.
 
892
        :param target_path: The path of target_entry, if known. If not known
 
893
            it will be looked up.
 
894
        :return: A tuple, item 0 of which is an iter_changes result tuple, and
 
895
            item 1 is True if there are any changes in the result tuple.
 
896
        """
 
897
        if source_entry is None:
 
898
            if target_entry is None:
 
899
                return None
 
900
            file_id = target_entry.file_id
 
901
        else:
 
902
            file_id = source_entry.file_id
 
903
        if source_entry is not None:
 
904
            source_versioned = True
 
905
            source_name = source_entry.name
 
906
            source_parent = source_entry.parent_id
 
907
            if source_path is None:
 
908
                source_path = self.source.id2path(file_id)
 
909
            source_kind, source_executable, source_stat = \
 
910
                self.source._comparison_data(source_entry, source_path)
 
911
        else:
 
912
            source_versioned = False
 
913
            source_name = None
 
914
            source_parent = None
 
915
            source_kind = None
 
916
            source_executable = None
 
917
        if target_entry is not None:
 
918
            target_versioned = True
 
919
            target_name = target_entry.name
 
920
            target_parent = target_entry.parent_id
 
921
            if target_path is None:
 
922
                target_path = self.target.id2path(file_id)
 
923
            target_kind, target_executable, target_stat = \
 
924
                self.target._comparison_data(target_entry, target_path)
 
925
        else:
 
926
            target_versioned = False
 
927
            target_name = None
 
928
            target_parent = None
 
929
            target_kind = None
 
930
            target_executable = None
 
931
        versioned = (source_versioned, target_versioned)
 
932
        kind = (source_kind, target_kind)
 
933
        changed_content = False
 
934
        if source_kind != target_kind:
 
935
            changed_content = True
 
936
        elif source_kind == 'file':
 
937
            if (self.source.get_file_sha1(file_id, source_path, source_stat) !=
 
938
                self.target.get_file_sha1(file_id, target_path, target_stat)):
 
939
                changed_content = True
 
940
        elif source_kind == 'symlink':
 
941
            if (self.source.get_symlink_target(file_id) !=
 
942
                self.target.get_symlink_target(file_id)):
 
943
                changed_content = True
 
944
            # XXX: Yes, the indentation below is wrong. But fixing it broke
 
945
            # test_merge.TestMergerEntriesLCAOnDisk.
 
946
            # test_nested_tree_subtree_renamed_and_modified. We'll wait for
 
947
            # the fix from bzr.dev -- vila 2009026
 
948
            elif source_kind == 'tree-reference':
 
949
                if (self.source.get_reference_revision(file_id, source_path)
 
950
                    != self.target.get_reference_revision(file_id, target_path)):
 
951
                    changed_content = True
 
952
        parent = (source_parent, target_parent)
 
953
        name = (source_name, target_name)
 
954
        executable = (source_executable, target_executable)
 
955
        if (changed_content is not False or versioned[0] != versioned[1]
 
956
            or parent[0] != parent[1] or name[0] != name[1] or
 
957
            executable[0] != executable[1]):
 
958
            changes = True
 
959
        else:
 
960
            changes = False
 
961
        return (file_id, (source_path, target_path), changed_content,
 
962
                versioned, parent, name, kind, executable), changes
 
963
 
836
964
    @needs_read_lock
837
965
    def compare(self, want_unchanged=False, specific_files=None,
838
966
        extra_trees=None, require_versioned=False, include_root=False,
853
981
            a PathsNotVersionedError will be thrown.
854
982
        :param want_unversioned: Scan for unversioned paths.
855
983
        """
856
 
        # NB: show_status depends on being able to pass in non-versioned files
857
 
        # and report them as unknown
858
984
        trees = (self.source,)
859
985
        if extra_trees is not None:
860
986
            trees = trees + tuple(extra_trees)
901
1027
        :param require_versioned: Raise errors.PathsNotVersionedError if a
902
1028
            path in the specific_files list is not versioned in one of
903
1029
            source, target or extra_trees.
 
1030
        :param specific_files: An optional list of file paths to restrict the
 
1031
            comparison to. When mapping filenames to ids, all matches in all
 
1032
            trees (including optional extra_trees) are used, and all children
 
1033
            of matched directories are included. The parents in the target tree
 
1034
            of the specific files up to and including the root of the tree are
 
1035
            always evaluated for changes too.
904
1036
        :param want_unversioned: Should unversioned files be returned in the
905
1037
            output. An unversioned file is defined as one with (False, False)
906
1038
            for the versioned pair.
908
1040
        lookup_trees = [self.source]
909
1041
        if extra_trees:
910
1042
             lookup_trees.extend(extra_trees)
 
1043
        # The ids of items we need to examine to insure delta consistency.
 
1044
        precise_file_ids = set()
 
1045
        changed_file_ids = []
911
1046
        if specific_files == []:
912
1047
            specific_file_ids = []
913
1048
        else:
914
1049
            specific_file_ids = self.target.paths2ids(specific_files,
915
1050
                lookup_trees, require_versioned=require_versioned)
 
1051
        if specific_files is not None:
 
1052
            # reparented or added entries must have their parents included
 
1053
            # so that valid deltas can be created. The seen_parents set
 
1054
            # tracks the parents that we need to have.
 
1055
            # The seen_dirs set tracks directory entries we've yielded.
 
1056
            # After outputting version object in to_entries we set difference
 
1057
            # the two seen sets and start checking parents.
 
1058
            seen_parents = set()
 
1059
            seen_dirs = set()
916
1060
        if want_unversioned:
917
1061
            all_unversioned = sorted([(p.split('/'), p) for p in
918
1062
                                     self.target.extras()
933
1077
        # can be extras. So the fake_entry is solely used to look up
934
1078
        # executable it values when execute is not supported.
935
1079
        fake_entry = InventoryFile('unused', 'unused', 'unused')
936
 
        for to_path, to_entry in to_entries_by_dir:
937
 
            while all_unversioned and all_unversioned[0][0] < to_path.split('/'):
 
1080
        for target_path, target_entry in to_entries_by_dir:
 
1081
            while (all_unversioned and
 
1082
                all_unversioned[0][0] < target_path.split('/')):
938
1083
                unversioned_path = all_unversioned.popleft()
939
 
                to_kind, to_executable, to_stat = \
 
1084
                target_kind, target_executable, target_stat = \
940
1085
                    self.target._comparison_data(fake_entry, unversioned_path[1])
941
1086
                yield (None, (None, unversioned_path[1]), True, (False, False),
942
1087
                    (None, None),
943
1088
                    (None, unversioned_path[0][-1]),
944
 
                    (None, to_kind),
945
 
                    (None, to_executable))
946
 
            file_id = to_entry.file_id
947
 
            to_paths[file_id] = to_path
 
1089
                    (None, target_kind),
 
1090
                    (None, target_executable))
 
1091
            source_path, source_entry = from_data.get(target_entry.file_id,
 
1092
                (None, None))
 
1093
            result, changes = self._changes_from_entries(source_entry,
 
1094
                target_entry, source_path=source_path, target_path=target_path)
 
1095
            to_paths[result[0]] = result[1][1]
948
1096
            entry_count += 1
949
 
            changed_content = False
950
 
            from_path, from_entry = from_data.get(file_id, (None, None))
951
 
            from_versioned = (from_entry is not None)
952
 
            if from_entry is not None:
953
 
                from_versioned = True
954
 
                from_name = from_entry.name
955
 
                from_parent = from_entry.parent_id
956
 
                from_kind, from_executable, from_stat = \
957
 
                    self.source._comparison_data(from_entry, from_path)
 
1097
            if result[3][0]:
958
1098
                entry_count += 1
959
 
            else:
960
 
                from_versioned = False
961
 
                from_kind = None
962
 
                from_parent = None
963
 
                from_name = None
964
 
                from_executable = None
965
 
            versioned = (from_versioned, True)
966
 
            to_kind, to_executable, to_stat = \
967
 
                self.target._comparison_data(to_entry, to_path)
968
 
            kind = (from_kind, to_kind)
969
 
            if kind[0] != kind[1]:
970
 
                changed_content = True
971
 
            elif from_kind == 'file':
972
 
                if (self.source.get_file_sha1(file_id, from_path, from_stat) !=
973
 
                    self.target.get_file_sha1(file_id, to_path, to_stat)):
974
 
                    changed_content = True
975
 
            elif from_kind == 'symlink':
976
 
                if (self.source.get_symlink_target(file_id) !=
977
 
                    self.target.get_symlink_target(file_id)):
978
 
                    changed_content = True
979
 
                # XXX: Yes, the indentation below is wrong. But fixing it broke
980
 
                # test_merge.TestMergerEntriesLCAOnDisk.
981
 
                # test_nested_tree_subtree_renamed_and_modified. We'll wait for
982
 
                # the fix from bzr.dev -- vila 2009026
983
 
                elif from_kind == 'tree-reference':
984
 
                    if (self.source.get_reference_revision(file_id, from_path)
985
 
                        != self.target.get_reference_revision(file_id, to_path)):
986
 
                        changed_content = True
987
 
            parent = (from_parent, to_entry.parent_id)
988
 
            name = (from_name, to_entry.name)
989
 
            executable = (from_executable, to_executable)
990
1099
            if pb is not None:
991
1100
                pb.update('comparing files', entry_count, num_entries)
992
 
            if (changed_content is not False or versioned[0] != versioned[1]
993
 
                or parent[0] != parent[1] or name[0] != name[1] or
994
 
                executable[0] != executable[1] or include_unchanged):
995
 
                yield (file_id, (from_path, to_path), changed_content,
996
 
                    versioned, parent, name, kind, executable)
997
 
 
 
1101
            if changes or include_unchanged:
 
1102
                if specific_file_ids is not None:
 
1103
                    new_parent_id = result[4][1]
 
1104
                    precise_file_ids.add(new_parent_id)
 
1105
                    changed_file_ids.append(result[0])
 
1106
                yield result
 
1107
            # Ensure correct behaviour for reparented/added specific files.
 
1108
            if specific_files is not None:
 
1109
                # Record output dirs
 
1110
                if result[6][1] == 'directory':
 
1111
                    seen_dirs.add(result[0])
 
1112
                # Record parents of reparented/added entries.
 
1113
                versioned = result[3]
 
1114
                parents = result[4]
 
1115
                if not versioned[0] or parents[0] != parents[1]:
 
1116
                    seen_parents.add(parents[1])
998
1117
        while all_unversioned:
999
1118
            # yield any trailing unversioned paths
1000
1119
            unversioned_path = all_unversioned.popleft()
1005
1124
                (None, unversioned_path[0][-1]),
1006
1125
                (None, to_kind),
1007
1126
                (None, to_executable))
1008
 
 
1009
 
        def get_to_path(to_entry):
1010
 
            if to_entry.parent_id is None:
1011
 
                to_path = '' # the root
1012
 
            else:
1013
 
                if to_entry.parent_id not in to_paths:
1014
 
                    # recurse up
1015
 
                    return get_to_path(self.target.inventory[to_entry.parent_id])
1016
 
                to_path = osutils.pathjoin(to_paths[to_entry.parent_id],
1017
 
                                           to_entry.name)
1018
 
            to_paths[to_entry.file_id] = to_path
1019
 
            return to_path
1020
 
 
 
1127
        # Yield all remaining source paths
1021
1128
        for path, from_entry in from_entries_by_dir:
1022
1129
            file_id = from_entry.file_id
1023
1130
            if file_id in to_paths:
1024
1131
                # already returned
1025
1132
                continue
1026
 
            if not file_id in self.target.all_file_ids():
 
1133
            if file_id not in self.target.all_file_ids():
1027
1134
                # common case - paths we have not emitted are not present in
1028
1135
                # target.
1029
1136
                to_path = None
1030
1137
            else:
1031
 
                to_path = get_to_path(self.target.inventory[file_id])
 
1138
                to_path = self.target.id2path(file_id)
1032
1139
            entry_count += 1
1033
1140
            if pb is not None:
1034
1141
                pb.update('comparing files', entry_count, num_entries)
1041
1148
            executable = (from_executable, None)
1042
1149
            changed_content = from_kind is not None
1043
1150
            # the parent's path is necessarily known at this point.
 
1151
            changed_file_ids.append(file_id)
1044
1152
            yield(file_id, (path, to_path), changed_content, versioned, parent,
1045
1153
                  name, kind, executable)
 
1154
        changed_file_ids = set(changed_file_ids)
 
1155
        if specific_file_ids is not None:
 
1156
            for result in self._handle_precise_ids(precise_file_ids,
 
1157
                changed_file_ids):
 
1158
                yield result
 
1159
 
 
1160
    def _get_entry(self, tree, file_id):
 
1161
        """Get an inventory entry from a tree, with missing entries as None.
 
1162
 
 
1163
        If the tree raises NotImplementedError on accessing .inventory, then
 
1164
        this is worked around using iter_entries_by_dir on just the file id
 
1165
        desired.
 
1166
 
 
1167
        :param tree: The tree to lookup the entry in.
 
1168
        :param file_id: The file_id to lookup.
 
1169
        """
 
1170
        try:
 
1171
            inventory = tree.inventory
 
1172
        except NotImplementedError:
 
1173
            # No inventory available.
 
1174
            try:
 
1175
                iterator = tree.iter_entries_by_dir(specific_file_ids=[file_id])
 
1176
                return iterator.next()[1]
 
1177
            except StopIteration:
 
1178
                return None
 
1179
        else:
 
1180
            try:
 
1181
                return inventory[file_id]
 
1182
            except errors.NoSuchId:
 
1183
                return None
 
1184
 
 
1185
    def _handle_precise_ids(self, precise_file_ids, changed_file_ids,
 
1186
        discarded_changes=None):
 
1187
        """Fill out a partial iter_changes to be consistent.
 
1188
 
 
1189
        :param precise_file_ids: The file ids of parents that were seen during
 
1190
            the iter_changes.
 
1191
        :param changed_file_ids: The file ids of already emitted items.
 
1192
        :param discarded_changes: An optional dict of precalculated
 
1193
            iter_changes items which the partial iter_changes had not output
 
1194
            but had calculated.
 
1195
        :return: A generator of iter_changes items to output.
 
1196
        """
 
1197
        # process parents of things that had changed under the users
 
1198
        # requested paths to prevent incorrect paths or parent ids which
 
1199
        # aren't in the tree.
 
1200
        while precise_file_ids:
 
1201
            precise_file_ids.discard(None)
 
1202
            # Don't emit file_ids twice
 
1203
            precise_file_ids.difference_update(changed_file_ids)
 
1204
            if not precise_file_ids:
 
1205
                break
 
1206
            # If the there was something at a given output path in source, we
 
1207
            # have to include the entry from source in the delta, or we would
 
1208
            # be putting this entry into a used path.
 
1209
            paths = []
 
1210
            for parent_id in precise_file_ids:
 
1211
                try:
 
1212
                    paths.append(self.target.id2path(parent_id))
 
1213
                except errors.NoSuchId:
 
1214
                    # This id has been dragged in from the source by delta
 
1215
                    # expansion and isn't present in target at all: we don't
 
1216
                    # need to check for path collisions on it.
 
1217
                    pass
 
1218
            for path in paths:
 
1219
                old_id = self.source.path2id(path)
 
1220
                precise_file_ids.add(old_id)
 
1221
            precise_file_ids.discard(None)
 
1222
            current_ids = precise_file_ids
 
1223
            precise_file_ids = set()
 
1224
            # We have to emit all of precise_file_ids that have been altered.
 
1225
            # We may have to output the children of some of those ids if any
 
1226
            # directories have stopped being directories.
 
1227
            for file_id in current_ids:
 
1228
                # Examine file_id
 
1229
                if discarded_changes:
 
1230
                    result = discarded_changes.get(file_id)
 
1231
                    old_entry = None
 
1232
                else:
 
1233
                    result = None
 
1234
                if result is None:
 
1235
                    old_entry = self._get_entry(self.source, file_id)
 
1236
                    new_entry = self._get_entry(self.target, file_id)
 
1237
                    result, changes = self._changes_from_entries(
 
1238
                        old_entry, new_entry)
 
1239
                else:
 
1240
                    changes = True
 
1241
                # Get this parents parent to examine.
 
1242
                new_parent_id = result[4][1]
 
1243
                precise_file_ids.add(new_parent_id)
 
1244
                if changes:
 
1245
                    if (result[6][0] == 'directory' and
 
1246
                        result[6][1] != 'directory'):
 
1247
                        # This stopped being a directory, the old children have
 
1248
                        # to be included.
 
1249
                        if old_entry is None:
 
1250
                            # Reusing a discarded change.
 
1251
                            old_entry = self._get_entry(self.source, file_id)
 
1252
                        for child in old_entry.children.values():
 
1253
                            precise_file_ids.add(child.file_id)
 
1254
                    changed_file_ids.add(result[0])
 
1255
                    yield result
1046
1256
 
1047
1257
 
1048
1258
class MultiWalker(object):