bpo-1635741: Clean sysdict and builtins of interpreter#21605
Conversation
|
The master benchmark: After this PR: |
Sorry, something went wrong.
|
@vstinner Hi, victor, pls take a look after your vacation, thanks. |
Sorry, something went wrong.
|
thanks Dong-hee Na for your label ;) |
Sorry, something went wrong.
What is this benchmark? Which code did you run? |
Sorry, something went wrong.
Hm, This should not be called as benchmark. I compare the refleak of this PR with the refleak of master branch. I use debug mode to test it. Because the testcase in https://bugs.python.org/issue1635741#msg355187 use |
Sorry, something went wrong.
|
Since few people care about subinterpreters, I don't think that it's worth it to document this internal change in a NEWS entry: I added "skip news" label. |
Sorry, something went wrong.
vstinner
left a comment
There was a problem hiding this comment.
LGTM.
Sorry, something went wrong.
|
thanks, victor. |
Sorry, something went wrong.
https://bugs.python.org/issue1635741