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
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
Use designated initializers for spec structs
  • Loading branch information
@encukou
encukou committed Dec 8, 2020
commit b8c37bf4d44cf30eb4f4cd0fdf95765888d01677
29 changes: 12 additions & 17 deletions Modules/_csv.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -524,11 +524,10 @@ static PyType_Slot Dialect_Type_slots[] = {
};

PyType_Spec Dialect_Type_spec = {
"_csv.Dialect",
sizeof(DialectObj),
0,
Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE, /* tp_flags */
Dialect_Type_slots,
.name = "_csv.Dialect",
.basicsize = sizeof(DialectObj),
.flags = Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE,
.slots = Dialect_Type_slots,
};


Expand Down Expand Up @@ -960,12 +959,10 @@ static PyType_Slot Reader_Type_slots[] = {
};

PyType_Spec Reader_Type_spec = {
"_csv.reader",
sizeof(ReaderObj),
0,
Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE |
Py_TPFLAGS_HAVE_GC,
Reader_Type_slots
.name = "_csv.reader",
.basicsize = sizeof(ReaderObj),
.flags = Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE | Py_TPFLAGS_HAVE_GC,
.slots = Reader_Type_slots
};


Expand Down Expand Up @@ -1381,12 +1378,10 @@ static PyType_Slot Writer_Type_slots[] = {
};

PyType_Spec Writer_Type_spec = {
"_csv.writer",
sizeof(WriterObj),
0,
Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE |
Py_TPFLAGS_HAVE_GC,
Writer_Type_slots,
.name = "_csv.writer",
.basicsize = sizeof(WriterObj),
.flags = Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE | Py_TPFLAGS_HAVE_GC,
.slots = Writer_Type_slots,
};


Expand Down