Description

encukou

opened on Jul 11, 2022

Issue body actions

The -X showrefcount option currently prints the number of references at exit.
Once we get the number to 0, we should test that it stays that way. It would be nice to add a mode, say -X showrefcount=assert0 or -X checkrefcount, that exits with nonzero if refs/blocks remain. Then it can be used in subprocess tests.

cc @vstinner

Activity

1. encukou

   added

   type-featureA feature request or enhancementA feature request or enhancement

   on Jul 11, 2022
2. jkloth commented on Jul 11, 2022

   

   jkloth

   on Jul 11, 2022

   Contributor

   More actions

   There is an existing test for 0 leaked refcounts: test_embed.test_no_memleak.

   It currently only ensures that leaks do not occur within just an init/fini cycle.
3. vstinner commented on Aug 3, 2022

   

   vstinner

   on Aug 3, 2022

   Member

   More actions

   I'm not sure if such feature would be useful today, since there are still many Python stdlib extensions which leak memory at exit :-(

   • Convert static types to heap types: use PyType_FromSpec() #84258
   • Clear static types in Py_Finalize() for embedded Python #90575 (comment)

   Well, at least last time I checked. Maybe things went better since I checked.
4. encukou commented on Aug 4, 2022

   

   encukou

   on Aug 4, 2022

   MemberAuthor

   More actions

   Well, there are some extensions/tests that don't leak memory, and the option could be used to ensure those don't start leaking.
5. iritkatriel

   added

   interpreter-core(Objects, Python, Grammar, and Parser dirs)(Objects, Python, Grammar, and Parser dirs)

   on Nov 27, 2023

Issue actions