mscdex / node-ncurses

An ncurses binding for node.js
MIT License
386 stars 25 forks source link

Evaluating value of hasMouse causes node to die #9

Closed holizz closed 12 years ago

holizz commented 13 years ago
> require('ncurses').hasMouse
node: /usr/include/node/node_object_wrap.h:51: static T* node::ObjectWrap::Unwrap(v8::Handle<v8::Object>) [with T = Window]: Assertion `handle->InternalFieldCount() > 0' failed.
zsh: abort      node

node 0.4.11-1 (packaged by Arch)

node-ncurses 0.2.0 (from npm)

mscdex commented 13 years ago

Try the master branch of node-ncurses and see if that works for you

ghost commented 12 years ago

built from master (last commit 20fca97d9bd04b98fa060a88b585711c8888d9c5) using: node-waf configure build

=>node

require('./ncurses').hasMouse node: /usr/local/include/node/node_object_wrap.h:51: static T* node::ObjectWrap::Unwrap(v8::Handlev8::Object) [with T = Window]: Assertion `handle->InternalFieldCount() > 0' failed. Aborted

=>node

require('./ncurses') node: /usr/local/include/node/node_object_wrap.h:51: static T* node::ObjectWrap::Unwrap(v8::Handlev8::Object) [with T = Window]: Assertion `handle->InternalFieldCount() > 0' failed. Aborted

=>node -v v0.4.3

=>uname -a Linux ydmm-ubuntu 3.0.0-13-generic-pae #22-Ubuntu SMP Wed Nov 2 15:17:35 UTC 2011 i686 i686 i386 GNU/Linux

and this is still an issue, can we git this re-opened?

really want to try this bad boy out, thanks for your help!

ghost commented 12 years ago

Can confirm this is an issue on my mac machine as well as a random IRCer can replicate after building:

=>node

require('./ncurses'); Assertion failed: (handle->InternalFieldCount() > 0), function Unwrap, file /usr/local/include/node/node_object_wrap.h, line 51. Abort trap

Darwin deshawnmbp.local 10.8.0 Darwin Kernel Version 10.8.0: Tue Jun 7 16:33:36 PDT 2011; root:xnu-1504.15.3~1/RELEASE_I386 i386

So this issue seems to be non-mouse related but simply requiring ncurses causes the error.

Maybe even rename this issue?: Assertion handle->InternalFieldCount() failure

ghost commented 12 years ago

Let me know if i shouldn't create a new issue and title it correctly in order to get this issue in open status.

mscdex commented 12 years ago

Don't worry about it, I think I know what the problem is but I won't be in a position to push a tested fix until next week.

mscdex commented 12 years ago

Please re-test with the latest version in npm.