rjfarmer / gfort2py

82%

LAST BUILD ON BRANCH main

branch: main

CHANGE BRANCH

x

Reset

• main
• arm
• bugfix/build-issues
• bugfix/complex-dtype
• bugfix/doc-update
• bugfix/mypy-precommit
• bugfix/quad-array-return
• bugfix/return-allocatable-char
• bugfix/return-allocatable-char2
• dependabot/github_actions/actions/cache-4
• dependabot/github_actions/actions/cache-5
• dependabot/github_actions/actions/cache-5.0.5
• dependabot/github_actions/actions/checkout-4
• dependabot/github_actions/actions/checkout-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/checkout-6.0.2
• dependabot/github_actions/actions/checkout-6.0.3
• dependabot/github_actions/actions/setup-python-4
• dependabot/github_actions/actions/setup-python-5
• dependabot/github_actions/actions/setup-python-6
• dependabot/github_actions/astral-sh/setup-uv-7
• dependabot/github_actions/astral-sh/setup-uv-8.0.0
• dependabot/github_actions/astral-sh/setup-uv-8.1.0
• dependabot/github_actions/astral-sh/setup-uv-8.2.0
• dependabot/github_actions/cygwin/cygwin-install-action-5
• dependabot/github_actions/cygwin/cygwin-install-action-6
• dependabot/github_actions/cygwin/cygwin-install-action-6.0.2
• dependabot/github_actions/egor-tensin/cleanup-path-4
• dependabot/github_actions/egor-tensin/cleanup-path-5
• dependabot/github_actions/egor-tensin/cleanup-path-5.0.1
• dependabot/github_actions/egor-tensin/cleanup-path-5.0.2
• dependabot/github_actions/github/codeql-action-4
• dependabot/github_actions/github/codeql-action-4.35.1
• dependabot/github_actions/github/codeql-action-4.35.2
• dependabot/github_actions/github/codeql-action-4.35.3
• dependabot/github_actions/github/codeql-action-4.35.4
• dependabot/github_actions/github/codeql-action-4.35.5
• dependabot/github_actions/github/codeql-action-4.36.0
• dependabot/github_actions/github/codeql-action-4.36.1
• dependabot/github_actions/github/codeql-action/upload-sarif-4.36.2
• dependabot/github_actions/psf/black-26.5.1
• dependabot/pip/numpy-gte-1.13.0-and-lt-2.2
• dependabot/pip/numpy-gte-1.13.0-and-lt-2.3
• dev-3.0
• feature/proc-pointers
• feature/qint
• feature/runtime-args
• feature/typebound-procs
• feature/windows-make
• fix-assumed-shape-ctype-leak
• gf3
• maint/cleanup
• maint/refactor-arg-handling
• pre-commit-ci-update-config
• v2.0.0
• v2.1.0
• v2.2.0
• v2.2.1
• v2.3.0
• v2.4.0
• v2.4.1
• v2.4.2
• v2.5.0
• v2.6.0
• v2.6.1
• v2.6.2
• v3.0.0
• v3.0.1

Build # 28711741961

Build Type

push

github

Committed by web-flow

Commit Message

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.

Coverage Stats

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