aio-libs / multidict / 35703746757

87%

push

github

Move consistency check helpers into md_debug.h (#1513)

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

## What do these changes do?

Moves the debug-only `ASSERT_CONSISTENT` macro and the
`_md_check_consistency()` / `_md_dump()` helpers out of `hashtable.h`
into a new `multidict/_multilib/md_debug.h` header. Every file that
calls `ASSERT_CONSISTENT` (`hashtable.h`, `views.h`, `_multidict.c`) now
includes it directly. The function bodies are moved verbatim; the
forward declarations in `hashtable.h` are no longer needed. The pure
Python backend is unaffected.

## Are there changes in behavior for the user?

No.

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

No, it shrinks `hashtable.h` and keeps debug helpers separate from the
hash table logic.

## Related issue number

N/A

## Checklist

- [x] I think the code is well written
- [ ] Unit tests for the changes exist: N/A, code move only; covered by
the existing suite under debug builds
- [x] Documentation reflects the changes
- [ ] If you provide code modification, please add yourself to
`CONTRIBUTORS.txt`: N/A
- [x] Add a new news fragment into the `CHANGES/` folder
- [x] `make doc-spelling` passes and any new technical words are added
to `docs/spelling_wordlist.txt`

Drafted with Claude Code (Claude Opus 5); reviewed by @asvetlov.

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

Tests, `python -m pytest tests -q -k "not test_leak"`:
- 3.13.2 debug build (`MULTIDICT_DEBUG_BUILD=1`, asserts on): 2149
passed, 19 skipped
- 3.14.7t debug build: 2137 passed, 31 skipped
- 3.13.2 release build: 2149 passed, 19 skipped
- Pure Python (`--no-c-extensions`): 1070 passed, 15 skipped

Lint: pre-commit on changed files passed; `make doc-spelling` passed.
</details>

762 of 1524 branches covered (50.0%)

Branch coverage included in aggregate %.

6714 of 6995 relevant lines covered (95.98%)

1.92 hits per line