Visitar URL original
gh-132413: Clear weakref to _datetime after modules are finalized by neonene · Pull Request #136152 · python/cpython · GitHub
Skip to content

gh-132413: Clear weakref to _datetime after modules are finalized #136152

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.

Sign up for GitHub

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

Closed
neonene wants to merge 13 commits into from
Closed

gh-132413: Clear weakref to _datetime after modules are finalized #136152

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
ff10134
NEWS
neonene Jun 30, 2025
4f83cb0
Add test
neonene Jun 30, 2025
4e83b5b
Avoid weakref being cleared by GC
neonene Jun 30, 2025
c4b1c62
Do not hold module in tuple
neonene Jul 1, 2025
4b1b45c
GC-untrack/track
neonene Jul 1, 2025
c5a2f41
Merge branch 'main' into wr-132413
neonene Jul 1, 2025
e33f704
Sort weaklist
neonene Jul 1, 2025
263bdb2
Add non-NULL check
neonene Jul 1, 2025
fdfadf0
Simply do not GC-untrack/track
neonene Jul 1, 2025
26c6425
Add a missing NULL check
neonene Jul 1, 2025
4807c36
Merge branch 'main' into wr-132413
neonene Jul 1, 2025
a5f2521
move test to test_gc.py
neonene Jul 3, 2025
d7c36b9
add and move test case
neonene Jul 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
add and move test case
  • Loading branch information
@neonene
neonene committed Jul 8, 2025
commit d7c36b960c675e5772f8cc00f167c71de4fa4f2d
23 changes: 0 additions & 23 deletions Lib/test/test_gc.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1517,29 +1517,6 @@ def test_ast_fini(self):
""")
assert_python_ok("-c", code)

def test_weakref_to_module_at_shutdown(self):
# gh-132413: Weakref gets cleared before modules are finalized
code = textwrap.dedent("""
import os # any module other than sys

def gen():
import weakref
wr = weakref.ref(os)
try:
yield
finally:
print(
os is not None,
os is wr()
)

it = gen()
next(it)
# quirk: Shutdown starts, then the finally block above follows
""")
res = assert_python_ok('-c', code)
self.assertEqual(res.out.rstrip(), b'True True')


def setUpModule():
global enabled, debug
Expand Down
44 changes: 44 additions & 0 deletions Lib/test/test_weakref.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1044,6 +1044,50 @@ def callback(obj):
stderr = res.err.decode("ascii", "backslashreplace")
self.assertNotRegex(stderr, "_Py_Dealloc: Deallocator of type 'TestObj'")

def test_module_at_shutdown(self):
# gh-132413: Weakref gets cleared by gc before modules are finalized
code = textwrap.dedent("""
import os # any module other than sys
import weakref
wr = weakref.ref(os)

def gen():
try:
yield
finally:
print(
os is not None,
os is wr()
)

it = gen()
next(it)
print('fini', end=' ')
""")
with self.subTest("gen"):
res = script_helper.assert_python_ok('-c', code)
self.assertEqual(res.out.rstrip(), b'fini True True')

code = textwrap.dedent("""
import os
import weakref
wr = weakref.ref(os)

class C:
def __del__(self):
print(
os is not None,
os is wr()
)

l = [C()]
l.append(l)
print('fini', end=' ')
""")
with self.subTest("del"):
res = script_helper.assert_python_ok('-c', code)
self.assertEqual(res.out.rstrip(), b'fini True True')


class SubclassableWeakrefTestCase(TestBase):

Expand Down