Skip to content

Commit 0b9fc94

Browse files
XhmikosRXhmikosR
committed
Remove unused variables.
1 parent 8b52eeb commit 0b9fc94

23 files changed

+45
-45
lines changed

‎.jshintrc‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
"noempty": true,
1111
"nonbsp": true,
1212
"quotmark": "double",
13+
"unused": true,
1314
"undef": true,
1415
"globals": {
1516
"CSSLint": true,

‎Gruntfile.js‎

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
/*global module:false*/
1+
/* jshint evil:true, node:true */
2+
23
module.exports = function(grunt) {
34

45
// Project configuration.
@@ -221,11 +222,9 @@ module.exports = function(grunt) {
221222

222223
//Run the YUITest suite
223224
grunt.registerMultiTask("yuitest", "Run the YUITests for the project", function() {
224-
/*jshint evil:true, node:true*/
225225

226-
var start = Date.now();
227226
var YUITest = require("yuitest");
228-
var CSSLint = require("./build/csslint-node").CSSLint;
227+
var CSSLint = require("./build/csslint-node").CSSLint; // jshint ignore:line
229228
var files = this.filesSrc;
230229
var TestRunner = YUITest.TestRunner;
231230
var done = this.async();
@@ -375,7 +374,7 @@ module.exports = function(grunt) {
375374
grunt.util.spawn({
376375
cmd: "git",
377376
args: ["tag"]
378-
}, function(error, result, code) {
377+
}, function(error, result) {
379378
//Find the latest git tag
380379
var tags = result.stdout.split("\n"),
381380
semver = tags[0].replace("v","").split("."),
@@ -386,7 +385,7 @@ module.exports = function(grunt) {
386385
//A simple array sort can't be used because of the comparison of
387386
//the strings "0.9.9" > "0.9.10"
388387
for (var i = 1, len = tags.length; i < len; i++) {
389-
semver = tags[i].replace("v","").split(".");
388+
semver = tags[i].replace("v", "").split(".");
390389

391390
var currentMajor = parseInt(semver[0], 10);
392391
if (currentMajor < major) {
@@ -418,7 +417,7 @@ module.exports = function(grunt) {
418417
grunt.util.spawn({
419418
cmd: "git",
420419
args: ["log", "--pretty=format:'* %s (%an)'", lastTag + "..HEAD"]
421-
}, function(error, result, code) {
420+
}, function(error, result) {
422421
var prettyPrint = result.stdout.split("'\n'").join("\n").replace(/\"$/, "").replace(/^\"/, "");
423422

424423
grunt.verbose.writeln().write(prettyPrint).writeln();
@@ -446,7 +445,7 @@ module.exports = function(grunt) {
446445
cmd: "java",
447446
args: ["-jar", "lib/js.jar", "lib/yuitest-rhino-cli.js", "build/csslint.js", filepath],
448447
opts: {stdio: "inherit"}
449-
}, function(error, result, code) {
448+
}, function() {
450449
progress--;
451450
if (progress === 0) {
452451
done();

‎src/cli/common.js‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
* provides environment-specific functionality.
44
*/
55

6+
/* exported cli */
7+
68
function cli(api){
79

810
var globalOptions = {

‎src/cli/node.js‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,9 @@
22
* CSSLint Node.js Command Line Interface
33
*/
44

5-
/*jshint node:true*/
6-
/*global cli*/
5+
/* jshint node:true */
6+
/* global cli */
7+
/* exported CSSLint */
78

89
var fs = require("fs"),
910
path = require("path"),

‎src/core/CSSLint.js‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,10 @@
44
* @static
55
* @extends parserlib.util.EventTarget
66
*/
7-
/*global parserlib, Reporter*/
7+
8+
/* global parserlib, Reporter */
9+
/* exported CSSLint */
10+
811
var CSSLint = (function(){
912

1013
var rules = [],
@@ -170,8 +173,7 @@ var CSSLint = (function(){
170173
*/
171174
api.verify = function(text, ruleset){
172175

173-
var i = 0,
174-
len = rules.length,
176+
var i = 0,
175177
reporter,
176178
lines,
177179
report,

‎src/formatters/checkstyle-xml.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@
6969
* @param options {Object} (UNUSED for now) specifies special handling of output
7070
* @return {String} output for results
7171
*/
72-
formatResults: function(results, filename, options) {
72+
formatResults: function(results, filename/*, options*/) {
7373
var messages = results.messages,
7474
output = [];
7575

@@ -91,7 +91,7 @@
9191

9292
if (messages.length > 0) {
9393
output.push("<file name=\""+filename+"\">");
94-
CSSLint.Util.forEach(messages, function (message, i) {
94+
CSSLint.Util.forEach(messages, function (message) {
9595
//ignore rollups for now
9696
if (!message.rollup) {
9797
output.push("<error line=\"" + message.line + "\" column=\"" + message.col + "\" severity=\"" + message.type + "\"" +

‎src/formatters/compact.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ CSSLint.addFormatter({
4444
return options.quiet ? "" : filename + ": Lint Free!";
4545
}
4646

47-
CSSLint.Util.forEach(messages, function(message, i) {
47+
CSSLint.Util.forEach(messages, function(message) {
4848
if (message.rollup) {
4949
output += filename + ": " + capitalize(message.type) + " - " + message.message + "\n";
5050
} else {

‎src/formatters/csslint-xml.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ CSSLint.addFormatter({
2626
* @param options {Object} (UNUSED for now) specifies special handling of output
2727
* @return {String} output for results
2828
*/
29-
formatResults: function(results, filename, options) {
29+
formatResults: function(results, filename/*, options*/) {
3030
var messages = results.messages,
3131
output = [];
3232

@@ -51,7 +51,7 @@ CSSLint.addFormatter({
5151

5252
if (messages.length > 0) {
5353
output.push("<file name=\""+filename+"\">");
54-
CSSLint.Util.forEach(messages, function (message, i) {
54+
CSSLint.Util.forEach(messages, function (message) {
5555
if (message.rollup) {
5656
output.push("<issue severity=\"" + message.type + "\" reason=\"" + escapeSpecialCharacters(message.message) + "\" evidence=\"" + escapeSpecialCharacters(message.evidence) + "\"/>");
5757
} else {

‎src/formatters/junit-xml.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ CSSLint.addFormatter({
2626
* @param options {Object} (UNUSED for now) specifies special handling of output
2727
* @return {String} output for results
2828
*/
29-
formatResults: function(results, filename, options) {
29+
formatResults: function(results, filename/*, options*/) {
3030

3131
var messages = results.messages,
3232
output = [],
@@ -72,7 +72,7 @@ CSSLint.addFormatter({
7272

7373
if (messages.length > 0) {
7474

75-
messages.forEach(function (message, i) {
75+
messages.forEach(function (message) {
7676

7777
// since junit has no warning class
7878
// all issues as errors

‎src/formatters/lint-xml.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ CSSLint.addFormatter({
2626
* @param options {Object} (UNUSED for now) specifies special handling of output
2727
* @return {String} output for results
2828
*/
29-
formatResults: function(results, filename, options) {
29+
formatResults: function(results, filename/*, options*/) {
3030
var messages = results.messages,
3131
output = [];
3232

@@ -52,7 +52,7 @@ CSSLint.addFormatter({
5252
if (messages.length > 0) {
5353

5454
output.push("<file name=\""+filename+"\">");
55-
CSSLint.Util.forEach(messages, function (message, i) {
55+
CSSLint.Util.forEach(messages, function (message) {
5656
if (message.rollup) {
5757
output.push("<issue severity=\"" + message.type + "\" reason=\"" + escapeSpecialCharacters(message.message) + "\" evidence=\"" + escapeSpecialCharacters(message.evidence) + "\"/>");
5858
} else {

0 commit comments

Comments
 (0)