Skip to content

bpo-1635741: Convert _sre types to heap types and establish module state (PEP 384) #23393

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
corona10 merged 15 commits into from
Nov 20, 2020
Merged

bpo-1635741: Convert _sre types to heap types and establish module state (PEP 384) #23393

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
78b9d53
Add convenience macro
Nov 3, 2020
ce746b3
Convert _sre pattern type to heap type
Nov 3, 2020
529142d
Convert _sre match type to heap type
Nov 3, 2020
7583104
Convert _sre scanner type to heap type
Nov 3, 2020
575fccf
Establish global module state and add types to it
Nov 3, 2020
f82463c
Convert global state to module state
Nov 3, 2020
63ad0f1
Convert _sre to multi-phase initialisation
Nov 3, 2020
27134f0
Add NEWS
Nov 19, 2020
1c925b9
Address Heimes' review: improve module state naming
Nov 19, 2020
12bf771
Merge GH-23101
Nov 19, 2020
0f10f7d
Update clinic
Nov 19, 2020
bdd1338
Address Heimes' review: Drop using _PyType_GetModuleByDef
Nov 19, 2020
a6bc481
Adjust idlelib tiptest
Nov 19, 2020
2fd3bd1
Revert "Adjust idlelib tiptest"
Nov 19, 2020
d9ca7a6
Merge branch 'master' into bpo-1635741/sre
Nov 20, 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
Convert _sre scanner type to heap type
  • Loading branch information
Erlend E. Aasland
Erlend E. Aasland committed Nov 19, 2020
commit 7583104c401e6c2329961151c69827b523134c6a
56 changes: 19 additions & 37 deletions Modules/_sre.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -257,13 +257,13 @@ static PyObject *pattern_scanner(PatternObject *, PyObject *, Py_ssize_t, Py_ssi
module _sre
class _sre.SRE_Pattern "PatternObject *" "Pattern_Type"
class _sre.SRE_Match "MatchObject *" "Match_Type"
class _sre.SRE_Scanner "ScannerObject *" "&Scanner_Type"
class _sre.SRE_Scanner "ScannerObject *" "Scanner_Type"
[clinic start generated code]*/
/*[clinic end generated code: output=da39a3ee5e6b4b0d input=b0230ec19a0deac8]*/

static PyTypeObject *Pattern_Type = NULL;
static PyTypeObject *Match_Type = NULL;
static PyTypeObject Scanner_Type;
static PyTypeObject *Scanner_Type = NULL;

/*[clinic input]
_sre.getcodesize -> int
Expand Down Expand Up @@ -2393,9 +2393,12 @@ pattern_new_match(PatternObject* pattern, SRE_STATE* state, Py_ssize_t status)
static void
scanner_dealloc(ScannerObject* self)
{
PyTypeObject *tp = Py_TYPE(self);

state_fini(&self->state);
Py_XDECREF(self->pattern);
PyObject_DEL(self);
Py_DECREF(tp);
}

/*[clinic input]
Expand Down Expand Up @@ -2479,7 +2482,7 @@ pattern_scanner(PatternObject *self, PyObject *string, Py_ssize_t pos, Py_ssize_
ScannerObject* scanner;

/* create scanner object */
scanner = PyObject_New(ScannerObject, &Scanner_Type);
scanner = PyObject_New(ScannerObject, Scanner_Type);
if (!scanner)
return NULL;
scanner->pattern = NULL;
Expand Down Expand Up @@ -2699,36 +2702,18 @@ static PyMemberDef scanner_members[] = {
{NULL} /* Sentinel */
};

static PyTypeObject Scanner_Type = {
PyVarObject_HEAD_INIT(NULL, 0)
"_" SRE_MODULE ".SRE_Scanner",
sizeof(ScannerObject), 0,
(destructor)scanner_dealloc,/* tp_dealloc */
0, /* tp_vectorcall_offset */
0, /* tp_getattr */
0, /* tp_setattr */
0, /* tp_as_async */
0, /* tp_repr */
0, /* tp_as_number */
0, /* tp_as_sequence */
0, /* tp_as_mapping */
0, /* tp_hash */
0, /* tp_call */
0, /* tp_str */
0, /* tp_getattro */
0, /* tp_setattro */
0, /* tp_as_buffer */
Py_TPFLAGS_DEFAULT, /* tp_flags */
0, /* tp_doc */
0, /* tp_traverse */
0, /* tp_clear */
0, /* tp_richcompare */
0, /* tp_weaklistoffset */
0, /* tp_iter */
0, /* tp_iternext */
scanner_methods, /* tp_methods */
scanner_members, /* tp_members */
0, /* tp_getset */
static PyType_Slot scanner_slots[] = {
{Py_tp_dealloc, scanner_dealloc},
{Py_tp_methods, scanner_methods},
{Py_tp_members, scanner_members},
{0, NULL},
};

static PyType_Spec scanner_spec = {
.name = "_" SRE_MODULE ".SRE_Scanner",
.basicsize = sizeof(ScannerObject),
.flags = Py_TPFLAGS_DEFAULT,
.slots = scanner_slots,
};

static PyMethodDef _functions[] = {
Expand Down Expand Up @@ -2762,17 +2747,14 @@ PyMODINIT_FUNC PyInit__sre(void)
PyObject* d;
PyObject* x;

/* Patch object types */
if (PyType_Ready(&Scanner_Type))
return NULL;

m = PyModule_Create(&sremodule);
if (m == NULL)
return NULL;

/* Create heap types */
CREATE_TYPE(m, Pattern_Type, &pattern_spec);
CREATE_TYPE(m, Match_Type, &match_spec);
CREATE_TYPE(m, Scanner_Type, &scanner_spec);

d = PyModule_GetDict(m);

Expand Down