Issue 1443865: gc.get_count() and other changes (original) (raw)

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/42983

classification

process

Created on 2006-03-06 04:09 by barry, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Messages (5)
msg49655 - (view)	Author: Barry A. Warsaw (barry) *	Date: 2006-03-06 04:09
This patch adds gc.get_count() as a companion to gc.get_threshold(). It also adds an optional argument to gc.collect() to allow you to specify the generation to collect. This patch also fixes a couple of gcc warnings.
msg49656 - (view)	Author: Neal Norwitz (nnorwitz) *	Date: 2006-03-06 23:42
Logged In: YES user_id=33168 I fixed the 2 warnings with %ld, so that part of the patch isn't necessary any longer. Instead of setting genarg = -999, can't you just set it to NUM_GENERATIONS - 1 and get rid of the check for -999? Doesn't ValueError make more sense than an IndexError? Isn't ValueError typically used for invalid args? Should gc_collect be: METH_VARARGS | METH_KEYWORDS ?
msg49657 - (view)	Author: Barry A. Warsaw (barry) *	Date: 2006-03-07 03:26
Logged In: YES user_id=12800 All good points. Here's a revised patch, with test cases and doc updates.
msg49658 - (view)	Author: Neal Norwitz (nnorwitz) *	Date: 2006-03-07 05:19
Logged In: YES user_id=33168 Oooooo, tests and doc, very nice. Don't forget to add a NEWS entry. Also the doc should have \versionadded, \versionchanged for the new method/added param. Neil was fine with the concept and no one else objected, so I'd say check it in.
msg49659 - (view)	Author: Barry A. Warsaw (barry) *	Date: 2006-03-07 09:46
Logged In: YES user_id=12800 Thanks! Committed in r42880.