Skip to content

gh-104265 Disallow instantiation of _csv.Reader and _csv.Writer #104266

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
erlend-aasland merged 13 commits into from
May 7, 2023
Merged

gh-104265 Disallow instantiation of _csv.Reader and _csv.Writer #104266

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
ae69f9b
Disallow instantiation and subtyping of `_csv.Reader` and `_csv.Write…
chgnrdv May 7, 2023
7d1fcd8
added test
chgnrdv May 7, 2023
aa34fae
changed tests to use check_disallow_instantiation, separated tests
chgnrdv May 7, 2023
6c45ff8
reverted Py_TPFLAGS_BASETYPE flag deletion, removed corresponding tests
chgnrdv May 7, 2023
7d9ec2a
📜🤖 Added by blurb_it.
blurb-it[bot] May 7, 2023
19bddd0
Update 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
b3f4a65
Update 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
a022e91
Update 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
52aacfd
📜🤖 Added by blurb_it.
blurb-it[bot] May 7, 2023
d31d214
Delete 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
2804744
Update Misc/NEWS.d/next/Library/2023-05-07-19-56-45.gh-issue-104265.f…
erlend-aasland May 7, 2023
8255467
applied suggestion by erlend-aasland
chgnrdv May 7, 2023
73609a8
Merge branch '_csv-make-Reader-Writer-types-non-instantiable' of gith…
chgnrdv May 7, 2023
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
Next Next commit
Disallow instantiation and subtyping of _csv.Reader and `_csv.Write… 
…r` types

Set `Py_TPFLAGS_DISALLOW_INSTANTIATION` and unset `Py_TPFLAGS_BASETYPE` flags on `Reader` and `Writer` types to prevent their instantiation and subtyping
  • Loading branch information
@chgnrdv
chgnrdv committed May 7, 2023
commit ae69f9b8e4a8987ff814006f9b3731e0185d320b
8 changes: 4 additions & 4 deletions Modules/_csv.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -999,8 +999,8 @@ static PyType_Slot Reader_Type_slots[] = {
PyType_Spec Reader_Type_spec = {
.name = "_csv.reader",
.basicsize = sizeof(ReaderObj),
.flags = (Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE | Py_TPFLAGS_HAVE_GC |
Py_TPFLAGS_IMMUTABLETYPE),
.flags = (Py_TPFLAGS_DEFAULT | Py_TPFLAGS_HAVE_GC |
Py_TPFLAGS_IMMUTABLETYPE | Py_TPFLAGS_DISALLOW_INSTANTIATION),
.slots = Reader_Type_slots
};

Expand Down Expand Up @@ -1430,8 +1430,8 @@ static PyType_Slot Writer_Type_slots[] = {
PyType_Spec Writer_Type_spec = {
.name = "_csv.writer",
.basicsize = sizeof(WriterObj),
.flags = (Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE | Py_TPFLAGS_HAVE_GC |
Py_TPFLAGS_IMMUTABLETYPE),
.flags = (Py_TPFLAGS_DEFAULT | Py_TPFLAGS_HAVE_GC |
Py_TPFLAGS_IMMUTABLETYPE | Py_TPFLAGS_DISALLOW_INSTANTIATION),
.slots = Writer_Type_slots,
};

Expand Down