jslintrb 0.2.0 → 0.3.0

data/README.rdoc

@@ -17,10 +17,14 @@ top-level project directory that sets options for my project, then one
 in my spec directory (like a test directory) which adds the globals
 that exist for the tests but not for the core application code.
 
- The file is JSON-formatted. See the
+The file is JSON-formatted. See the
 jslint documentation (http://www.jslint.com/lint.html) for allowed
 options.
 
+jslintrb adds support for JavaScript with statements. They are parsed
+but produce a warning. The "withstmt" option will suppress that
+warning. Note that this does not endorse the use of with statements.
+
 == Copyright
 
 jslintrb copyright (c) 2010 Steven Parkes. See LICENSE for details.

data/bin/jslintrb

@@ -10,7 +10,8 @@ ENV["JSLINTRB_JS_PATH"] =
 
 args = [ "env",
          "JSLINT_FILES=#{ARGV.join(':')}",
-        "johnson",
+         "JOHNSON_RUNTIME=tracemonkey",
+         "johnson",
          File.join(JSLINT_DIR,"fulljslint.js"),
          File.join(JSLINT_DIR,"json2.js"),
          File.join(JSLINT_DIR,"jslintrb.js") ]

data/lib/jslintrb/fulljslint.js

@@ -330,7 +330,8 @@ var JSLINT = (function () {
             strict     : true, // require the "use strict"; pragma
             sub        : true, // if all forms of subscript notation are tolerated
             white      : true, // if strict whitespace rules apply
-            widget     : true  // if the Yahoo Widgets globals should be predefined
+            widget     : true, // if the Yahoo Widgets globals should be predefined
+            withstmt   : true  // if with statements should be allowd
         },
 
 // browser contains a set of global names which are commonly provided by a
@@ -4583,6 +4584,28 @@ loop:   for (;;) {
                 return p;
             }
         }
+    };
+
+    function withparams() {
+        var i, t = nexttoken, p = [];
+        advance('(');
+        nospace();
+        if (nexttoken.id === ')') {
+            advance(')');
+            nospace(prevtoken, token);
+            return;
+        }
+        for (;;) {
+            i = parse(20);
+            p.push(i);
+            if (nexttoken.id === ',') {
+                comma();
+            } else {
+                advance(')', t);
+                nospace(prevtoken, token);
+                return p;
+            }
+        }
     }
 
     function doFunction(i) {
@@ -4667,6 +4690,15 @@ loop:   for (;;) {
         return this;
     });
 
+    blockstmt('with', function () {
+        if (!option.withstmt) {
+          warning("Don't use with statements");
+        }
+        withparams();
+        block(true);
+        return this;
+    });
+
     blockstmt('try', function () {
         var b, e, s;
         if (option.adsafe) {
@@ -5149,6 +5181,7 @@ loop:   for (;;) {
                 o.sub     = false;
                 o.undef   = true;
                 o.widget  = false;
+                o.withstmt= false;
                 predefined.Date = null;
                 predefined['eval'] = null;
                 predefined.Function = null;

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: jslintrb
 version: !ruby/object:Gem::Version 
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors: 
 - Steven Parkes
@@ -20,7 +20,7 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: "0"
+        version: 1.1.2.6
     version: 
 description: jslintrb is a packaged version of Douglas Crockford's JSLint JavaScript code checker. jslintrb uses SpiderMonkey via the Johnson Ruby gem to interpret the JSLint javascript.
 email: smparkes@smparkes.net