← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tree.py

  • Committer: John Arbash Meinel
  • Date: 2009-06-16 13:57:14 UTC
  • mto: This revision was merged to the branch mainline in revision 4449.
  • Revision ID: john@arbash-meinel.com-20090616135714-8o7jdtqqsfuv914z
The new code removes a get_parent_map call.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tree.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2009 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
22
23
 
23
24
import bzrlib
24
25
from bzrlib import (
29
30
    osutils,
30
31
    revision as _mod_revision,
31
32
    rules,
 
33
    symbol_versioning,
32
34
    )
33
35
from bzrlib.decorators import needs_read_lock
34
 
from bzrlib.errors import BzrError, NoSuchId
 
36
from bzrlib.errors import BzrError, BzrCheckError, NoSuchId
35
37
from bzrlib import errors
36
 
from bzrlib.inventory import InventoryFile
 
38
from bzrlib.inventory import Inventory, InventoryFile
37
39
from bzrlib.inter import InterObject
38
40
from bzrlib.osutils import fingerprint_file
 
41
import bzrlib.revision
39
42
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
40
 
from bzrlib.trace import note
 
43
from bzrlib.trace import mutter, note
41
44
 
42
45
 
43
46
class Tree(object):
97
100
    def iter_changes(self, from_tree, include_unchanged=False,
98
101
                     specific_files=None, pb=None, extra_trees=None,
99
102
                     require_versioned=True, want_unversioned=False):
100
 
        """See InterTree.iter_changes"""
101
103
        intertree = InterTree.get(from_tree, self)
102
104
        return intertree.iter_changes(include_unchanged, specific_files, pb,
103
105
            extra_trees, require_versioned, want_unversioned=want_unversioned)
133
135
        return self.has_id(file_id)
134
136
 
135
137
    def has_or_had_id(self, file_id):
 
138
        if file_id == self.inventory.root.file_id:
 
139
            return True
136
140
        return self.inventory.has_id(file_id)
137
141
 
138
142
    def is_ignored(self, filename):
170
174
        return self.bzrdir.is_control_filename(filename)
171
175
 
172
176
    @needs_read_lock
173
 
    def iter_entries_by_dir(self, specific_file_ids=None, yield_parents=False):
 
177
    def iter_entries_by_dir(self, specific_file_ids=None):
174
178
        """Walk the tree in 'by_dir' order.
175
179
 
176
180
        This will yield each entry in the tree as a (path, entry) tuple.
193
197
 
194
198
        The yield order (ignoring root) would be::
195
199
          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, yield_parents=yield_parents)
 
202
            specific_file_ids=specific_file_ids)
203
203
 
204
204
    def iter_references(self):
205
205
        if self.supports_tree_reference():
222
222
    def path_content_summary(self, path):
223
223
        """Get a summary of the information about path.
224
224
 
225
 
        All the attributes returned are for the canonical form, not the
226
 
        convenient form (if content filters are in use.)
227
 
 
228
225
        :param path: A relative path within the tree.
229
226
        :return: A tuple containing kind, size, exec, sha1-or-link.
230
227
            Kind is always present (see tree.kind()).
231
 
            size is present if kind is file and the size of the 
232
 
                canonical form can be cheaply determined, None otherwise.
 
228
            size is present if kind is file, None otherwise.
233
229
            exec is None unless kind is file and the platform supports the 'x'
234
230
                bit.
235
231
            sha1-or-link is the link target if kind is symlink, or the sha1 if
404
400
            bit_iter = iter(path.split("/"))
405
401
            for elt in bit_iter:
406
402
                lelt = elt.lower()
407
 
                new_path = None
408
403
                for child in self.iter_children(cur_id):
409
404
                    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
405
                        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.  
 
406
                        if child_base.lower() == lelt:
424
407
                            cur_id = child
425
 
                            new_path = osutils.pathjoin(cur_path, child_base)
 
408
                            cur_path = osutils.pathjoin(cur_path, child_base)
426
409
                            break
427
 
                        elif child_base.lower() == lelt:
428
 
                            cur_id = child
429
 
                            new_path = osutils.pathjoin(cur_path, child_base)
430
410
                    except NoSuchId:
431
411
                        # before a change is committed we can see this error...
432
412
                        continue
433
 
                if new_path:
434
 
                    cur_path = new_path
435
413
                else:
436
414
                    # got to the end of this directory and no entries matched.
437
415
                    # Return what matched so far, plus the rest as specified.
459
437
        raise NotImplementedError(self.annotate_iter)
460
438
 
461
439
    def _get_plan_merge_data(self, file_id, other, base):
462
 
        from bzrlib import versionedfile
 
440
        from bzrlib import merge, versionedfile
463
441
        vf = versionedfile._PlanMergeVersionedFile(file_id)
464
442
        last_revision_a = self._get_file_revision(file_id, vf, 'this:')
465
443
        last_revision_b = other._get_file_revision(file_id, vf, 'other:')
520
498
            parent_keys = [(file_id, self._file_revision(t, file_id)) for t in
521
499
                self._iter_parent_trees()]
522
500
            vf.add_lines((file_id, last_revision), parent_keys,
523
 
                         self.get_file_lines(file_id))
 
501
                         self.get_file(file_id).readlines())
524
502
            repo = self.branch.repository
525
503
            base_vf = repo.texts
526
504
        else:
582
560
            yield child.file_id
583
561
 
584
562
    def lock_read(self):
585
 
        """Lock this tree for multiple read only operations.
586
 
        
587
 
        :return: A bzrlib.lock.LogicalLockResult.
588
 
        """
589
563
        pass
590
564
 
591
565
    def revision_tree(self, revision_id):
719
693
                for path in path_names:
720
694
                    yield searcher.get_items(path)
721
695
 
 
696
    @needs_read_lock
722
697
    def _get_rules_searcher(self, default_searcher):
723
698
        """Get the RulesSearcher for this tree given the default one."""
724
699
        searcher = default_searcher
852
827
        new_pending = set()
853
828
        for file_id in pending:
854
829
            for tree in trees:
855
 
                if not tree.has_or_had_id(file_id):
 
830
                if not tree.has_id(file_id):
856
831
                    continue
857
832
                for child_id in tree.iter_children(file_id):
858
833
                    if child_id not in interesting_ids:
873
848
    will pass through to InterTree as appropriate.
874
849
    """
875
850
 
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
 
 
882
851
    _optimisers = []
883
852
 
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
 
 
964
853
    @needs_read_lock
965
854
    def compare(self, want_unchanged=False, specific_files=None,
966
855
        extra_trees=None, require_versioned=False, include_root=False,
981
870
            a PathsNotVersionedError will be thrown.
982
871
        :param want_unversioned: Scan for unversioned paths.
983
872
        """
 
873
        # NB: show_status depends on being able to pass in non-versioned files
 
874
        # and report them as unknown
984
875
        trees = (self.source,)
985
876
        if extra_trees is not None:
986
877
            trees = trees + tuple(extra_trees)
1027
918
        :param require_versioned: Raise errors.PathsNotVersionedError if a
1028
919
            path in the specific_files list is not versioned in one of
1029
920
            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.
1036
921
        :param want_unversioned: Should unversioned files be returned in the
1037
922
            output. An unversioned file is defined as one with (False, False)
1038
923
            for the versioned pair.
1040
925
        lookup_trees = [self.source]
1041
926
        if extra_trees:
1042
927
             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 = []
1046
928
        if specific_files == []:
1047
929
            specific_file_ids = []
1048
930
        else:
1049
931
            specific_file_ids = self.target.paths2ids(specific_files,
1050
932
                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()
1060
933
        if want_unversioned:
1061
934
            all_unversioned = sorted([(p.split('/'), p) for p in
1062
935
                                     self.target.extras()
1077
950
        # can be extras. So the fake_entry is solely used to look up
1078
951
        # executable it values when execute is not supported.
1079
952
        fake_entry = InventoryFile('unused', 'unused', 'unused')
1080
 
        for target_path, target_entry in to_entries_by_dir:
1081
 
            while (all_unversioned and
1082
 
                all_unversioned[0][0] < target_path.split('/')):
 
953
        for to_path, to_entry in to_entries_by_dir:
 
954
            while all_unversioned and all_unversioned[0][0] < to_path.split('/'):
1083
955
                unversioned_path = all_unversioned.popleft()
1084
 
                target_kind, target_executable, target_stat = \
 
956
                to_kind, to_executable, to_stat = \
1085
957
                    self.target._comparison_data(fake_entry, unversioned_path[1])
1086
958
                yield (None, (None, unversioned_path[1]), True, (False, False),
1087
959
                    (None, None),
1088
960
                    (None, unversioned_path[0][-1]),
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]
 
961
                    (None, to_kind),
 
962
                    (None, to_executable))
 
963
            file_id = to_entry.file_id
 
964
            to_paths[file_id] = to_path
1096
965
            entry_count += 1
1097
 
            if result[3][0]:
 
966
            changed_content = False
 
967
            from_path, from_entry = from_data.get(file_id, (None, None))
 
968
            from_versioned = (from_entry is not None)
 
969
            if from_entry is not None:
 
970
                from_versioned = True
 
971
                from_name = from_entry.name
 
972
                from_parent = from_entry.parent_id
 
973
                from_kind, from_executable, from_stat = \
 
974
                    self.source._comparison_data(from_entry, from_path)
1098
975
                entry_count += 1
 
976
            else:
 
977
                from_versioned = False
 
978
                from_kind = None
 
979
                from_parent = None
 
980
                from_name = None
 
981
                from_executable = None
 
982
            versioned = (from_versioned, True)
 
983
            to_kind, to_executable, to_stat = \
 
984
                self.target._comparison_data(to_entry, to_path)
 
985
            kind = (from_kind, to_kind)
 
986
            if kind[0] != kind[1]:
 
987
                changed_content = True
 
988
            elif from_kind == 'file':
 
989
                if (self.source.get_file_sha1(file_id, from_path, from_stat) !=
 
990
                    self.target.get_file_sha1(file_id, to_path, to_stat)):
 
991
                    changed_content = True
 
992
            elif from_kind == 'symlink':
 
993
                if (self.source.get_symlink_target(file_id) !=
 
994
                    self.target.get_symlink_target(file_id)):
 
995
                    changed_content = True
 
996
                # XXX: Yes, the indentation below is wrong. But fixing it broke
 
997
                # test_merge.TestMergerEntriesLCAOnDisk.
 
998
                # test_nested_tree_subtree_renamed_and_modified. We'll wait for
 
999
                # the fix from bzr.dev -- vila 2009026
 
1000
                elif from_kind == 'tree-reference':
 
1001
                    if (self.source.get_reference_revision(file_id, from_path)
 
1002
                        != self.target.get_reference_revision(file_id, to_path)):
 
1003
                        changed_content = True
 
1004
            parent = (from_parent, to_entry.parent_id)
 
1005
            name = (from_name, to_entry.name)
 
1006
            executable = (from_executable, to_executable)
1099
1007
            if pb is not None:
1100
1008
                pb.update('comparing files', entry_count, num_entries)
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])
 
1009
            if (changed_content is not False or versioned[0] != versioned[1]
 
1010
                or parent[0] != parent[1] or name[0] != name[1] or
 
1011
                executable[0] != executable[1] or include_unchanged):
 
1012
                yield (file_id, (from_path, to_path), changed_content,
 
1013
                    versioned, parent, name, kind, executable)
 
1014
 
1117
1015
        while all_unversioned:
1118
1016
            # yield any trailing unversioned paths
1119
1017
            unversioned_path = all_unversioned.popleft()
1124
1022
                (None, unversioned_path[0][-1]),
1125
1023
                (None, to_kind),
1126
1024
                (None, to_executable))
1127
 
        # Yield all remaining source paths
 
1025
 
 
1026
        def get_to_path(to_entry):
 
1027
            if to_entry.parent_id is None:
 
1028
                to_path = '' # the root
 
1029
            else:
 
1030
                if to_entry.parent_id not in to_paths:
 
1031
                    # recurse up
 
1032
                    return get_to_path(self.target.inventory[to_entry.parent_id])
 
1033
                to_path = osutils.pathjoin(to_paths[to_entry.parent_id],
 
1034
                                           to_entry.name)
 
1035
            to_paths[to_entry.file_id] = to_path
 
1036
            return to_path
 
1037
 
1128
1038
        for path, from_entry in from_entries_by_dir:
1129
1039
            file_id = from_entry.file_id
1130
1040
            if file_id in to_paths:
1131
1041
                # already returned
1132
1042
                continue
1133
 
            if not self.target.has_id(file_id):
 
1043
            if not file_id in self.target.all_file_ids():
1134
1044
                # common case - paths we have not emitted are not present in
1135
1045
                # target.
1136
1046
                to_path = None
1137
1047
            else:
1138
 
                to_path = self.target.id2path(file_id)
 
1048
                to_path = get_to_path(self.target.inventory[file_id])
1139
1049
            entry_count += 1
1140
1050
            if pb is not None:
1141
1051
                pb.update('comparing files', entry_count, num_entries)
1148
1058
            executable = (from_executable, None)
1149
1059
            changed_content = from_kind is not None
1150
1060
            # the parent's path is necessarily known at this point.
1151
 
            changed_file_ids.append(file_id)
1152
1061
            yield(file_id, (path, to_path), changed_content, versioned, parent,
1153
1062
                  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
1256
1063
 
1257
1064
 
1258
1065
class MultiWalker(object):