[Python-checkins] r45375 - python/trunk/Lib/test/leakers/README.txt

[neal.norwitz]

Author: neal.norwitz
Date: Fri Apr 14 07:35:39 2006
New Revision: 45375

Modified:
   python/trunk/Lib/test/leakers/README.txt
Log:
Update notes to address cyclic garbage and what should happen when a test is fixed.

Modified: python/trunk/Lib/test/leakers/README.txt
==============================================================================
--- python/trunk/Lib/test/leakers/README.txt	(original)
+++ python/trunk/Lib/test/leakers/README.txt	Fri Apr 14 07:35:39 2006
@@ -5,6 +5,15 @@
 doesn't increase, the bug has been fixed and the file should be removed
 from the repository.
 
+Note:  be careful to check for cyclic garbage.  Sometimes it may be helpful
+to define the leak function like:
+
+def leak():
+    def inner_leak():
+        # this is the function that leaks, but also creates cycles
+    inner_leak()
+    gc.collect() ; gc.collect() ; gc.collect()
+
 Here's an example interpreter session for test_gestalt which still leaks:
 
 >>> from test.leakers.test_gestalt import leak
@@ -17,3 +26,7 @@
 [28940 refs]
 >>> 
 
+Once the leak is fixed, the test case should be moved into an appropriate
+test (even if it was originally from the test suite).  This ensures the
+regression doesn't happen again.  And if it does, it should be easier
+to track down.