mirror of
https://github.com/python/cpython.git
synced 2025-08-04 08:59:19 +00:00
Make the sqlite tests pass.
This commit is contained in:
parent
bd4a63e091
commit
6d21456137
7 changed files with 28 additions and 31 deletions
|
@ -435,10 +435,8 @@ void _pysqlite_set_result(sqlite3_context* context, PyObject* py_val)
|
|||
} else if (PyString_Check(py_val)) {
|
||||
sqlite3_result_text(context, PyString_AsString(py_val), -1, SQLITE_TRANSIENT);
|
||||
} else if (PyUnicode_Check(py_val)) {
|
||||
stringval = PyUnicode_AsUTF8String(py_val);
|
||||
if (stringval) {
|
||||
sqlite3_result_text(context, PyString_AsString(stringval), -1, SQLITE_TRANSIENT);
|
||||
Py_DECREF(stringval);
|
||||
sqlite3_result_text(context, PyUnicode_AsString(stringval), -1, SQLITE_TRANSIENT);
|
||||
}
|
||||
} else {
|
||||
/* TODO: raise error */
|
||||
|
@ -1094,7 +1092,7 @@ pysqlite_connection_create_collation(pysqlite_Connection* self, PyObject* args)
|
|||
goto finally;
|
||||
}
|
||||
|
||||
if (!PyArg_ParseTuple(args, "O!O:create_collation(name, callback)", &PyString_Type, &name, &callable)) {
|
||||
if (!PyArg_ParseTuple(args, "O!O:create_collation(name, callback)", &PyUnicode_Type, &name, &callable)) {
|
||||
goto finally;
|
||||
}
|
||||
|
||||
|
|
|
@ -248,7 +248,7 @@ PyObject* _pysqlite_build_column_name(const char* colname)
|
|||
if ((*pos == '[') && (pos > colname) && (*(pos-1) == ' ')) {
|
||||
pos--;
|
||||
}
|
||||
return PyString_FromStringAndSize(colname, pos - colname);
|
||||
return PyUnicode_FromStringAndSize(colname, pos - colname);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -372,8 +372,10 @@ PyObject* _pysqlite_fetch_one_row(pysqlite_Cursor* self)
|
|||
}
|
||||
} else if (self->connection->text_factory == (PyObject*)&PyString_Type) {
|
||||
converted = PyString_FromString(val_str);
|
||||
} else if (self->connection->text_factory == (PyObject*)&PyBytes_Type) {
|
||||
converted = PyBytes_FromStringAndSize(val_str, strlen(val_str));
|
||||
} else {
|
||||
converted = PyObject_CallFunction(self->connection->text_factory, "s", val_str);
|
||||
converted = PyObject_CallFunction(self->connection->text_factory, "y", val_str);
|
||||
}
|
||||
} else {
|
||||
/* coltype == SQLITE_BLOB */
|
||||
|
@ -746,17 +748,13 @@ PyObject* pysqlite_cursor_executescript(pysqlite_Cursor* self, PyObject* args)
|
|||
return NULL;
|
||||
}
|
||||
|
||||
if (PyString_Check(script_obj)) {
|
||||
script_cstr = PyString_AsString(script_obj);
|
||||
} else if (PyUnicode_Check(script_obj)) {
|
||||
script_str = PyUnicode_AsUTF8String(script_obj);
|
||||
if (!script_str) {
|
||||
if (PyUnicode_Check(script_obj)) {
|
||||
script_cstr = PyUnicode_AsString(script_obj);
|
||||
if (!script_cstr) {
|
||||
return NULL;
|
||||
}
|
||||
|
||||
script_cstr = PyString_AsString(script_str);
|
||||
} else {
|
||||
PyErr_SetString(PyExc_ValueError, "script argument must be unicode or string.");
|
||||
PyErr_SetString(PyExc_ValueError, "script argument must be unicode.");
|
||||
return NULL;
|
||||
}
|
||||
|
||||
|
|
|
@ -86,8 +86,8 @@ PyObject* pysqlite_row_subscript(pysqlite_Row* self, PyObject* idx)
|
|||
item = PyTuple_GetItem(self->data, _idx);
|
||||
Py_XINCREF(item);
|
||||
return item;
|
||||
} else if (PyString_Check(idx)) {
|
||||
key = PyString_AsString(idx);
|
||||
} else if (PyUnicode_Check(idx)) {
|
||||
key = PyUnicode_AsString(idx);
|
||||
|
||||
nitems = PyTuple_Size(self->description);
|
||||
|
||||
|
|
|
@ -113,7 +113,8 @@ int pysqlite_statement_bind_parameter(pysqlite_Statement* self, int pos, PyObjec
|
|||
rc = sqlite3_bind_text(self->st, pos, string, -1, SQLITE_TRANSIENT);
|
||||
} else if PyUnicode_Check(parameter) {
|
||||
stringval = PyUnicode_AsUTF8String(parameter);
|
||||
string = PyString_AsString(stringval);
|
||||
string = PyBytes_AsString(stringval);
|
||||
|
||||
rc = sqlite3_bind_text(self->st, pos, string, -1, SQLITE_TRANSIENT);
|
||||
Py_DECREF(stringval);
|
||||
} else {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue