← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: Martin Pool
  • Date: 2008-04-24 07:22:53 UTC
  • mto: This revision was merged to the branch mainline in revision 3415.
  • Revision ID: mbp@sourcefrog.net-20080424072253-opmjij7xfy38w27f
Remove every assert statement from bzrlib!

Depending on the context they are:

 * turned into an explicit if/raise of either AssertionError 
   or something more specific -- particularly where they protect
   programming interfaces, complex invariants, or data file integrity
 * removed, if they're redundant with a later check, not protecting
   a meaningful invariant
 * turned into a selftest method on tests

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/index.py
830
830
                trim_start = data.find('\n') + 1
831
831
            else:
832
832
                trim_start = data.find('\n', trim_start) + 1
833
 
            assert trim_start != 0, 'no \n was present'
 
833
            if not (trim_start != 0):
 
834
                raise AssertionError('no \n was present')
834
835
            # print 'removing start', offset, trim_start, repr(data[:trim_start])
835
836
        if not end_adjacent:
836
837
            # work around python bug in rfind
838
839
                trim_end = data.rfind('\n') + 1
839
840
            else:
840
841
                trim_end = data.rfind('\n', None, trim_end) + 1
841
 
            assert trim_end != 0, 'no \n was present'
 
842
            if not (trim_end != 0):
 
843
                raise AssertionError('no \n was present')
842
844
            # print 'removing end', offset, trim_end, repr(data[trim_end:])
843
845
        # adjust offset and data to the parseable data.
844
846
        trimmed_data = data[trim_start:trim_end]
845
 
        assert trimmed_data, 'read unneeded data [%d:%d] from [%d:%d]' % (
846
 
            trim_start, trim_end, offset, offset + len(data))
 
847
        if not (trimmed_data):
 
848
            raise AssertionError('read unneeded data [%d:%d] from [%d:%d]' 
 
849
                % (trim_start, trim_end, offset, offset + len(data)))
847
850
        if trim_start:
848
851
            offset += trim_start
849
852
        # print "parsing", repr(trimmed_data)
867
870
            if line == '':
868
871
                # must be at the end
869
872
                if self._size:
870
 
                    assert self._size == pos + 1, "%s %s" % (self._size, pos)
 
873
                    if not (self._size == pos + 1):
 
874
                        raise AssertionError("%s %s" % (self._size, pos))
871
875
                trailers += 1
872
876
                continue
873
877
            elements = line.split('\0')
958
962
            for offset, data in readv_data:
959
963
                if self._bisect_nodes is None:
960
964
                    # this must be the start
961
 
                    assert offset == 0
 
965
                    if not (offset == 0):
 
966
                        raise AssertionError()
962
967
                    offset, data = self._parse_header_from_bytes(data)
963
968
                # print readv_ranges, "[%d:%d]" % (offset, offset + len(data))
964
969
                self._parse_region(offset, data)