← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/revisiontree.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-09-01 08:02:42 UTC
  • mfrom: (5390.3.3 faster-revert-593560)
  • Revision ID: pqm@pqm.ubuntu.com-20100901080242-esg62ody4frwmy66
(spiv) Avoid repeatedly calling self.target.all_file_ids() in
 InterTree.iter_changes. (Andrew Bennetts)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/revisiontree.py
1
 
# Copyright (C) 2005, 2007 Canonical Ltd
 
1
# Copyright (C) 2006-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
20
20
 
21
21
from bzrlib import (
22
22
    errors,
23
 
    osutils,
24
23
    revision,
25
 
    symbol_versioning,
26
24
    tree,
27
25
    )
28
26
 
45
43
        self._rules_searcher = None
46
44
 
47
45
    def supports_tree_reference(self):
48
 
        return True
 
46
        return getattr(self._repository._format, "supports_tree_reference",
 
47
            False)
49
48
 
50
49
    def get_parent_ids(self):
51
50
        """See Tree.get_parent_ids.
86
85
                      default_revision=revision.CURRENT_REVISION):
87
86
        """See Tree.annotate_iter"""
88
87
        text_key = (file_id, self.inventory[file_id].revision)
89
 
        annotations = self._repository.texts.annotate(text_key)
 
88
        annotator = self._repository.texts.get_annotator()
 
89
        annotations = annotator.annotate_flat(text_key)
90
90
        return [(key[-1], line) for key, line in annotations]
91
91
 
92
92
    def get_file_size(self, file_id):
101
101
 
102
102
    def get_file_mtime(self, file_id, path=None):
103
103
        ie = self._inventory[file_id]
104
 
        revision = self._repository.get_revision(ie.revision)
 
104
        try:
 
105
            revision = self._repository.get_revision(ie.revision)
 
106
        except errors.NoSuchRevision:
 
107
            raise errors.FileTimestampUnavailable(self.id2path(file_id))
105
108
        return revision.timestamp
106
109
 
107
110
    def is_executable(self, file_id, path=None):
113
116
    def has_filename(self, filename):
114
117
        return bool(self.inventory.path2id(filename))
115
118
 
116
 
    def list_files(self, include_root=False):
 
119
    def list_files(self, include_root=False, from_dir=None, recursive=True):
117
120
        # The only files returned by this are those from the version
118
 
        entries = self.inventory.iter_entries()
119
 
        # skip the root for compatability with the current apis.
120
 
        if self.inventory.root is not None and not include_root:
 
121
        inv = self.inventory
 
122
        if from_dir is None:
 
123
            from_dir_id = None
 
124
        else:
 
125
            from_dir_id = inv.path2id(from_dir)
 
126
            if from_dir_id is None:
 
127
                # Directory not versioned
 
128
                return
 
129
        entries = inv.iter_entries(from_dir=from_dir_id, recursive=recursive)
 
130
        if inv.root is not None and not include_root and from_dir is None:
121
131
            # skip the root for compatability with the current apis.
122
132
            entries.next()
123
133
        for path, entry in entries:
125
135
 
126
136
    def get_symlink_target(self, file_id):
127
137
        ie = self._inventory[file_id]
128
 
        return ie.symlink_target;
 
138
        # Inventories store symlink targets in unicode
 
139
        return ie.symlink_target
129
140
 
130
141
    def get_reference_revision(self, file_id, path=None):
131
142
        return self.inventory[file_id].reference_revision
163
174
        return set(self._repository.get_ancestry(self._revision_id,
164
175
                                                 topo_sorted=False))
165
176
 
 
177
    def is_locked(self):
 
178
        return self._repository.is_locked()
 
179
 
166
180
    def lock_read(self):
167
181
        self._repository.lock_read()
 
182
        return self
168
183
 
169
184
    def __repr__(self):
170
185
        return '<%s instance at %x, rev_id=%r>' % (
232
247
        lookup_trees = [self.source]
233
248
        if extra_trees:
234
249
             lookup_trees.extend(extra_trees)
 
250
        # The ids of items we need to examine to insure delta consistency.
 
251
        precise_file_ids = set()
 
252
        discarded_changes = {}
235
253
        if specific_files == []:
236
254
            specific_file_ids = []
237
255
        else:
238
256
            specific_file_ids = self.target.paths2ids(specific_files,
239
257
                lookup_trees, require_versioned=require_versioned)
240
 
 
241
258
        # FIXME: It should be possible to delegate include_unchanged handling
242
259
        # to CHKInventory.iter_changes and do a better job there -- vila
243
260
        # 20090304
244
 
        if include_unchanged:
245
 
            changed_file_ids = []
 
261
        changed_file_ids = set()
246
262
        for result in self.target.inventory.iter_changes(self.source.inventory):
247
 
            if (specific_file_ids is not None
248
 
                and not result[0] in specific_file_ids):
249
 
                # CHKMap.iter_changes is clean and fast. Better filter out
250
 
                # the specific files *after* it did its job.
251
 
                continue
 
263
            if specific_file_ids is not None:
 
264
                file_id = result[0]
 
265
                if file_id not in specific_file_ids:
 
266
                    # A change from the whole tree that we don't want to show yet.
 
267
                    # We may find that we need to show it for delta consistency, so
 
268
                    # stash it.
 
269
                    discarded_changes[result[0]] = result
 
270
                    continue
 
271
                new_parent_id = result[4][1]
 
272
                precise_file_ids.add(new_parent_id)
252
273
            yield result
253
 
            if include_unchanged:
254
 
                # Keep track of yielded results (cheaper than building the
255
 
                # whole inventory).
256
 
                changed_file_ids.append(result[0])
 
274
            changed_file_ids.add(result[0])
 
275
        if specific_file_ids is not None:
 
276
            for result in self._handle_precise_ids(precise_file_ids,
 
277
                changed_file_ids, discarded_changes=discarded_changes):
 
278
                yield result
257
279
        if include_unchanged:
258
280
            # CHKMap avoid being O(tree), so we go to O(tree) only if
259
281
            # required to.
260
282
            # Now walk the whole inventory, excluding the already yielded
261
283
            # file ids
262
 
            def not_a_change(file_id, relpath, parent, kind, executable):
263
 
                return (file_id,
264
 
                        (relpath, relpath), # Not renamed
265
 
                        False, # Not modified
266
 
                        (True, True), # Still  versioned
267
 
                        (executable, executable))
268
284
            changed_file_ids = set(changed_file_ids)
269
285
            for relpath, entry in self.target.inventory.iter_entries():
270
286
                if (specific_file_ids is not None
272
288
                    continue
273
289
                if not entry.file_id in changed_file_ids:
274
290
                    yield (entry.file_id,
275
 
                           (relpath, relpath),
276
 
                           False,
277
 
                           (True, True),
 
291
                           (relpath, relpath), # Not renamed
 
292
                           False, # Not modified
 
293
                           (True, True), # Still  versioned
278
294
                           (entry.parent_id, entry.parent_id),
279
295
                           (entry.name, entry.name),
280
296
                           (entry.kind, entry.kind),