From e33aef7b159a24789cfa8f1fcf4462f6a4bab464 Mon Sep 17 00:00:00 2001
From: "Gregory P. Smith" <greg@mad-scientist.com>
Date: Tue, 13 Jan 2004 19:59:57 +0000
Subject: [PATCH] __init__.py:  keep it compatible with older python (True and
 False == 1 and 0) test_basics.py: updated for the set_get_returns_none()
 default of 2 change.

---
 Lib/bsddb/__init__.py         |  4 ++--
 Lib/bsddb/test/test_basics.py | 19 +++++++++++++++----
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/Lib/bsddb/__init__.py b/Lib/bsddb/__init__.py
index 99499c56430..9c59b00b2c7 100644
--- a/Lib/bsddb/__init__.py
+++ b/Lib/bsddb/__init__.py
@@ -184,7 +184,7 @@ class _DBWithCursor(_iter_mixin):
     # BerkeleyDB deadlocks (due to being opened with DB_INIT_LOCK
     # and DB_THREAD to be thread safe) when intermixing database
     # operations that use the cursor internally with those that don't.
-    def _closeCursors(self, save=True):
+    def _closeCursors(self, save=1):
         if self.dbc:
             c = self.dbc
             self.dbc = None
@@ -223,7 +223,7 @@ class _DBWithCursor(_iter_mixin):
         del self.db[key]
 
     def close(self):
-        self._closeCursors(save=False)
+        self._closeCursors(save=0)
         if self.dbc is not None:
             self.dbc.close()
         v = 0
diff --git a/Lib/bsddb/test/test_basics.py b/Lib/bsddb/test/test_basics.py
index 93a7fb75916..d757b34185e 100644
--- a/Lib/bsddb/test/test_basics.py
+++ b/Lib/bsddb/test/test_basics.py
@@ -290,7 +290,7 @@ class BasicTestCase(unittest.TestCase):
 
     #----------------------------------------
 
-    def test03_SimpleCursorStuff(self, get_raises_error=0, set_raises_error=1):
+    def test03_SimpleCursorStuff(self, get_raises_error=0, set_raises_error=0):
         if verbose:
             print '\n', '-=' * 30
             print "Running %s.test03_SimpleCursorStuff (get_error %s, set_error %s)..." % \
@@ -459,9 +459,20 @@ class BasicTestCase(unittest.TestCase):
                   self.__class__.__name__
 
         old = self.d.set_get_returns_none(0)
-        assert old == 1
+        assert old == 2
         self.test03_SimpleCursorStuff(get_raises_error=1, set_raises_error=1)
 
+    def test03b_SimpleCursorWithGetReturnsNone1(self):
+        # same test but raise exceptions instead of returning None
+        if verbose:
+            print '\n', '-=' * 30
+            print "Running %s.test03b_SimpleCursorStuffWithoutGetReturnsNone..." % \
+                  self.__class__.__name__
+
+        old = self.d.set_get_returns_none(1)
+        self.test03_SimpleCursorStuff(get_raises_error=0, set_raises_error=1)
+
+
     def test03c_SimpleCursorGetReturnsNone2(self):
         # same test but raise exceptions instead of returning None
         if verbose:
@@ -469,10 +480,10 @@ class BasicTestCase(unittest.TestCase):
             print "Running %s.test03c_SimpleCursorStuffWithoutSetReturnsNone..." % \
                   self.__class__.__name__
 
+        old = self.d.set_get_returns_none(1)
+        assert old == 2
         old = self.d.set_get_returns_none(2)
         assert old == 1
-        old = self.d.set_get_returns_none(2)
-        assert old == 2
         self.test03_SimpleCursorStuff(get_raises_error=0, set_raises_error=0)
 
     #----------------------------------------