geojs 1.14.4 → 1.14.6

package/src/util/common.js

@@ -20,7 +20,7 @@ var m_timingData = {},
  * it finds.
  *
  * @param {...*} var_args Any number of arguments.
- * @returns {number} The first numeric argument, or `undefined` if there are no
+ * @returns {number?} The first numeric argument, or `undefined` if there are no
  *      numeric arguments.
  * @private
  */
@@ -31,6 +31,7 @@ function setNumeric() {
       return arguments[i];
     }
   }
+  return undefined;
 }
 
 /**
@@ -88,7 +89,7 @@ var util = {
    * @param {geo.point2D} vert0 Vertex 0 of the triangle.
    * @param {geo.point2D} vert1 Vertex 1 (x direction) of the triangle.
    * @param {geo.point2D} vert2 Vertex 2 (y direction) of the triangle.
-   * @returns {geo.point2D} The point in the triangle basis, or `undefined`
+   * @returns {geo.point2D?} The point in the triangle basis, or `undefined`
    *    if the triangle is degenerate.
    * @memberof geo.util
    */
@@ -103,6 +104,7 @@ var util = {
     if (det) {
       return {x: (x * d - y * b) / det, y: (x * -c + y * a) / det};
     }
+    return undefined;
   },
 
   /**
@@ -188,7 +190,7 @@ var util = {
    * Return a random string of length n || 8.  The string consists of
    * mixed-case ASCII alphanumerics.
    *
-   * @param {number} [n=8] The length of the string to return.
+   * @param {number} [n] The length of the string to return.
    * @returns {string} A string of random characters.
    * @memberof geo.util
    */
@@ -252,7 +254,7 @@ var util = {
    *
    * @param {number} start The start integer.
    * @param {number} end The end integer.
-   * @param {number} [step=1] The step.
+   * @param {number} [step] The step.
    * @returns {number[]} An array of integers.
    * @memberof geo.util
    */
@@ -331,10 +333,10 @@ var util = {
    * @param {geo.vgl.geometryData} geom The geometry to reference and modify.
    * @param {string} srcName The name of the source.
    * @param {number} len The number of elements for the array.
-   * @param {number} [allowLarger=0.2] If the existing buffer is larger than
+   * @param {number} [allowLarger] If the existing buffer is larger than
    *    requested, don't reallocate it unless it exceeds the size of
    *    `len * (1 + allowLarger)`.
-   * @param {number} [allocateLarger=0.1] If reallocating an existing buffer,
+   * @param {number} [allocateLarger] If reallocating an existing buffer,
    *    allocate `len * (1 + allocateLarger)` to reduce the need to reallocate
    *    on subsequent calls.  If this is the first allocation (the previous
    *    size was 0), `len` is allocated.
@@ -483,7 +485,7 @@ var util = {
    * @param {geo.actionRecord[]} actions A list of actions to compare to the
    *    inputs and modifiers.  The first action that matches will be
    *    returned.
-   * @returns {geo.actionRecord} A matching action or `undefined`.
+   * @returns {geo.actionRecord?} A matching action or `undefined`.
    * @memberof geo.util
    */
   actionMatch: function (inputs, modifiers, actions) {
@@ -512,6 +514,7 @@ var util = {
     })) {
       return matched;
     }
+    return undefined;
   },
 
   /**
@@ -550,9 +553,9 @@ var util = {
     var minLevel = Math.min(0, Math.floor(Math.log(Math.min(
           (mapW || tileWidth) / tileWidth,
           (mapH || tileHeight) / tileHeight)) / Math.log(2))),
-        maxLevel = Math.ceil(Math.log(Math.max(
+        maxLevel = Math.max(0, Math.ceil(Math.log(Math.max(
           width / tileWidth,
-          height / tileHeight)) / Math.log(2));
+          height / tileHeight)) / Math.log(2)));
     var mapParams = {
       node: node,
       ingcs: '+proj=longlat +axis=esu',
@@ -918,7 +921,7 @@ var util = {
    *    minimum.
    * @param {number} [max] If specified, use this instead of calculating the
    *    maximum.
-   * @param {boolean} [limit=false] If truthy, if `min` is specified, the
+   * @param {boolean} [limit] If truthy, if `min` is specified, the
    *    returned `min` will be the larger of the specified value and the
    *    computed value, and if `max` is specified, the returned value will
    *    be the smaller of the specified value and the computed value.
@@ -994,7 +997,7 @@ var util = {
 
   /**
    * Recursively merge two objects.  This is intended to replace
-   * util.deepMerge(target, ...sources).
+   * $.extend(target, ...sources).
    *
    * @param {object} target target object to modify.
    * @param {object} sources object(s) to merge into the target.
@@ -1397,9 +1400,9 @@ var util = {
    *
    * @param {boolean} [stop] Falsy to start tracking, truthy to start tracking.
    * @param {boolean} [reset] If truthy, reset the statistics.
-   * @param {number} [threshold=15] If present, set the threshold in
+   * @param {number} [threshold] If present, set the threshold in
    *    milliseconds used in tracking slow callbacks.
-   * @param {number} [keep=200] If present, set the number of recent frame
+   * @param {number} [keep] If present, set the number of recent frame
    *    times to track.
    * @memberof geo.util
    */

package/src/util/distanceGrid.js

@@ -55,7 +55,6 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  * @copyright 2012, David Leaver
  */
 
-var $ = require('jquery');
 var L = {};
 L.Util = {
     // return unique ID of an object
@@ -181,7 +180,7 @@ DistanceGrid.prototype = {
 
     /* return the point coordinates contained in the structure */
     contents: function () {
-        return $.map(this._objectPoint, function (val) { return val; });
+        return Object.values(this._objectPoint);
     },
 
     _getCoord: function (x) {

package/src/util/mockVGL.js

@@ -1,5 +1,3 @@
-/* eslint-disable camelcase */
-
 var webglRenderer = require('../webgl/webglRenderer');
 
 var _renderWindow, _supported;
@@ -11,7 +9,7 @@ module.exports = {};
  * state.  Use restoreWebglRenderer to unmock.  Call vgl.mockCounts() to get
  * the number of times different webGL functions have been called.
  *
- * @param {boolean} [supported=true] If false, then the webgl renderer will
+ * @param {boolean} [supported] If false, then the webgl renderer will
  *      indicate that this is an unsupported browser environment.
  * @alias geo.util.mockWebglRenderer
  */

package/src/util/polyops.js

@@ -26,7 +26,7 @@ var util = require('../util/common');
  * @property {number} [epsilon1] A precision value to use when computing the
  *   operation.  If not specified, this is computed from the range of values in
  *   ``poly1``.  It is used for processing ``poly1`` and the general operation.
- * @property {number} [epsilon1] A precision value to use when processing
+ * @property {number} [epsilon2] A precision value to use when processing
  *   ``poly2``.  If not specified, this is computed from the range of values in
  *   ``poly2``.
  * @property {string|object} [style] If specified, the preferred output style.

package/src/util/throttle.js

@@ -37,7 +37,7 @@
  * @alias geo.util.throttle
  * @param {number} delay A zero-or-greater delay in milliseconds. For event
  *    callbacks, values around 100 or 250 (or even higher) are most useful.
- * @param {boolean} [no_trailing=false] If no_trailing is
+ * @param {boolean} [no_trailing] If no_trailing is
  *    true, callback will only execute every `delay` milliseconds while the
  *    throttled-function is being called. If no_trailing is false or
  *    unspecified, callback will be executed one final time after the last
@@ -173,7 +173,7 @@ var throttle = function (delay, no_trailing, callback, accumulator, debounce_mod
  * @method geo.util.debounce
  * @param {number} delay A zero-or-greater delay in milliseconds. For event
  *    callbacks, values around 100 or 250 (or even higher) are most useful.
- * @param {boolean} [at_begin=false] If at_begin is false or
+ * @param {boolean} [at_begin] If at_begin is false or
  *    unspecified, callback will only be executed `delay` milliseconds after
  *    the last debounced-function call. If at_begin is true, callback will be
  *    executed only at the first debounced-function call. (After the

package/src/vgl/boundingObject.js

@@ -67,7 +67,7 @@ vgl.boundingObject = function () {
    */
   this.setBounds = function (minX, maxX, minY, maxY, minZ, maxZ) {
     if (!this.hasValidBounds([minX, maxX, minY, maxY, minZ, maxZ])) {
-      return;
+      return undefined;
     }
 
     m_bounds[0] = minX;

package/src/vgl/groupNode.js

@@ -54,6 +54,7 @@ vgl.groupNode = function () {
         return true;
       }
     }
+    return false;
   };
 
   /**

package/src/vgl/material.js

@@ -25,7 +25,7 @@ vgl.material = function () {
   /**
    * Return bin number for the material.
    *
-   * @default 100
+   * @default
    * @returns {number}
    */
   this.binNumber = function () {

package/src/vgl/object.js

@@ -5,7 +5,7 @@ var timestamp = require('../timestamp');
  * Create a new instance of class object.
  *
  * @class
- * @alias vgl.object.
+ * @alias vgl.object
  * @returns {vgl.object}
  */
 vgl.object = function () {

package/src/webgl/layer.js

@@ -85,7 +85,7 @@ var webgl_layer = function () {
    * renderers for multiple layers, rerendering can be delayed.
    *
    * @param {geo.webgl.webglRenderer} newRenderer The renderer to move to.
-   * @param {boolean} [rerender=false] If truthy, rerender after the switch.
+   * @param {boolean} [rerender] If truthy, rerender after the switch.
    * @returns {this}
    * @memberof geo.webgl.layer
    */

package/src/webgl/markerFeature.js

@@ -275,7 +275,7 @@ var webgl_markerFeature = function (arg) {
    *    each arrays.
    * @param {array} styleArray If keyOrObject is a string, an array of values
    *    for the style.  If keyOrObject is an object, this parameter is ignored.
-   * @param {boolean} [refresh=false] `true` to redraw the feature when it has
+   * @param {boolean} [refresh] `true` to redraw the feature when it has
    *    been updated.  If an object with styles is passed, the redraw is only
    *    done once.
    * @returns {this}

package/src/webgl/pointFeature.js

@@ -250,7 +250,7 @@ var webgl_pointFeature = function (arg) {
    *    each arrays.
    * @param {array} styleArray If keyOrObject is a string, an array of values
    *    for the style.  If keyOrObject is an object, this parameter is ignored.
-   * @param {boolean} [refresh=false] `true` to redraw the feature when it has
+   * @param {boolean} [refresh] `true` to redraw the feature when it has
    *    been updated.  If an object with styles is passed, the redraw is only
    *    done once.
    * @returns {this}

package/src/webgl/quadFeature.js

@@ -508,7 +508,7 @@ var webgl_quadFeature = function (arg) {
    * @param {string} shaderType One of `image_vertex`, `image_fragment`,
    *   `color_vertex`, or `color_fragment`.
    * @param {string} shaderCode The shader program.
-   * @returns {this} The class instance on success, undefined in an unknown
+   * @returns {this?} The class instance on success, undefined in an unknown
    *    shaderType was specified.
    */
   this.setShader = function (shaderType, shaderCode) {
@@ -518,7 +518,7 @@ var webgl_quadFeature = function (arg) {
       case 'color_vertex': vertexShaderColor = shaderCode; break;
       case 'color_fragment': fragmentShaderColor = shaderCode; break;
       default:
-        return;
+        return undefined;
     }
     return m_this;
   };

package/src/webgl/webglRenderer.js

@@ -319,7 +319,7 @@ var checkedWebGL;
 webglRenderer.supported = function () {
   if (checkedWebGL === undefined) {
     /* This is extracted from what Modernizr uses. */
-    var canvas, ctx, exts; // eslint-disable-line no-unused-vars
+    var canvas, ctx, exts;
     try {
       canvas = document.createElement('canvas');
       /** @type {WebGLRenderingContext} */