aio-libs / multidict / 33988650647

87%

push

github

Fix heap type reference leak (#1415)

<!-- Thank you for your contribution! -->

## What do these changes do?

Fixed C-extension multidict and proxy deallocators to release the
heap-type reference owned by each instance. Added regression coverage
for dictionary and proxy type reference counts.

## Are there changes in behavior for the user?

No API behavior changed. Repeated construction and destruction of these
C-extension objects no longer retains type references.

## Is it a substantial burden for the maintainers to support this?

No. The cleanup follows the existing deallocation pattern used by
multidict views and iterators.

## Related issue number

N/A

## Checklist

- [x] I think the code is well written
- [x] Unit tests for the changes exist
- [x] Documentation reflects the changes
- [ ] If you provide code modification, please add yourself to
`CONTRIBUTORS.txt` N/A, this repository does not contain this file
- [x] Add a new news fragment into the `CHANGES/` folder

<details>
<summary>Agent run details (optional, for reviewers)</summary>

Tests: `python -m pytest -q` passed with the C extension, 1661 tests
passed.

Tests: `python -m pytest -q --no-c-extensions` passed with the
pure-Python implementation, 850 tests passed.

Lint: targeted pre-commit hooks passed.

Docs: `make doc-spelling` is blocked by five pre-existing misspellings
in generated `CHANGES.rst`; this fragment introduced none.
</details>

Drafted with GitHub Copilot App 1.0.83-5; reviewed by Andrew Svetlov.

---------

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

627 of 1254 branches covered (50.0%)

Branch coverage included in aggregate %.

16 of 20 new or added lines in 1 file covered. (80.0%)

4067 of 4246 relevant lines covered (95.78%)

1.92 hits per line