Author vstinner
Recipients bkabrda, rkuska, serhiy.storchaka, vstinner
Date 2015-12-02.10:33:16
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1449052396.37.0.632316347088.issue19527@psf.upfronthosting.co.za>
In-reply-to
Content
00141-fix-tests_with_COUNT_ALLOCS-v5.patch: please don't do that! It makes tests much more verbose for a compilation option which is hidden and probably almost never used in the wild. The option has no configuration option for example.

*If* you really want to keep the feature, I would prefer to make it more visible (add a configuration option) and disable the output at exit by default. It's better to add a new "-X showcountallocs" as it was done with "-X showrefcount". Before, we had to fix a lot of unit tests (as 00141-fix-tests_with_COUNT_ALLOCS-v5.patch) to strip the "[xxx refs]" from stderr, it was very annoying.

"Python 3.4 now has sys.getallocatedblocks() and a new tracemalloc module which are compiled by default."

IMHO these two debug features superseded COUNT_ALLOCS. Please try to convince me of the use case of this very old debug feature.
History
Date User Action Args
2015-12-02 10:33:16vstinnersetrecipients: + vstinner, serhiy.storchaka, bkabrda, rkuska
2015-12-02 10:33:16vstinnersetmessageid: <1449052396.37.0.632316347088.issue19527@psf.upfronthosting.co.za>
2015-12-02 10:33:16vstinnerlinkissue19527 messages
2015-12-02 10:33:16vstinnercreate