Merge pull request #6339 from fredj/cleanup

Code cleanup

config/example.json

@@ -28,7 +28,6 @@
     ],
     "jscomp_off": [
       "lintChecks",
-      "analyzerChecks",
       "missingProvide",
       "unknownDefines"
     ],

config/examples-all.json

@@ -27,8 +27,7 @@
       "*"
     ],
     "jscomp_off": [
-      "lintChecks",
+      "lintChecks"
-      "analyzerChecks"
     ],
     "extra_annotation_name": [
       "api", "observable"

config/ol.json

@@ -18,8 +18,7 @@
       "*"
     ],
     "jscomp_off": [
-      "lintChecks",
+      "lintChecks"
-      "analyzerChecks"
     ],
     "extra_annotation_name": [
       "api", "observable"

doc/tutorials/closure.md

@@ -230,8 +230,7 @@ Here is a version of `config.json` with more compilation checks enabled:
     ],
     "jscomp_off": [
       "unknownDefines",
-      "lintChecks",
+      "lintChecks"
-      "analyzerChecks"
     ],
     "extra_annotation_name": [
       "api", "observable"

src/ol/kinetic.js

@@ -103,15 +103,6 @@ ol.Kinetic.prototype.end = function() {
 };
 
 
-/**
- * @private
- * @return {number} Duration of animation (milliseconds).
- */
-ol.Kinetic.prototype.getDuration_ = function() {
-  return Math.log(this.minVelocity_ / this.initialVelocity_) / this.decay_;
-};
-
-
 /**
  * @return {number} Total distance travelled (pixels).
  */

src/ol/renderer/canvas/tilelayer.js

@@ -52,12 +52,6 @@ ol.renderer.canvas.TileLayer = function(tileLayer) {
    */
   this.tmpExtent = ol.extent.createEmpty();
 
-  /**
-   * @private
-   * @type {ol.TileCoord}
-   */
-  this.tmpTileCoord_ = [0, 0, 0];
-
   /**
    * @private
    * @type {ol.TileRange}

src/ol/view.js

@@ -217,7 +217,7 @@ ol.View.prototype.animate = function(var_args) {
   }
   var series = [];
   for (var i = 0; i < animationCount; ++i) {
-    var options = /** @type olx.AnimationOptions */ (arguments[i]);
+    var options = /** @type {olx.AnimationOptions} */ (arguments[i]);
 
     var animation = /** @type {ol.ViewAnimation} */ ({
       start: start,