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
Use Py_CLEAR rather than Py_XDECREF
  • Loading branch information
@encukou
encukou committed Nov 13, 2020
commit 23f844ad71e4b3084b6d3510c220a740b52044ea
38 changes: 19 additions & 19 deletions Modules/_csv.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -316,15 +316,15 @@ static void
Dialect_dealloc(DialectObj *self)
{
PyTypeObject *tp = Py_TYPE(self);
Py_XDECREF(self->lineterminator);
Py_CLEAR(self->lineterminator);
tp->tp_free((PyObject *)self);
Py_DECREF(tp);
}

static void
Dialect_finalize(DialectObj *self)
{
Py_XDECREF(self->lineterminator);
Py_CLEAR(self->lineterminator);
}

static char *dialect_kws[] = {
Expand Down Expand Up @@ -402,7 +402,7 @@ dialect_new(PyTypeObject *type, PyObject *args, PyObject *kwargs)

self = (DialectObj *)type->tp_alloc(type, 0);
if (self == NULL) {
Py_XDECREF(dialect);
Py_CLEAR(dialect);
Comment thread
encukou marked this conversation as resolved.
return NULL;
}
self->lineterminator = NULL;
Expand Down Expand Up @@ -466,16 +466,16 @@ dialect_new(PyTypeObject *type, PyObject *args, PyObject *kwargs)
ret = (PyObject *)self;
Py_INCREF(self);
err:
Py_XDECREF(self);
Py_XDECREF(dialect);
Py_XDECREF(delimiter);
Py_XDECREF(doublequote);
Py_XDECREF(escapechar);
Py_XDECREF(lineterminator);
Py_XDECREF(quotechar);
Py_XDECREF(quoting);
Py_XDECREF(skipinitialspace);
Py_XDECREF(strict);
Py_CLEAR(self);
Py_CLEAR(dialect);
Py_CLEAR(delimiter);
Py_CLEAR(doublequote);
Py_CLEAR(escapechar);
Py_CLEAR(lineterminator);
Py_CLEAR(quotechar);
Py_CLEAR(quoting);
Py_CLEAR(skipinitialspace);
Py_CLEAR(strict);
return ret;
}

Expand Down Expand Up @@ -885,9 +885,9 @@ Reader_dealloc(ReaderObj *self)
{
PyTypeObject *tp = Py_TYPE(self);
PyObject_GC_UnTrack(self);
Py_XDECREF(self->dialect);
Py_XDECREF(self->input_iter);
Py_XDECREF(self->fields);
Py_CLEAR(self->dialect);
Py_CLEAR(self->input_iter);
Py_CLEAR(self->fields);
if (self->field != NULL)
PyMem_Free(self->field);
PyObject_GC_Del(self);
Expand All @@ -897,9 +897,9 @@ Reader_dealloc(ReaderObj *self)
static void
Reader_finalize(ReaderObj *self)
{
Py_XDECREF(self->dialect);
Py_XDECREF(self->input_iter);
Py_XDECREF(self->fields);
Py_CLEAR(self->dialect);
Py_CLEAR(self->input_iter);
Py_CLEAR(self->fields);
if (self->field != NULL) {
PyMem_Free(self->field);
Comment thread
encukou marked this conversation as resolved.
}
Expand Down