bpo-33418: Add tp_clear for function object by methane · Pull Request #8058 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.8k
Merged
Merged
Conversation
Member
Without tp_clear, GC can't break cyclic reference.
It will cause memory leak when cyclic reference is
created intentionally.
the-knights-who-say-ni
added
the
CLA signed
label
bedevere-bot
added
the
awaiting merge
label
Without tp_clear, GC can't break cyclic reference. It will cause memory leak when cyclic reference is created intentionally.
methane
force-pushed
the
func_clear
branch
from
2741a21 to
fd8f66c
Compare
serhiy-storchaka
approved these changes
serhiy-storchaka
commented
Jul 3, 2018
serhiy-storchaka commented
Member
I have doubts about backporting this change. This can expose bugs when a ref cycle be broken at unexpected place.
methane
removed
needs backport to 2.7
labels
methane
merged commit
3c45240
into
python:master
bedevere-bot
removed
the
awaiting merge
label
methane
deleted the
func_clear
branch
Yhg1s pushed a commit that referenced this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment