[Python-checkins] r57378 - in python/trunk/Lib: bsddb/__init__.py test/test_bsddb.py
gregory.p.smith
python-checkins at python.org
Fri Aug 24 07:11:38 CEST 2007
Author: gregory.p.smith
Date: Fri Aug 24 07:11:38 2007
New Revision: 57378
Modified:
python/trunk/Lib/bsddb/__init__.py
python/trunk/Lib/test/test_bsddb.py
Log:
Fix bug 1725856.
Modified: python/trunk/Lib/bsddb/__init__.py
==============================================================================
--- python/trunk/Lib/bsddb/__init__.py (original)
+++ python/trunk/Lib/bsddb/__init__.py Fri Aug 24 07:11:38 2007
@@ -274,12 +274,16 @@
def first(self):
self._checkOpen()
+ # fix 1725856: don't needlessly try to restore our cursor position
+ self.saved_dbc_key = None
self._checkCursor()
rv = _DeadlockWrap(self.dbc.first)
return rv
def last(self):
self._checkOpen()
+ # fix 1725856: don't needlessly try to restore our cursor position
+ self.saved_dbc_key = None
self._checkCursor()
rv = _DeadlockWrap(self.dbc.last)
return rv
Modified: python/trunk/Lib/test/test_bsddb.py
==============================================================================
--- python/trunk/Lib/test/test_bsddb.py (original)
+++ python/trunk/Lib/test/test_bsddb.py Fri Aug 24 07:11:38 2007
@@ -127,6 +127,22 @@
items.append(self.f.previous())
self.assertSetEquals(items, self.d.items())
+ def test_first_while_deleting(self):
+ # Test for bug 1725856
+ self.assert_(len(self.d) >= 2, "test requires >=2 items")
+ for _ in self.d:
+ key = self.f.first()[0]
+ del self.f[key]
+ self.assertEqual([], self.f.items(), "expected empty db after test")
+
+ def test_last_while_deleting(self):
+ # Test for bug 1725856's evil twin
+ self.assert_(len(self.d) >= 2, "test requires >=2 items")
+ for _ in self.d:
+ key = self.f.last()[0]
+ del self.f[key]
+ self.assertEqual([], self.f.items(), "expected empty db after test")
+
def test_set_location(self):
self.assertEqual(self.f.set_location('e'), ('e', self.d['e']))
More information about the Python-checkins
mailing list