aio-libs / multidict / 35654102862

88%

push

github

Split deferred decref and compiler macros into their own headers (#1510)

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

## What do these changes do?

Moves `md_deferred_decref_t` and its helpers out of `hashtable.h` into a
new `multidict/_multilib/deferred_decref.h`, and the `HT_UNLIKELY` /
`HT_COLD` / `HT_ALWAYS_INLINE` macros out of `htkeys.h` into a new
`compiler.h`. The `md_` prefix is dropped from the deferred decref names
(`deferred_decref_t`, `deferred_decref_push()`, ...) and the `HT_`
prefix from the macros. Otherwise the code is moved verbatim. 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 `htkeys.h` and keeps self-contained
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, pure code move covered by
the existing suite
- [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 (`pytest -q -k "not test_leak"`, CPython 3.14t):
C extension release build: 2097 passed, 31 skipped
C extension `MULTIDICT_DEBUG_BUILD=1`: 2097 passed, 31 skipped
Pure Python (`--no-c-extensions`): 1056 passed, 9 skipped
Lint: pre-commit clang-format passed; `make doc-spelling` passed.
</details>

762 of 1524 branches covered (50.0%)

Branch coverage included in aggregate %.

6616 of 6897 relevant lines covered (95.93%)

1.92 hits per line