Skip to content

Commit 57fb87d

Browse files
author
David Lawson
committed
_hash.pyx: fix string hashing in 3.13
_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)
1 parent b9dce0e commit 57fb87d

2 files changed

Lines changed: 7 additions & 0 deletions

File tree

‎capnpy/_hash.h‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,4 +30,10 @@ typedef struct {
3030
// Unused in Python 3, but has to exist during cython compilation
3131
_Py_HashSecret_custom HashSecret;
3232

33+
// Required for 3.13 because it's now hidden and C will assume the
34+
// return type is int rather than long
35+
#if PY_VERSION_HEX > 0x030D0000 && PY_VERSION_HEX < 0x030E0000
36+
Py_hash_t _Py_HashBytes(const void *src, Py_ssize_t len);
37+
#endif
38+
3339
#endif

‎capnpy/_hash.pyx‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ Reimplementation of CPython's hashing algorithm for many builtin
33
types. The idea is that if you have C variables, you can compute fast hashes
44
*without* having to allocate real Python object
55
"""
6+
from cpython.version cimport PY_VERSION_HEX
67

78
cdef extern from "Python.h":
89
int PY_MAJOR_VERSION

0 commit comments

Comments
 (0)