changeset: 75120:cf12fe9ce2b0 parent: 75118:3a701916ba8a parent: 75119:e67715b87131 user: Petri Lehtinen date: Tue Feb 21 13:59:34 2012 +0200 files: Lib/sqlite3/test/userfunctions.py Misc/ACKS Misc/NEWS Modules/_sqlite/connection.c description: Merge branch '3.2' Closes #8033. diff -r 3a701916ba8a -r cf12fe9ce2b0 Lib/sqlite3/test/userfunctions.py --- a/Lib/sqlite3/test/userfunctions.py Tue Feb 21 09:29:10 2012 +0200 +++ b/Lib/sqlite3/test/userfunctions.py Tue Feb 21 13:59:34 2012 +0200 @@ -37,6 +37,8 @@ return None def func_returnblob(): return b"blob" +def func_returnlonglong(): + return 1<<31 def func_raiseexception(): 5/0 @@ -50,6 +52,8 @@ return type(v) is type(None) def func_isblob(v): return isinstance(v, (bytes, memoryview)) +def func_islonglong(v): + return isinstance(v, int) and v>= 1<<31 class AggrNoStep: def __init__(self): @@ -127,6 +131,7 @@ self.con.create_function("returnfloat", 0, func_returnfloat) self.con.create_function("returnnull", 0, func_returnnull) self.con.create_function("returnblob", 0, func_returnblob) + self.con.create_function("returnlonglong", 0, func_returnlonglong) self.con.create_function("raiseexception", 0, func_raiseexception) self.con.create_function("isstring", 1, func_isstring) @@ -134,6 +139,7 @@ self.con.create_function("isfloat", 1, func_isfloat) self.con.create_function("isnone", 1, func_isnone) self.con.create_function("isblob", 1, func_isblob) + self.con.create_function("islonglong", 1, func_islonglong) def tearDown(self): self.con.close() @@ -200,6 +206,12 @@ self.assertEqual(type(val), bytes) self.assertEqual(val, b"blob") + def CheckFuncReturnLongLong(self): + cur = self.con.cursor() + cur.execute("select returnlonglong()") + val = cur.fetchone()[0] + self.assertEqual(val, 1<<31) + def CheckFuncException(self): cur = self.con.cursor() try: @@ -239,6 +251,12 @@ val = cur.fetchone()[0] self.assertEqual(val, 1) + def CheckParamLongLong(self): + cur = self.con.cursor() + cur.execute("select islonglong(?)", (1<<42,)) + val = cur.fetchone()[0] + self.assertEqual(val, 1) + class AggregateTests(unittest.TestCase): def setUp(self): self.con = sqlite.connect(":memory:") diff -r 3a701916ba8a -r cf12fe9ce2b0 Misc/ACKS --- a/Misc/ACKS Tue Feb 21 09:29:10 2012 +0200 +++ b/Misc/ACKS Tue Feb 21 13:59:34 2012 +0200 @@ -242,6 +242,7 @@ Erik Demaine John Dennis Roger Dev +Philippe Devalkeneer Raghuram Devarakonda Caleb Deveraux Catherine Devlin diff -r 3a701916ba8a -r cf12fe9ce2b0 Misc/NEWS --- a/Misc/NEWS Tue Feb 21 09:29:10 2012 +0200 +++ b/Misc/NEWS Tue Feb 21 13:59:34 2012 +0200 @@ -479,6 +479,9 @@ Library ------- +- Issue #8033: sqlite3: Fix 64-bit integer handling in user functions + on 32-bit architectures. Initial patch by Philippe Devalkeneer. + - HTMLParser is now able to handle slashes in the start tag. - Issue #13641: Decoding functions in the base64 module now accept ASCII-only diff -r 3a701916ba8a -r cf12fe9ce2b0 Modules/_sqlite/connection.c --- a/Modules/_sqlite/connection.c Tue Feb 21 09:29:10 2012 +0200 +++ b/Modules/_sqlite/connection.c Tue Feb 21 13:59:34 2012 +0200 @@ -484,7 +484,6 @@ void _pysqlite_set_result(sqlite3_context* context, PyObject* py_val) { - long longval; const char* buffer; Py_ssize_t buflen; @@ -493,8 +492,7 @@ } else if (py_val == Py_None) { sqlite3_result_null(context); } else if (PyLong_Check(py_val)) { - longval = PyLong_AsLong(py_val); - sqlite3_result_int64(context, (PY_LONG_LONG)longval); + sqlite3_result_int64(context, PyLong_AsLongLong(py_val)); } else if (PyFloat_Check(py_val)) { sqlite3_result_double(context, PyFloat_AsDouble(py_val)); } else if (PyUnicode_Check(py_val)) { @@ -519,7 +517,6 @@ sqlite3_value* cur_value; PyObject* cur_py_value; const char* val_str; - PY_LONG_LONG val_int; Py_ssize_t buflen; args = PyTuple_New(argc); @@ -531,8 +528,7 @@ cur_value = argv[i]; switch (sqlite3_value_type(argv[i])) { case SQLITE_INTEGER: - val_int = sqlite3_value_int64(cur_value); - cur_py_value = PyLong_FromLong((long)val_int); + cur_py_value = PyLong_FromLongLong(sqlite3_value_int64(cur_value)); break; case SQLITE_FLOAT: cur_py_value = PyFloat_FromDouble(sqlite3_value_double(cur_value));

AltStyle によって変換されたページ (->オリジナル) /