summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVictor Stinner <victor.stinner@gmail.com>2016-08-22 21:33:13 (GMT)
committerVictor Stinner <victor.stinner@gmail.com>2016-08-22 21:33:13 (GMT)
commit463b86a881ab9afc4c24f4a669582626977708d7 (patch)
treecf0434ac97f18bbc29973f9883ce10451f0aaa8d
parent155ea65e5c88d250a752ee5321860ef11ede4085 (diff)
downloadcpython-463b86a881ab9afc4c24f4a669582626977708d7.zip
cpython-463b86a881ab9afc4c24f4a669582626977708d7.tar.gz
cpython-463b86a881ab9afc4c24f4a669582626977708d7.tar.bz2
Issue #27809: Use _PyObject_FastCallDict()
Modify: * init_subclass() * builtin___build_class__() Fix also a bug in init_subclass(): check for super() failure.
-rw-r--r--Objects/typeobject.c28
-rw-r--r--Python/bltinmodule.c12
2 files changed, 15 insertions, 25 deletions
diff --git a/Objects/typeobject.c b/Objects/typeobject.c
index 544d0b5..63bfd66 100644
--- a/Objects/typeobject.c
+++ b/Objects/typeobject.c
@@ -7007,30 +7007,28 @@ set_names(PyTypeObject *type)
static int
init_subclass(PyTypeObject *type, PyObject *kwds)
{
- PyObject *super, *func, *tmp, *tuple;
+ PyObject *super, *func, *result;
+ PyObject *args[2] = {(PyObject *)type, (PyObject *)type};
+
+ super = _PyObject_FastCall((PyObject *)&PySuper_Type, args, 2);
+ if (super == NULL) {
+ return -1;
+ }
- super = PyObject_CallFunctionObjArgs((PyObject *) &PySuper_Type,
- type, type, NULL);
func = _PyObject_GetAttrId(super, &PyId___init_subclass__);
Py_DECREF(super);
-
- if (func == NULL)
+ if (func == NULL) {
return -1;
-
- tuple = PyTuple_New(0);
- if (tuple == NULL) {
- Py_DECREF(func);
- return 0;
}
- tmp = PyObject_Call(func, tuple, kwds);
- Py_DECREF(tuple);
- Py_DECREF(func);
- if (tmp == NULL)
+ result = _PyObject_FastCallDict(func, NULL, 0, kwds);
+ Py_DECREF(func);
+ if (result == NULL) {
return -1;
+ }
- Py_DECREF(tmp);
+ Py_DECREF(result);
return 0;
}
diff --git a/Python/bltinmodule.c b/Python/bltinmodule.c
index 1cdc0e2..b22867e 100644
--- a/Python/bltinmodule.c
+++ b/Python/bltinmodule.c
@@ -155,16 +155,8 @@ builtin___build_class__(PyObject *self, PyObject *args, PyObject *kwds)
}
}
else {
- PyObject *pargs = PyTuple_Pack(2, name, bases);
- if (pargs == NULL) {
- Py_DECREF(prep);
- Py_DECREF(meta);
- Py_XDECREF(mkw);
- Py_DECREF(bases);
- return NULL;
- }
- ns = PyEval_CallObjectWithKeywords(prep, pargs, mkw);
- Py_DECREF(pargs);
+ PyObject *pargs[2] = {name, bases};
+ ns = _PyObject_FastCallDict(prep, pargs, 2, mkw);
Py_DECREF(prep);
}
if (ns == NULL) {