← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/workingtree_4.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2007-10-08 08:12:00 UTC
  • mfrom: (2858.2.5 remove-checks)
  • Revision ID: pqm@pqm.ubuntu.com-20071008081200-04a0byt5lo5tyft9
Remove old calls to safe_file_id and safe_revision_id (mbp)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/workingtree_4.py
453
453
 
454
454
    def has_id(self, file_id):
455
455
        state = self.current_dirstate()
456
 
        file_id = osutils.safe_file_id(file_id)
457
456
        row, parents = self._get_entry(file_id=file_id)
458
457
        if row is None:
459
458
            return False
463
462
    @needs_read_lock
464
463
    def id2path(self, file_id):
465
464
        "Convert a file-id to a path."
466
 
        file_id = osutils.safe_file_id(file_id)
467
465
        state = self.current_dirstate()
468
466
        entry = self._get_entry(file_id=file_id)
469
467
        if entry == (None, None):
477
475
 
478
476
            Note: The caller is expected to take a read-lock before calling this.
479
477
            """
480
 
            file_id = osutils.safe_file_id(file_id)
481
478
            entry = self._get_entry(file_id=file_id, path=path)
482
479
            if entry == (None, None):
483
480
                return False
489
486
            Note: The caller is expected to take a read-lock before calling this.
490
487
            """
491
488
            if not path:
492
 
                file_id = osutils.safe_file_id(file_id)
493
489
                path = self.id2path(file_id)
494
490
            mode = os.lstat(self.abspath(path)).st_mode
495
491
            return bool(stat.S_ISREG(mode) and stat.S_IEXEC & mode)
1004
1000
 
1005
1001
        WorkingTree4 supplies revision_trees for any basis tree.
1006
1002
        """
1007
 
        revision_id = osutils.safe_revision_id(revision_id)
1008
1003
        dirstate = self.current_dirstate()
1009
1004
        parent_ids = dirstate.get_parent_ids()
1010
1005
        if revision_id not in parent_ids:
1017
1012
    @needs_tree_write_lock
1018
1013
    def set_last_revision(self, new_revision):
1019
1014
        """Change the last revision in the working tree."""
1020
 
        new_revision = osutils.safe_revision_id(new_revision)
1021
1015
        parents = self.get_parent_ids()
1022
1016
        if new_revision in (NULL_REVISION, None):
1023
1017
            assert len(parents) < 2, (
1041
1035
        :param revision_ids: The revision_ids to set as the parent ids of this
1042
1036
            working tree. Any of these may be ghosts.
1043
1037
        """
1044
 
        revision_ids = [osutils.safe_revision_id(r) for r in revision_ids]
1045
1038
        trees = []
1046
1039
        for revision_id in revision_ids:
1047
1040
            try:
1073
1066
        # convert absent trees to the null tree, which we convert back to
1074
1067
        # missing on access.
1075
1068
        for rev_id, tree in parents_list:
1076
 
            rev_id = osutils.safe_revision_id(rev_id)
1077
1069
            _mod_revision.check_not_reserved_id(rev_id)
1078
1070
            if tree is not None:
1079
1071
                real_trees.append((rev_id, tree))
1154
1146
            return
1155
1147
        state = self.current_dirstate()
1156
1148
        state._read_dirblocks_if_needed()
1157
 
        ids_to_unversion = set()
1158
 
        for file_id in file_ids:
1159
 
            ids_to_unversion.add(osutils.safe_file_id(file_id))
 
1149
        ids_to_unversion = set(file_ids)
1160
1150
        paths_to_unversion = set()
1161
1151
        # sketch:
1162
1152
        # check if the root is to be unversioned, if so, assert for now.
1269
1259
        These trees get an initial random root id, if their repository supports
1270
1260
        rich root data, TREE_ROOT otherwise.
1271
1261
        """
1272
 
        revision_id = osutils.safe_revision_id(revision_id)
1273
1262
        if not isinstance(a_bzrdir.transport, LocalTransport):
1274
1263
            raise errors.NotLocalUrl(a_bzrdir.transport.base)
1275
1264
        transport = a_bzrdir.get_workingtree_transport(self)
1341
1330
 
1342
1331
    def __init__(self, dirstate, revision_id, repository):
1343
1332
        self._dirstate = dirstate
1344
 
        self._revision_id = osutils.safe_revision_id(revision_id)
 
1333
        self._revision_id = revision_id
1345
1334
        self._repository = repository
1346
1335
        self._inventory = None
1347
1336
        self._locked = 0
1399
1388
        """
1400
1389
        if file_id is None and path is None:
1401
1390
            raise errors.BzrError('must supply file_id or path')
1402
 
        file_id = osutils.safe_file_id(file_id)
1403
1391
        if path is not None:
1404
1392
            path = path.encode('utf8')
1405
1393
        parent_index = self._get_parent_index()