Skip to content

PEP 667: Clarify impact on PyEval_GetLocals #3809

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

Closed
ncoghlan wants to merge 6 commits into from
Closed

PEP 667: Clarify impact on PyEval_GetLocals #3809

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
78d16e3
PEP 667: Clarify impact on PyEval_GetLocals
ncoghlan Jun 2, 2024
71a8024
Clear borrowed reference in example
ncoghlan Jun 2, 2024
f899d74
Merge branch 'main' into pep-667-clarify-pyeval_getlocals
ncoghlan Jun 21, 2024
37d7ef4
Add link to SC decision on PyEval_GetLocals
ncoghlan Jun 21, 2024
8aa2e35
Fix custom link syntax
ncoghlan Jun 21, 2024
83d8028
Remove stray underscore
ncoghlan Jun 21, 2024
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
Clear borrowed reference in example
  • Loading branch information
@ncoghlan
ncoghlan committed Jun 2, 2024
commit 71a80245a86ce20ebb0ad9c30cfe003cba6556ba
1 change: 1 addition & 0 deletions peps/pep-0667.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -271,6 +271,7 @@ In these cases, the original code should be replaced with::
goto error_handler;
}
locals = PyFrame_GetLocals(frame);
frame = NULL; // Minimise visibility of borrowed reference
if (locals == NULL) {
goto error_handler;
}
Expand Down