Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34k
gh-110752: Reset ceval.eval_breaker to 0 in interpreter_clear#110753
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
gh-110752: Reset ceval.eval_breaker to 0 in interpreter_clear#110753
Uh oh!
There was an error while loading. Please reload this page.
Conversation
gaogaotiantian commented Oct 12, 2023 • edited by bedevere-app bot
Loading Uh oh!
There was an error while loading. Please reload this page.
edited by bedevere-app bot
Uh oh!
There was an error while loading. Please reload this page.
markshannon commented Oct 12, 2023
Looks good |
bedevere-bot commented Oct 12, 2023
|
bedevere-bot commented Oct 12, 2023
|
I tried to read the code and I believe at this point all the threads should be done so we don't need atomic operations.
Also no test was added for now. The test requires interpreter finalization so it needs to be in its own process.