Skip to content

Commit

Permalink
[[CHORE]] Remove unused internal variable
Browse files Browse the repository at this point in the history
The `urls` variable was included in the initial release of JSLint [1]. It was
later removed via [1], before the first public release of JSHint. JSHint has
never used this value.

[1] ca120a7
[2] 528bd2e
  • Loading branch information
jugglinmike authored and rwaldron committed Aug 20, 2019
1 parent c81bf3a commit d015657
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions src/jshint.js
Expand Up @@ -86,8 +86,6 @@ var JSHINT = (function() {
membersOnly,
predefined, // Global variables defined by option

urls,

extraModules = [],
emitter = new events.EventEmitter();

Expand Down Expand Up @@ -6339,7 +6337,6 @@ var JSHINT = (function() {
});

functions = [state.funct];
urls = [];
member = {};
membersOnly = null;
inblock = false;
Expand Down Expand Up @@ -6537,11 +6534,6 @@ var JSHINT = (function() {
data.implieds = impliedGlobals;
}

if (urls.length > 0) {
/* istanbul ignore next */
data.urls = urls;
}

globals = state.funct["(scope)"].getUsedOrDefinedGlobals();
if (globals.length > 0) {
data.globals = globals;
Expand Down

0 comments on commit d015657

Please sign in to comment.