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-39778: Don't traverse weak-reference lists OrderedDict's tp_traverse and tp_clear #18749
Conversation
Codecov Report
@@ Coverage Diff @@
## master #18749 +/- ##
=========================================
Coverage 82.13% 82.13%
=========================================
Files 1956 1955 -1
Lines 590143 584845 -5298
Branches 44489 44503 +14
=========================================
- Hits 484725 480389 -4336
+ Misses 95762 94804 -958
+ Partials 9656 9652 -4
Continue to review full report at Codecov.
|
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! Thanks 😃
Thanks @pablogsal for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7. |
Thanks @pablogsal for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8. |
Sorry @pablogsal, I had trouble checking out the |
…rse and tp_clear (pythonGH-18749) Objects do not own weak references to them directly through the __weakref__ list so these do not need to be traversed by the GC. (cherry picked from commit 0c2b509) Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
GH-18755 is a backport of this pull request to the 3.7 branch. |
…_traverse and tp_clear (pythonGH-18749) Objects do not own weak references to them directly through the __weakref__ list so these do not need to be traversed by the GC. (cherry picked from commit 0c2b509) Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
GH-18756 is a backport of this pull request to the 3.8 branch. |
https://bugs.python.org/issue39778