mirror of
https://github.com/python/cpython.git
synced 2025-08-04 00:48:58 +00:00
bpo-46236: Fix PyFunction_GetAnnotations() returned tuple. (GH-30409)
Automerge-Triggered-By: GH:pablogsal
(cherry picked from commit 46e4c257e7
)
Co-authored-by: Inada Naoki <songofacandy@gmail.com>
This commit is contained in:
parent
7e951f356e
commit
da8be157f4
2 changed files with 34 additions and 22 deletions
|
@ -221,6 +221,37 @@ PyFunction_SetClosure(PyObject *op, PyObject *closure)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static PyObject *
|
||||
func_get_annotation_dict(PyFunctionObject *op)
|
||||
{
|
||||
if (op->func_annotations == NULL) {
|
||||
return NULL;
|
||||
}
|
||||
if (PyTuple_CheckExact(op->func_annotations)) {
|
||||
PyObject *ann_tuple = op->func_annotations;
|
||||
PyObject *ann_dict = PyDict_New();
|
||||
if (ann_dict == NULL) {
|
||||
return NULL;
|
||||
}
|
||||
|
||||
assert(PyTuple_GET_SIZE(ann_tuple) % 2 == 0);
|
||||
|
||||
for (Py_ssize_t i = 0; i < PyTuple_GET_SIZE(ann_tuple); i += 2) {
|
||||
int err = PyDict_SetItem(ann_dict,
|
||||
PyTuple_GET_ITEM(ann_tuple, i),
|
||||
PyTuple_GET_ITEM(ann_tuple, i + 1));
|
||||
|
||||
if (err < 0) {
|
||||
return NULL;
|
||||
}
|
||||
}
|
||||
Py_SETREF(op->func_annotations, ann_dict);
|
||||
}
|
||||
Py_INCREF(op->func_annotations);
|
||||
assert(PyDict_Check(op->func_annotations));
|
||||
return op->func_annotations;
|
||||
}
|
||||
|
||||
PyObject *
|
||||
PyFunction_GetAnnotations(PyObject *op)
|
||||
{
|
||||
|
@ -228,7 +259,7 @@ PyFunction_GetAnnotations(PyObject *op)
|
|||
PyErr_BadInternalCall();
|
||||
return NULL;
|
||||
}
|
||||
return ((PyFunctionObject *) op) -> func_annotations;
|
||||
return func_get_annotation_dict((PyFunctionObject *)op);
|
||||
}
|
||||
|
||||
int
|
||||
|
@ -443,27 +474,7 @@ func_get_annotations(PyFunctionObject *op, void *Py_UNUSED(ignored))
|
|||
if (op->func_annotations == NULL)
|
||||
return NULL;
|
||||
}
|
||||
if (PyTuple_CheckExact(op->func_annotations)) {
|
||||
PyObject *ann_tuple = op->func_annotations;
|
||||
PyObject *ann_dict = PyDict_New();
|
||||
if (ann_dict == NULL) {
|
||||
return NULL;
|
||||
}
|
||||
|
||||
assert(PyTuple_GET_SIZE(ann_tuple) % 2 == 0);
|
||||
|
||||
for (Py_ssize_t i = 0; i < PyTuple_GET_SIZE(ann_tuple); i += 2) {
|
||||
int err = PyDict_SetItem(ann_dict,
|
||||
PyTuple_GET_ITEM(ann_tuple, i),
|
||||
PyTuple_GET_ITEM(ann_tuple, i + 1));
|
||||
|
||||
if (err < 0)
|
||||
return NULL;
|
||||
}
|
||||
Py_SETREF(op->func_annotations, ann_dict);
|
||||
}
|
||||
Py_INCREF(op->func_annotations);
|
||||
return op->func_annotations;
|
||||
return func_get_annotation_dict(op);
|
||||
}
|
||||
|
||||
static int
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue