Improves handling of EOL whitespace error reporting

rwaldron · dmethvin · commit b6a2dfbeb9ff · 2012-01-11T21:49:55.000-05:00
diff --git a/build/jshint-check.js b/build/jshint-check.js
@@ -25,12 +25,12 @@ if ( jshint( src, config ) ) {
 	jshint.errors.forEach(function( e ) {
 		if ( !e ) { return; }
 
-		var str = e.evidence ? e.evidence : "";
+		var str = e.evidence ? e.evidence : "",
+		character = e.character === true ? "EOL" : "C" + e.character;
 
 		if ( str ) {
 			str = str.replace( /\t/g, " " ).trim();
-
-			console.log( " [L" + e.line + ":C" + e.character + "] " + e.reason + "\n  " + str + "\n");
+			console.log( " [L" + e.line + ":" + character + "] " + e.reason + "\n  " + str + "\n");
 		}
 	});
 }

Original file line number	Diff line number	Diff line change
`@@ -25,12 +25,12 @@ if ( jshint( src, config ) ) {`
`25`	`25`	`jshint.errors.forEach(function( e ) {`
`26`	`26`	`if ( !e ) { return; }`
`27`	`27`
`28`		`- var str = e.evidence ? e.evidence : "";`
	`28`	`+ var str = e.evidence ? e.evidence : "",`
	`29`	`+ character = e.character === true ? "EOL" : "C" + e.character;`
`29`	`30`
`30`	`31`	`if ( str ) {`
`31`	`32`	`str = str.replace( /\t/g, " " ).trim();`
`32`		`-`
`33`		`- console.log( " [L" + e.line + ":C" + e.character + "] " + e.reason + "\n " + str + "\n");`
	`33`	`+ console.log( " [L" + e.line + ":" + character + "] " + e.reason + "\n " + str + "\n");`
`34`	`34`	`}`
`35`	`35`	`});`
`36`	`36`	`}`