andrew / node-xbox-controller

:video_game: [UNMAINTAINED] Interface for Xbox 360 game controller into Node.js
http://libraries.io/npm/xbox-controller
MIT License
270 stars 49 forks source link

Updated node-hid to 0.4.0 #34

Closed andrew closed 9 years ago

gordonturner commented 9 years ago

@andrew Looks good here (OSX 10.10.3, node v0.12.0).

No errors now, only 2 warnings:

child_process: customFds option is deprecated, use stdio instead.
  CC(target) Release/obj.target/hidapi/hidapi/mac/hid.o
../hidapi/mac/hid.c:255:20: warning: comparison of integers of different signs: 'CFIndex' (aka 'long') and 'size_t' (aka 'unsigned long') [-Wsign-compare]
                if (chars_copied == len)
                    ~~~~~~~~~~~~ ^  ~~~
../hidapi/mac/hid.c:295:20: warning: comparison of integers of different signs: 'CFIndex' (aka 'long') and 'size_t' (aka 'unsigned long') [-Wsign-compare]
                if (used_buf_len == len)
                    ~~~~~~~~~~~~ ^  ~~~
2 warnings generated.

These warnings were present before 0.4.0 though.

andrew commented 9 years ago

:ship: https://github.com/andrew/node-xbox-controller/releases/tag/v0.6.0

gordonturner commented 9 years ago

Thanks @andrew!