On Wed, 2020-05-13 at 13:14 +0100, Pablo Galindo Salgado wrote:
> > But again this is for PyObjects only.
>
> Not really, we check also memory blocks:
>
> https://github.com/python/cpython/blob/master/Lib/test/libregrtest/refleak.py#L72
>
> as long as you don't directly call malloc and use one o
Aah, thanks for pointing this out. As a quick side note: are these cool
regrtest features documented anywhere? I remember seeing some mention of
the -R 3:3 argument in the dev guide but nothing else.
Probably useful to have this documented somewhere in the dev guide.
On Wed, May 13, 2020, 5:14 AM
C extensions "should" use PyMem_Malloc/PyMem_Free to benefit of PEP
445 hooks like debug hooks and tracemalloc.
* Debug hooks (enabled by -X dev) helps to debug buffer overflow:
https://docs.python.org/dev/c-api/memory.html#c.PyMem_SetupDebugHooks
* tracemalloc: https://docs.python.org/dev/library
Hi,
Le mer. 13 mai 2020 à 02:04, Giampaolo Rodola' a écrit :
> I would like to discuss a proposal regarding one aspect which AFAIK is
> currently missing from cPython's test suite: the ability to detect memory
> leaks of functions implemented in the C extension modules.
test.regrtest can be us
> But again this is for PyObjects only.
Not really, we check also memory blocks:
https://github.com/python/cpython/blob/master/Lib/test/libregrtest/refleak.py#L72
as long as you don't directly call malloc and use one of the Python
specific APIs like PyMem_Malloc
then the reflect code should catc
On Wed, May 13, 2020 at 9:17 AM Ammar Askar wrote:
> > Py_DECREF calls in the C code
>
> I think this part specifically is already covered through refleak
> checks:
> https://github.com/python/cpython/blob/master/Lib/test/libregrtest/refleak.py
>
> Since it can involve the repetition of tests ma
> Py_DECREF calls in the C code
I think this part specifically is already covered through refleak
checks:
https://github.com/python/cpython/blob/master/Lib/test/libregrtest/refleak.py
Since it can involve the repetition of tests many times, these aren't
run on the CI though, they do get run on