Merge pull request phaserjs#4250 from jcyuan/Branch_v3.15.1

doc improvement & minor bug fix

Author: photonstorm

src/gameobjects/container/Container.js

@@ -601,8 +601,8 @@ var Container = new Class({
      * @method Phaser.GameObjects.Container#getFirst
      * @since 3.4.0
      *
-     * @param {string} [property] - The property to test on each Game Object in the Container.
-     * @param {*} [value] - The value to test the property against. Must pass a strict (`===`) comparison check.
+     * @param {string} property - The property to test on each Game Object in the Container.
+     * @param {*} value - The value to test the property against. Must pass a strict (`===`) comparison check.
      * @param {integer} [startIndex=0] - An optional start index to search from.
      * @param {integer} [endIndex=Container.length] - An optional end index to search up to (but not included)
      *

src/structs/List.js

@@ -10,10 +10,9 @@ var NOOP = require('../utils/NOOP');
 var StableSort = require('../utils/array/StableSort');
 
 /**
- * @callback EachListCallback
- * @generic I - [item]
+ * @callback EachListCallback<I>
  *
- * @param {*} item - The item which is currently being processed.
+ * @param {I} item - The item which is currently being processed.
  * @param {...*} [args] - Additional arguments that will be passed to the callback, after the child.
  */
 
@@ -271,7 +270,6 @@ var List = new Class({
      * @method Phaser.Structs.List#getFirst
      * @since 3.0.0
      *
-     * @genericUse {T} - [value]
      * @genericUse {T | null} - [$return]
      *
      * @param {string} property - The name of the property to test or a falsey value to have no criterion.

src/structs/Map.js

@@ -7,11 +7,10 @@
 var Class = require('../utils/Class');
 
 /**
- * @callback EachMapCallback
- * @generic E - [entry]
+ * @callback EachMapCallback<E>
  *
  * @param {string} key - The key of the Map entry.
- * @param {*} entry - The value of the Map entry.
+ * @param {E} entry - The value of the Map entry.
  *
  * @return {?boolean} The callback result.
  */

src/structs/Set.js

@@ -7,10 +7,9 @@
 var Class = require('../utils/Class');
 
 /**
- * @callback EachSetCallback
- * @generic E - [entry]
+ * @callback EachSetCallback<E>
  *
- * @param {*} entry - The Set entry.
+ * @param {E} entry - The Set entry.
  * @param {number} index - The index of the entry within the Set.
  *
  * @return {?boolean} The callback result.