This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: add gc.DEBUG_SAVEALL option, gc_str fix
Type: Stage:
Components: Interpreter Core Versions:
process
Status: closed Resolution:
Dependencies: Superseder:
Assigned To: nascheme Nosy List: gvanrossum, marangoz, nascheme, tim.peters
Priority: normal Keywords: patch

Created on 2000-09-04 01:13 by nascheme, last changed 2022-04-10 16:02 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
None nascheme, 2000-09-04 01:13 None
Messages (7)
msg34218 - (view) Author: Neil Schemenauer (nascheme) * (Python committer) Date: 2000-09-04 01:13
 
msg34219 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2000-09-22 14:08
*Unless* you fear that this would destabilize things (which are coming together so nicely), I'd suggest to check this in before beta2 goes out!
Accepted and Assigned to Neil for this reason.

It doesn't *look* like there's anything tricky going on, but I'll leave it to your judgement. If you don't want to check it in after all, please log in to sourceforge and change the status to Postponed.
msg34220 - (view) Author: Tim Peters (tim.peters) * (Python committer) Date: 2000-09-22 04:35
Is there a reason this patch is just sitting here?  *Looks* like all the concerns were addressed two weeks ago.
msg34221 - (view) Author: Neil Schemenauer (nascheme) * (Python committer) Date: 2000-09-04 14:05
- regrtest will print the repr of cyclic garbage found if -l option is specified and -q is not
- put else on newline in accordance with Python style
- constrain debugging options during test_gc (don't enabled DEBUG_LEAK)
- fix test_gc to remove garbage from gc.garbage
msg34222 - (view) Author: Neil Schemenauer (nascheme) * (Python committer) Date: 2000-09-22 13:58
Is this patch is okay for 2.0?  If so I can check it in and close the patch.  I was planning to wait for post 2.0.
msg34223 - (view) Author: Neil Schemenauer (nascheme) * (Python committer) Date: 2000-09-22 15:45
Checked in.  Guys, _please_ review my diffs.  The original patch did not apply 
cleanly.  Also,  I found some problems with the patch which I tried to fix.
msg34224 - (view) Author: Vladimir Marangozov (marangoz) * (Python triager) Date: 2000-09-04 03:13
Looks good. I approve the str fix. One nit: "else" clauses usually go
to a new line in the Python source. Make it "else if" on a new line.

Questions:

- DEBUG_LEAK now includes SAVEALL. The regression test suite sets
   DEBUG_LEAK by default. It used to report a bunch of objects during
   test_gc. Now it will collect the detected cycles in the suite. Should
   regrtest.py report them somehow?

- After the regression test, gc.garbage contains an A instance and an
   empty list. I think they're created by test_gc. Can't they be cleared?
   We can assign gc.garbage = [], but this doesn't clear anything.

- test_gc/test_saveall has gc.set_debug(0), followed immediately by
   gc.set_debug(gc.DEBUG_SAVEALL).
   Is this on purpose or is it a typo? BTW, I don't quite understand why
   this function does what it does... Could you elaborate on it?
History
Date User Action Args
2022-04-10 16:02:20adminsetgithub: 33035
2000-09-04 01:13:57naschemecreate