Issue 34146: PyCData_Type.tp_hash doesn't use PyObject_HashNotImplemented

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/78327

classification

Title:	PyCData_Type.tp_hash doesn't use PyObject_HashNotImplemented
Type:		Stage:
Components:	ctypes	Versions:	Python 3.8

process

Status:	open	Resolution:
Dependencies:		Superseder:
Assigned To:		Nosy List:	bup
Priority:	normal	Keywords:

Created on 2018-07-18 10:41 by bup, last changed 2022-04-11 14:59 by admin.

Messages (1)
msg321878 - (view)	Author: Dan Snider (bup) *	Date: 2018-07-18 10:41
So not only does the function it does use (`PyCData_nohash`) prevent PyType_Ready from setting _ctypes._SimpleCData.__dict__["__hash__"] to `None`, (contrary to to every other unhashable type in the standard library) but the error message it returns is less detailed. It seems like a big deal if this were to be changed as it could break backwards compatibility so I won't open PR myself as I'm still not that familiar with the contribution process.

msg321878 - (view)

Author: Dan Snider (bup) *

Date: 2018-07-18 10:41

So not only does the function it does use (`PyCData_nohash`) prevent PyType_Ready from setting _ctypes._SimpleCData.__dict__["__hash__"] to `None`, (contrary to to every other unhashable type in the standard library) but the error message it returns is less detailed.

It seems like a big deal if this were to be changed as it could break backwards compatibility so I won't open PR myself as I'm still not that familiar with the contribution process.

History
Date	User	Action	Args
2022-04-11 14:59:03	admin	set	github: 78327
2018-07-18 10:41:57	bup	create