FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-131510: Use PyUnstable_Unicode_GET_CACHED_HASH() by vstinner · Pull Request #141520 · python/cpython · GitHub

Repository navigation

gh-131510: Use PyUnstable_Unicode_GET_CACHED_HASH() - #141520

Merged
encukou merged 1 commit into
python:mainfrom
vstinner:cached_hash
Nov 14, 2025
Merged

encukou merged 1 commit into
python:mainfrom
vstinner:cached_hash

Conversation

vstinner commented Nov 13, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Replace code accessing directly to PyASCIIObject.hash with PyUnstable_Unicode_GET_CACHED_HASH().

Remove redundant "assert(PyUnicode_Check(op))" from PyUnstable_Unicode_GET_CACHED_HASH(), _PyASCIIObject_CAST() already implements the check.

Replace code accessing directly to PyASCIIObject.hash with
PyUnstable_Unicode_GET_CACHED_HASH().

Remove redundant "assert(PyUnicode_Check(op))" from
PyUnstable_Unicode_GET_CACHED_HASH(), _PyASCIIObject_CAST() already
implements the check.

Copy link
Copy Markdown
Member Author

cc @encukou

encukou merged commit 3bacae5 into python:main Nov 14, 2025
48 checks passed

encukou commented Nov 14, 2025

Copy link
Copy Markdown
Member

Thank you!

vstinner deleted the cached_hash branch December 3, 2025 15:36
StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
…41520)

Replace code that directly accesses PyASCIIObject.hash with
PyUnstable_Unicode_GET_CACHED_HASH().

Remove redundant "assert(PyUnicode_Check(op))" from
PyUnstable_Unicode_GET_CACHED_HASH(), _PyASCIIObject_CAST() already
implements the check.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL