-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
bpo-33451: Close .pyc files before calling PyEval_EvalCode() #7884
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.
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
bpo-33451: Close .pyc files before calling PyEval_EvalCode() #7884
Conversation
.pyc files were kept open longer than necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (I explicitly checked that run_pyc_file is a private helper function inside pythonrun.c, so there's no externally visible change in behaviour other than the intended one of closing the pyc file before we start executing the code)
Thanks @ZackerySpytz for the PR, and @ncoghlan for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6. |
Thanks @ZackerySpytz for the PR, and @ncoghlan for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7. |
GH-7888 is a backport of this pull request to the 3.7 branch. |
…-7884) Directly executed pyc files were being kept open longer than necessary. (cherry picked from commit ea73775) Co-authored-by: Zackery Spytz <[email protected]>
GH-7889 is a backport of this pull request to the 3.6 branch. |
…-7884) Directly executed pyc files were being kept open longer than necessary. (cherry picked from commit ea73775) Co-authored-by: Zackery Spytz <[email protected]>
Directly executed pyc files were being kept open longer than necessary. (cherry picked from commit ea73775) Co-authored-by: Zackery Spytz <[email protected]>
Directly executed pyc files were being kept open longer than necessary. (cherry picked from commit ea73775) Co-authored-by: Zackery Spytz <[email protected]>
.pyc files were kept open longer than necessary.
https://bugs.python.org/issue33451