-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ReferenceError: convertedFunction is not defined #10
Comments
Hi Jason, Does it work in the browser version? Thanks Best Regards! Nishant Das Patnaik On Fri, Jan 31, 2014 at 3:39 AM, Jason notifications@github.com wrote:
|
Hey, No, neither seem to work unfortunately. The browser version hangs and crashes the tab / window. Really cool effort for this project. |
The array just isn't defined anywhere in the file. Not sure what that line is supposed to say... |
It is defined on analyzer.js file and that was included in HTML. On Sat, Mar 29, 2014 at 2:41 AM, Ben Toews notifications@github.com wrote:
|
Yeah, the non-node version seems to work better. |
No further development happened from the core team.....That is the reason, On Sat, Mar 29, 2014 at 4:08 PM, Ben Toews notifications@github.com wrote:
|
Add ** exports.convertedFunction= convertedFunction;** in analyzer.js |
I just tried out the node server.js with some code (about 2k loc):
ReferenceError: convertedFunction is not defined
/home/jason/playground/jsprime/jsprime-node/engine.js:149
convertedFunction.push(newFunction2);
^
ReferenceError: convertedFunction is not defined
at Object.asignFunctionReturnValue (/home/jason/playground/jsprime/jsprime-node/engine.js:149:13)
at IncomingMessage. (/home/jason/playground/jsprime/jsprime-node/server.js:71:18)
at IncomingMessage.EventEmitter.emit (events.js:92:17)
at _stream_readable.js:920:16
at process._tickCallback (node.js:415:13)
Any ideas?
Thanks
The text was updated successfully, but these errors were encountered: