rsingla / jslint4java

Automatically exported from code.google.com/p/jslint4java
Other
0 stars 0 forks source link

2.0.0: jslint4java reports whitespace errors which jslint website does not #67

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run the attached jslintTest.js file through jslint4Java ant task with the 
browser and confusion options.
2. Copy the contents of the file into jslint.com.

What is the expected output? 
No errors - which is what jslint.com reports..

What do you see instead?
   [jslint] C:\Users\Edward\workspaces\TrailHunger\ContentDeliveryNetwork\WebContent\r25.9\js\page\jslintTest.js:8:5: Expected '}' at column 1, not column 5.
   [jslint]         } else {
   [jslint]     ^
   [jslint] C:\Users\Edward\workspaces\TrailHunger\ContentDeliveryNetwork\WebContent\r25.9\js\page\jslintTest.js:9:13: Expected 'this' at column 1, not column 13.
   [jslint]             this.garminUploadPresenter.displayNoPlugin();
   [jslint]             ^
   [jslint] C:\Users\Edward\workspaces\TrailHunger\ContentDeliveryNetwork\WebContent\r25.9\js\page\jslintTest.js:10:5: Expected '}' at column 1, not column 5.
   [jslint]         }
   [jslint]     ^

What version of the product are you using? On what operating system?
2.0.0 Windows XP

Original issue reported on code.google.com by ejsa...@gmail.com on 4 Sep 2011 at 1:34

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks.  I've verified this locally.

% java -jar ~/jslint4java-2.0.0.jar --browser --confusion jslintTest.js
jslint:jslintTest.js:8:5:Expected '}' at column 1, not column 5.
jslint:jslintTest.js:9:13:Expected 'this' at column 1, not column 13.
jslint:jslintTest.js:10:5:Expected '}' at column 1, not column 5.

I think that this has actually been fixed already as issue 62, and 
independently by Doug Crockford 
(https://github.com/douglascrockford/JSLint/commit/4310f67e).  Suffice to say 
that this will be fixed in the next release.

There are two workarounds until then:

- Specify --white.
- Download the latest jslint.js and use the "jslint" attribute to point at it.

Original comment by d...@happygiraffe.net on 8 Sep 2011 at 7:24