Skip to content

Commit 0d7fb4c

Browse files
committed
Make jshint warn on unused variables.
Also remove a whole ton of them.
1 parent c52b986 commit 0d7fb4c

39 files changed

Lines changed: 52 additions & 286 deletions

.jshintrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
"quotmark": false,
1616
"regexp": false,
1717
"undef": true,
18-
"unused": false,
18+
"unused": "vars",
1919
"strict": true,
2020
"trailing": true,
2121
"asi": false,

spec/Map/unionRectanglesSpec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
/*global require,describe,it,expect,beforeEach*/
3+
/*global require,describe,it,expect*/
44

55
var Rectangle = require('../../third_party/cesium/Source/Core/Rectangle');
66

spec/ViewModels/CsvItemViewModelSpec.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
/*global require,describe,it,expect,beforeEach*/
44

55
var ApplicationViewModel = require('../../src/ViewModels/ApplicationViewModel');
6-
var ImageryLayerItemViewModel = require('../../src/ViewModels/ImageryLayerItemViewModel');
76
var CatalogItemViewModel = require('../../src/ViewModels/CatalogItemViewModel');
87
var CsvItemViewModel = require('../../src/ViewModels/CsvItemViewModel');
98

@@ -25,7 +24,7 @@ describe('CsvDataItemViewModel', function() {
2524

2625
it('throws if constructed without an application', function() {
2726
expect(function() {
28-
var viewModel = new CsvItemViewModel();
27+
var viewModel = new CsvItemViewModel(); // jshint ignore:line
2928
}).toThrow();
3029
});
3130

spec/ViewModels/WebMapServiceDataItemViewModelSpec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ describe('WebMapServiceDataItemViewModel', function() {
2525

2626
it('throws if constructed without an application', function() {
2727
expect(function() {
28-
var viewModel = new WebMapServiceItemViewModel();
28+
var viewModel = new WebMapServiceItemViewModel(); // jshint ignore:line
2929
}).toThrow();
3030
});
3131

src/Core/inherit.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
'use strict';
22

3-
/*global require*/
4-
53
var inherit = function(base, derived) {
64
function F() {}
75
F.prototype = base.prototype;

src/Map/Dataset.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,6 @@ Dataset.prototype._processVariables = function () {
182182
*/
183183
Dataset.prototype.loadJson = function (jsonTable) {
184184

185-
var dataObject = { positions: [], dataValues: [] };
186185
this.dataShape = undefined;
187186

188187
//create the variable set

src/Map/LeafletVisualizer.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,16 @@
11

22
"use strict";
33

4-
/*global require,L,URI,$,Document,alert,console*/
4+
/*global require,L*/
55
var AssociativeArray = require('../../third_party/cesium/Source/Core/AssociativeArray');
66
var Cartesian2 = require('../../third_party/cesium/Source/Core/Cartesian2');
7-
var Cartesian3 = require('../../third_party/cesium/Source/Core/Cartesian3');
87
var Color = require('../../third_party/cesium/Source/Core/Color');
98
var defined = require('../../third_party/cesium/Source/Core/defined');
109
var destroyObject = require('../../third_party/cesium/Source/Core/destroyObject');
1110
var DeveloperError = require('../../third_party/cesium/Source/Core/DeveloperError');
1211
var Property = require('../../third_party/cesium/Source/DataSources/Property');
1312
var Ellipsoid = require('../../third_party/cesium/Source/Core/Ellipsoid');
1413
var CesiumMath = require('../../third_party/cesium/Source/Core/Math');
15-
var loadImage = require('../../third_party/cesium/Source/Core/loadImage');
1614
var writeTextToCanvas = require('../../third_party/cesium/Source/Core/writeTextToCanvas');
1715

1816

src/Map/TableDataSource.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/*global require,$*/
1+
/*global require*/
22
"use strict";
33

44
var Dataset = require('./Dataset');
@@ -11,7 +11,6 @@ And writes a czml file for it to display
1111

1212
//TODO: DOCUMENT using model in GeoJsonDataSource
1313

14-
var defaultValue = require('../../third_party/cesium/Source/Core/defaultValue');
1514
var defined = require('../../third_party/cesium/Source/Core/defined');
1615
var CzmlDataSource = require('../../third_party/cesium/Source/DataSources/CzmlDataSource');
1716
var Color = require('../../third_party/cesium/Source/Core/Color');

src/Map/VarType.js

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,5 @@
1-
/*global require*/
2-
"use strict";
3-
4-
/*!
5-
* Copyright(c) 2012-2013 National ICT Australia Limited (NICTA). All rights reserved.
6-
*/
1+
'use strict';
72

83
var VarType = {LON: 0, LAT: 1, ALT: 2, TIME: 3, SCALAR: 4, ENUM: 5 };
94

10-
115
module.exports = VarType;
12-
13-
14-

src/Map/gmlToGeoJson.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
/*global require,$*/
3+
/*global $*/
44

55
function gmlToGeoJson(xml) {
66
var json = $.xml2json(xml);

0 commit comments

Comments
 (0)