Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

python: Use Py_hash_t instead of long in hdr_hash #2409

Closed
wants to merge 1 commit into from

Conversation

kraj
Copy link

@kraj kraj commented Mar 3, 2023

Fixes
python/header-py.c:744:2: error: incompatible function pointer types initializing 'hashfunc' (aka 'int (*)(struct _object *)') with an expression of type 'long (PyObject *)' (aka 'long (struct _object )') [-Wincompatible-function-pointer-types]
| hdr_hash, /
tp_hash */
| ^~~~~~~~

Fixes
python/header-py.c:744:2: error: incompatible function pointer types initializing 'hashfunc' (aka 'int (*)(struct _object *)') with an expression of type 'long (PyObject *)' (aka 'long (struct _object *)') [-Wincompatible-function-pointer-types]
|         hdr_hash,                       /* tp_hash */
|         ^~~~~~~~

Signed-off-by: Khem Raj <raj.khem@gmail.com>
@pmatilai
Copy link
Member

pmatilai commented Mar 7, 2023

Thanks for the backport, but we generally only accept PR's on master branch, backports are cherry-picked at the time of release-creation.

@pmatilai pmatilai closed this Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants