← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/medium.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/smart/medium.py
77
77
 
78
78
        This sets the _push_back_buffer to the given bytes.
79
79
        """
80
 
        assert self._push_back_buffer is None, (
81
 
            "_push_back called when self._push_back_buffer is %r"
82
 
            % (self._push_back_buffer,))
 
80
        if self._push_back_buffer is not None:
 
81
            raise AssertionError(
 
82
                "_push_back called when self._push_back_buffer is %r"
 
83
                % (self._push_back_buffer,))
83
84
        if bytes == '':
84
85
            return
85
86
        self._push_back_buffer = bytes
86
87
 
87
88
    def _get_push_back_buffer(self):
88
 
        assert self._push_back_buffer != '', (
89
 
            '%s._push_back_buffer should never be the empty string, '
90
 
            'which can be confused with EOF' % (self,))
 
89
        if self._push_back_buffer == '':
 
90
            raise AssertionError(
 
91
                '%s._push_back_buffer should never be the empty string, '
 
92
                'which can be confused with EOF' % (self,))
91
93
        bytes = self._push_back_buffer
92
94
        self._push_back_buffer = None
93
95
        return bytes
663
665
        This clears the _current_request on self._medium to allow a new 
664
666
        request to be created.
665
667
        """
666
 
        assert self._medium._current_request is self
 
668
        if self._medium._current_request is not self:
 
669
            raise AssertionError()
667
670
        self._medium._current_request = None
668
671
        
669
672
    def _finished_writing(self):