Closed astewart-xmos closed 6 years ago
Thank you for the submission.
I don't remember the details, but c8c6541 suggests that a Windows build failed trying to include stdbool.h. Could there be differences between Visual Studio versions? I would have most likely used 2015 Community.
payload_len_exceeds_control_packet_size
uses bool and I expect this on Windows comes windows.h.
I am happy to merge this as VocalFusion uses a tagged version. If we find this breaking Windows builds we can revisit the change on our side.
The find_xmos_device
error handling is welcome. Using exit was lazy of me.
There are two changes in this PR:
#include <stdbool.h>
indevice_access_usb.c
find_xmos_device()
indevice_access_usb.c
to return an error, instead of exiting the processcontrol_init_usb()
for Windows more similar to that of Mac or Linux, which return an error when the device is not found