Commit 63b0cb2f authored by Antoine Pitrou's avatar Antoine Pitrou

Fix py3k warnings in bsddb

parent 868817ea
This diff is collapsed.
...@@ -234,7 +234,7 @@ class DBShelf(MutableMapping): ...@@ -234,7 +234,7 @@ class DBShelf(MutableMapping):
# given nothing is passed to the extension module. That way # given nothing is passed to the extension module. That way
# an exception can be raised if set_get_returns_none is turned # an exception can be raised if set_get_returns_none is turned
# off. # off.
data = apply(self.db.get, args, kw) data = self.db.get(*args, **kw)
try: try:
return cPickle.loads(data) return cPickle.loads(data)
except (EOFError, TypeError, cPickle.UnpicklingError): except (EOFError, TypeError, cPickle.UnpicklingError):
...@@ -303,7 +303,7 @@ class DBShelfCursor: ...@@ -303,7 +303,7 @@ class DBShelfCursor:
def get(self, *args): def get(self, *args):
count = len(args) # a method overloading hack count = len(args) # a method overloading hack
method = getattr(self, 'get_%d' % count) method = getattr(self, 'get_%d' % count)
apply(method, args) method(*args)
def get_1(self, flags): def get_1(self, flags):
rec = self.dbc.get(flags) rec = self.dbc.get(flags)
......
...@@ -398,7 +398,7 @@ class bsdTableDB : ...@@ -398,7 +398,7 @@ class bsdTableDB :
# column names # column names
newcolumnlist = copy.copy(oldcolumnlist) newcolumnlist = copy.copy(oldcolumnlist)
for c in columns: for c in columns:
if not oldcolumnhash.has_key(c): if not c in oldcolumnhash:
newcolumnlist.append(c) newcolumnlist.append(c)
# store the table's new extended column list # store the table's new extended column list
...@@ -472,7 +472,7 @@ class bsdTableDB : ...@@ -472,7 +472,7 @@ class bsdTableDB :
raise TableDBError, "unknown table" raise TableDBError, "unknown table"
# check the validity of each column name # check the validity of each column name
if not self.__tablecolumns.has_key(table): if not table in self.__tablecolumns:
self.__load_column_info(table) self.__load_column_info(table)
for column in rowdict.keys() : for column in rowdict.keys() :
if not self.__tablecolumns[table].count(column): if not self.__tablecolumns[table].count(column):
...@@ -540,7 +540,7 @@ class bsdTableDB : ...@@ -540,7 +540,7 @@ class bsdTableDB :
# error # error
dataitem = None dataitem = None
dataitem = mappings[column](dataitem) dataitem = mappings[column](dataitem)
if dataitem <> None: if dataitem != None:
self.db.put( self.db.put(
_data_key(table, column, rowid), _data_key(table, column, rowid),
dataitem, txn=txn) dataitem, txn=txn)
...@@ -615,7 +615,7 @@ class bsdTableDB : ...@@ -615,7 +615,7 @@ class bsdTableDB :
argument and returning a boolean. argument and returning a boolean.
""" """
try: try:
if not self.__tablecolumns.has_key(table): if not table in self.__tablecolumns:
self.__load_column_info(table) self.__load_column_info(table)
if columns is None: if columns is None:
columns = self.__tablecolumns[table] columns = self.__tablecolumns[table]
...@@ -639,7 +639,7 @@ class bsdTableDB : ...@@ -639,7 +639,7 @@ class bsdTableDB :
argument and returning a boolean. argument and returning a boolean.
""" """
# check the validity of each column name # check the validity of each column name
if not self.__tablecolumns.has_key(table): if not table in self.__tablecolumns:
self.__load_column_info(table) self.__load_column_info(table)
if columns is None: if columns is None:
columns = self.tablecolumns[table] columns = self.tablecolumns[table]
...@@ -709,28 +709,24 @@ class bsdTableDB : ...@@ -709,28 +709,24 @@ class bsdTableDB :
# extract the rowid from the key # extract the rowid from the key
rowid = key[-_rowid_str_len:] rowid = key[-_rowid_str_len:]
if not rejected_rowids.has_key(rowid): if not rowid in rejected_rowids:
# if no condition was specified or the condition # if no condition was specified or the condition
# succeeds, add row to our match list. # succeeds, add row to our match list.
if not condition or condition(data): if not condition or condition(data):
if not matching_rowids.has_key(rowid): if not rowid in matching_rowids:
matching_rowids[rowid] = {} matching_rowids[rowid] = {}
if savethiscolumndata: if savethiscolumndata:
matching_rowids[rowid][column] = data matching_rowids[rowid][column] = data
else: else:
if matching_rowids.has_key(rowid): if rowid in matching_rowids:
del matching_rowids[rowid] del matching_rowids[rowid]
rejected_rowids[rowid] = rowid rejected_rowids[rowid] = rowid
key, data = cur.next() key, data = cur.next()
except db.DBError, dberror: except db.DBError, dberror:
if sys.version_info[0] < 3 : if dberror.args[0] != db.DB_NOTFOUND:
if dberror[0] != db.DB_NOTFOUND: raise
raise
else :
if dberror.args[0] != db.DB_NOTFOUND:
raise
continue continue
cur.close() cur.close()
...@@ -743,7 +739,7 @@ class bsdTableDB : ...@@ -743,7 +739,7 @@ class bsdTableDB :
if len(columns) > 0: if len(columns) > 0:
for rowid, rowdata in matching_rowids.items(): for rowid, rowdata in matching_rowids.items():
for column in columns: for column in columns:
if rowdata.has_key(column): if column in rowdata:
continue continue
try: try:
rowdata[column] = self.db.get( rowdata[column] = self.db.get(
...@@ -815,13 +811,10 @@ class bsdTableDB : ...@@ -815,13 +811,10 @@ class bsdTableDB :
txn.commit() txn.commit()
txn = None txn = None
if self.__tablecolumns.has_key(table): if table in self.__tablecolumns:
del self.__tablecolumns[table] del self.__tablecolumns[table]
except db.DBError, dberror: except db.DBError, dberror:
if txn: if txn:
txn.abort() txn.abort()
if sys.version_info[0] < 3 : raise TableDBError(dberror.args[1])
raise TableDBError, dberror[1]
else :
raise TableDBError, dberror.args[1]
...@@ -61,7 +61,7 @@ def DeadlockWrap(function, *_args, **_kwargs): ...@@ -61,7 +61,7 @@ def DeadlockWrap(function, *_args, **_kwargs):
""" """
sleeptime = _deadlock_MinSleepTime sleeptime = _deadlock_MinSleepTime
max_retries = _kwargs.get('max_retries', -1) max_retries = _kwargs.get('max_retries', -1)
if _kwargs.has_key('max_retries'): if 'max_retries' in _kwargs:
del _kwargs['max_retries'] del _kwargs['max_retries']
while True: while True:
try: try:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment