search

natevw / pi-spi

Simple RasPi SPI library (node.js)
93 stars 14 forks source link

Node 0.12.x compatibility via NAN #15

Closed natevw closed 9 years ago

natevw commented 9 years ago

I've rewritten the native bindings to use https://github.com/rvagg/nan which now lets it compile on both 0.10 (and perhaps earlier) and 0.12 (and perhaps forks…).

This also adds a bit of polish to arguments handling, with JS-level type checking and a default value for readcount on spi.transfer.

This should fix https://github.com/natevw/pi-spi/issues/14 and https://github.com/natevw/pi-spi/issues/8

natevw commented 9 years ago

Confirmed that (with some various fixes) the stub version now builds on Win32. Mostly just need to double-check that the echo test works and ship it!

natevw commented 9 years ago

this was merged (via rebase…)