Skip to content

gh-132775: Expand the Capability of Interpreter.call() #133484

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
ericsnowcurrently merged 13 commits into from
May 30, 2025
Merged

gh-132775: Expand the Capability of Interpreter.call() #133484

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
cc387f8
Add _PyXI_session_result.
ericsnowcurrently May 10, 2025
b906d4a
Add _PyXI_Preserve().
ericsnowcurrently May 8, 2025
08307a0
Support full calls in interp_call().
ericsnowcurrently Apr 25, 2025
d173ec0
Use _PyXI_Preserve().
ericsnowcurrently May 8, 2025
a9612e3
Update the Interpreter.call() docstring.
ericsnowcurrently May 26, 2025
5d7141d
Improve handling of xidata-to-object for scripts and calls.
ericsnowcurrently May 26, 2025
293a4dc
Merge branch 'main' into interp-call-expansion
ericsnowcurrently May 26, 2025
c25d9e1
Always load the missing module in XI data situations.
ericsnowcurrently May 26, 2025
66bed1d
Fix refleaks.
ericsnowcurrently May 28, 2025
3ed3423
Fix _PyFunction_VerifyStateless().
ericsnowcurrently May 28, 2025
17beeda
Fix exc propagation and an excinfo memory leak.
ericsnowcurrently May 29, 2025
2682157
Drop a stale comment.
ericsnowcurrently May 29, 2025
37c41f8
Merge branch 'main' into interp-call-expansion
ericsnowcurrently May 29, 2025
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
Fix _PyFunction_VerifyStateless().
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed May 28, 2025
commit 3ed342338d8d82d9bc2417689d1dcffaf213ca5b
34 changes: 20 additions & 14 deletions Objects/funcobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1264,26 +1264,32 @@ _PyFunction_VerifyStateless(PyThreadState *tstate, PyObject *func)
}
// Disallow __defaults__.
PyObject *defaults = PyFunction_GET_DEFAULTS(func);
if (defaults != NULL && defaults != Py_None && PyDict_Size(defaults) > 0)
{
_PyErr_SetString(tstate, PyExc_ValueError, "defaults not supported");
return -1;
if (defaults != NULL) {
assert(PyTuple_Check(defaults)); // per PyFunction_New()
if (PyTuple_GET_SIZE(defaults) > 0) {
_PyErr_SetString(tstate, PyExc_ValueError,
"defaults not supported");
return -1;
}
}
// Disallow __kwdefaults__.
PyObject *kwdefaults = PyFunction_GET_KW_DEFAULTS(func);
if (kwdefaults != NULL && kwdefaults != Py_None
&& PyDict_Size(kwdefaults) > 0)
{
_PyErr_SetString(tstate, PyExc_ValueError,
"keyword defaults not supported");
return -1;
if (kwdefaults != NULL) {
assert(PyDict_Check(kwdefaults)); // per PyFunction_New()
if (PyDict_Size(kwdefaults) > 0) {
_PyErr_SetString(tstate, PyExc_ValueError,
"keyword defaults not supported");
return -1;
}
}
// Disallow __closure__.
PyObject *closure = PyFunction_GET_CLOSURE(func);
if (closure != NULL && closure != Py_None && PyTuple_GET_SIZE(closure) > 0)
{
_PyErr_SetString(tstate, PyExc_ValueError, "closures not supported");
return -1;
if (closure != NULL) {
assert(PyTuple_Check(closure)); // per PyFunction_New()
if (PyTuple_GET_SIZE(closure) > 0) {
_PyErr_SetString(tstate, PyExc_ValueError, "closures not supported");
return -1;
}
}
// Check the code.
PyCodeObject *co = (PyCodeObject *)PyFunction_GET_CODE(func);
Expand Down