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 to multi-phase initialisation
  • Loading branch information
Erlend E. Aasland committed Nov 19, 2020
commit 63ad0f103b41bb36360d19e110cf53b25aa11c10
70 changes: 55 additions & 15 deletions Modules/_sre.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2796,12 +2796,35 @@ static PyMethodDef _functions[] = {
{NULL, NULL}
};

static struct PyModuleDef sremodule = {
.m_base = PyModuleDef_HEAD_INIT,
.m_name = "_" SRE_MODULE,
.m_size = sizeof(sre_module_state),
.m_methods = _functions,
};
static int
sre_traverse(PyObject *module, visitproc visit, void *arg)
{
sre_module_state *state = get_sre_module_state(module);

Py_VISIT(state->Pattern_Type);
Py_VISIT(state->Match_Type);
Py_VISIT(state->Scanner_Type);

return 0;
}

static int
sre_clear(PyObject *module)
{
sre_module_state *state = get_sre_module_state(module);

Py_CLEAR(state->Pattern_Type);
Py_CLEAR(state->Match_Type);
Py_CLEAR(state->Scanner_Type);

return 0;
}

static void
sre_free(void *module)
{
sre_clear((PyObject *)module);
}

#define CREATE_TYPE(m, type, spec) \
do { \
Expand All @@ -2811,17 +2834,13 @@ do { \
} \
} while (0)

PyMODINIT_FUNC PyInit__sre(void)
static int
sre_exec(PyObject *m)
{
sre_module_state *state;
PyObject* m;
PyObject* d;
PyObject* x;

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

/* Create heap types */
state = get_sre_module_state(m);
CREATE_TYPE(m, state->Pattern_Type, &pattern_spec);
Expand Down Expand Up @@ -2859,11 +2878,32 @@ PyMODINIT_FUNC PyInit__sre(void)
PyDict_SetItemString(d, "copyright", x);
Py_DECREF(x);
}
return m;
return 0;

error:
Py_XDECREF(m);
return NULL;
return -1;
}

static PyModuleDef_Slot sre_slots[] = {
{Py_mod_exec, sre_exec},
{0, NULL},
};

static struct PyModuleDef sremodule = {
.m_base = PyModuleDef_HEAD_INIT,
.m_name = "_" SRE_MODULE,
.m_size = sizeof(sre_module_state),
.m_methods = _functions,
.m_slots = sre_slots,
.m_traverse = sre_traverse,
.m_free = sre_free,
.m_clear = sre_clear,
};

PyMODINIT_FUNC
PyInit__sre(void)
{
return PyModuleDef_Init(&sremodule);
}

/* vim:ts=4:sw=4:et
Expand Down