Fix and fixture for issue #16

albertosantini / node-rio

Integration with Rserve, a TCP/IP server for R framework

https://github.com/albertosantini/node-conpa

MIT License

176 stars 35 forks source link

Fix and fixture for issue #16 #17

Closed manuelsantillan closed 10 years ago

albertosantini commented 10 years ago

Sigh. :(

$ node rio_bug.js
err is Eval failed with error code 127
Res ?? undefined

(on windows box without your patch)

manuelsantillan commented 10 years ago

For some reason the bug does not exist on a windows box, possibly because Rserve can cope with 2-byte padding on Windows (the connection is closed at the other end).

However on Mac OS X it is a bug, and this fixes it (we've tested on macbook air's and macbook pros). Don't know about Linux, though.