Skip to content

bpo-14935: Remove static state from the _csv module #23224

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
encukou merged 12 commits into from
Dec 15, 2020
Merged

bpo-14935: Remove static state from the _csv module #23224

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
a2a0e6f
Remove static state from the _csv module
encukou May 12, 2020
09802b2
Use PyModule_AddObjectRef and PyTuple_Pack
encukou Nov 10, 2020
1665ac7
Rename m_state to module_state
encukou Nov 13, 2020
23f844a
Use Py_CLEAR rather than Py_XDECREF
encukou Nov 13, 2020
8d1f95d
Use `module` rather than `m`
encukou Nov 13, 2020
2c60895
Factor out calling _PyType_GetModuleByDef & PyModule_GetState
encukou Nov 13, 2020
e69d0a1
Set freed pointers to NULL
encukou Nov 13, 2020
fb93e7f
Only call get_csv_state up to once per function
encukou Nov 13, 2020
b62987b
Pass module_state directly to _call_dialect
encukou Nov 13, 2020
f12a768
Add NULL check for _csv_state_from_type
encukou Dec 8, 2020
f302029
Use Py_NewRef where appropriate
encukou Dec 8, 2020
b8c37bf
Use designated initializers for spec structs
encukou Dec 8, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Factor out calling _PyType_GetModuleByDef & PyModule_GetState
  • Loading branch information
@encukou
encukou committed Nov 13, 2020
commit 2c60895481efc07f69b770ace33ebf13e63a2505
37 changes: 20 additions & 17 deletions Modules/_csv.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -340,6 +340,22 @@ static char *dialect_kws[] = {
NULL
};

static _csvstate *
_csv_state_from_type(PyTypeObject *type, const char *name)
{
PyObject *module = _PyType_GetModuleByDef(type, &_csvmodule);
if (module == NULL) {
return NULL;
}
_csvstate *module_state = PyModule_GetState(module);
if (module_state == NULL) {
PyErr_Format(PyExc_SystemError,
"%s: No _csv module state found", name);
Comment thread
encukou marked this conversation as resolved.
return NULL;
}
return module_state;
}

static PyObject *
dialect_new(PyTypeObject *type, PyObject *args, PyObject *kwargs)
{
Expand Down Expand Up @@ -368,16 +384,7 @@ dialect_new(PyTypeObject *type, PyObject *args, PyObject *kwargs)
&strict))
return NULL;

PyObject *module = _PyType_GetModuleByDef(type, &_csvmodule);
if (module == NULL) {
return NULL;
}
_csvstate *module_state = PyModule_GetState(module);
if (module_state == NULL) {
return PyErr_Format(PyExc_SystemError,
"dialect_new: No _csv module state found");
}

_csvstate *module_state = _csv_state_from_type(type, "dialect_new");

if (dialect != NULL) {
if (PyUnicode_Check(dialect)) {
Expand Down Expand Up @@ -810,14 +817,10 @@ Reader_iternext(ReaderObj *self)
const void *data;
PyObject *lineobj;

PyObject *module = _PyType_GetModuleByDef(Py_TYPE(self), &_csvmodule);
if (module == NULL) {
return NULL;
}
_csvstate *module_state = PyModule_GetState(module);
_csvstate *module_state = _csv_state_from_type(Py_TYPE(self),
"Reader.__next__");
if (module_state == NULL) {
return PyErr_Format(PyExc_SystemError,
"dialect_new: No _csv module state found");
return NULL;
}

if (parse_reset(self) < 0)
Expand Down