search

pearu / pylibnidaqmx

a Python wrapper to libnidaqmx library
Other
10 stars 9 forks source link

TaskHandle is a void* and should be used instead of uInt32 #52

Closed hvraven closed 8 years ago

hvraven commented 8 years ago

at least in NIDAQmx 15.1 TaskHandle is defined as a void* and not as a uInt32. In practice this makes no difference on 32 bit system (or 64bit systems running 32bit Python), however with 64bit bit it does.

This change additonally let's Task inherit from TaskHandle instead of uInt32 (which are the same for 32 bit systems)

This is tested on a Windows 7 64-bit using 64bit WinPythin. The system is running a set of 6353 cards currently using CounterOutputs and AnalogOutputs. This is not tested on 32 bit python, but it shouldn't have any changes on these systems.

hoechenberger commented 8 years ago

Great! I think we should also test this on 32-bit Python before merging, though.