← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/revisiontree.py

  • Committer: John Arbash Meinel
  • Date: 2010-05-11 10:45:26 UTC
  • mto: This revision was merged to the branch mainline in revision 5225.
  • Revision ID: john@arbash-meinel.com-20100511104526-zxnstcxta22hzw2n
Implement a compiled extension for parsing the text key out of a CHKInventory value.

Related to bug #562666. This seems to shave 5-10% out of the time spent doing a complete
branch of bzr.dev/launchpad/etc.

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
 
103
101
 
104
102
    def get_file_mtime(self, file_id, path=None):
105
103
        ie = self._inventory[file_id]
106
 
        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))
107
108
        return revision.timestamp
108
109
 
109
110
    def is_executable(self, file_id, path=None):
242
243
        lookup_trees = [self.source]
243
244
        if extra_trees:
244
245
             lookup_trees.extend(extra_trees)
 
246
        # The ids of items we need to examine to insure delta consistency.
 
247
        precise_file_ids = set()
 
248
        discarded_changes = {}
245
249
        if specific_files == []:
246
250
            specific_file_ids = []
247
251
        else:
248
252
            specific_file_ids = self.target.paths2ids(specific_files,
249
253
                lookup_trees, require_versioned=require_versioned)
250
 
 
251
254
        # FIXME: It should be possible to delegate include_unchanged handling
252
255
        # to CHKInventory.iter_changes and do a better job there -- vila
253
256
        # 20090304
254
 
        if include_unchanged:
255
 
            changed_file_ids = []
 
257
        changed_file_ids = set()
256
258
        for result in self.target.inventory.iter_changes(self.source.inventory):
257
 
            if (specific_file_ids is not None
258
 
                and not result[0] in specific_file_ids):
259
 
                # CHKMap.iter_changes is clean and fast. Better filter out
260
 
                # the specific files *after* it did its job.
261
 
                continue
 
259
            if specific_file_ids is not None:
 
260
                file_id = result[0]
 
261
                if file_id not in specific_file_ids:
 
262
                    # A change from the whole tree that we don't want to show yet.
 
263
                    # We may find that we need to show it for delta consistency, so
 
264
                    # stash it.
 
265
                    discarded_changes[result[0]] = result
 
266
                    continue
 
267
                new_parent_id = result[4][1]
 
268
                precise_file_ids.add(new_parent_id)
262
269
            yield result
263
 
            if include_unchanged:
264
 
                # Keep track of yielded results (cheaper than building the
265
 
                # whole inventory).
266
 
                changed_file_ids.append(result[0])
 
270
            changed_file_ids.add(result[0])
 
271
        if specific_file_ids is not None:
 
272
            for result in self._handle_precise_ids(precise_file_ids,
 
273
                changed_file_ids, discarded_changes=discarded_changes):
 
274
                yield result
267
275
        if include_unchanged:
268
276
            # CHKMap avoid being O(tree), so we go to O(tree) only if
269
277
            # required to.