[issue14785] Add sys._debugmallocstats()

Changes by Dave Malcolm : -- keywords: -needs review resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker ___ __

[issue14785] Add sys._debugmallocstats()

Roundup Robot added the comment: New changeset d63a80abfbec by David Malcolm in branch 'default': Issue #14785: Add sys._debugmallocstats() to help debug low-level memory allocation issues http://hg.python.org/cpython/rev/d63a80abfbec -- nosy: +python-dev _

[issue14785] Add sys._debugmallocstats()

Dave Malcolm added the comment: Thanks. I'm attaching an updated version of the patch, wrapping all new C entrypoints within a #ifndef Py_LIMITED_API I also moved the existing _PyObject_DebugMallocStats() entrypoint to within a #ifndef Py_LIMITED_API. As noted above, it is not documented (a

[issue14785] Add sys._debugmallocstats()

Antoine Pitrou added the comment: One other nit: the C API functions shouldn't be included in the limited API. -- ___ Python tracker ___

[issue14785] Add sys._debugmallocstats()

Dave Malcolm added the comment: Updated version of the patch, using test.script_helper.assert_python_ok() and adding a NEWS entry -- Added file: http://bugs.python.org/file25579/add-debug-malloc-stats-v2.patch ___ Python tracker

[issue14785] Add sys._debugmallocstats()

Antoine Pitrou added the comment: Nice idea. I don't see any obvious problem with the patch, except that the test should reuse test.script_helper.assert_python_ok(). -- nosy: +pitrou ___ Python tracker __

[issue14785] Add sys._debugmallocstats()

New submission from Dave Malcolm : I'm attaching a patch which generalizes the at-exit PYTHONMALLOCSTATS memory usage report, so that it's available in a regular build and can be triggered from Python, by calling: sys._debugmallocstats() This can be useful when debugging memory usage issues