Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32.4k
gh-106320: Move private _PyHash API to the internal C API#107026
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
* No longer export most private _PyHash symbols, only export the ones which are needed by shared extensions.* Modules/_xxtestfuzz/fuzzer.c now uses the internal C API.
`Modules/_xxtestfuzz/fuzzer.c` in CPython started using private API inpython/cpython@89f9875#diff-a22ac7579f23068e0c6bcacf1086ee962fdf8e68725cf0510b42d0a389cf1efd.This made fuzzing builds failhttps://bugs.chromium.org/p/oss-fuzz/issues/detail?id=60831.Including needed header files fixes the error.
`Modules/_xxtestfuzz/fuzzer.c` in CPython started using private API inpython/cpython@89f9875#diff-a22ac7579f23068e0c6bcacf1086ee962fdf8e68725cf0510b42d0a389cf1efd.This made fuzzing builds failhttps://bugs.chromium.org/p/oss-fuzz/issues/detail?id=60831.Including needed header files fixes the error.
Again, as in#108604, the gmpy2 uses some parts of this API, e.g. to hash mpz like int. Meanwhile I didaleaxit/gmpy#441 to workaround removed API.@vstinner, what we could do instead? |
Would you mind to open a new issue, give the exhaustive list of functions that you need, and which API do you expect? |
…publicThis partially revertspython#107026.
_Py_HashBytes was hidden in 3.13 (python/cpython#107026).> C assumes that any function with an "implicit declaration" returns an int, which'll be smaller than a Py_hash_t, hence the different results. (https://groups.google.com/g/cython-users/c/YQkU8oc8oSM/m/nVJ_IX8KAgAJ)
_Py_HashBytes was hidden in 3.13 (python/cpython#107026).> C assumes that any function with an "implicit declaration" returns an int, which'll be smaller than a Py_hash_t, hence the different results. (https://groups.google.com/g/cython-users/c/YQkU8oc8oSM/m/nVJ_IX8KAgAJ)
Uh oh!
There was an error while loading.Please reload this page.