← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/repository.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2008-05-08 07:05:00 UTC
  • mfrom: (3376.2.15 no-asserts)
  • Revision ID: pqm@pqm.ubuntu.com-20080508070500-9zyyvsk0eev20t4w
(mbp) remove and disallow assert statements

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/repository.py
95
95
        if committer is None:
96
96
            self._committer = self._config.username()
97
97
        else:
98
 
            assert isinstance(committer, basestring), type(committer)
99
98
            self._committer = committer
100
99
 
101
100
        self.new_inventory = Inventory(None)
328
327
            if kind != parent_entry.kind:
329
328
                store = True
330
329
        if kind == 'file':
331
 
            assert content_summary[2] is not None, \
332
 
                "Files must not have executable = None"
 
330
            if content_summary[2] is None:
 
331
                raise ValueError("Files must not have executable = None")
333
332
            if not store:
334
333
                if (# if the file length changed we have to store:
335
334
                    parent_entry.text_size != content_summary[1] or
501
500
        :returns: The validator(which is a sha1 digest, though what is sha'd is
502
501
            repository format specific) of the serialized inventory.
503
502
        """
504
 
        assert self.is_in_write_group()
 
503
        if not self.is_in_write_group():
 
504
            raise AssertionError("%r not in write group" % (self,))
505
505
        _mod_revision.check_not_reserved_id(revision_id)
506
 
        assert inv.revision_id is None or inv.revision_id == revision_id, \
507
 
            "Mismatch between inventory revision" \
508
 
            " id and insertion revid (%r, %r)" % (inv.revision_id, revision_id)
509
 
        assert inv.root is not None
 
506
        if not (inv.revision_id is None or inv.revision_id == revision_id):
 
507
            raise AssertionError(
 
508
                "Mismatch between inventory revision"
 
509
                " id and insertion revid (%r, %r)"
 
510
                % (inv.revision_id, revision_id))
 
511
        if inv.root is None:
 
512
            raise AssertionError()
510
513
        inv_lines = self._serialise_inventory_to_lines(inv)
511
514
        inv_vf = self.get_inventory_weave()
512
515
        return self._inventory_add_lines(inv_vf, revision_id, parents,
1095
1098
                raise errors.InvalidRevisionId(revision_id=rev_id, branch=self)
1096
1099
        revs = self._revision_store.get_revisions(revision_ids,
1097
1100
                                                  self.get_transaction())
1098
 
        for rev in revs:
1099
 
            assert not isinstance(rev.revision_id, unicode)
1100
 
            for parent_id in rev.parent_ids:
1101
 
                assert not isinstance(parent_id, unicode)
1102
1101
        return revs
1103
1102
 
1104
1103
    @needs_read_lock
1501
1500
 
1502
1501
        :return: An iterator of inventories.
1503
1502
        """
1504
 
        assert None not in revision_ids
1505
 
        assert _mod_revision.NULL_REVISION not in revision_ids
 
1503
        if ((None in revision_ids)
 
1504
            or (_mod_revision.NULL_REVISION in revision_ids)):
 
1505
            raise ValueError('cannot get null revision inventory')
1506
1506
        return self._iter_inventories(revision_ids)
1507
1507
 
1508
1508
    def _iter_inventories(self, revision_ids):
1536
1536
    def get_inventory_xml(self, revision_id):
1537
1537
        """Get inventory XML as a file object."""
1538
1538
        try:
1539
 
            assert isinstance(revision_id, str), type(revision_id)
1540
1539
            iw = self.get_inventory_weave()
1541
1540
            return iw.get_text(revision_id)
1542
1541
        except IndexError:
2446
2445
        target_ids = set(self.target.all_revision_ids())
2447
2446
        if revision_id is not None:
2448
2447
            source_ids = self.source.get_ancestry(revision_id)
2449
 
            assert source_ids[0] is None
 
2448
            if source_ids[0] is not None:
 
2449
                raise AssertionError()
2450
2450
            source_ids.pop(0)
2451
2451
        else:
2452
2452
            source_ids = self.source.all_revision_ids()
2613
2613
        # - RBC 20060209
2614
2614
        if revision_id is not None:
2615
2615
            source_ids = self.source.get_ancestry(revision_id)
2616
 
            assert source_ids[0] is None
 
2616
            if source_ids[0] is not None:
 
2617
                raise AssertionError()
2617
2618
            source_ids.pop(0)
2618
2619
        else:
2619
2620
            source_ids = self.source._all_possible_ids()
2682
2683
        """See InterRepository.missing_revision_ids()."""
2683
2684
        if revision_id is not None:
2684
2685
            source_ids = self.source.get_ancestry(revision_id)
2685
 
            assert source_ids[0] is None
 
2686
            if source_ids[0] is not None:
 
2687
                raise AssertionError()
2686
2688
            source_ids.pop(0)
2687
2689
        else:
2688
2690
            source_ids = self.source.all_revision_ids()
2795
2797
            return self._walk_to_common_revisions([revision_id])
2796
2798
        elif revision_id is not None:
2797
2799
            source_ids = self.source.get_ancestry(revision_id)
2798
 
            assert source_ids[0] is None
 
2800
            if source_ids[0] is not None:
 
2801
                raise AssertionError()
2799
2802
            source_ids.pop(0)
2800
2803
        else:
2801
2804
            source_ids = self.source.all_revision_ids()
2962
2965
        # Is source's model compatible with target's model?
2963
2966
        source._ensure_real()
2964
2967
        real_source = source._real_repository
2965
 
        assert not isinstance(real_source, remote.RemoteRepository), (
2966
 
            "We don't support remote repos backed by remote repos yet.")
 
2968
        if isinstance(real_source, remote.RemoteRepository):
 
2969
            raise NotImplementedError(
 
2970
                "We don't support remote repos backed by remote repos yet.")
2967
2971
        return InterRepository._same_model(real_source, target)
2968
2972
 
2969
2973
    @needs_write_lock