diff options
author | Victor Stinner <victor.stinner@gmail.com> | 2016-12-05 16:04:32 (GMT) |
---|---|---|
committer | Victor Stinner <victor.stinner@gmail.com> | 2016-12-05 16:04:32 (GMT) |
commit | 7bfb42d5b7721ca26e33050d025fec5c43c00058 (patch) | |
tree | c1c91a2a34361474de2c02388c8f91d4333f2ea5 /Modules/_collectionsmodule.c | |
parent | d77e5b7211e8daf22f2b3e0df124393bca504c38 (diff) | |
download | cpython-7bfb42d5b7721ca26e33050d025fec5c43c00058.zip cpython-7bfb42d5b7721ca26e33050d025fec5c43c00058.tar.gz cpython-7bfb42d5b7721ca26e33050d025fec5c43c00058.tar.bz2 |
Issue #28858: Remove _PyObject_CallArg1() macro
Replace
_PyObject_CallArg1(func, arg)
with
PyObject_CallFunctionObjArgs(func, arg, NULL)
Using the _PyObject_CallArg1() macro increases the usage of the C stack, which
was unexpected and unwanted. PyObject_CallFunctionObjArgs() doesn't have this
issue.
Diffstat (limited to 'Modules/_collectionsmodule.c')
-rw-r--r-- | Modules/_collectionsmodule.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Modules/_collectionsmodule.c b/Modules/_collectionsmodule.c index cbf5b50..b8b7584 100644 --- a/Modules/_collectionsmodule.c +++ b/Modules/_collectionsmodule.c @@ -538,7 +538,8 @@ deque_copy(PyObject *deque) return NULL; } if (old_deque->maxlen < 0) - return _PyObject_CallArg1((PyObject *)(Py_TYPE(deque)), deque); + return PyObject_CallFunctionObjArgs((PyObject *)(Py_TYPE(deque)), + deque, NULL); else return PyObject_CallFunction((PyObject *)(Py_TYPE(deque)), "Oi", deque, old_deque->maxlen, NULL); |