Skip to content

Commit

Permalink
Fixed #1305: Removed unused code from checkstyle.js
Browse files Browse the repository at this point in the history
Signed-off-by: Anton Kovalyov <anton@kovalyov.net>
  • Loading branch information
andersjanmyr authored and valueof committed Oct 20, 2013
1 parent 6038e4e commit c7de483
Showing 1 changed file with 1 addition and 35 deletions.
36 changes: 1 addition & 35 deletions src/reporters/checkstyle.js
Expand Up @@ -15,7 +15,7 @@ module.exports =
"<": "&lt;",
">": "&gt;"
},
file, fileName, i, issue, globals, unuseds, errorMessage;
fileName, i, issue, errorMessage;

opts = opts || {};

Expand Down Expand Up @@ -51,40 +51,6 @@ module.exports =
});
});

data.forEach(function (result) {
file = data.file;
globals = result.implieds;
unuseds = result.unused;

// Register the file
result.file = result.file.replace(/^\.\//, '');
if (!files[result.file]) {
files[result.file] = [];
}

if (globals) {
globals.forEach(function (global) {
files[result.file].push({
severity: 'warning',
line: global.line,
column: 0,
message: "Implied global '" + global.name + "'",
source: 'jshint.implied-globals'
});
});
}
if (unuseds) {
unuseds.forEach(function (unused) {
files[result.file].push({
severity: 'warning',
line: unused.line,
column: 0,
message: "Unused variable: '" + unused.name + "'",
source: 'jshint.implied-unuseds'
});
});
}
});

out.push("<?xml version=\"1.0\" encoding=\"utf-8\"?>");
out.push("<checkstyle version=\"4.3\">");
Expand Down

0 comments on commit c7de483

Please sign in to comment.