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.

Sign up for GitHub

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

Jump to bottom

Jshint error lost if reason detail is not a java.lang.String #26

Open
joelkamentz opened this issue Jun 17, 2013 · 0 comments
Open

Jshint error lost if reason detail is not a java.lang.String #26

joelkamentz opened this issue Jun 17, 2013 · 0 comments

Comments

@joelkamentz
Copy link

Using 0.3.6-SNAPSHOT-deps from ede60c0, I see the following error message when running ant-jshint:

"Problem casting JShint error variable for previous error. See issue (#1) (org.mozilla.javascript.ConsString cannot be cast to java.lang.String)"

https://github.com/philmander/ant-jshint/blob/master/src/main/java/com/philmander/jshint/JsHintRunner.java line 229 is throwing a ClassCastException while trying to get details for a jshint error. In the process of handling that exception, the jshint error information is lost (not written to ant-jshint output file).

While the detail object is not a String, calling toString() on it yields:
"Too many errors. (50% scanned)."
codecop added a commit to codecop/ant-jshint that referenced this issue Jun 27, 2013
@codecop
Fix for Issue philmander#26 - calling toString on a Sting just return…
eeabaef 
…s this, so no

inpact for regular case but fixes the problem when too many errors were
found (because then it seems a constant instead of a String is
returned).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@joelkamentz