Re: [Rpm-maint] [rpm-software-management/rpm] python: Use Py_hash_t instead of long in hdr_hash (PR #2408)
> What tooling/versions is giving this error? I don't doubt that Py_hash_t is > the more proper type here, just that I've never seen the compiler complain > about this. clang16+ -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2408#issuecomment-1458361863 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
[Rpm-maint] [rpm-software-management/rpm] python: Use Py_hash_t instead of long in hdr_hash (PR #2409)
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 */ | ^~~~ You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/2409 -- Commit Summary -- * python: Use Py_hash_t instead of long in hdr_hash -- File Changes -- M python/header-py.c (4) -- Patch Links -- https://github.com/rpm-software-management/rpm/pull/2409.patch https://github.com/rpm-software-management/rpm/pull/2409.diff -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2409 You are receiving this because you are subscribed to this thread. Message ID: rpm-software-management/rpm/pull/2...@github.com ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
[Rpm-maint] [rpm-software-management/rpm] python: Use Py_hash_t instead of long in hdr_hash (PR #2408)
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 */ | ^~~~ You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/2408 -- Commit Summary -- * python: Use Py_hash_t instead of long in hdr_hash -- File Changes -- M python/header-py.c (4) -- Patch Links -- https://github.com/rpm-software-management/rpm/pull/2408.patch https://github.com/rpm-software-management/rpm/pull/2408.diff -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2408 You are receiving this because you are subscribed to this thread. Message ID: rpm-software-management/rpm/pull/2...@github.com ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
