Release v0.9.6

nzakas · nzakas · commit 12397f0b1a7a · 2012-02-10T14:40:47.000-08:00
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,9 @@
+February 10, 2012 - v0.9.6
+
+* Make sure line numbers are correct for vendor-prefix rule (fixes #238) (Nicholas C. Zakas)
+* fix line and column numbers for compatible vendor prefixes rule (fixes #236) (Nicholas C. Zakas)
+* Fixed JavaScript compatibility issues in several places. Everything now works in IE < 9. (Nicholas C. Zakas)
+* Updated parser (fixes #234) (Nicholas C. Zakas)
 
 February 3, 2012 - v0.9.5
 
@@ -239,6 +245,8 @@ June 15, 2011 - v0.1.0
 
 
 
+
+
 
 
 
diff --git a/release/csslint-node.js b/release/csslint-node.js
@@ -21,7 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 */
-/* Build time: 10-February-2012 02:23:58 */
+/* Build time: 10-February-2012 02:34:15 */
 
 /*!
 Parser-Lib
@@ -8597,11 +8597,11 @@ CSSLint.addRule({
                 actual = needsStandard[i].actual;
 
                 if (!properties[needed]){               
-                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", event.line, event.col, rule);
+                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                 } else {
                     //make sure standard property is last
                     if (properties[needed][0].pos < properties[actual][0].pos){
-                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", event.line, event.col, rule);
+                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                     }
                 }
             }
diff --git a/release/csslint-rhino.js b/release/csslint-rhino.js
@@ -21,7 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 */
-/* Build time: 10-February-2012 02:23:58 */
+/* Build time: 10-February-2012 02:34:15 */
 var CSSLint = (function(){
 
 /*!
@@ -8598,11 +8598,11 @@ CSSLint.addRule({
                 actual = needsStandard[i].actual;
 
                 if (!properties[needed]){               
-                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", event.line, event.col, rule);
+                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                 } else {
                     //make sure standard property is last
                     if (properties[needed][0].pos < properties[actual][0].pos){
-                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", event.line, event.col, rule);
+                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                     }
                 }
             }
diff --git a/release/csslint-tests.js b/release/csslint-tests.js
@@ -1734,10 +1734,12 @@ background: -ms-linear-gradient(top, #1e5799 ,#2989d8 ,#207cca ,#7db9e8 );
         name: "Vendor Prefix Errors",
 
         "Using -moz-border-radius without border-radius should result in one warning": function(){
-            var result = CSSLint.verify("h1 { -moz-border-radius: 5px; }", { "vendor-prefix": 1 });
+            var result = CSSLint.verify("h1 {\n    -moz-border-radius: 5px; \n}", { "vendor-prefix": 1 });
             Assert.areEqual(1, result.messages.length);
             Assert.areEqual("warning", result.messages[0].type);
             Assert.areEqual("Missing standard property 'border-radius' to go along with '-moz-border-radius'.", result.messages[0].message);
+            Assert.areEqual(2, result.messages[0].line);
+            Assert.areEqual(5, result.messages[0].col);
         },
 
         "Using -webkit-border-radius without border-radius should result in one warning": function(){
@@ -1755,10 +1757,13 @@ background: -ms-linear-gradient(top, #1e5799 ,#2989d8 ,#207cca ,#7db9e8 );
         },
 
         "Using -moz-border-radius after  border-radius should result in one warning": function(){
-            var result = CSSLint.verify("h1 { border-radius: 5px; -moz-border-radius: 5px; }", { "vendor-prefix": 1 });
+            var result = CSSLint.verify("h1 { \nborder-radius: 5px; \n    -moz-border-radius: 5px; }", { "vendor-prefix": 1 });
             Assert.areEqual(1, result.messages.length);
             Assert.areEqual("warning", result.messages[0].type);
             Assert.areEqual("Standard property 'border-radius' should come after vendor-prefixed property '-moz-border-radius'.", result.messages[0].message);
+            Assert.areEqual(3, result.messages[0].line);
+            Assert.areEqual(5, result.messages[0].col);
+            
         },
 
         "Using -webkit-border-bottom-left-radius with border-bottom-left-radius should not result in a warning.": function(){
diff --git a/release/csslint-worker.js b/release/csslint-worker.js
@@ -21,7 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 */
-/* Build time: 10-February-2012 02:23:58 */
+/* Build time: 10-February-2012 02:34:15 */
 
 /*!
 Parser-Lib
@@ -8597,11 +8597,11 @@ CSSLint.addRule({
                 actual = needsStandard[i].actual;
 
                 if (!properties[needed]){               
-                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", event.line, event.col, rule);
+                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                 } else {
                     //make sure standard property is last
                     if (properties[needed][0].pos < properties[actual][0].pos){
-                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", event.line, event.col, rule);
+                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                     }
                 }
             }
diff --git a/release/csslint-wsh.js b/release/csslint-wsh.js
@@ -21,7 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 */
-/* Build time: 10-February-2012 02:23:58 */
+/* Build time: 10-February-2012 02:34:15 */
 var CSSLint = (function(){
 
 /*!
@@ -8598,11 +8598,11 @@ CSSLint.addRule({
                 actual = needsStandard[i].actual;
 
                 if (!properties[needed]){               
-                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", event.line, event.col, rule);
+                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                 } else {
                     //make sure standard property is last
                     if (properties[needed][0].pos < properties[actual][0].pos){
-                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", event.line, event.col, rule);
+                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                     }
                 }
             }
diff --git a/release/csslint.js b/release/csslint.js
@@ -21,7 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 */
-/* Build time: 10-February-2012 02:23:58 */
+/* Build time: 10-February-2012 02:34:15 */
 var CSSLint = (function(){
 
 /*!
@@ -8598,11 +8598,11 @@ CSSLint.addRule({
                 actual = needsStandard[i].actual;
 
                 if (!properties[needed]){               
-                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", event.line, event.col, rule);
+                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                 } else {
                     //make sure standard property is last
                     if (properties[needed][0].pos < properties[actual][0].pos){
-                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", event.line, event.col, rule);
+                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                     }
                 }
             }
diff --git a/release/npm/cli.js b/release/npm/cli.js
@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-/* Build time: 10-February-2012 02:23:58 */
+/* Build time: 10-February-2012 02:34:15 */
 
 /*
  * Encapsulates all of the CLI functionality. The api argument simply
diff --git a/release/npm/lib/csslint-node.js b/release/npm/lib/csslint-node.js
@@ -21,7 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 */
-/* Build time: 10-February-2012 02:23:58 */
+/* Build time: 10-February-2012 02:34:15 */
 
 /*!
 Parser-Lib
@@ -8597,11 +8597,11 @@ CSSLint.addRule({
                 actual = needsStandard[i].actual;
 
                 if (!properties[needed]){               
-                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", event.line, event.col, rule);
+                    reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                 } else {
                     //make sure standard property is last
                     if (properties[needed][0].pos < properties[actual][0].pos){
-                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", event.line, event.col, rule);
+                        reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);
                     }
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN`
`21`	`21`	`THE SOFTWARE.`
`22`	`22`
`23`	`23`	`*/`
`24`		`-/* Build time: 10-February-2012 02:23:58 */`
	`24`	`+/* Build time: 10-February-2012 02:34:15 */`
`25`	`25`
`26`	`26`	`/*!`
`27`	`27`	`Parser-Lib`
`@@ -8597,11 +8597,11 @@ CSSLint.addRule({`
`8597`	`8597`	`actual = needsStandard[i].actual;`
`8598`	`8598`
`8599`	`8599`	`if (!properties[needed]){`
`8600`		`- reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", event.line, event.col, rule);`
	`8600`	`+ reporter.report("Missing standard property '" + needed + "' to go along with '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);`
`8601`	`8601`	`} else {`
`8602`	`8602`	`//make sure standard property is last`
`8603`	`8603`	`if (properties[needed][0].pos < properties[actual][0].pos){`
`8604`		`- reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", event.line, event.col, rule);`
	`8604`	`+ reporter.report("Standard property '" + needed + "' should come after vendor-prefixed property '" + actual + "'.", properties[actual][0].name.line, properties[actual][0].name.col, rule);`
`8605`	`8605`	`}`
`8606`	`8606`	`}`
`8607`	`8607`	`}`