search

fivdi / epoll

A low-level Node.js binding for the Linux epoll API
MIT License
84 stars 13 forks source link

Epoll.EPOLLET.toString(16) returns -80000000 #2

Closed fivdi closed 10 years ago

fivdi commented 10 years ago

Currently Epoll.EPOLLET is -0x80000000 but it should be 0x80000000. This results in the calls to args[1]->IsUint32() in the Add and Modify methods rejecting Epoll.EPOLLET as an invalid argument.

fivdi commented 10 years ago

This was an issue on ARM, works as expected on Intel. Fixed with 76627737d34e18719b5e1b264cf5ebcf00280f22