[issue7211] select module - kevent ident field 64 bit issue

Michael Broghton report at bugs.python.org
Mon Oct 26 23:34:34 CET 2009


Michael Broghton <mbroughton at advanis.ca> added the comment:

I'm not sure how to patch this so that it will work on both 32 and 64
bit systems. Issues:

1. What would be an appropriate member type for ident in
kqueue_event_members? It seems like T_PYSSIZET might work. Otherwise, I
am guessing that this will involve some #if's.

2. I think the format spec in kqueue_event_repr needs to change. It
seems like this will also require some #if's.

3. kqueue_event_init uses PyObject_AsFileDescriptor to set the ident
field. This should be doing a PyLong_Check first to see if
PyLong_AsSomething would be more appropriate.

4. I think the type of the result variable in kqueue_event_richcompare
needs to be changed to long long int.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue7211>
_______________________________________


More information about the Python-bugs-list mailing list