Skip to content
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

crash caused by commit 6e940d27 #697

Closed
mlippert opened this issue Jul 2, 2014 · 2 comments
Closed

crash caused by commit 6e940d27 #697

mlippert opened this issue Jul 2, 2014 · 2 comments

Comments

@mlippert
Copy link

mlippert commented Jul 2, 2014

Jeff, I just saw that you had closed #379, so I went to test it out.

(And since there isn't much opportunity to say this elsewhere, I just wanted you to know how much I (and I'm sure many others) appreciate the time and effort you put into this tool!)

The HEAD of master (19867a0) crashed, so I rolled back a few commits and then walked forward until it crashed again. This was in the commit just before the HEAD (6e940d2) do not show the default values for enums (#689)
Here's the top of the traceback (I don't think the rest is useful, but I can get it if you need it)

Generating output files...org.mozilla.javascript.EcmaError: TypeError: Cannot call method "forEach" of undefined (<eval'ed string>#1(Function)#66(Function)#20(Function)#10)
        at org.mozilla.javascript.ScriptRuntime.constructError(ScriptRuntime.java:3689)
        at org.mozilla.javascript.ScriptRuntime.constructError(ScriptRuntime.java:3667)
        at org.mozilla.javascript.ScriptRuntime.typeError(ScriptRuntime.java:3695)
        at org.mozilla.javascript.ScriptRuntime.typeError2(ScriptRuntime.java:3714)
        at org.mozilla.javascript.ScriptRuntime.undefCallError(ScriptRuntime.java:3731)
        at org.mozilla.javascript.ScriptRuntime.getPropFunctionAndThisHelper(ScriptRuntime.java:2257)
        at org.mozilla.javascript.ScriptRuntime.getPropFunctionAndThis(ScriptRuntime.java:2250)
        at org.mozilla.javascript.Interpreter.interpretLoop(Interpreter.java:1317)
        at script.anonymous(<eval'ed string>#1(Function)#66(Function)#20(Function):10)
        at org.mozilla.javascript.Interpreter.interpret(Interpreter.java:815)
        at org.mozilla.javascript.InterpretedFunction.call(InterpretedFunction.java:109)
        at org.mozilla.javascript.ScriptRuntime.applyOrCall(ScriptRuntime.java:2429)
        at org.mozilla.javascript.BaseFunction.execIdCall(BaseFunction.java:269)
        at org.mozilla.javascript.IdFunctionObject.call(IdFunctionObject.java:97)
        at org.mozilla.javascript.optimizer.OptRuntime.callN(OptRuntime.java:52)
        at org.mozilla.javascript.gen.file__localpath_jsdoc_node_modules_underscore_underscore_js_6._c_anonymous_148(Unknown Source)
        at org.mozilla.javascript.gen.file__localpath_jsdoc_node_modules_underscore_underscore_js_6.call(Unknown Source)

If you want me to look through the code I'm creating the doc for to find specifically how I'm using some annotation, let me know (I'm guessing it will have something to do w/ @enum 😄 .

@hegemonic
Copy link
Contributor

Thanks for the kind words, @mlippert!

I found a way to reproduce this issue, and it should now be fixed on master. If you still see this problem after updating, please let me know, and I'll take a look.

@mlippert
Copy link
Author

mlippert commented Jul 2, 2014

Thanks @hegemonic, super quick response. Tested the fix on my code base and the problem is fixed.
(and the mixes in annotations show up in the doc as well).

Mike

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants