[issue41796] _ast module state should be made per interpreter

STINNER Victor added the comment: The leak is fixed, I close again the issue. Thanks for the report Pablo. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue41796] _ast module state should be made per interpreter

STINNER Victor added the comment: New changeset fd957c124c1d9c5eaf61f7af8cf266bafcb1 by Victor Stinner in branch 'master': bpo-41796: Call _PyAST_Fini() earlier to fix a leak (GH-23131) https://github.com/python/cpython/commit/fd957c124c1d9c5eaf61f7af8cf266bafcb1 --

[issue41796] _ast module state should be made per interpreter

Change by STINNER Victor : -- pull_requests: +22047 stage: resolved -> patch review pull_request: https://github.com/python/cpython/pull/23131 ___ Python tracker ___

[issue41796] _ast module state should be made per interpreter

Change by Batuhan Taskaya : -- nosy: +BTaskaya ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41796] _ast module state should be made per interpreter

STINNER Victor added the comment: There is a leak (I just marked bpo-42250 as duplicate of this issue): https://buildbot.python.org/all/#/builders/205/builds/83 OK .. test_ast leaked [23640, 23636, 23640] references, sum=70916 test_ast leaked [7932, 7930, 7932] memory blocks, sum=23794 1

[issue41796] _ast module state should be made per interpreter

Change by STINNER Victor : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue41796] _ast module state should be made per interpreter

STINNER Victor added the comment: New changeset 5cf4782a2630629d0978bf4cf6b6340365f449b2 by Victor Stinner in branch 'master': bpo-41796: Make _ast module state per interpreter (GH-23024) https://github.com/python/cpython/commit/5cf4782a2630629d0978bf4cf6b6340365f449b2 --

[issue41796] _ast module state should be made per interpreter

Change by hai shi : -- nosy: +shihai1991 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41796] _ast module state should be made per interpreter

Change by STINNER Victor : -- keywords: +patch pull_requests: +21938 stage: -> patch review pull_request: https://github.com/python/cpython/pull/23024 ___ Python tracker ___

[issue41796] _ast module state should be made per interpreter

STINNER Victor added the comment: > I started this here: > https://github.com/python/cpython/commit/60960cba606573450e76934c954787419524147d > Feel free to take that code. Oh thanks! I was looking for your work, but I failed to find it. It is really helpful! I started to hack asdl_c.py

[issue41796] _ast module state should be made per interpreter

Petr Viktorin added the comment: I started this here: https://github.com/python/cpython/commit/60960cba606573450e76934c954787419524147d Feel free to take that code. -- nosy: +petr.viktorin ___ Python tracker

[issue41796] _ast module state should be made per interpreter

New submission from STINNER Victor : In September 2019, the _ast extension module was converted to PEP 384 (stable ABI): bpo-38113. In bpo-41631, I moved the _ast module state back to a global state, rather than a regular module state, to fix multiple bugs. The state should be made per