[PROTON-1868] [Python] pn_handle_t binding does not reliably work with Win64

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: proton-c-0.24.0
• Component/s: python-binding, ruby-binding
• Assignee: Andrew Stitcher
• Reporter: Andrew Stitcher
• Created: 26/Jun/18 05:17
• Updated: 04/Jul/18 15:57
• Resolved: 26/Jun/18 14:19

Description

pn_handle_t is treated by swig as identical to uintptr_t which is defined as unsigned long. However under Win64 this is a 32 bit int which is not long enough to fit all 64 bit addresses.

The swig code should be changed to either define uintptr_t as unsigned long long for 64 bit builds which is always 64 bits (on all platforms).

Or we should use a custom typemap for pn_handle_t which uses PyLong_AsVoidPtr & PyLong_FromVoidPtr.