Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 0.3, 0.4
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Plugin should be updated to use latest version of http://code.google.com/p/jslint4java/. Major changes in jslint4java:

      Version 1.4.7

      • changes in options: adds the continue option, whilst removing eqeqeq, immed and laxbreak options
      • warning messages are changed and should be updated

      Version 2.0.0

      • The nature of the boolean options has been inverted--beware!
        • bitwise now enables the use of bitwise operators.
        • newcap now means "ignore capitalisation of constructors"
        • plusplus now means "incr" and "decr" operators are tolerated.
        • regexp now means allow the use of "." in regexes.
        • undef now means you can declare variables out of order.
        • white now means acceptance of non-compliant whitespace.
      • New options:
        • sloppy, which is the inverse of strict.
        • confusion, allows inconsistent types.
        • vars, to tolerate multiple var declarations per function (replaces onevar).
        • eqeq, to tolerate "==" and "!=".
        • node, to predefine node.js globals.
        • properties, to require declaration of all properties.
        • unparam, to allow unused parameters.
      • Removed options:
        • onevar
        • strict

      Version 2.0.2

      • Removed option:
        • adsafe
        • confusion
        • safe
      • New Options
        • anon

        Issue Links

          Activity

          Hide
          Jakob Kruse added a comment -

          This is really, really important! Current JSLint is from 2010, and uses a completely different in-file options set than current versions.

          Show
          Jakob Kruse added a comment - This is really, really important! Current JSLint is from 2010, and uses a completely different in-file options set than current versions.
          Hide
          Fabio Gavilondo added a comment -

          Another vote. Really, really important!

          It's not just about the new options. jslint4java 1.4.6 (and 1.4.7) fail to parse JavaScript files encoded in UTF-8Y (aka UTF-8 BOM) as these versions don't recognize the byte order mark at the beginning of the file and give up, with the rather cryptic message:

          Unexpected ".

          I'm not sure if this is a problem with the old version of Rhino, or JSLint itself, but jslint4java 2.0.2 does not have that problem.

          Given that nearly all Windows tools (Visual Studio, Notepad...) use UTF-8Y as their default encoding, and a lot of people do JS development on Windows it would be really important to upgrade the plugin to a JSLint version that can deal with this encoding.

          Show
          Fabio Gavilondo added a comment - Another vote. Really, really important! It's not just about the new options. jslint4java 1.4.6 (and 1.4.7) fail to parse JavaScript files encoded in UTF-8Y (aka UTF-8 BOM) as these versions don't recognize the byte order mark at the beginning of the file and give up, with the rather cryptic message: Unexpected ". I'm not sure if this is a problem with the old version of Rhino, or JSLint itself, but jslint4java 2.0.2 does not have that problem. Given that nearly all Windows tools (Visual Studio, Notepad...) use UTF-8Y as their default encoding, and a lot of people do JS development on Windows it would be really important to upgrade the plugin to a JSLint version that can deal with this encoding.
          Hide
          Eddie Webb added a comment -

          Anyone using jslint in a Java project? I can't understand how to allow both static analysis to occur on same project - SONARPLUGINS-1802

          Show
          Eddie Webb added a comment - Anyone using jslint in a Java project? I can't understand how to allow both static analysis to occur on same project - SONARPLUGINS-1802

            People

            • Assignee:
              Unassigned
              Reporter:
              Anonymous
            • Votes:
              4 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: