rjfarmer / gfort2py / 28711741961

82%

push

github

Cache assumed-shape array descriptor ctype to fix per-call memory leak (#138)

* Cache assumed-shape array descriptor ctype to fix per-call memory leak

ftype_assumed_shape.ctype built three fresh ctypes.Structure subclasses
(_bounds14, _dtype_type, _fAllocArray) on every access. CPython never frees
Structure subclasses, so every call to a procedure with an assumed-shape
array argument leaked memory (~16 KB/call, measured) and created new
Structure subclasses without bound.

The descriptor layout depends only on (ndims, is_64bit()), so build the
class once and cache it keyed on that tuple. Behaviour is unchanged.

Adds a regression test that fails before this change (600 new
ctypes.Structure subclasses over 200 calls) and passes after.

6 of 6 new or added lines in 1 file covered. (100.0%)

2632 of 3197 relevant lines covered (82.33%)

0.82 hits per line