← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/lru_cache.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/lru_cache.py
109
109
        self._queue = new_queue
110
110
        # All entries should be of the same size. There should be one entry in
111
111
        # queue for each entry in cache, and all refcounts should == 1
112
 
        assert (len(self._queue) == len(self._cache) ==
113
 
                len(self._refcount) == sum(self._refcount.itervalues()))
 
112
        if not (len(self._queue) == len(self._cache) ==
 
113
                len(self._refcount) == sum(self._refcount.itervalues())):
 
114
            raise AssertionError()
114
115
 
115
116
    def _remove(self, key):
116
117
        """Remove an entry, making sure to maintain the invariants."""